-
Notifications
You must be signed in to change notification settings - Fork 936
feat: Add "state" command to pull and push workspace state #1264
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
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1264 +/- ##
==========================================
+ Coverage 65.90% 66.01% +0.10%
==========================================
Files 275 276 +1
Lines 18034 18146 +112
Branches 216 216
==========================================
+ Hits 11886 11979 +93
- Misses 4913 4925 +12
- Partials 1235 1242 +7
Continue to review full report at Codecov.
|
var buildName string | ||
cmd := &cobra.Command{ | ||
Use: "pull <workspace> [file]", | ||
RunE: func(cmd *cobra.Command, args []string) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this needs a cobra.MinimumNArgs(1)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
It's possible for a workspace to become in an invalid state. This is something we'll detect for jobs, and allow monitoring of. These commands will allow admins to manually reconcile state.
It's possible for a workspace to become in an invalid state. This is something we'll detect for jobs, and allow monitoring of. These commands will allow admins to manually reconcile state.
It's possible for a workspace to become in an invalid state.
This is something we'll detect for jobs, and allow monitoring of.
These commands will allow admins to manually reconcile state.
When the state is pushed, should we require a new workspace build to be created? That seems like a better approach than replacing it.