search

SUSE / BCI-tests

This repository contains the tests for the SUSE Base Container Images
Apache License 2.0
10 stars 22 forks source link

WIP: add minimal smoke test for postfix container image #487

Closed Priyankasaggu11929 closed 3 months ago

Priyankasaggu11929 commented 5 months ago

PR adds basic smoke test for postfix container image (re: https://github.com/SUSE/BCI-dockerfile-generator/pull/1046)

To test (with postfix container image built in linked PR 1046):

OS_VERSION=tumbleweed TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/tumbleweed/tumbleweed-1046/ tox -e postfix

OS_VERSION=15.6 TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/sle-15-sp6/6-1046/ tox -e postfix

Known issue with:

OS_VERSION=15.5 TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/sle-15-sp5/5-1046/ tox -e postfix

# fails with 

/bin/sh -c \'ps -eo pid,cmd | grep \'"\'"\'^ *1 \'"\'"\' | sed \'"\'"\'s/^ *1 //\'"\'"\'\'', _stdout=b'', _stderr=b'/bin/sh: ps: command not found\n')

(missing ps command in suse/sle15:15.5)

Priyankasaggu11929 commented 5 months ago

I'm expecting following 12 tests to fail, till postfix container image is available on registry.suse.com, with listed tags

Priyankasaggu11929 commented 5 months ago

formatted code & rebased PR. Thanks!

dirkmueller commented 3 months ago

@Priyankasaggu11929 could you please rebase?

Priyankasaggu11929 commented 3 months ago

@Priyankasaggu11929 could you please rebase?

rebased the PR @dirkmueller

Priyankasaggu11929 commented 3 months ago

@dcermak @dirkmueller

I'm still working on testing ldap lookup on SP5/6 postfix container with 389-ds.

But if the current state of tests in this PR are ok to merge, then I will send follow up PR for 389-ds testing.

dirkmueller commented 3 months ago

@dcermak @dirkmueller

I'm still working on testing ldap lookup on SP5/6 postfix container with 389-ds.

But if the current state of tests in this PR are ok to merge, then I will send follow up PR for 389-ds testing.

yeah, that isn't a blocking issue for me. I'll re-review it but we first need to get https://github.com/SUSE/BCI-dockerfile-generator/pull/1046 in

Priyankasaggu11929 commented 3 months ago

https://github.com/SUSE/BCI-dockerfile-generator/pull/1046 is merged now.

The postfix tests are failing in the CI. I'll debug & send changes.

dcermak commented 3 months ago

The SLE 15.6 based container fails to become healthy. Looks like this might be the issue:

/entrypoint/entrypoint.sh: line 400: /usr/sbin/syslogd: No such file or directory

I guess we need to add this package to the container or switch the logging framework (although there seems to be a syslogd implementation based on busybox, so we could try that).

Priyankasaggu11929 commented 3 months ago

Postfix container tests starting failing in the CI.

Testing again after re-adding HEALTHCHECK(s).

Update - adding HEALTHCHECK(s) fixed the failing postfix tests.

Priyankasaggu11929 commented 3 months ago

@dirkmueller, @dcermak, all CI tests are green now except the 6 failing IBS-released tests.

Pinging for final review/approve.

dcermak commented 3 months ago

Thanks a lot for your work Priyanka!