Skip to content

[CLOUDGA-29108] Add interactive confirmation dialogue about disabling backup replication#313

Merged
pranbansal02 merged 1 commit intomainfrom
CLOUDGA-29108
Sep 5, 2025
Merged

[CLOUDGA-29108] Add interactive confirmation dialogue about disabling backup replication#313
pranbansal02 merged 1 commit intomainfrom
CLOUDGA-29108

Conversation

@pranbansal02
Copy link
Copy Markdown
Contributor

Summary

[CLOUDGA-29108] Add interactive confirmation dialogue about disabling backup replication
Also added force flag to bypass the prompt for non-interactive usage

Testing

When replication is enabled on YBM but flag not passed on CLI:

$./ybm cluster update --cluster-name gp-cli --region-info region=asia-east2,num-nodes=1,num-cores=2,disk-size-gb=50,vpc=gcp --region-info region=asia-northeast3,num-nodes=1,num-cores=2,disk-size-gb=50,vpc=gcp --region-info region=europe-central2,num-nodes=1,num-cores=2,disk-size-gb=50,vpc=gcp
A newer version is available. Please upgrade to the latest version v0.1.27
? The field 'backup-replication-gcp-target' is not passed for the regions. This will disable backup replication for cluster gp-cli : Are you sure you want to proceed? No
command aborted


With force flag:

$./ybm cluster update --cluster-name gp-cli --force --region-info region=asia-east2,num-nodes=1,num-cores=2,disk-size-gb=50,vpc=gcp --region-info region=asia-northeast3,num-nodes=1,num-cores=2,disk-size-gb=50,vpc=gcp --region-info region=europe-central2,num-nodes=1,num-cores=2,disk-size-gb=50,vpc=gcp
A newer version is available. Please upgrade to the latest version v0.1.27
The cluster gp-cli is being updated and will begin updating if there are changes
Name      Tier        Version          State     Health    Provider   Regions         Nodes     Node Res.(Vcpu/Mem/DiskGB/IOPS)   Connection Pooling
gp-cli    Dedicated   2024.2.4.0-b89   QUEUED    💚        GCP        asia-east2,+2   3         2 / 8GB / 50GB / -                ❌

@pranbansal02 pranbansal02 requested a review from a team as a code owner September 5, 2025 06:16
Copy link
Copy Markdown
Collaborator

@atharva-yb atharva-yb left a comment

Choose a reason for hiding this comment

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

LGTM

@pranbansal02 pranbansal02 merged commit 5cd2bc7 into main Sep 5, 2025
1 check passed
@pranbansal02 pranbansal02 deleted the CLOUDGA-29108 branch September 5, 2025 10:31
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.

3 participants