Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add config value endpointCA for private S3 such as MinIO #230

Merged
merged 2 commits into from
Mar 26, 2024

Conversation

PseudoResonance
Copy link
Contributor

This PR adds a chart values to endpointCA in both the backup and recovery sections, allowing a user to specify a private CA. The value is stored in a secret, which can either be created manually, or will be created automatically if the create option is true.

Resolves #229

Signed-off-by: PseudoResonance <kaio11604@gmail.com>
@itay-grudev
Copy link
Collaborator

I am aware of this. I already have a solution for it as part of my work on a KUTTL test suite here, where I needed to add endpointCA because I am using MinIO for the backup tests.

That being said, your solution seems to be better.

Copy link
Collaborator

@itay-grudev itay-grudev left a comment

Choose a reason for hiding this comment

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

Can you please run: make schema docs and commit that?

Signed-off-by: PseudoResonance <kaio11604@gmail.com>
@itay-grudev itay-grudev merged commit 181e782 into cloudnative-pg:main Mar 26, 2024
4 checks passed
itay-grudev added a commit that referenced this pull request Mar 27, 2024
* Added the ability to exclude specific PrometheusRules by @itay-grudev (#232)
* Added config value endpointCA for private S3 such as MinIO by @PseudoResonance (#229, #230)
* Bug Fix: Severity of CNPGClusterLowDiskSpaceCritical should be critical not warning by @baurmatt (#223)
* New `backup.barmanObjectStore.wal` and `backup.barmanObjectStore.data` to support disabling encryption by @itay-grudev (#198, #221)

Signed-off-by: Itay Grudev <itay.grudev@essentim.com>
itay-grudev added a commit that referenced this pull request Mar 27, 2024
* Added the ability to exclude specific PrometheusRules by @itay-grudev (#232)
* Added config value endpointCA for private S3 such as MinIO by @PseudoResonance (#229, #230)
* Bug Fix: Severity of CNPGClusterLowDiskSpaceCritical should be critical not warning by @baurmatt (#223)
* New `backup.barmanObjectStore.wal` and `backup.barmanObjectStore.data` to support disabling encryption by @itay-grudev (#198, #221)

Signed-off-by: Itay Grudev <itay.grudev@essentim.com>
Co-authored-by: Itay Grudev <itay.grudev@essentim.com>
@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( cluster ) Related to the cluster chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to specify endpointCA in cluster chart
2 participants