Skip to content

Commit 298d86d

Browse files
committed
remove map fields from BaseComponentSpec
1 parent 6df5425 commit 298d86d

File tree

6 files changed

+5
-13
lines changed

6 files changed

+5
-13
lines changed

operator/src/main/java/com/datastax/oss/kaap/crds/BaseComponentSpec.java

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,6 @@ public abstract class BaseComponentSpec<T> extends ValidableSpec<T> implements W
4646
protected String image;
4747
@JsonPropertyDescription(CRDConstants.DOC_IMAGE_PULL_POLICY)
4848
private String imagePullPolicy;
49-
@JsonPropertyDescription(CRDConstants.DOC_NODE_SELECTORS)
50-
protected Map<String, String> nodeSelectors;
5149
@JsonPropertyDescription(CRDConstants.DOC_REPLICAS)
5250
protected Integer replicas;
5351
@JsonPropertyDescription("Pod disruption budget configuration.")
@@ -60,16 +58,6 @@ public abstract class BaseComponentSpec<T> extends ValidableSpec<T> implements W
6058
private NodeAffinity nodeAffinity;
6159
@JsonPropertyDescription(CRDConstants.DOC_ANTIAFFINITY)
6260
private AntiAffinityConfig antiAffinity;
63-
@JsonPropertyDescription(CRDConstants.DOC_ANNOTATIONS)
64-
private Map<String, String> annotations;
65-
@JsonPropertyDescription(CRDConstants.DOC_POD_ANNOTATIONS)
66-
private Map<String, String> podAnnotations;
67-
@JsonPropertyDescription(CRDConstants.DOC_LABELS)
68-
private Map<String, String> labels;
69-
@JsonPropertyDescription(CRDConstants.DOC_POD_LABELS)
70-
private Map<String, String> podLabels;
71-
@JsonPropertyDescription(CRDConstants.DOC_POD_MATCH_LABELS)
72-
private Map<String, String> matchLabels;
7361
@JsonPropertyDescription(CRDConstants.DOC_IMAGE_PULL_SECRETS)
7462
private List<LocalObjectReference> imagePullSecrets;
7563
@JsonPropertyDescription(CRDConstants.DOC_CONTAINER_ENV)
@@ -88,7 +76,6 @@ public void applyDefaults(GlobalSpec globalSpec) {
8876
if (image == null) {
8977
image = globalSpec.getImage();
9078
}
91-
nodeSelectors = ConfigUtil.mergeMaps(globalSpec.getNodeSelectors(), nodeSelectors);
9279
if (imagePullPolicy == null) {
9380
imagePullPolicy = globalSpec.getImagePullPolicy();
9481
}

operator/src/main/java/com/datastax/oss/kaap/crds/bookkeeper/BookKeeperSetSpec.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,7 @@ public static class ServiceConfig {
172172
@Override
173173
public void applyDefaults(GlobalSpec globalSpec) {
174174
super.applyDefaults(globalSpec);
175+
nodeSelectors = ConfigUtil.mergeMaps(globalSpec.getNodeSelectors(), nodeSelectors);
175176
super.applyFsGroup0IfSecurityContextMissing();
176177
if (replicas == null) {
177178
replicas = 3;

operator/src/main/java/com/datastax/oss/kaap/crds/broker/BrokerSetSpec.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,7 @@ public static class TransactionCoordinatorInitJobConfig {
215215
@Override
216216
public void applyDefaults(GlobalSpec globalSpec) {
217217
super.applyDefaults(globalSpec);
218+
nodeSelectors = ConfigUtil.mergeMaps(globalSpec.getNodeSelectors(), nodeSelectors);
218219

219220
if (gracePeriod == null) {
220221
gracePeriod = 60;

operator/src/main/java/com/datastax/oss/kaap/crds/function/FunctionsWorkerSpec.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,7 @@ public static class ServiceConfig {
156156
@Override
157157
public void applyDefaults(GlobalSpec globalSpec) {
158158
super.applyDefaults(globalSpec);
159+
nodeSelectors = ConfigUtil.mergeMaps(globalSpec.getNodeSelectors(), nodeSelectors);
159160
super.applyFsGroup0IfSecurityContextMissing();
160161

161162
if (replicas == null) {

operator/src/main/java/com/datastax/oss/kaap/crds/proxy/ProxySetSpec.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,7 @@ public static class WebSocketConfig {
181181
@Override
182182
public void applyDefaults(GlobalSpec globalSpec) {
183183
super.applyDefaults(globalSpec);
184+
nodeSelectors = ConfigUtil.mergeMaps(globalSpec.getNodeSelectors(), nodeSelectors);
184185
if (replicas == null) {
185186
replicas = 3;
186187
}

operator/src/main/java/com/datastax/oss/kaap/crds/zookeeper/ZooKeeperSpec.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ public static class MetadataInitializationJobConfig {
149149
@Override
150150
public void applyDefaults(GlobalSpec globalSpec) {
151151
super.applyDefaults(globalSpec);
152+
nodeSelectors = ConfigUtil.mergeMaps(globalSpec.getNodeSelectors(), nodeSelectors);
152153
super.applyFsGroup0IfSecurityContextMissing();
153154
if (podManagementPolicy == null) {
154155
podManagementPolicy = "Parallel";

0 commit comments

Comments
 (0)