RHOAIENG-17305: chore(tests): IPv4 compatibility test(s), checking that we did not break the single-stack IPv4 case #835
Conversation
|
Skipping CI for Draft Pull Request. |
jiridanek
force-pushed
the
jd_ipv6_test
branch
2 times, most recently
from
6d441b5 to
7cb73c8
Compare
jiridanek
force-pushed
the
jd_ipv6_test
branch
from
7cb73c8 to
c9c055f
Compare
jiridanek
force-pushed
the
jd_ipv6_test
branch
from
c9c055f to
9dafdbd
Compare
jiridanek
changed the title
| # try to grab logs regardless of whether container started or not | ||
| stdout, stderr = container.get_logs() | ||
| for line in stdout.splitlines() + stderr.splitlines(): | ||
| logging.debug(line) |
There was a problem hiding this comment.
another level of the test could be to check that log doesn't contain any error or warning message; but let's add it in a separate PR eventually
There was a problem hiding this comment.
|
/lgtm |
|
/override ci/prow/images |
|
@jiridanek: Overrode contexts on behalf of jiridanek: ci/prow/images In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/approve |
|
@jiridanek: Overrode contexts on behalf of jiridanek: ci/prow/images In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jiridanek The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
follow-up to
oc versionin the test, which can be run with ci testing #829Description
Check each workbench image opens up a port successfully. There's a general case (default network config) and an ipv4-only case.
How Has This Been Tested?
Merge criteria: