Skip to content

Commit 4c2eab1

Browse files
committed
Merge branch 'fix-generation-status-update-main' into 'main'
Fix domain status observedGeneration not updated when mii online update See merge request weblogic-cloud/weblogic-kubernetes-operator!4962
2 parents b494894 + 1232ef7 commit 4c2eab1

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

operator/src/main/java/oracle/kubernetes/operator/DomainStatusUpdater.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@
8585
import static oracle.kubernetes.operator.ProcessingConstants.DOMAIN_TOPOLOGY;
8686
import static oracle.kubernetes.operator.ProcessingConstants.MII_DYNAMIC_UPDATE;
8787
import static oracle.kubernetes.operator.ProcessingConstants.MII_DYNAMIC_UPDATE_RESTART_REQUIRED;
88+
import static oracle.kubernetes.operator.ProcessingConstants.MII_DYNAMIC_UPDATE_SUCCESS;
8889
import static oracle.kubernetes.operator.ProcessingConstants.SERVER_HEALTH_MAP;
8990
import static oracle.kubernetes.operator.ProcessingConstants.SERVER_STATE_MAP;
9091
import static oracle.kubernetes.operator.WebLogicConstants.RUNNING_STATE;
@@ -424,10 +425,12 @@ static class DomainStatusUpdaterContext {
424425
private DomainStatus newStatus;
425426
private final List<EventData> newEvents = new ArrayList<>();
426427
final boolean endOfProcessing;
428+
private Packet packet;
427429

428430
DomainStatusUpdaterContext(Packet packet, DomainStatusUpdaterStep domainStatusUpdaterStep) {
429431
info = DomainPresenceInfo.fromPacket(packet).orElseThrow();
430432
isMakeRight = MakeRightDomainOperation.isMakeRight(packet);
433+
this.packet = packet;
431434
this.domainStatusUpdaterStep = domainStatusUpdaterStep;
432435
endOfProcessing = (Boolean) packet.getOrDefault(ProcessingConstants.END_OF_PROCESSING, Boolean.FALSE);
433436
}
@@ -538,7 +541,10 @@ Step createUpdateSteps(Step next) {
538541
if (!isStatusUnchanged()) {
539542
result.add(createDomainStatusReplaceStep());
540543
} else {
541-
if (endOfProcessing && isMakeRight) {
544+
boolean successFullDynamicUpdate = MII_DYNAMIC_UPDATE_SUCCESS.equals(packet.get(MII_DYNAMIC_UPDATE))
545+
|| MII_DYNAMIC_UPDATE_RESTART_REQUIRED.equals(packet.get(MII_DYNAMIC_UPDATE));
546+
547+
if ((endOfProcessing && isMakeRight) || successFullDynamicUpdate) {
542548
Optional.ofNullable(createDomainStatusObservedGenerationReplaceStep()).ifPresent(result::add);
543549
}
544550
}

0 commit comments

Comments
 (0)