weblogic-kubernetes-operator icon indicating copy to clipboard operation
weblogic-kubernetes-operator copied to clipboard

Enhance the MII sample pod wait script for 4.0

Open tbarnes-us opened this issue 2 years ago • 2 comments

Changes:

  • Rely on a domain reaching its 'Complete' condition instead of requiring user to pass in the expected pod count
  • Check that the domain generation and observedGeneration match
  • Check that the domain api version is at least v9
  • Fail fast by default if a 'Failed' condition is detected.

tbarnes-us avatar Aug 04 '22 19:08 tbarnes-us

NOTE: This is converted to 'draft' as there are issues in the new 4.0 operator conditions that in turn make this change unstable (as this change depends on conditions). I have reproduced using the latest Operator version and have filed a corresponding internal JIRA task.

tbarnes-us avatar Aug 04 '22 19:08 tbarnes-us

Minor side comments about potential further minor improvements (lower priority - may be added as time permits):

  • Simplify: Replace use of temporary files by populating internal results into an env var instead.
  • Clarify: Update usage to clearly mention that the script checks generation vs. observedGeneration.
  • Simplify: Create log helper function for replacing 'echo timestamp ...' style lines.
  • Fix: Usage should get defaults from default constants - it is currently using vars that might change when the command line is processed.

tbarnes-us avatar Aug 24 '22 15:08 tbarnes-us

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
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Sep 13 '22 12:09 sonarqubecloud[bot]

Closing - moved to different branch/pull OWLS-88344-new

tbarnes-us avatar Sep 14 '22 19:09 tbarnes-us