Skip to content

Commit fc30f53

Browse files
committed
[PLAT-20841] Clockbound YNP config param is wrongly set during create universe
Summary: The clockbound param is not set correctly in the YNP config during create universe. This got introduced in D52182 / D50614 Added a fix so that clockbound param value is picked from persisted universe details. Test Plan: Manually verified the clockbound param is set correctly and universe comes up with clockbound. Reviewers: anijhawan, nsingh, agunta Reviewed By: agunta Subscribers: yugaware Differential Revision: https://phorge.dev.yugabyte.com/D53064
1 parent 062bd93 commit fc30f53

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

managed/src/main/java/com/yugabyte/yw/commissioner/tasks/payload/YNPConfigGenerator.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,6 @@ private void populateFromNodeDetails(ConfigParams params, ObjectNode rootNode) {
185185
if (node.cloudInfo.private_ip != null) {
186186
ynpNode.put("node_ip", node.cloudInfo.private_ip);
187187
}
188-
ynpNode.put("is_configure_clockbound", userIntent.isUseClockbound());
189188
ynpNode.put("configure_cgroup", Util.configureCgroup(userIntent, provider, true, confGetter));
190189
DeviceInfo deviceInfo = userIntent.getDeviceInfoForNode(node);
191190
if (deviceInfo.mountPoints != null) {
@@ -249,6 +248,9 @@ private void populateFromUniverse(ConfigParams params, ObjectNode rootNode) {
249248
ObjectNode ynpNode = (ObjectNode) rootNode.get("ynp");
250249
setCommunicationPorts(ynpNode, universe.getUniverseDetails().communicationPorts);
251250
ynpNode.put("is_ybcontroller_disabled", !universe.getUniverseDetails().isEnableYbc());
251+
ynpNode.put(
252+
"is_configure_clockbound",
253+
universe.getUniverseDetails().getPrimaryCluster().userIntent.isUseClockbound());
252254
Customer customer = Customer.getOrBadRequest(params.getProvider().getCustomerUUID());
253255
boolean enableEarlyoomFeature =
254256
confGetter.getConfForScope(customer, CustomerConfKeys.enableEarlyoomFeature);

0 commit comments

Comments
 (0)