UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Support CAbundle via Helm Values

Merged tunde requested to merge issue-329 into main
All threads resolved!

This PR fixes issue 329 .

cc: @gosim

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Contributor

    we are pasting the entire cert(base64 encoded) content in the cacert field in the values.yaml.

  • Contributor

    reminder to pull from 'main' (new chart & image versions)

  • tunde added 7 commits

    added 7 commits

    Compare with previous version

  • Author Contributor

    @bhearn done!

  • Micah Nagel added 1 commit

    added 1 commit

    Compare with previous version

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel added 3 commits

    added 3 commits

    Compare with previous version

  • Micah Nagel added 1 commit

    added 1 commit

    Compare with previous version

  • Micah Nagel resolved all threads

    resolved all threads

  • LGTM. Will need to implement the upstream solution whenever that comes available, but this is a good interim solution.

    Another note: based on the env name, I'm assuming this only supports AWS at the moment - which is ok - just something to note if someone else is testing in a different cloud environment. I was unable to find any upstream docs alluding to the envs needed for other cloud providers.

  • Micah Nagel approved this merge request

    approved this merge request

  • Micah Nagel mentioned in commit 2ef50d1c

    mentioned in commit 2ef50d1c

  • merged

  • Please register or sign in to reply
    Loading