demo: remove system tenant DDL restrictions

This makes demo different from the current out of box experience where
these settings.

Epic: none
Release note: None

Author: stevendanna

pkg/cli/democluster/BUILD.bazel

@@ -37,7 +37,6 @@ go_library(
         "//pkg/sql/catalog/catalogkeys",
         "//pkg/sql/isql",
         "//pkg/sql/sem/catconstants",
-        "//pkg/sql/sqlclustersettings",
         "//pkg/storage/fs",
         "//pkg/testutils/serverutils",
         "//pkg/testutils/serverutils/regionlatency",
@@ -78,7 +77,6 @@ go_test(
         "//pkg/security/securityassets",
         "//pkg/security/securitytest",
         "//pkg/server",
-        "//pkg/sql/sqlclustersettings",
         "//pkg/storage/fs",
         "//pkg/testutils",
         "//pkg/testutils/serverutils/regionlatency",

pkg/cli/democluster/demo_cluster.go

@@ -39,7 +39,6 @@ import (
 	"github.com/cockroachdb/cockroach/pkg/sql/catalog/catalogkeys"
 	"github.com/cockroachdb/cockroach/pkg/sql/isql"
 	"github.com/cockroachdb/cockroach/pkg/sql/sem/catconstants"
-	"github.com/cockroachdb/cockroach/pkg/sql/sqlclustersettings"
 	"github.com/cockroachdb/cockroach/pkg/storage/fs"
 	"github.com/cockroachdb/cockroach/pkg/testutils/serverutils"
 	"github.com/cockroachdb/cockroach/pkg/testutils/serverutils/regionlatency"
@@ -457,15 +456,6 @@ func (c *transientCluster) Start(ctx context.Context) (err error) {
 					return err
 				}
 			}
-
-			for _, s := range []string{
-				string(sqlclustersettings.RestrictAccessToSystemInterface.Name()),
-				string(sql.TipUserAboutSystemInterface.Name()),
-			} {
-				if _, err := ie.Exec(ctx, "restrict-system-interface", nil, fmt.Sprintf(`SET CLUSTER SETTING %s = true`, s)); err != nil {
-					return err
-				}
-			}
 		}
 
 		// Prepare the URL for use by the SQL shell.

pkg/cli/democluster/demo_cluster_test.go

@@ -22,7 +22,6 @@ import (
 	"github.com/cockroachdb/cockroach/pkg/security/securityassets"
 	"github.com/cockroachdb/cockroach/pkg/security/securitytest"
 	"github.com/cockroachdb/cockroach/pkg/server"
-	"github.com/cockroachdb/cockroach/pkg/sql/sqlclustersettings"
 	"github.com/cockroachdb/cockroach/pkg/storage/fs"
 	"github.com/cockroachdb/cockroach/pkg/testutils"
 	"github.com/cockroachdb/cockroach/pkg/testutils/serverutils/regionlatency"
@@ -309,9 +308,6 @@ func TestTransientClusterMultitenant(t *testing.T) {
 	ctx, cancel = c.stopper.WithCancelOnQuiesce(ctx)
 	defer cancel()
 
-	// Ensure CREATE TABLE below works properly.
-	sqlclustersettings.RestrictAccessToSystemInterface.Override(ctx, &c.firstServer.SystemLayer().ClusterSettings().SV, false)
-
 	testutils.RunTrueAndFalse(t, "forSecondaryTenant", func(t *testing.T, forSecondaryTenant bool) {
 		url, err := c.getNetworkURLForServer(ctx, 0,
 			true /* includeAppName */, serverSelection(forSecondaryTenant))