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: missing prod label for cronjob (among others?) #564

Closed
wants to merge 10 commits into from

Conversation

DerekRoberts
Copy link
Member

@DerekRoberts DerekRoberts commented Jan 17, 2024


Thanks for the PR!

Any successful deployments (not always required) will be available below.

Once merged, code will be promoted and handed off to following workflow run.

@MCatherine1994
Copy link
Contributor

Screen Shot 2024-01-17 at 9 44 44 AM Hi Derek, just wondering are we still going to use "test" tag image in prod deployment? if we already promote image with tag "prod" in the previous step, maybe we can use the prod tag?

@DerekRoberts
Copy link
Member Author

DerekRoberts commented Jan 17, 2024

@MCatherine1994 No! That was a mistake. Thanks for the catch. :)

@MCatherine1994
Copy link
Contributor

Hi Derek, just wondering in the openshift deployment yaml for the api, the cron job image is using image: ${REGISTRY}/${PROMOTE}, and the deployment config image is using image: ${NAME}-${ZONE}-${COMPONENT}:${ZONE}-api from image stream, I wonder shall we make them the same? or you'll have another pr for that?

@DerekRoberts
Copy link
Member Author

@MCatherine1994 That's doable! I'd prefer to merge this and put the variable and image registry changes in another PR, since they're more of an improvement and less of a break/fix.

@DerekRoberts
Copy link
Member Author

Rolled into #568. Closing.

@DerekRoberts DerekRoberts deleted the bug/missingProdLabels branch January 18, 2024 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

bug: API and cronjob using different images
2 participants