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

Improvement to container build jobs #5536

Merged
merged 6 commits into from
Jun 25, 2024

Conversation

jrjohnson
Copy link
Member

@jrjohnson jrjohnson commented Jun 15, 2024

  • Disabled fast failing to containers will build even if one fails
  • Added slack messaging when there is a failure
  • Use the new GitHub API for passing outputs between jobs
  • Updated nginx
  • Changed our health check
  • Used some new options to change the way we download and run the containers

- Disabled fast failing to containers will build even if one fails
- Added slack messaging when there is a failure
- Use the new GitHub API for passing outputs between jobs
Always try and start every container, even if the one before failed and
output the docker process list every time so we can see what might be
failing in a test.
Not sure why, but this startup has slowed down.
localhost stopped working, I'm not 100% clear on why.
This has fallen way behind, the good news is that nginx now provides a
"stable-alpine" image so we don't have to remember to update the version
in the future.
We have some failures here and I'm not sure why. Playing around with
downloading less and putting them in a different place.
@jrjohnson jrjohnson marked this pull request as ready for review June 25, 2024 20:37
Copy link
Member

@stopfstedt stopfstedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stopfstedt stopfstedt merged commit 9c34178 into ilios:master Jun 25, 2024
32 checks passed
@jrjohnson jrjohnson deleted the improved-container-builds branch June 25, 2024 21:20
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.

None yet

2 participants