Skip to content

Back off CA validation on OpenSearch#275

Merged
dbutenhof merged 1 commit intocloud-bulldozer:mainfrom
dbutenhof:cafix
Oct 8, 2025
Merged

Back off CA validation on OpenSearch#275
dbutenhof merged 1 commit intocloud-bulldozer:mainfrom
dbutenhof:cafix

Conversation

@dbutenhof
Copy link
Copy Markdown
Collaborator

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

This problem turned out to be a bit more difficult than I'd hoped, since we build our deployment containers on GitHub Actions with external runners which can't reach the internal Red Hat CA URL. Ultimately the solution would have to be grabbing and installing the CA bundle at container startup (e.g., in the entrypoint); but for now I'm going to declare (temporary) defeat and restore the verify_certs=False.

Related Tickets & Documents

PANDA-1005 Red Hat CA chain

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

The real problem is that it's difficult to test GitHub Actions outside the normal triggers, which means we don't see container build problems until merging onto main. And, in this case, a local test wouldn't really help since I can't reproduce the GitHub Action Runner environment enough to be sure it makes any difference.

This problem turned out to be a bit more difficult than I'd hoped, since we
build our deployment containers on GitHub Actions with external runners which
can't reach the internal Red Hat CA URL. Ultimately the solution would have to
be grabbing and installing the CA bundle at container startup (e.g., in the
entrypoint); but for now I'm going to declare (temporary) defeat and restore
the `verify_certs=False`.
@dbutenhof dbutenhof self-assigned this Sep 30, 2025
Copy link
Copy Markdown
Collaborator

@MVarshini MVarshini left a comment

Choose a reason for hiding this comment

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

LGTM!

@dbutenhof dbutenhof merged commit 6c10507 into cloud-bulldozer:main Oct 8, 2025
10 checks passed
@dbutenhof dbutenhof deleted the cafix branch October 8, 2025 13:47
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.

2 participants