Skip to content

DM-48647: Update ap_verify default to use ghcr #1067

Merged
roceb merged 2 commits intomainfrom
tickets/DM-48647
Apr 30, 2025
Merged

DM-48647: Update ap_verify default to use ghcr #1067
roceb merged 2 commits intomainfrom
tickets/DM-48647

Conversation

@roceb
Copy link
Copy Markdown
Contributor

@roceb roceb commented Mar 22, 2025

No description provided.

@roceb roceb requested a review from ktlim March 24, 2025 15:06
@roceb roceb force-pushed the tickets/DM-48647 branch from 6b9a912 to 385cd69 Compare April 3, 2025 23:14
@roceb roceb force-pushed the tickets/DM-48647 branch from 778d0ba to 0752371 Compare April 30, 2025 16:44
@roceb roceb force-pushed the tickets/DM-48647 branch from 0752371 to 50b6656 Compare April 30, 2025 19:44
parameters {
stringParam('REF', null, 'Git "ref" of ap_verify to attempt to build. Default uses version in release docker image.')
stringParam('DOCKER_IMAGE', 'lsstsqre/centos:d_latest', 'Explicit name of release docker image including tag.')
stringParam('DOCKER_IMAGE', 'ghcr.io/lsst/scipipe', 'Explicit name of release docker image including tag.')
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this has to have a tag. d_latest should carry over.

Copy link
Copy Markdown
Contributor

@ktlim ktlim left a comment

Choose a reason for hiding this comment

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

Please fix the tag, otherwise this is fine.

@roceb roceb merged commit 6d135ba into main Apr 30, 2025
4 checks passed
@roceb roceb deleted the tickets/DM-48647 branch April 30, 2025 21:59
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