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

Add end-to-end tests for net-lb-app-ext-regional #1962

Merged
merged 4 commits into from
Jan 6, 2024

Conversation

wiktorn
Copy link
Collaborator

@wiktorn wiktorn commented Jan 6, 2024

  • add E2E tests for net-lb-app-ext-regional
  • add tier for external addresses in modules/net-address, as regional lb supports only STANDARD network
  • rename fixture fixtures/compute-mig-bc.tf to fixtures/compute-vm-group-bc.tf as it creates unmanaged group
  • add proxy-only subnets to E2E harness

Checklist

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 wiktorn-e2e_test_fixturization branch from 12efb77 to 021555c Compare January 6, 2024 10:06
HTTP to HTTPS needs work, now fails with:
Error: Error creating ForwardingRule: googleapi: Error 400: Invalid value for field 'resource.IPAddress': '34.160.52.156'. Invalid IP address specified., invalid

  with module.ralb-test-0-redirect.google_compute_forwarding_rule.default,
  on fabric/modules/net-lb-app-ext-regional/main.tf line 32, in resource "google_compute_forwarding_rule" "default":
  32: resource "google_compute_forwarding_rule" "default" {
@wiktorn wiktorn force-pushed the wiktorn-e2e_test_fixturization branch from 021555c to f734503 Compare January 6, 2024 10:09
@wiktorn
Copy link
Collaborator Author

wiktorn commented Jan 6, 2024

E2E test run

@wiktorn wiktorn force-pushed the wiktorn-e2e_test_fixturization branch from f734503 to dc7c2e6 Compare January 6, 2024 10:43
@wiktorn wiktorn marked this pull request as ready for review January 6, 2024 11:05
@wiktorn wiktorn merged commit 2a66fca into master Jan 6, 2024
13 checks passed
@wiktorn wiktorn deleted the wiktorn-e2e_test_fixturization branch January 6, 2024 11:05
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