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
feat(helmv3): support for private repositories and registries in lock files #13726
Conversation
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.
Looks good so far
lib/manager/helmv3/artifacts.ts
Outdated
// get repositories and registries defined in the package file | ||
const packages = yaml.load(newPackageFileContent) as ChartDefinition; //TODO #9610 | ||
const locks = yaml.load( | ||
existingLockFileContent.toString() |
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.
Why toString? Should not be required.
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.
existingLockFileContent
is a Buffer
and the load
function only accepts the content as string.
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.
Add utf-8 encoding to the readLocalFile
then you'll get a string. 😉
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.
My thought has been that it is "safer" to compare the Buffers at https://github.com/renovatebot/renovate/pull/13726/files#diff-c2af1ba9c74f4a977e199d28c1e555ace1ce248ab708c0a5874fa5e5e2cd13c7R160, as encoding can not affect it then.
But works for me.
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
🎉 This PR is included in version 31.51.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes:
Adds login commands for private registries and repositories, if a fitting host rule has been found
Context:
Closes #13524
Closes #10659
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via:
https://github.com/secustor/renovate-helm-oci