Skip to content

test: Remove e2e test #933

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

Merged
merged 1 commit into from
Dec 5, 2024
Merged

test: Remove e2e test #933

merged 1 commit into from
Dec 5, 2024

Conversation

henrywang
Copy link
Collaborator

This PR removed e2e test here because QE already has daily e2e test for bootc latest main copr build.

xref: #868

@henrywang
Copy link
Collaborator Author

The integration test failure is not caused by this PR. It'll be fixed in another PR later.

xref: bootc-dev#868

Signed-off-by: Xiaofeng Wang <henrywangxf@me.com>
@cgwalters
Copy link
Collaborator

BTW I asked and the recent constant e2e failures here are because it's unable to provision virt-capable metal machines in the target cloud.

I would like to investigate again using nested virt in GCP at least instead as we've had pretty decent luck with using that for at least sanity testing.

@henrywang
Copy link
Collaborator Author

henrywang commented Dec 4, 2024

I would like to investigate again using nested virt in GCP at least instead as we've had pretty decent luck with using that for at least sanity testing.

I use GCP VM as Github action runner to run RHEL for Edge test (nested virt required) before. It works. (x86_64 only, arm GCP VM does not support nested virt)

We can drop testing farm and use tmt to run test on GCP VM directly, like tmt run provision -h connect --guest x.x.x.x --user root --key x/private_key plan xxx. And GCP is CNCF supported cloud.

@henrywang henrywang merged commit 4690143 into bootc-dev:main Dec 5, 2024
24 of 26 checks passed
@henrywang henrywang deleted the remove_e2e branch December 5, 2024 05:12
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.

2 participants