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.

Sign up for GitHub

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

Jump to bottom

Small fixes to FAST Networking stage with NVAs #1273

Merged
merged 3 commits into from
Mar 23, 2023
Merged

Small fixes to FAST Networking stage with NVAs #1273

merged 3 commits into from
Mar 23, 2023

Conversation

simonebruzzechesse
Copy link
Collaborator

Updates in this PR:

  • Removed old startup-script template file used with previous version of the NVA (replaced by cloud init and cos)
  • Shorten Service Account name for test-resources (previous one was exceeding max length)
  • Aligned boot disk configuration for compatibility with newer version of the compute-vm module (added nested initialize_params in boot_disk parameter)
  • Fixed internet connectivity from spokes VM due to missing sNAT on the untrusted network interface of the NVAs (iptable nat configuration was available in the previous version of the NVA via startup script but that logic was not ported in the newer version of the module). Added enable_masquerading (from newer version of the simple-nva module) parameter on the untrusted network interface which solves the connectivity issue.

@simonebruzzechesse simonebruzzechesse marked this pull request as ready for review March 22, 2023 21:36
@ludoo ludoo merged commit c4c4688 into master Mar 23, 2023
@ludoo ludoo deleted the bruzz/fix-nva-stage branch March 23, 2023 08:57
lcaggio pushed a commit that referenced this pull request May 5, 2023
@simonebruzzechesse @lcaggio
Small fixes to FAST Networking stage with NVAs (#1273)
9a658b0 
* fix issue with test-resources and internet connectivity from spokes

* terraform fmt

* removed reference to startup-script in README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LucaPrete LucaPrete LucaPrete approved these changes

@ludoo ludoo ludoo approved these changes

@skalolazka skalolazka Awaiting requested review from skalolazka

Assignees
No one assigned
Labels
on:FAST
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@simonebruzzechesse @ludoo @LucaPrete