Skip to content

[Hackaton] Introduce Web Push#5277

Open
blackheaven wants to merge 11 commits into
developfrom
gdifolco/hackaton/web-push
Open

[Hackaton] Introduce Web Push#5277
blackheaven wants to merge 11 commits into
developfrom
gdifolco/hackaton/web-push

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven blackheaven requested review from a team as code owners June 18, 2026 10:01
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 18, 2026
@blackheaven blackheaven force-pushed the gdifolco/hackaton/web-push branch from f17e16f to 047d1a9 Compare June 18, 2026 14:41
The gundeck-integration helm test pod mounts gundeck-config (whose
configmap emits postgresqlPassword: /etc/wire/gundeck/secrets/pgPassword
when gundeck.secrets.pgPassword is set) but did not mount the
gundeck-secrets volume. This caused createEnv -> initPostgresPool ->
initCredentials to fail with 'File doesn't exist' for the two tests
that call withSettingsOverrides (storeNotificationsEvenWhenRedisIsDown,
testRedisMigration).

Mirrors the existing brig-secrets pattern in brig-integration.yaml and
the earlier fixes in gundeck/deployment.yaml and
integration/integration-integration.yaml.
@blackheaven blackheaven force-pushed the gdifolco/hackaton/web-push branch from 047d1a9 to 8e05230 Compare June 18, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants