-
Notifications
You must be signed in to change notification settings - Fork 183
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
Rely on index checksum for HelmRepository reconcile operation #267
Rely on index checksum for HelmRepository reconcile operation #267
Conversation
4dca0c7
to
9ee6f68
Compare
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.
Couple of minor nitpicks, other than that looking good 🌻
9ee6f68
to
0440a5c
Compare
Should be ready to merge, sorry for all the back and forth. ;) |
@@ -209,11 +209,16 @@ func (r *HelmRepositoryReconciler) reconcile(ctx context.Context, repository sou | |||
return sourcev1.HelmRepositoryNotReady(repository, sourcev1.IndexationFailedReason, err.Error()), err | |||
} | |||
|
|||
// return early on unchanged generation | |||
indexBytes, err := yaml.Marshal(chartRepo.Index) |
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.
indexBytes, err := yaml.Marshal(chartRepo.Index) | |
indexBytes, err := yaml.Marshal(&chartRepo.Index) |
This will be the last requested change, promised 🖐️
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.
Good catch, here we go!
Fixes fluxcd#256 Signed-off-by: Léo Martinez <leo84.martinez@gmail.com>
0440a5c
to
c1512d3
Compare
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.
Thank you very much @martinezleoml 🍇
Fixes #256