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

Push and pull 'nightly' tag Docker images in nightly test job #25619

Merged
merged 6 commits into from
Jul 30, 2024

Conversation

riftEmber
Copy link
Member

@riftEmber riftEmber commented Jul 23, 2024

Adjust the nightly Docker testing script to push (and pull) nightly tags of the images.

This incidentally makes the chapel-gasnet and chapel-gasnet-smp images build off of the just-built Chapel base image. Previously it was building off of the latest tag from the last release, not accessing the local copy of latest due to behavior of the buildx builder.

While here, adjust formatting of util/cron/test-docker.bash script for clarity.

Resolves https://github.com/Cray/chapel-private/issues/6436.

[reviewer info placeholder]

Testing:

  • manual run succeeds and pushes nightly images on chapelmac-m1
  • build exits with error if Chapel base image can't build
  • (pre-merge reminder) update Dockerhub page to note the meaning of the nightly tag

@riftEmber riftEmber requested a review from tzinsky July 23, 2024 20:42
@riftEmber riftEmber changed the title Push and pull 'nightly' Docker images in nightly test job Push and pull 'nightly' tag Docker images in nightly test job Jul 23, 2024
@riftEmber riftEmber force-pushed the test-docker-nightly branch 2 times, most recently from f532e32 to ee0c010 Compare July 24, 2024 15:38
Also adjusts the gasnet and gasnet-smp Dockerfiles to build
off of the Chapel base image's nightly tag (rather than latest)
so they use the just-built result.

Signed-off-by: Anna Rift <[email protected]>
Dashes not underscores...

Signed-off-by: Anna Rift <[email protected]>
@riftEmber riftEmber force-pushed the test-docker-nightly branch from 1c9f6a9 to e2b1613 Compare July 29, 2024 15:47
@arezaii
Copy link
Contributor

arezaii commented Jul 30, 2024

make command casing consistent in Dockerfile

I think #25637 already addressed the case difference issue

Copy link
Contributor

@arezaii arezaii left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @riftEmber

@riftEmber
Copy link
Member Author

I think #25637 already addressed the case difference issue

Ah, that explains why my last rebase skipped my commit doing the same thing 😄

@riftEmber riftEmber merged commit 1f5ed02 into chapel-lang:main Jul 30, 2024
7 checks passed
@riftEmber riftEmber deleted the test-docker-nightly branch July 30, 2024 16: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.

2 participants