-
Notifications
You must be signed in to change notification settings - Fork 463
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 reuse of the functionality that checks for skaffold dependencies #8630
Allow reuse of the functionality that checks for skaffold dependencies #8630
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.
Thanks a lot for the PR!
It looks good, I just have one idea about keeping the previous behaviour so that all mismatches between the actual and current dependencies is displayed in a single invocation of the script.
/test pull-gardener-e2e-kind |
/lgtm |
LGTM label has been added. Git tree hash: cadc7d98d9035830aad539711ce2fbd24dbb497d
|
22c4acd
to
c592b8b
Compare
/lgtm |
LGTM label has been added. Git tree hash: 50d19442ec61dc2dd7e2928106b2b5a84a29e98f
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: plkokanov 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 |
* go.mod: Update github.com/gardener/gardener to v1.82.2 * make revendor * Clean up TODO related to the skaffold version After gardener/gardener#8634 we no longer need to overwrite the skaffold version * Use the `hack/check-skaffold-deps-for-binary.sh` from Gardener scripts See gardener/gardener#8630
How to categorize this PR?
/area dev-productivity
/kind enhancement
What this PR does / why we need it:
In the registry-cache extension we also had to specify ko dependencies in the skaffold file to workaround GoogleContainerTools/skaffold#7836. See gardener/gardener-extension-registry-cache#57.
We also forked the
hack/check-skaffold-deps.sh
script and adapted it for us.This PR makes possible reuse of the core logic so that other extensions using skaffold (such as shoot-rsyslog-relp extension, cc @plkokanov) can easily reuse the logic for checking skaffold deps.
Which issue(s) this PR fixes:
Part of gardener/gardener-extension-registry-cache#3
Special notes for your reviewer:
See example usage in ialidzhikov/gardener-extension-registry-cache@7f47b4b
Release note: