-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add --include-cluster-resources flag for restores #147
Add --include-cluster-resources flag for restores #147
Conversation
f37fd45
to
9c4e15a
Compare
pkg/restore/restore_test.go
Outdated
@@ -394,6 +401,24 @@ func TestRestoreResourceForNamespace(t *testing.T) { | |||
restorers: map[schema.GroupResource]restorers.ResourceRestorer{schema.GroupResource{Resource: "foo-resource"}: newFakeCustomRestorer()}, | |||
expectedObjs: toUnstructured(newTestConfigMap().WithArkLabel("my-restore").ConfigMap), | |||
}, | |||
// NEW |
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.
Remove
9c4e15a
to
b7d8ef5
Compare
@skriss I was just thinking that if you're creating a |
Or make it an optional bool (pointer) |
Yeah, I think defaulting to include makes sense. I think I like using an optional bool that defaults to true if not specified better; it's more consistent with the corresponding flag on backups. |
Yeah, let's go with the optional bool that defaults to true. |
440d12c
to
59b4ec9
Compare
Signed-off-by: Steve Kriss <steve@heptio.com>
…rue) Signed-off-by: Steve Kriss <steve@heptio.com>
59b4ec9
to
a7cc587
Compare
Fixes #136