Skip to content

Commit

Permalink
Add --no-progress to wait-for sub command (#2093)
Browse files Browse the repository at this point in the history
## Description
Simply adds --no-progress option to **zarf tools wait-for**.
Wanted this for cases where zarf tools wait-for is used to wait on nodes
joining, can take a couple of minutes depending on the workflow and the
spinners would flood the log a couple times per second.

Fixes #2091

## 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

---------

Co-authored-by: Wayne Starr <[email protected]>
  • Loading branch information
WeaponX314 and Racer159 authored Oct 24, 2023
1 parent 9619449 commit 4478cd6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ zarf tools wait-for { KIND | PROTOCOL } { NAME | SELECTOR | URI } { CONDITION |
```
-h, --help help for wait-for
-n, --namespace string Specify the namespace of the resources to wait for.
--no-progress Disable fancy UI progress bars, spinners, logos, etc
--timeout string Specify the timeout duration for the wait command. (default "5m")
```

Expand Down
1 change: 1 addition & 0 deletions src/cmd/tools/wait.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,4 +53,5 @@ func init() {
toolsCmd.AddCommand(waitForCmd)
waitForCmd.Flags().StringVar(&waitTimeout, "timeout", "5m", lang.CmdToolsWaitForFlagTimeout)
waitForCmd.Flags().StringVarP(&waitNamespace, "namespace", "n", "", lang.CmdToolsWaitForFlagNamespace)
waitForCmd.Flags().BoolVar(&message.NoProgress, "no-progress", false, lang.RootCmdFlagNoProgress)
}

0 comments on commit 4478cd6

Please sign in to comment.