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

feat( cluster ): Adds support for recovery.mode=import #475

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

itay-grudev
Copy link
Collaborator

@itay-grudev itay-grudev commented Dec 19, 2024

The original definition for pgBaseBackupSource is extracted into a template {{- include "cluster.externalSourceCluster" (list) so that it can be reused by both pgBaseBackupSource and importSource.

Closes: #424, #425

Closes: #424, #425
Signed-off-by: Itay Grudev <itay@verito.digital>
@itay-grudev itay-grudev requested a review from phisco December 19, 2024 02:42
Signed-off-by: Itay Grudev <itay@verito.digital>
@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label Dec 19, 2024
@capuche2412
Copy link

Hi there! 👋

I just wanted to bring this topic up again, as I believe it would be a really nice addition to the project. It aligns well with the existing work, and I think it could bring some great value. 😊

Please let me know if you need any help! Thanks for considering this!

charts/cluster/README.md Outdated Show resolved Hide resolved
charts/cluster/README.md Outdated Show resolved Hide resolved
Copy link

@pchovelon pchovelon left a comment

Choose a reason for hiding this comment

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

Some suggestions

charts/cluster/values.yaml Outdated Show resolved Hide resolved
charts/cluster/values.yaml Show resolved Hide resolved
Co-authored-by: Pierrick <139142330+pchovelon@users.noreply.github.com>
Signed-off-by: Itay Grudev <itay+github.com@grudev.com>
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.

Can't set externalClusters
3 participants