Skip to content

tests: ensure client certificate is always being created - #2870

Merged
sxd merged 3 commits into
mainfrom
dev/fix_connectivity_e2e
Sep 26, 2023
Merged

sxd merged 3 commits into
mainfrom
dev/fix_connectivity_e2e

Conversation

@sxd

@sxd sxd commented Sep 25, 2023

Copy link
Copy Markdown
Contributor

The connectivity test were failing due to a dependency of a previous test that should create the proper secret with the required certificate. Now, we create the certificate everytime in the test to keep the tests independent.

@github-actions github-actions Bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.19 labels Sep 25, 2023
@github-actions

Copy link
Copy Markdown
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

@github-actions

Copy link
Copy Markdown
Contributor

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

The connectivity test were failing due to a dependency of a previous test
that should create the proper secret with the required certificate. Now,
we create the certificate everytime in the test to keep the tests
independent.

Signed-off-by: Jonathan Gonzalez V <[email protected]>
@sxd
sxd force-pushed the dev/fix_connectivity_e2e branch from 61dd1d2 to 4470ab6 Compare September 25, 2023 21:33
@github-actions

Copy link
Copy Markdown
Contributor

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions

Copy link
Copy Markdown
Contributor

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@sxd sxd added the ok to merge 👌 This PR can be merged label Sep 26, 2023
@NiccoloFei
NiccoloFei force-pushed the dev/fix_connectivity_e2e branch from 1e3052d to 369b204 Compare September 26, 2023 13:11
Signed-off-by: Niccolò Fei <[email protected]>
@NiccoloFei
NiccoloFei force-pushed the dev/fix_connectivity_e2e branch from 369b204 to 247bef0 Compare September 26, 2023 13:12
@NiccoloFei NiccoloFei changed the title chore: create user certificate for test every time tests: ensure client certificate is always being created Sep 26, 2023
@sxd

sxd commented Sep 26, 2023

Copy link
Copy Markdown
Contributor Author

/ok-to-merge

@sxd
sxd merged commit 7125b96 into main Sep 26, 2023
@sxd
sxd deleted the dev/fix_connectivity_e2e branch September 26, 2023 14:04
cnpg-bot pushed a commit that referenced this pull request Sep 26, 2023
The connectivity test were failing due to a dependency of a previous test
that should create the proper secret with the required certificate. Now,
we create the certificate every time in the test to keep the tests
independent.

Signed-off-by: Jonathan Gonzalez V <[email protected]>
Signed-off-by: Niccolò Fei <[email protected]>
Co-authored-by: Niccolò Fei <[email protected]>
(cherry picked from commit 7125b96)
cnpg-bot pushed a commit that referenced this pull request Sep 26, 2023
The connectivity test were failing due to a dependency of a previous test
that should create the proper secret with the required certificate. Now,
we create the certificate every time in the test to keep the tests
independent.

Signed-off-by: Jonathan Gonzalez V <[email protected]>
Signed-off-by: Niccolò Fei <[email protected]>
Co-authored-by: Niccolò Fei <[email protected]>
(cherry picked from commit 7125b96)
Lennart01 pushed a commit to Lennart01/cloudnative-pg that referenced this pull request Sep 30, 2023
…-pg#2870)

The connectivity test were failing due to a dependency of a previous test
that should create the proper secret with the required certificate. Now,
we create the certificate every time in the test to keep the tests
independent.

Signed-off-by: Jonathan Gonzalez V <[email protected]>
Signed-off-by: Niccolò Fei <[email protected]>
Co-authored-by: Niccolò Fei <[email protected]>
Signed-off-by: Lennart Krauch <[email protected]>
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 no-issue ok to merge 👌 This PR can be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants