Skip to content

fix(dev): stop vault-k8s release from claiming e2e-tests NodePorts#4571

Merged
divineforest merged 1 commit into
mainfrom
chore/fix-e2e-vault-port-conflict
May 12, 2026
Merged

fix(dev): stop vault-k8s release from claiming e2e-tests NodePorts#4571
divineforest merged 1 commit into
mainfrom
chore/fix-e2e-vault-port-conflict

Conversation

@divineforest
Copy link
Copy Markdown
Contributor

@divineforest divineforest commented May 12, 2026

The vault-k8s and e2e-tests Tilt resources share the helm/e2e-tests chart. Components mcpExampleOAuth/mcpServerJwks/mcpServerIdJag default to enabled with pinned NodePorts (30082-30084), so installing both releases fails with "provided port is already allocated". vault-k8s only needs Vault, so disable those components alongside the existing wiremock/keycloak overrides.


Archestra Contributor

The vault-k8s and e2e-tests Tilt resources share the helm/e2e-tests
chart. Components mcpExampleOAuth/mcpServerJwks/mcpServerIdJag default
to enabled with pinned NodePorts (30082-30084), so installing both
releases fails with "provided port is already allocated". vault-k8s
only needs Vault, so disable those components alongside the existing
wiremock/keycloak overrides.
@divineforest divineforest enabled auto-merge May 12, 2026 08:26
@divineforest divineforest added this pull request to the merge queue May 12, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 12, 2026
@divineforest divineforest merged commit 953a696 into main May 12, 2026
80 of 81 checks passed
@divineforest divineforest deleted the chore/fix-e2e-vault-port-conflict branch May 12, 2026 09:10
iskhakov added a commit that referenced this pull request May 12, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.2.42](platform-v1.2.41...platform-v1.2.42)
(2026-05-12)


### Bug Fixes

* **dev:** stop vault-k8s release from claiming e2e-tests NodePorts
([#4571](#4571))
([953a696](953a696))


### Miscellaneous Chores

* vault readonly migration
([#4528](#4528))
([8df421b](8df421b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: archestra-ci[bot] <222894074+archestra-ci[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Ildar Iskhakov <ildar6v@gmail.com>
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.

1 participant