ci: maintainer review bypass for solo merges - #125
Closed
bradnussbaum wants to merge 8 commits into
Closed
Conversation
Operators can set discovery_type=K8S with label selector and service port name so cluster members resolve from the Kubernetes API instead of static address lists. Includes resolver, config validation, and IT coverage against a stubbed API.
Enable durable, evolvable causal-cluster replication under large payloads and rolling upgrades by negotiating RAFT_1/RAFT_2 and chunking large replicated content on the v2 path.
Productize dbms.listConnections and dbms.killConnection(s) on StandardNetworkConnectionTracker so admins can inspect and terminate client sessions, with ConnectionTrackingIT covering Bolt/HTTP/HTTPS.
Prove concurrent commits still return distinct up-to-date bookmarks while one transaction is mid-apply, so drivers can rely on read-your-writes.
Cover hostname verification reject/accept paths and legacy policy behavior, with SecureClient/Server SslPolicy overloads for policy-backed secure paths.
Reload tokens/schema before index init, disable metrics on temp recovery DBs, and register store entity counters so fulltext indexes survive backup restore and cluster catchup.
Re-enable CostMorsel acceptance with a refreshed blacklist, restore the morsel experimental notification on the selection path, and align MorselRuntimeAcceptanceTest with current runtime behavior.
GitHub forbids self-approval of PRs. Keep one approving review for everyone else, list the maintainer in bypass_pull_request_allowances, and still enforce status checks via enforce_admins.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@bradnussbaumtobypass_pull_request_allowancesso the solo maintainer can merge their own PRs (GitHub forbids self-approve)enforce_admins: true)Test plan
dev-quality-gateis green without a second approver