flip flag for easier testing

Signed-off-by: Priya Bibra <[email protected]>
slackhq · Sep 7, 2023 · 8cc0ecd · 8cc0ecd
1 parent f3d3ae0
commit 8cc0ecd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/go/vt/vtgate/vstream_manager_test.go b/go/vt/vtgate/vstream_manager_test.go
@@ -1196,6 +1196,7 @@ func TestVStreamIdleHeartbeat(t *testing.T) {
 }
 
 func TestVstreamCopy(t *testing.T) {
+	*allowVstreamCopy = false
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 

diff --git a/go/vt/vtgate/vtgate.go b/go/vt/vtgate/vtgate.go
@@ -71,7 +71,7 @@ var (
 	defaultDDLStrategy   = flag.String("ddl_strategy", string(schema.DDLStrategyDirect), "Set default strategy for DDL statements. Override with @@ddl_strategy session variable")
 	dbDDLPlugin          = flag.String("dbddl_plugin", "fail", "controls how to handle CREATE/DROP DATABASE. use it if you are using your own database provisioning service")
 	noScatter            = flag.Bool("no_scatter", false, "when set to true, the planner will fail instead of producing a plan that includes scatter queries")
-	allowVstreamCopy     = flag.Bool("allow_vstream_copy", false, "when set to false, vstream copy will not be allowed - temporary until we can properly support RDONLY for this")
+	allowVstreamCopy     = flag.Bool("allow_vstream_copy", true, "when set to false, vstream copy will not be allowed - temporary until we can properly support RDONLY for this")
 
 	// TODO(deepthi): change these two vars to unexported and move to healthcheck.go when LegacyHealthcheck is removed