Skip to content

test: fix tests remaining stuck waiting for portforward#6987

Merged
mnencia merged 1 commit intomainfrom
dev/6985
Feb 26, 2025
Merged

test: fix tests remaining stuck waiting for portforward#6987
mnencia merged 1 commit intomainfrom
dev/6985

Conversation

@fcanovai
Copy link
Contributor

Error out if ForwardPorts exists with error, instead of going into a blocking select.

Closes #6985

@fcanovai fcanovai added bug 🐛 Something isn't working E2E tests E2E tests tickets for easy triage on release process labels Feb 26, 2025
@cnpg-bot cnpg-bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.22 release-1.24 release-1.25 labels Feb 26, 2025
@github-actions
Copy link
Contributor

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@fcanovai
Copy link
Contributor Author

/test d=push tl=1

@github-actions
Copy link
Contributor

@fcanovai, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/13540924112

Error out if ForwardPorts exists with error, instead of going into a
blocking select.

Closes #6985

Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
@mnencia
Copy link
Member

mnencia commented Feb 26, 2025

/ok-to-merge

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Feb 26, 2025
@mnencia mnencia merged commit 6e46e6c into main Feb 26, 2025
33 of 34 checks passed
@mnencia mnencia deleted the dev/6985 branch February 26, 2025 12:58
cnpg-bot pushed a commit that referenced this pull request Feb 26, 2025
Error out if ForwardPorts exists with error, instead of going into a
blocking select.

Closes #6985

Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
(cherry picked from commit 6e46e6c)
cnpg-bot pushed a commit that referenced this pull request Feb 26, 2025
Error out if ForwardPorts exists with error, instead of going into a
blocking select.

Closes #6985

Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
(cherry picked from commit 6e46e6c)
cnpg-bot pushed a commit that referenced this pull request Feb 26, 2025
Error out if ForwardPorts exists with error, instead of going into a
blocking select.

Closes #6985

Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
(cherry picked from commit 6e46e6c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-requested ◀️ This pull request should be backported to all supported releases bug 🐛 Something isn't working E2E tests E2E tests tickets for easy triage on release process ok to merge 👌 This PR can be merged release-1.22 release-1.24 release-1.25

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: e2e can get stuck if port-forward fails

4 participants