Skip to content

fix: e2e tests#4847

Merged
Skarlso merged 2 commits intomainfrom
gc/fix/e2e-tests
May 27, 2025
Merged

fix: e2e tests#4847
Skarlso merged 2 commits intomainfrom
gc/fix/e2e-tests

Conversation

@gusfcarvalho
Copy link
Copy Markdown
Member

@gusfcarvalho gusfcarvalho commented May 27, 2025

I think e2e tests on main are now broken after a PR that had them broken was merged :D

Signed-off-by: Gustavo Carvalho <gustavo@externalsecrets.com>
Signed-off-by: Gustavo Carvalho <gustavo@externalsecrets.com>
@gusfcarvalho gusfcarvalho requested a review from a team as a code owner May 27, 2025 11:59
@gusfcarvalho gusfcarvalho requested a review from moolen May 27, 2025 11:59
@sonarqubecloud
Copy link
Copy Markdown

@Skarlso
Copy link
Copy Markdown
Contributor

Skarlso commented May 27, 2025

@gusfcarvalho huh which one was that?

@Skarlso
Copy link
Copy Markdown
Contributor

Skarlso commented May 27, 2025

Oh I see :D

@gusfcarvalho
Copy link
Copy Markdown
Member Author

gusfcarvalho commented May 27, 2025

which one was that?

it had a failure on starting the flux engine because generatrostates crds were owned by the previous test suite (probably because of lack of cleanup - hence this fix). But when I ran again to confirm the issue, scaleway decided to go bonkers lol

@Skarlso Skarlso merged commit a47a323 into main May 27, 2025
20 checks passed
@Skarlso Skarlso deleted the gc/fix/e2e-tests branch May 27, 2025 13:27
pepordev pushed a commit to pepordev/external-secrets that referenced this pull request Jun 11, 2025
* fix: pending generator states makes next suite fail

Signed-off-by: Gustavo Carvalho <gustavo@externalsecrets.com>

* fix: fix remaining suites

Signed-off-by: Gustavo Carvalho <gustavo@externalsecrets.com>

---------

Signed-off-by: Gustavo Carvalho <gustavo@externalsecrets.com>
Signed-off-by: Pedro Parra Ortega <pedro.parraortega@enreach.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.

2 participants