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

Correct branch specify for GH Actions #58

Merged
merged 2 commits into from
Mar 13, 2024
Merged

Correct branch specify for GH Actions #58

merged 2 commits into from
Mar 13, 2024

Conversation

rnc
Copy link
Contributor

@rnc rnc commented Mar 12, 2024

No description provided.

@rnc
Copy link
Contributor Author

rnc commented Mar 12, 2024

@jmtd This keeps hanging - do you know why?

@jmtd
Copy link
Collaborator

jmtd commented Mar 12, 2024

We experience the exact same problem in the OpenJDK repo. I don't know why it's hanging but I did narrow it down to with which command the hang is occurring. the docker library function exec_start sometimes never returns. Their docs imply it's subject to a default API timeout but that isn't taking effect.

I've got a branch that we use to try and improve things where I fork before calling exec_start and watchdog that from the parent process. It doens't fix it , but it means the test that's wedged times out and doesn't stop the whole test run.

v1...jmtd:behave-test-steps:v1

We've been running with this for a while so perhaps I should raise a PR.

@jmtd
Copy link
Collaborator

jmtd commented Mar 12, 2024

oh it's worth configuring the GHA timeout to be much lower than the default (6 hours?) too: https://github.com/jboss-container-images/openjdk/blob/ubi9/.github/workflows/image-workflow-template.yml#L13

@rnc rnc merged commit 2ba86ad into cekit:v1 Mar 13, 2024
1 check passed
@rnc rnc deleted the ACTION branch March 13, 2024 09:41
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