Skip to content

Commit

Permalink
safekeeper: fix typo in allowlist for /profile/heap (#10186)
Browse files Browse the repository at this point in the history
  • Loading branch information
erikgrinaker authored Dec 18, 2024
1 parent 1d12efc commit 1668d39
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion safekeeper/src/http/routes.rs
Original file line number Diff line number Diff line change
Expand Up @@ -564,7 +564,7 @@ pub fn make_router(
if conf.http_auth.is_some() {
router = router.middleware(auth_middleware(|request| {
const ALLOWLIST_ROUTES: &[&str] =
&["/v1/status", "/metrics", "/profile/cpu", "profile/heap"];
&["/v1/status", "/metrics", "/profile/cpu", "/profile/heap"];
if ALLOWLIST_ROUTES.contains(&request.uri().path()) {
None
} else {
Expand Down

1 comment on commit 1668d39

@github-actions
Copy link

Choose a reason for hiding this comment

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

6450 tests run: 6163 passed, 2 failed, 285 skipped (full report)


Failures on Postgres 17

# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_pageserver_small_inmemory_layers[debug-pg17-False] or test_idle_checkpoints[debug-pg17]"
Flaky tests (7)

Postgres 17

Postgres 16

  • test_pgdata_import_smoke[None-1024-RelBlockSize.MULTIPLE_RELATION_SEGMENTS]: release-arm64

Postgres 15

  • test_pgdata_import_smoke[None-1024-RelBlockSize.MULTIPLE_RELATION_SEGMENTS]: release-arm64
  • test_nbtree_pagesplit_cycleid: release-arm64

Postgres 14

  • test_pgdata_import_smoke[None-1024-RelBlockSize.MULTIPLE_RELATION_SEGMENTS]: release-arm64
  • test_pgdata_import_smoke[8-1024-RelBlockSize.MULTIPLE_RELATION_SEGMENTS]: release-arm64
  • test_physical_replication_config_mismatch_max_locks_per_transaction: release-x86-64

Test coverage report is not available

The comment gets automatically updated with the latest test results
1668d39 at 2024-12-18T16:51:20.312Z :recycle:

Please sign in to comment.