Skip to content

Conversation

henrywang
Copy link
Collaborator

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 <henrywangxf@me.com>
Signed-off-by: Colin Walters <walters@verbum.org>
And add a retry loop.

Signed-off-by: Colin Walters <walters@verbum.org>
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 <walters@verbum.org>
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 <henrywangxf@me.com>
@henrywang
Copy link
Collaborator 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
Collaborator 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 bootc-dev:main Mar 7, 2024
@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