-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Fix registry-creds addon failure with ImageRepository #9733
Conversation
/ok-to-test |
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.
would u plz put the out (failure) Before this PR in the description too?
kvm2 Driver |
Sure, it's done |
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 for fixing this
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alonyb, medyagh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR fixes #9722
It's not necessary make a reverse (#9725).
Problem was that addon was not loading data in this func to deployment file.
func (m *BinAsset) loadData(isTemplate bool) error
because isTemplate was always in false.
This PR only change a cred bool var to true in addon class (
pkg/minikube/assets/addons.go
)In this PR #9551 this value should also have been changed.
Note: Since if you are going to use data load with jinja template as
{{.ImageRepository}}
in yaml file, you have to set theisTemplate
variable intotrue
, in case you are going to set it manually like "repository/nameImagev1.0
", you can setisTemplate
intofalse
. All this is done in addons.goNow it's working