fix: add --no-install for activate command (#536)

to fix #534

Signed-off-by: Peter Pan <Peter.Pan@daocloud.io>
This commit is contained in:
Peter Pan
2023-07-03 18:36:02 +08:00
committed by GitHub
parent 5a983c4a0a
commit 1f5462c80b
3 changed files with 12 additions and 5 deletions

2
cmd/cache/list.go vendored
View File

@@ -42,7 +42,7 @@ var listCmd = &cobra.Command{
color.Red("Error: %v", err)
os.Exit(1)
}
for _, name := range names {
println(name)
}

View File

@@ -21,6 +21,10 @@ import (
"github.com/spf13/viper"
)
var (
skipInstall bool
)
// activateCmd represents the activate command
var activateCmd = &cobra.Command{
Use: "activate [integration]",
@@ -39,7 +43,7 @@ var activateCmd = &cobra.Command{
integration := integration.NewIntegration()
// Check if the integation exists
err := integration.Activate(integrationName, namespace, activeFilters)
err := integration.Activate(integrationName, namespace, activeFilters, skipInstall)
if err != nil {
color.Red("Error: %v", err)
return
@@ -51,5 +55,6 @@ var activateCmd = &cobra.Command{
func init() {
IntegrationCmd.AddCommand(activateCmd)
activateCmd.Flags().BoolVarP(&skipInstall, "no-install", "s", false, "Only activate the integration filter without installing the filter (for example, if that filter plugin is already deployed in cluster, we do not need to re-install it again)")
}

View File

@@ -66,7 +66,7 @@ func (*Integration) Get(name string) (IIntegration, error) {
return integrations[name], nil
}
func (*Integration) Activate(name string, namespace string, activeFilters []string) error {
func (*Integration) Activate(name string, namespace string, activeFilters []string, skipInstall bool) error {
if _, ok := integrations[name]; !ok {
return errors.New("integration not found")
}
@@ -83,8 +83,10 @@ func (*Integration) Activate(name string, namespace string, activeFilters []stri
viper.Set("active_filters", uniqueFilters)
if err := integrations[name].Deploy(namespace); err != nil {
return err
if !skipInstall {
if err := integrations[name].Deploy(namespace); err != nil {
return err
}
}
if err := viper.WriteConfig(); err != nil {