Added code to prompt user when target-db-version flag not mentioned (#…

…2249) * Added code to prompt user when target-db-schema flag not mentioned * Modified the analyze-schema test adding the --yes flag. * Made suggested changes and removes previous changes. --------- Co-authored-by: Atharva Kurhade <[email protected]>
yugabyte · Jan 28, 2025 · 396bece · 396bece
1 parent aebbe93
commit 396bece
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/migtests/scripts/functions.sh b/migtests/scripts/functions.sh
@@ -357,6 +357,7 @@ export_data() {
 analyze_schema() {
 	args="--export-dir ${EXPORT_DIR}
 		--send-diagnostics=false
+		--yes
 	"
     yb-voyager analyze-schema ${args} $*
 }

diff --git a/yb-voyager/cmd/assessMigrationCommand.go b/yb-voyager/cmd/assessMigrationCommand.go
@@ -1700,9 +1700,15 @@ func validateAssessmentMetadataDirFlag() {
 
 func validateAndSetTargetDbVersionFlag() error {
 	if targetDbVersionStrFlag == "" {
-		targetDbVersion = ybversion.LatestStable
-		return nil
+		if utils.AskPrompt("No target-db-version has been specified.\nDo you want to continue with the latest stable YugabyteDB version:", ybversion.LatestStable.String()) {
+			targetDbVersion = ybversion.LatestStable
+			return nil
+		} else {
+			utils.ErrExit("Aborting..")
+			return nil
+		}
 	}
+
 	var err error
 	targetDbVersion, err = ybversion.NewYBVersion(targetDbVersionStrFlag)