Skip to content

Fix 4096#18

Open
adrianriobo wants to merge 3 commits into
mainfrom
fix-4096
Open

Fix 4096#18
adrianriobo wants to merge 3 commits into
mainfrom
fix-4096

Conversation

@adrianriobo

Copy link
Copy Markdown
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 <ariobolo@redhat.com>
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 <ariobolo@redhat.com>
….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 <ariobolo@redhat.com>
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