Page MenuHomeSoftware Heritage

terraform/azure: Add a new graph vm
AbandonedPublic

Authored by ardumont on Thu, Aug 1, 6:27 PM.

Details

Reviewers
seirl
ftigeot
Group Reviewers
Reviewers
Summary

This is not complete yet though as the location/vm size/current quota need to
match. And this is not the case yet:

  • Standard_H16m_Promo (16vcpus): is not available in westeurope (north europe, uk, etc...). "west us" seems fine though. [1]
  • only 8 vcpus left in our current sponsorship

Still, i think the gist of it is fine.

[1]
Apparently, trying to create resources located in different locations does not work (up to the network).
When creating the vm, we have intermediary resources creation failures.
Due to apparently, resources which do not see each other.
So if we are to create that node elsewhere then westeurope, we might have more than 1 vm to create but as many resources as we already have in that new location (for the things to
work and communicate).

Test Plan
terraform plan

This will output more than just that new vm (since the vault state is not
shared).

Diff Detail

Repository
rSPRE sysadm-provisioning
Branch
add-graph-vm
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7115
Build 10031: arc lint + arc unit

Event Timeline

ardumont created this revision.Thu, Aug 1, 6:27 PM
ardumont updated this revision to Diff 6076.Thu, Aug 1, 6:29 PM

Fix

I did not see i moved around the vault.tf file and committed that.
This is now fixed

ardumont edited the summary of this revision. (Show Details)Thu, Aug 1, 6:34 PM
ardumont edited the summary of this revision. (Show Details)
ardumont planned changes to this revision.EditedThu, Aug 1, 7:28 PM

In the end, vm created manually through the azure portal.

So if we are to create that node elsewhere then westeurope, we might have more than 1 vm to create but as many resources as we already have in that new location (for the things to work and communicate).

That created a bunch of resources located in the same region (as hypothesized above in the description, location: west us).

Also the vcpus quota got unstuck by @seirl requesting some more vcpus (which got accepted).

ardumont abandoned this revision.Fri, Aug 2, 7:34 AM
ardumont updated this revision to Diff 6077.Fri, Aug 2, 7:49 AM

Update/Rebase and plug to another branch

This is still incomplete but can serve as basis for later, might be?

ardumont abandoned this revision.Fri, Aug 2, 7:50 AM