Skip to content

Conversation

@xiancao
Copy link
Contributor

@xiancao xiancao commented May 2, 2022

@hzhao-github
Copy link
Contributor

LGTM

@maggiehe00
Copy link
Contributor

I'm not sure why we need to change method checkDomainEventContainsExpectedMsg of K8sEvnets in release/3.4 to make it different than main branch. Can you please explain.

@xiancao
Copy link
Contributor Author

xiancao commented May 2, 2022

I'm not sure why we need to change method checkDomainEventContainsExpectedMsg of K8sEvnets in release/3.4 to make it different than main branch. Can you please explain.

checkDomainEventContainsExpectedMsg() method only checks the first event that matches the event type, reason, timestamp and etc. For DOMAIN_PROCESSING_FAILED type, there could be multiple events. The logic need to make sure that it checks all matching events.

Copy link
Contributor

@maggiehe00 maggiehe00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sonarqubecloud
Copy link

sonarqubecloud bot commented May 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@rjeberhard rjeberhard merged commit d34ef3d into release/3.4 May 2, 2022
@rjeberhard rjeberhard deleted the xc-97159-2 branch May 2, 2022 20:54
rjeberhard pushed a commit to rjeberhard/weblogic-kubernetes-operator that referenced this pull request Apr 14, 2023
…le#3022)

* backport triage ItMiiAuxiliaryImage test failure in release/3.4
rjeberhard pushed a commit that referenced this pull request Apr 19, 2023
* backport triage ItMiiAuxiliaryImage test failure in release/3.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants