Skip to content

Commit

Permalink
Dont allow allocate columnar for unsupported deployers
Browse files Browse the repository at this point in the history
  • Loading branch information
willbroadbelt authored and brett19 committed Aug 15, 2024
1 parent 2836165 commit 5758f38
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
4 changes: 3 additions & 1 deletion deployment/caodeploy/deployer.go
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,9 @@ func (d *Deployer) NewCluster(ctx context.Context, def *clusterdef.Cluster) (dep
if err != nil {
return nil, errors.Wrap(err, "failed to detect whether we are using openshift")
}

if def.Columnar {
return nil, errors.New("columnar is not supported for caodeploy")
}
clusterID := cbdcuuid.New()
namespace := "cbdc2-" + clusterID.String()

Expand Down
3 changes: 3 additions & 0 deletions deployment/localdeploy/deployer.go
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,9 @@ func (d *Deployer) NewCluster(ctx context.Context, def *clusterdef.Cluster) (dep
if len(def.NodeGroups) != 1 || def.NodeGroups[0].Count != 1 {
return nil, errors.New("local deployment only supports a single node")
}
if def.Columnar {
return nil, errors.New("columnar is not supported for local deploy")
}

nodeGrp := def.NodeGroups[0]

Expand Down

0 comments on commit 5758f38

Please sign in to comment.