Skip to content

Commit 7cc0ade

Browse files
committed
OCM-15159 | feat: Introduce -y for migrating cluster network type
1 parent 97b962b commit 7cc0ade

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

cmd/edit/cluster/cmd.go

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -738,7 +738,7 @@ func run(cmd *cobra.Command, _ []string) {
738738
// SDN -> OVN Migration
739739
var migrateNetworkType bool
740740
// Only prompt user with migrating the cluster's network type when it is not OVN-Kubernetes
741-
if cmd.Flags().Changed(ocm.NetworkTypeFlagName) && networkType == ocm.NetworkTypeOvn {
741+
if cmd.Flags().Changed(ocm.NetworkTypeFlagName) && networkType == ocm.NetworkTypeOvn && !confirm.Yes() {
742742
interactive.SetEnabled(true)
743743
}
744744
if interactive.Enabled() && args.networkType != "" {
@@ -782,12 +782,15 @@ func run(cmd *cobra.Command, _ []string) {
782782
}
783783

784784
if cmd.Flags().Changed(ocm.NetworkTypeFlagName) && networkType == ocm.NetworkTypeOvn {
785+
if !confirm.Yes() && migrateNetworkType {
786+
migrateNetworkType, err = confirmMigration()
785787

786-
migrateNetworkType, err = confirmMigration()
787-
788-
if err != nil {
789-
r.Reporter.Errorf("%s", err)
790-
os.Exit(1)
788+
if err != nil {
789+
r.Reporter.Errorf("%s", err)
790+
os.Exit(1)
791+
}
792+
} else if confirm.Yes() {
793+
migrateNetworkType = true
791794
}
792795
}
793796

0 commit comments

Comments
 (0)