Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 4096 #18

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix 4096 #18

wants to merge 3 commits into from

Conversation

adrianriobo
Copy link
Owner

Fixes: Issue #N

Relates to: Issue #N, PR #N, ...

Solution/Idea

Describe in plain English what you solved and how. For instance, Added start command to CRC so the user can create a VM and set-up a single-node OpenShift cluster on it with one command. It requires blablabla...

Proposed changes

List main as well as consequential changes you introduced or had to introduce.

  1. Add start command.
  2. Add setup as prerequisite to start.
  3. ...

Testing

What is the bottom-line functionality that needs testing? Describe in pseudo-code or in English. Use verifiable statements that tie your changes to existing functionality.

  1. start succeeds first time after setup succeeded
  2. stdout contains ... if start succeeded
  3. stderr contains ... after start failed
  4. status returns ... if start succeeded
  5. status returns ... if start failed
  6. start fails after start succeeded or after status says CRC is Running
  7. ...

As part of commit crc-org@ae4e57f we changed the e2e and integration images; as ubi9 based images
were used to build the e2e and integration binaries those can not be executed on RHEL8 target hosts. This fix change the builders to use ubi8 based images

Signed-off-by: Adrian Riobo Lorenzo <[email protected]>
The scripts used to run the integration tests require to set several environment variables to customize the execution; the way the script try to set them and run the integration binary inheriting them was not right. This fix export those variables and execute the integration binary properly

Signed-off-by: Adrian Riobo Lorenzo <[email protected]>
….com/ubi8/httpd-24:latest

This image was partially used within e2e, now we extend its usage to all the remainig places

Signed-off-by: Adrian Riobo Lorenzo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant