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

test: update integration README #377

Merged
merged 4 commits into from
Mar 7, 2024

Conversation

henrywang
Copy link
Contributor

Use --tls-verify=false in podman build, don't need CERT_URL secret any more.

@cgwalters
Copy link
Collaborator

cgwalters commented Mar 6, 2024

Looks like that run hit

Maximum test time '40m' exceeded

...hmm, maybe the pull was slow? I did a blind attempt to add a pull before the run (with retries) to more cleanly separate network issues from install issues.

henrywang and others added 3 commits March 6, 2024 16:44
Signed-off-by: Xiaofeng Wang <[email protected]>
Signed-off-by: Colin Walters <[email protected]>
And add a retry loop.

Signed-off-by: Colin Walters <[email protected]>
Not sure what was going wrong, we really need to collect
the logs in this scenario instead of a hard timeout.

Signed-off-by: Colin Walters <[email protected]>
Github has job status update. Same job will have two status if
testing farm updates PR status. That does not make sense.

Github job status update can't be disabled, so testing farm
PR job status update has to be disabled.

Signed-off-by: Xiaofeng Wang <[email protected]>
@henrywang
Copy link
Contributor Author

@cgwalters There're two test status for one testing farm case (github job status and testing farm status). This does not make sense. Github job status update can't be disabled, so testing farm PR job status update has to be disabled.

BTW: I filed two testing-farm-as-github-action github action bugs sclorg/testing-farm-as-github-action#129 and sclorg/testing-farm-as-github-action#130.

@henrywang
Copy link
Contributor Author

@cgwalters Can we use centos stream 9 image to test runuser -u bin? That might workaround this failure.

@cgwalters
Copy link
Collaborator

@cgwalters Can we use centos stream 9 image to test runuser -u bin? That might workaround this failure.

See #379 for that - it's unrelated and nonblocking.

@henrywang henrywang requested a review from cgwalters March 7, 2024 13:14
@cgwalters
Copy link
Collaborator

Cool, my untested ansible change worked 😄

@cgwalters cgwalters merged commit 4e76c33 into containers:main Mar 7, 2024
19 of 20 checks passed
@henrywang henrywang deleted the test_integration branch March 7, 2024 13:32
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