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

chore!: remove walredo_process_kind config option & kind type #7756

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

problame
Copy link
Contributor

refs #7753

Preceding PR #7754
laid out the plan, this one wraps it up.

problame added 2 commits May 14, 2024 13:51
refs #7753

This PR is step (1) of removing sync walredo from Pageserver.

Changes:
* Remove the sync impl
* If sync is configured, warn! and use async instead
* Remove the metric that exposes `kind`
* Remove the tenant status API that exposes `kind`

Future Work
-----------

After we've released this change to prod and are sure we won't
roll back, we will

1. update the prod Ansible to remove the config flag from the prod
   pageserver.toml.
2. remove the remaining `kind` code in pageserver

These two changes need no release inbetween.

See  #7753 for details.
refs #7753

Preceding PR #7754
laid out the plan, this one wraps it up.
Copy link

github-actions bot commented May 14, 2024

3186 tests run: 3047 passed, 0 failed, 139 skipped (full report)


Code coverage* (full report)

  • functions: 31.5% (6569 of 20862 functions)
  • lines: 48.4% (50792 of 104933 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
d966ea4 at 2024-06-04T15:39:24.164Z :recycle:

Base automatically changed from problame/async-walredo/remove-sync-impl-part-1 to main May 15, 2024 13:04
…o/remove-sync-impl-part-2

Conflicts:
	pageserver/src/config.rs
	pageserver/src/walredo/process.rs
@problame problame marked this pull request as ready for review June 5, 2024 09:48
@problame problame requested a review from a team as a code owner June 5, 2024 09:48
@problame problame requested a review from skyzh June 5, 2024 09:48
@problame problame changed the title [DO NOT MERGE YET] chore!: remove walredo_process_kind config option & kind type chore!: remove walredo_process_kind config option & kind type Jun 5, 2024
@problame problame requested review from koivunej and removed request for skyzh June 5, 2024 09:48
Copy link
Member

@koivunej koivunej left a comment

Choose a reason for hiding this comment

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

Yep, looking as expected.

@problame problame merged commit 83ab14e into main Jun 5, 2024
65 checks passed
@problame problame deleted the problame/async-walredo/remove-sync-impl-part-2 branch June 5, 2024 12:21
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