Skip to content
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

Implement new test cmd w/a --force-yolo option and improve the discovery for zarf pacakge development helpers #2169

Closed
Racer159 opened this issue Nov 30, 2023 · 0 comments · Fixed by #2170
Labels
enhancement ✨ New feature or request
Milestone

Comments

@Racer159
Copy link
Contributor

Racer159 commented Nov 30, 2023

Is your feature request related to a problem? Please describe.

As Ezra I want to have a cleaner workflow for creating and testing packages during development so that I can test their logic and flow without having to run zarf p c and zarf p d every time.

Describe the solution you'd like

  • Given I have a package definition in a folder
  • When I run zarf dev test package/folder
  • Then Zarf creates the package and deploys the package in one fell swoop without an intermediate tarball
  • And When I run zarf dev test package/folder --force-yolo
  • Then Zarf again creates and deploys the package but forces the package to be in YOLO mode and will not pull images/repos into the package (without the author needing to change the YOLO key in the definition)

Describe alternatives you've considered

We could add a flag on deploy or create but having them combines allows for tweaks to happen faster with fewer disk writes in between runs.

Additional context

This command will allow us to add testing specific flags and features for Zarf packages, and as a part of this change we should remove the dev alias from internal and instead make prepare an alias of dev so that "developing Zarf packages" is the primary wording going forward for the package definition writing process.

@Racer159 Racer159 added the enhancement ✨ New feature or request label Nov 30, 2023
@Racer159 Racer159 changed the title Implement a --force-yolo option and improve the discovery for zarf pacakge development helpers Implement new test cmd w/a --force-yolo option and improve the discovery for zarf pacakge development helpers Nov 30, 2023
@Racer159 Racer159 added this to the (2023.12.05) milestone Nov 30, 2023
@Racer159 Racer159 added the uds label Nov 30, 2023
@Racer159 Racer159 modified the milestones: (2023.12.05), (2023.12.19) Dec 12, 2023
Racer159 added a commit that referenced this issue Dec 15, 2023
…ure `zarf prepare` (#2170)

## Description

Adds a `zarf dev deploy` command that will create + deploy a local
package in one shot.

## Related Issue

Fixes #2169 
Fixes #2098 

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow)
followed

---------

Signed-off-by: razzle <harry@razzle.cloud>
Co-authored-by: Wayne Starr <Racer159@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant