-
Notifications
You must be signed in to change notification settings - Fork 64
🐛 fix: don't template registry+v1 manifests #1979
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: don't template registry+v1 manifests #1979
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Joe Lanford <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1979 +/- ##
==========================================
+ Coverage 69.10% 69.14% +0.03%
==========================================
Files 79 79
Lines 7011 7023 +12
==========================================
+ Hits 4845 4856 +11
- Misses 1885 1886 +1
Partials 281 281
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I wouldn't worry about the upgrade-e2e CI, but it seems weird to me that we are only now seeing the sanity/lint errors, and those look easy enough to resolve. |
/override upgrade-e2e |
@grokspawn: Overrode contexts on behalf of grokspawn: upgrade-e2e In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
88e57f2
to
9811d1b
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: grokspawn 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 |
Description
Some registry+v1 manifests may actually contain Go Template strings that are meant to survive and actually persist into etcd (e.g. to be used as a templated configuration for another component). In order to avoid applying templating logic to registry+v1's static manifests, we create the manifests as Files, and then template those files via simple Templates.
Reviewer Checklist