nanovms / ops

ops - build and run nanos unikernels

Home Page:https://ops.city

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

region in config should be separate between img and the created instances

unusualevent opened this issue · comments

the region for uploading an image in the config for vultr is ewr1, but the closest "region" for creating an instance is "ewr".

the keys between the image upload and the instance creation should be distinct.

currently I have to set the region in the CLI option (for the instance region) to overwrite the config.json (which holds the image region).

regions and zones are a bit conflated in various providers (iirc mainly because of aws volumes) - so we basically force labeling zones everywhere and then infer regions from that (see https://github.com/nanovms/ops/blob/master/provider/aws/aws.go#L41 )

I think keeping with this behavior vultr would want the same functionality if it isn't using it

fair - is it possible to update your documentation to clarify that?

e.g. changing the docs:

  • that the region/zone in the CLI overrides the one in the file
  • that you should set the region/zone of the image in the config.json, and specify the region of the instance in the cli args?

vs changing the code:

  • rename Region/Zone to ImageRegion/ImageZone and InstanceZone/RegionZone keys in the config.json

we definitely can - any prs to https://github.com/nanovms/ops-documentation once approved go live instantly

oh I'm definitely not a wordsmith.

w/ #1473 you can use 'ewr1' for both - this follows the convention we're using for other providers

then the docs should be updated to change it from "ewr" to "ewr1"