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

Allow including the previous image tag in the commit message template #437

Closed
Tracked by #4712
darkowlzz opened this issue Sep 1, 2022 · 1 comment · Fixed by #647
Closed
Tracked by #4712

Allow including the previous image tag in the commit message template #437

darkowlzz opened this issue Sep 1, 2022 · 1 comment · Fixed by #647
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@darkowlzz
Copy link
Contributor

Discussed in fluxcd/flux2#3027

Originally posted by tun0 August 24, 2022
The notifications we currently have with Flux v1 (with help of fluxcloud) mention both old and new image tags.

With Flux v2, only the new image tag is mentioned.

It would be very nice if the previous tag would be made available as well in the notification object.

@darkowlzz darkowlzz transferred this issue from fluxcd/flux2 Sep 1, 2022
@darkowlzz darkowlzz added the enhancement New feature or request label Sep 1, 2022
@darkowlzz darkowlzz transferred this issue from fluxcd/image-automation-controller Sep 14, 2022
@darkowlzz darkowlzz transferred this issue from fluxcd/image-reflector-controller Sep 14, 2022
@darkowlzz
Copy link
Contributor Author

darkowlzz commented Sep 14, 2022

Based on the discussion in fluxcd/image-reflector-controller#311 (comment) , this is not related to the image selected by the ImagePolicy, but it's about the old value being replaced by the new value in manifest file in git.
An ImageUpdateAutomation notification can include the information about the old and new value in the manifest that it updated.

@pjbgf pjbgf added the help wanted Extra attention is needed label Oct 10, 2022
@stefanprodan stefanprodan changed the title Include previous image tag in imageupdateautomation notifications Allow including the previous image tag in the commit message template Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
Status: Since Last Dev Meeting
Development

Successfully merging a pull request may close this issue.

2 participants