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.

Sign up for GitHub

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

Jump to bottom

feat: use image digests for conditional rollouts with HELM, initial attempt #1528

Merged
merged 8 commits into from
Oct 26, 2023
Merged

feat: use image digests for conditional rollouts with HELM, initial attempt #1528

merged 8 commits into from
Oct 26, 2023

Conversation

mishraomp
Copy link
Collaborator

@mishraomp mishraomp commented Oct 25, 2023 •
edited by github-actions bot
Loading

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are promoted to:

@mishraomp mishraomp linked an issue Oct 25, 2023 that may be closed by this pull request
feat: get digests from builder action #1513
Closed
DerekRoberts
DerekRoberts approved these changes Oct 25, 2023
View reviewed changes
Copy link
Member

@DerekRoberts DerekRoberts left a comment

Choose a reason for hiding this comment

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

Looking forward to this! :)

@mishraomp
fix:
b4d4288 
1. update the helm template version to align with new way of handling digest and tags
2. fix the name of the needs
@mishraomp
feat:
8135e6d 
1. Add digest for TEST and PROD deploys as well to avoid unnecessary rollouts.
@mishraomp
feat:
5244ca2 
1. upgrade to 2.0.0 for builder.
@mishraomp mishraomp marked this pull request as ready for review October 26, 2023 01:18
@mishraomp mishraomp enabled auto-merge (squash) October 26, 2023 01:18
@mishraomp mishraomp merged commit 52cefa8 into main Oct 26, 2023
16 checks passed
@mishraomp mishraomp deleted the 1513-feat-get-digests-from-builder-action branch October 26, 2023 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DerekRoberts DerekRoberts DerekRoberts approved these changes

Assignees
No one assigned
Labels
None yet
Projects
DevOps (NR)
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: get digests from builder action
2 participants
@mishraomp @DerekRoberts