Skip to content

Conversation

@nikita-vanyasin
Copy link
Contributor

@nikita-vanyasin nikita-vanyasin commented Jun 15, 2022

The EnsureCRD implementation was slightly adjusted to return error if requested, so the command can exit with 1 if the problem occurs.

@nikita-vanyasin nikita-vanyasin changed the title Add 'crd install' subcommand to new 'ops' binary [Feature] Add 'crd install' subcommand to new 'ops' binary Jun 15, 2022
@nikita-vanyasin nikita-vanyasin changed the title [Feature] Add 'crd install' subcommand to new 'ops' binary [Feature] Add 'crd install' subcommand Jun 15, 2022
@nikita-vanyasin nikita-vanyasin force-pushed the feature/cmd-ops-crd-install branch from bbd698e to d7efc22 Compare June 15, 2022 07:13
cmd/cmd.go Outdated
"github.com/spf13/cobra"
flag "github.com/spf13/pflag"
appsv1 "k8s.io/api/apps/v1"
v1 "k8s.io/api/core/v1"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, use import names as they are used everywhere

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was just import auto-ordering, but I corrected them anyway 👌

@nikita-vanyasin nikita-vanyasin force-pushed the feature/cmd-ops-crd-install branch from d7efc22 to f5e9b20 Compare June 22, 2022 08:43
@nikita-vanyasin nikita-vanyasin requested a review from ajanikow June 22, 2022 09:12
@nikita-vanyasin nikita-vanyasin force-pushed the feature/cmd-ops-crd-install branch from f5e9b20 to 5b380fa Compare July 12, 2022 07:47
@nikita-vanyasin nikita-vanyasin changed the title [Feature] Add 'crd install' subcommand [Feature] GT-40 Add 'crd install' subcommand Jul 12, 2022
@nikita-vanyasin nikita-vanyasin force-pushed the feature/cmd-ops-crd-install branch from 5b380fa to c146cd5 Compare July 18, 2022 07:09
@ajanikow ajanikow merged commit 24d981c into master Jul 18, 2022
@ajanikow ajanikow deleted the feature/cmd-ops-crd-install branch July 18, 2022 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants