PR#816: Add AMI regions ca-central-1 and eu-west-2 - fedora-websites

fedora-websites

#816 Add AMI regions ca-central-1 and eu-west-2

Merged 5 years ago by robyduck. Opened 5 years ago by sinnykumari.

sinnykumari/fedora-websites add-ami-regions into f28

Add AMI regions ca-central-1 and eu-west-2

Sinny Kumari • 5 years ago

44b7c49

build.d/globalvar.py

file modified

+3 -1

		`@@ -322,12 +322,14 @@`
		`region(u'US West (N. California)', u'California', u'us-west-1'),`
		`region(u'EU West (Ireland)', u'Ireland', u'eu-west-1'),`
		`region(u'EU Central (Frankfurt)', u'Frankfurt', u'eu-central-1'),`
		`+ region(u'EU West (London)', u'London', u'eu-west-2'),`
		`region(u'Asia Pacific SE (Singapore)', u'Singapore', u'ap-southeast-1'),`
		`region(u'Asia Pacific NE (Tokyo)', u'Tokyo', u'ap-northeast-1'),`
		`region(u'Asia Pacific SE (Sydney)', u'Sydney', u'ap-southeast-2'),`
		`region(u'South America East (Sāo Paulo)', u'Saopaolo', u'sa-east-1'),`
		`region(u'Asia Pacific (Seoul)', u'Seoul', u'ap-northeast-2'),`
		`- region(u'Asia Pacific (Mumbai)', u'Mumbai', u'ap-south-1')`
		`+ region(u'Asia Pacific (Mumbai)', u'Mumbai', u'ap-south-1'),`
		`+ region(u'Canada (Central)', u'Central', u'ca-central-1')`
		`]`

		`path_stats={`

tools/fedimg_vars_lib.py

file modified

+2 -1

		`@@ -116,7 +116,8 @@`

		`def mocked_fedimg(templates):`
		`regions = ['us-east-1', 'ap-northeast-1', 'sa-east-1', 'ap-southeast-1', 'ap-southeast-2',`
		`- 'us-west-2', 'us-west-1', 'eu-central-1', 'eu-west-1', 'ap-northeast-2', 'ap-south-1']`
		`+ 'us-west-2', 'us-west-1', 'eu-central-1', 'eu-west-1', 'ap-northeast-2',`
		`+ 'ap-south-1', 'ca-central-1', 'eu-west-2']`
		`mockdata = {}`
		`for region in regions:`
		`mockdata[region] = 'ami-mocked'`

sinnykumari commented 5 years ago

Signed-off-by: Sinny Kumari sinny@redhat.com

Latest AMIs including F28 RC 1.1 are also available for ca-central-1 and eu-west-2 regions.

dustymabe commented 5 years ago

right, and the two that are missing at us-east-2 and eu-west-3 right?

dustymabe commented 5 years ago

LGTM - this is just for cloud base, right?

sinnykumari commented 5 years ago

right, and the two that are missing at us-east-2 and eu-west-3 right?

correct

sinnykumari commented 5 years ago

LGTM - this is just for cloud base, right?

Both files are common to cloud base and Atomic. So, changes will reflect at both places.
But this shouldn't affect Atomic Host AMIs regions on website because we only update regions for which AMIs are available. check at https://pagure.io/fedora-websites/blob/master/f/getfedora.org/build/fedimg_vars.py#_100 . Looks fine to me when I tested locally.