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

Fix CI/CD flakiness with docker compose command style update #667

Merged

Conversation

aj-stein-gsa
Copy link
Contributor

@aj-stein-gsa aj-stein-gsa commented Sep 10, 2024

Committer Notes

More details about GHA runner compatibility with old versus new docker compose commands in this discussion post. May resolve #643.

https://github.com/orgs/community/discussions/27185

All Submissions:

By submitting a pull request, you are agreeing to provide this contribution under the CC0 1.0 Universal public domain dedication.

@aj-stein-gsa aj-stein-gsa self-assigned this Sep 10, 2024
@aj-stein-gsa aj-stein-gsa marked this pull request as ready for review September 10, 2024 14:43
@aj-stein-gsa aj-stein-gsa requested a review from a team as a code owner September 10, 2024 14:43
Copy link
Contributor Author

@aj-stein-gsa aj-stein-gsa left a comment

Choose a reason for hiding this comment

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

I did not realized I moved around the OSCAL submodule. To quote a wise artist: so just chill till the next episode!

@aj-stein-gsa aj-stein-gsa marked this pull request as draft September 10, 2024 15:33
More details about GHA runner compatibility with old versus new docker
compose commands in this discussion post.

https://github.com/orgs/community/discussions/27185
Not all, but many, actions were pinned to old versions by tag and to an
explicit SHA1 hash from the repo that are old enough to cause GHA
error notices. Upgraded from these to current SHA1 hash for the most
(692973e3d937129bcbf40652eb9f2f61becf3332) for recent v4.1.7 release.

> The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

> The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Source: https://github.com/GSA/fedramp-automation/actions/runs/10783353551
This directive is now obsolete, causes more noise in the GHA workflow
run logs.

https://forums.docker.com/t/docker-compose-yml-version-is-obsolete/141313
@aj-stein-gsa aj-stein-gsa force-pushed the 643-content-processing-cicd-is-flaky branch from 92a8552 to e4461b7 Compare September 10, 2024 15:42
@aj-stein-gsa aj-stein-gsa marked this pull request as ready for review September 10, 2024 15:43
@aj-stein-gsa
Copy link
Contributor Author

I did not realized I moved around the OSCAL submodule. To quote a wise artist: so just chill till the next episode!

And fixed, moving back from draft to ready.

@aj-stein-gsa aj-stein-gsa merged commit 6096faa into GSA:develop Sep 10, 2024
7 checks passed
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.

1 participant