Skip to content

MLE-28392: Update Helm utility image to redhat/ubi9:9.7#336

Merged
vitalykorolev merged 2 commits into
developfrom
MLE-28392_update-ubi9-util-image
Apr 24, 2026
Merged

MLE-28392: Update Helm utility image to redhat/ubi9:9.7#336
vitalykorolev merged 2 commits into
developfrom
MLE-28392_update-ubi9-util-image

Conversation

@vitalykorolev
Copy link
Copy Markdown
Collaborator

@vitalykorolev vitalykorolev commented Apr 12, 2026

Summary

Update the Helm chart utility (init) container image from redhat/ubi9:9.6 to redhat/ubi9:9.7.

Change

  • charts/values.yaml: bump initContainers.utilContainer.image from redhat/ubi9:9.6 to redhat/ubi9:9.7

  • Updated TLS app server configuration (configure_tls in configmap-scripts.yaml). After each ssl-certificate-template PUT, the script now polls https://localhost:{port}/ every 2 seconds (up to 60 seconds) and only proceeds once the server responds over HTTPS, rather than using a fixed sleep. This makes the timing self-correcting across environments.

The PUT response code is also captured and logged, with an automatic retry on non-204 responses.

The ssl-allow-tls PUT was removed in test. SSL is always enabled internally once a certificate template is assigned.

Validation

Jenkins CI to be validated with MarkLogic 11 and 12 (root and rootless).

Jira: MLE-28392

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Helm chart’s utility (init) container base image to the next UBI9 release to keep the chart aligned with the intended Red Hat UBI patch level.

Changes:

  • Bump initContainers.utilContainer.image default from redhat/ubi9:9.6 to redhat/ubi9:9.7.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread charts/values.yaml
@vitalykorolev vitalykorolev removed the request for review from barkhachoithani April 14, 2026 17:22
- Bump initContainers.utilContainer.image from redhat/ubi9:9.6 to 9.7
  in charts/values.yaml and README.md
- In ML12, assigning ssl-certificate-template to an app server no longer
  automatically sets ssl-allow-tls=true (unlike ML11). Fix configure_tls
  to explicitly set ssl-allow-tls=true via http://8002 BEFORE assigning
  the cert template for each server (App-Services, Admin, Manage), while
  http://8002 is still plaintext. This is harmless on ML11.
@vitalykorolev vitalykorolev force-pushed the MLE-28392_update-ubi9-util-image branch from 8452da2 to ac4a9cf Compare April 14, 2026 17:23
Copy link
Copy Markdown
Collaborator

@sumanthravipati sumanthravipati left a comment

Choose a reason for hiding this comment

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

looks good

@vitalykorolev vitalykorolev requested review from rwinieski and removed request for rwinieski April 18, 2026 16:09
- Replace fixed sleep 5s with active HTTPS polling loop (waits up to
  60s per app server) to handle slower startup on Jenkins/UBI 9.7
- Capture HTTP response code with retry logic for ssl-certificate-template
  PUT instead of fire-and-forget
- Remove ssl-allow-tls PUT (not a valid/configurable property on ML12)
- Update test assertion from ssl-allow-tls to ssl-certificate-template
  (ssl-allow-tls is absent from Manage API responses on ML12+)
@vitalykorolev vitalykorolev force-pushed the MLE-28392_update-ubi9-util-image branch from 4e1927e to ed9d40d Compare April 18, 2026 19:30
Copy link
Copy Markdown
Collaborator

@pengzhouml pengzhouml left a comment

Choose a reason for hiding this comment

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

Looks good to me

@vitalykorolev vitalykorolev merged commit 3d46811 into develop Apr 24, 2026
3 checks passed
@vitalykorolev vitalykorolev deleted the MLE-28392_update-ubi9-util-image branch April 24, 2026 17:08
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.

5 participants