Repository navigation
feat: add Zarf package disassembly/reassembly flows - #1520
Conversation
|
since I know is'a comin' - @greptileai review |
|
|
Want to improve UX a little so moved back to draft - specifically thinking about how a chart edit might be needed... the zarf yaml I think can also use Zarf's helpers to be formatted a little cleaner and I want to explore that too. |
|
@greptileai review again on the final shape |
bradclawsie
left a comment
There was a problem hiding this comment.
Great work. I've reviewed and did a guided reading of data flow. I round-tripped a zarf-init package from disassembly to reassembly (and then deployed the rebuilt package into a fresh cluster).
I had one edge case in another comment, but I am totally fine with applying that patch on my own if you have other things you want to move on to.
Otherwise, this is ready to merge. Let me know how you would like to proceed, and thanks again for this work.
slaskawi
left a comment
There was a problem hiding this comment.
LGTM! Let's get this in!
joelmccoy
left a comment
There was a problem hiding this comment.
Really late getting to this.. Took a very first pass and will do a deeper one soon, but wanted to callout the decision to feature flag. I disagree with this and would like to discuss. Left a comment on this and would love input from @slaskawi @chance-coleman @eoghanriley
Description
This PR adds a Zarf package disassembly command to
uds-cli- the goal is to take a built Zarf package and disassemble it into re-creatable source code that can be modified entirely offline. This is possible today manually withzarf tools archivercommands but this makes the process a bit more streamlined and intentional.Related Issue
Fixes #N/A
Type of change
Checklist before merging