Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed e2e tests for alloydb module #2321

Merged
merged 3 commits into from
May 30, 2024
Merged

Fixed e2e tests for alloydb module #2321

merged 3 commits into from
May 30, 2024

Conversation

simonebruzzechesse
Copy link
Collaborator

Fixed issue with AlloyDB e2e tests. I had to add ignore_changes due to a bug in the API (not yet managed in the provider) causing an attribute not being set and plan never aligned with infrastructure state.


I applicable, I acknowledge that I have:

  • Read the contributing guide
  • Ran terraform fmt on all modified files
  • Regenerated the relevant README.md files using tools/tfdoc.py
  • Made sure all relevant tests pass

@wiktorn wiktorn force-pushed the bruzz-alloydb-e2e-fix branch from bfb9b77 to efd7076 Compare May 30, 2024 07:08
@wiktorn wiktorn force-pushed the bruzz-alloydb-e2e-fix branch from efd7076 to 53473c1 Compare May 30, 2024 07:12
@wiktorn
Copy link
Collaborator

wiktorn commented May 30, 2024

E2E run

@wiktorn wiktorn merged commit 941a35a into master May 30, 2024
13 checks passed
@wiktorn wiktorn deleted the bruzz-alloydb-e2e-fix branch May 30, 2024 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants