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.

Sign up for GitHub

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

Jump to bottom

feat: declarative publications and subscriptions #115

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

feat: declarative publications and subscriptions #115

wants to merge 67 commits into from

Conversation

gabriele-wolfox
Copy link

@gabriele-wolfox gabriele-wolfox commented Oct 10, 2024 •
edited
Loading

This patch allows the user to configure logical replication publications
and subscription using a Kubernetes API.

Based on cloudnative-pg#5325, to be merged after that.

Closes cloudnative-pg#5567

Signed-off-by: Leonardo Cecchi [email protected]

@github-actions GitHub Actions
Copy link

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@gabriele-wolfox
Copy link
Author

/test d=push tl=4

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11271847854

@gabriele-wolfox gabriele-wolfox changed the title feat: declarative database management feat: declarative publications and subscriptions Oct 10, 2024
@gabriele-wolfox
Copy link
Author

/test

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11273538165

@gabriele-wolfox
Copy link
Author

/test limit=local ft=declarative-pub-sub

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11325996119

@gabriele-wolfox
Copy link
Author

/test limit=local ft=declarative-pub-sub

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11329397733

@gabriele-wolfox
Copy link
Author

/test limit=local ft=declarative-pub-sub

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11329834553

@gabriele-wolfox
Copy link
Author

/test limit=local ft=declarative-pub-sub

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11330791289

@gabriele-wolfox
Copy link
Author

/test limit=local ft=declarative-pub-sub

@github-actions GitHub Actions
Copy link

@gabriele-wolfox, here's the link to the E2E on CNPG workflow run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11341354662

armru and others added 29 commits November 27, 2024 18:21
@armru @mnencia
fix: make alterSql handle parameters correctly
9be5a89 
Signed-off-by: Armando Ruocco <[email protected]>
@armru @NiccoloFei @mnencia
fix: create publication sql
fd3b6a5 
Signed-off-by: Armando Ruocco <[email protected]>

Co-authored-by: Niccolò Fei <[email protected]>
@armru @mnencia
test(subscription): add basic coverage
1b68c73 
Signed-off-by: Armando Ruocco <[email protected]>
@NiccoloFei @mnencia
chore: fixing order and linting
01d3775 
Signed-off-by: Niccolò Fei <[email protected]>
@mnencia
chore: review
1095e3b 
Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
chore: improve readability
5cf243c 
Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
chore: more fixes
9a9a5a9 
Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
test: review
424390a 
Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
chore: quote identifier in parameters
7d46774 
Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
chore: Remove the owner field
8780705 
Signed-off-by: Marco Nenciarini <[email protected]>
@armru @mnencia
chore: review
08264b6 
Signed-off-by: Armando Ruocco <[email protected]>
@armru @mnencia
refactor: remove unknown status
6133524 
Signed-off-by: Armando Ruocco <[email protected]>
@armru @mnencia
refactor: change toPublicationCreateSQL signature
58d0496 
Signed-off-by: Armando Ruocco <[email protected]>
@mnencia
revert: "refactor: remove unknown status"
eecc9de 
This reverts commit bc46b93.

Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
fix: handle finalizers on cluster deletion
54a37bb 
Signed-off-by: Marco Nenciarini <[email protected]>
@mnencia
refactor: finalizers
9562e96 
Signed-off-by: Marco Nenciarini <[email protected]>
@gbartolini @mnencia
docs: renamed page into Logical Replication
796c52f 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: intro
3b140fa 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: publications
2148e7e 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: review publications
984e788 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: subscriptions
4dea38d 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: remove superuser requirement from examples
a0d6ac8 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: example
838737a 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
review
89aec95 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: review publication API
a36bb57 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: review subscription api
978a45e 
Signed-off-by: Gabriele Bartolini <[email protected]>
@gbartolini @mnencia
docs: add operator capability level and index
7c08299 
Signed-off-by: Gabriele Bartolini <[email protected]>
@mnencia
chore: docs review
56b7f0c 
Signed-off-by: Marco Nenciarini <[email protected]>
@NiccoloFei @mnencia
docs: minor fixes
be0d8a2 
Signed-off-by: Niccolò Fei <[email protected]>
@mnencia mnencia force-pushed the declarative-pub-sub branch from d3b3afa to be0d8a2 Compare November 27, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsilvela jsilvela Awaiting requested review from jsilvela jsilvela is a code owner

@NiccoloFei NiccoloFei Awaiting requested review from NiccoloFei NiccoloFei is a code owner

@litaocdl litaocdl Awaiting requested review from litaocdl litaocdl is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
backport-requested ◀️ release-1.22 release-1.23 release-1.24
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add CRDs and controllers for Publication and Subscription
8 participants
@gabriele-wolfox @sxd @armru @mnencia @leonardoce @NiccoloFei @jsilvela @gbartolini