zarf
zarf copied to clipboard
Published
20 hours ago •
defenseunicorns
defenseunicorns
fix: adopt namespace metadata
Description
Changes behavior to keep existing metadata on namespaces rather than steamrolling them
Related Issue
Fixes #2489
Checklist before merging
- [ ] Test, docs, adr added or updated as needed
- [ ] Contributor Guide Steps followed
Deploy Preview for zarf-docs canceled.
| Name | Link |
|---|---|
| Latest commit | 86549e04f71cd5a0ed847e3ed327cc8c77fd9460 |
| Latest deploy log | https://app.netlify.com/sites/zarf-docs/deploys/66479b432707d100086c2f6a |
![netlify[bot] avatar](https://avatars.githubusercontent.com/in/13473?v=4 "Published by a pub.dev verified publisher"){kind=small}
We could use client set directly instead
I'm 50/50 on this. On one hand it's simpler, on the other hand I don't mind abstracting the calls away to a separate library (like cluster since we're going to merge k8s into it). It was nice when context was implemented for example since it was a common way to change all of our k8s calls in the same package. Gonna call it direct for now, and we'll see if anyone disagrees.