3198 snmp zoo instance

[cakekoa]

What does this PR do?

Fixes #3198.

• Adds a script for building the SNMP image
• Updates the terraform script to generate the SNMP instance

PR Checklist

• Have you added an explanation of what your changes do and why you'd like to include them?
• Is the TravisCI build passing?
• Was the CHANGELOG.md updated to reflect the changes?
• Was the documentation framework updated to reflect the changes?
• Have you checked that you haven't introduced any duplicate code?

Testing Checklist

• Added relevant unit tests?
• Do all unit tests pass?
• Do all end-to-end tests pass?
• Any other testing performed?

  Tested by {Running the Monkey locally with relevant config/running Island/...}

• If applicable, add screenshots or log transcripts of the feature working

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.02 ⚠️

Comparison is base (02ebfa6) 73.13% compared to head (23ef556) 73.11%.

❗ Current head 23ef556 differs from pull request most recent head 484b5c5. Consider uploading reports for the commit 484b5c5 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3237      +/-   ##
===========================================
- Coverage    73.13%   73.11%   -0.02%     
===========================================
  Files          467      467              
  Lines        13681    13675       -6     
===========================================
- Hits         10005     9998       -7     
- Misses        3676     3677       +1     

see 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[mssalvatore]

We should put this in the europe-west1-b zone, as it has much higher quotas than europe-west3-a.

[cakekoa]

That sounds good. I can re-generate the instances on europe-west1. Our config currently defaults to europe-west3 -- I'll update it to point to europe-west1 by default.

[mssalvatore]

Will pointing to europe-west1 by default have unintended consequences? The BB tests have a dictionary of regions to instances.

[cakekoa]

Ah, I see what is going on. Yes, it will cause issues. We have our networks split into different regions -- do you know why we do this?

Our networks:

• 10.2.0.* (tunneling2-main) europe-west3
• 10.2.1.* (tunneling-main) europe-west3
• 10.2.2.* (monkeyzoo-main) europe-west3
• 10.2.3.* (monkeyzoo-main-1) europe-west1
• 10.2.4.* (credential-reuse) europe-west1
• 10.2.5.* (credential-reuse2) europe-west1

[mssalvatore]

The different network segments are to allow for different types of testing. But the different regions are are because, initially, everything was in west3, but we hit the CPU quota, so we started building things in west1. There hasn't been time to migrate everything to west1.

[cakekoa]

Okay. I can put the instance into europe-west1. To fit with our existing network conventions, I should change the IP to 10.2.3.20