#816 Add AMI regions ca-central-1 and eu-west-2
Merged 2 years ago by robyduck. Opened 2 years ago by sinnykumari.
sinnykumari/fedora-websites add-ami-regions  into  f28

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={

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'

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.

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

LGTM - this is just for cloud base, right?

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

correct

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.

Pull-Request has been merged by robyduck

2 years ago