Skip to content

Bugfix: actions dockerhub push#590

Merged
scrasmussen merged 2 commits intoNCAR:mainfrom
scrasmussen:bugfix/actions-dockerhub-push
Jul 9, 2025
Merged

Bugfix: actions dockerhub push#590
scrasmussen merged 2 commits intoNCAR:mainfrom
scrasmussen:bugfix/actions-dockerhub-push

Conversation

@scrasmussen
Copy link
Copy Markdown
Member

SOURCE: Soren Rasmussen, NSF NCAR

DESCRIPTION OF CHANGES:

  • Move from Docker Buildx to build and push the docker image to basic CLI docker commands. The old version might be failing due to buildx taking too long and github times-out because there hasn't been output to the terminal in a set amount of time.

TESTS CONDUCTED:

  • PR will test and push to dtcenter/ccpp-scm:latest-test docker tag. Once the Github action succeeds the last commit of this PR should be removed before merging.

@scrasmussen scrasmussen force-pushed the bugfix/actions-dockerhub-push branch from b62bd77 to b684271 Compare June 4, 2025 19:56
@scrasmussen scrasmussen marked this pull request as draft June 4, 2025 19:56
@scrasmussen scrasmussen force-pushed the bugfix/actions-dockerhub-push branch 2 times, most recently from 2aee69d to 3d9d81e Compare June 4, 2025 20:09
@scrasmussen scrasmussen marked this pull request as ready for review June 4, 2025 20:36
@scrasmussen
Copy link
Copy Markdown
Member Author

Unfortunately (or fortunately for security) the secrets.DOCKERHUB_TOKEN variable isn't defined in PRs, so the only way to test this is after it is merged into main. Thus this PR is ready for review

@scrasmussen scrasmussen force-pushed the bugfix/actions-dockerhub-push branch from 3d9d81e to 9c90515 Compare July 3, 2025 21:53
@scrasmussen scrasmussen force-pushed the bugfix/actions-dockerhub-push branch 2 times, most recently from 0a5a36f to 023cdb3 Compare July 9, 2025 17:35
@scrasmussen scrasmussen force-pushed the bugfix/actions-dockerhub-push branch from 023cdb3 to a2a15c7 Compare July 9, 2025 18:50
@scrasmussen scrasmussen merged commit e731913 into NCAR:main Jul 9, 2025
13 checks passed
@scrasmussen scrasmussen deleted the bugfix/actions-dockerhub-push branch July 30, 2025 15:55
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.

3 participants