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

--confirm not respected when deploying zarf package #200

Closed
RothAndrew opened this issue Dec 13, 2021 · 4 comments · Fixed by #297
Closed

--confirm not respected when deploying zarf package #200

RothAndrew opened this issue Dec 13, 2021 · 4 comments · Fixed by #297
Assignees
Labels
bug 🐞 Something isn't working

Comments

@RothAndrew
Copy link
Contributor

What I saw:

If I deploy the Big Bang example, which has an optional component called kubescape, it prompts me to choose whether I want Kubescape or not

What I expected:

If I run zarf package deploy foobar.tar.gz --confirm I expect that it would never stop and prompt me to interact, because I used the --confirm flag. It would either make a choice itself or fatal out if it cant. If the --components flag was not used I'm thinking it should use the defaults that are set for each component.

@RothAndrew
Copy link
Contributor Author

Note: This is an existing finding from before the big PR from Friday. It's not an urgent priority before we cut the next release.

@RothAndrew
Copy link
Contributor Author

I'm also curious how we would non-interactively deploy the big bang example without kubescape. Since it is the only optional component omitting it would be the same as asking for the defaults, which in this case is "yes". I wonder if we had Zarf look for an empty string or something to decide that the user is saying "I don't want any of the components that are optional.

Possible example:

# option 1
zarf package deploy foobar.tar.zst --confirm --components ""
# option 2
zarf package deploy foobar.tar.zst --confirm --no-optional-components
# option 3
???

@jeff-mccoy
Copy link
Contributor

Note: This is an existing finding from before the big PR from Friday. It's not an urgent priority before we cut the next release.

Thanks for clarifying I was about to 🤦‍♂️

@RothAndrew
Copy link
Contributor Author

#271 should fix this. It's getting merged into #237, so when #237 merges this should get resolved.

@RothAndrew RothAndrew moved this from Planned to Under Review in Zarf Project Board Feb 4, 2022
@RothAndrew RothAndrew moved this from Under Review to Doing Now in Zarf Project Board Feb 4, 2022
@YrrepNoj YrrepNoj added the bug 🐞 Something isn't working label Feb 8, 2022
@YrrepNoj YrrepNoj moved this from Doing Now to Under Review in Zarf Project Board Feb 8, 2022
@YrrepNoj YrrepNoj linked a pull request Feb 8, 2022 that will close this issue
3 tasks
Repository owner moved this from Under Review to Done in Zarf Project Board Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants