-
Notifications
You must be signed in to change notification settings - Fork 951
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
chore: add notification when a release occurs #2112
Conversation
✅ Deploy Preview for karpenter-docs-prod ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -41,3 +41,13 @@ cosignImages() { | |||
COSIGN_EXPERIMENTAL=1 cosign sign ${COSIGN_FLAGS} ${CONTROLLER_DIGEST} | |||
COSIGN_EXPERIMENTAL=1 cosign sign ${COSIGN_FLAGS} ${WEBHOOK_DIGEST} | |||
} | |||
|
|||
notifyRelease(){ | |||
RELEASE_TYPE=$1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thoughts on not exposing the concept of RELEASE_TYPE and instead let it simply be the tag. Customers can delineate the release type by the name. This enables us to build new types of releases and just tag them as we see fit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I rather keep it, notifications can possibly used in many different ways, and it's good to make it easy for the client to not have to do extra work. For example a simple check for the release type can be done via text search making it easy for a basic client like a mail client to do different things depending on the type.
We should still be able to tag things the way we see fit. Also it's good to know what type of a tag was supposed to be released so that we can then compare it to what was actually release. This maybe helpful in troubleshooting and forensics.
Finally a blob of data is not as helpful, this makes the message more human friendly too, it's somewhat like the "kind" in Kubernetes, we are not just defining attributes but also specifying what this even is.
Fixes #
Adds a notification message when releases occur.
How was this change tested?
Does this change impact docs?
Release Note
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.