On Thu, Oct 15, 2015 at 02:31:48PM -0500, Dennis Gilmore wrote:
On Thursday, October 15, 2015 03:07:50 PM Ralph Bean wrote:
> ---
> roles/fedimg/templates/fedimg.cfg | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/roles/fedimg/templates/fedimg.cfg
> b/roles/fedimg/templates/fedimg.cfg index bafe26e..e372e53 100644
> --- a/roles/fedimg/templates/fedimg.cfg
> +++ b/roles/fedimg/templates/fedimg.cfg
> @@ -36,6 +36,7 @@ amis = us-east-1|RHEL|6.5|x86_64|ami-be6a98d6|aki-919dcaf8
> ap-northeast-1|RHEL|6.5|x86_64|ami-e7aee0e6|aki-176bf516
> ap-southeast-1|RHEL|6.5|x86_64|ami-c683df94|aki-503e7402
> ap-southeast-2|RHEL|6.5|x86_64|ami-41ra8f7b|aki-c362fff9
> + eu-central-1|RHEL|6.5|x86_64|ami-76eddb6b|aki-184c7a05
> eu-west-1|RHEL|6.5|x86_64|ami-81f23cf6|aki-52a34525
> sa-east-1|RHEL|6.5|x86_64|ami-b7ec43aa|aki-5553f448
> us-west-1|RHEL|6.5|x86_64|ami-fc393eb9|aki-880531cd
> @@ -44,6 +45,7 @@ amis = us-east-1|RHEL|6.5|x86_64|ami-be6a98d6|aki-919dcaf8
> ap-northeast-1|RHEL|6.5|i386|ami-c7bff1c6|aki-136bf512
> ap-southeast-1|RHEL|6.5|i386|ami-9eb8e4cc|aki-ae3973fc
> ap-southeast-2|RHEL|6.5|i386|ami-87f194bd|aki-cd62fff7
> + eu-central-1|RHEL|6.5|i386|ami-76eddb6b|aki-184c7a05
> eu-west-1|RHEL|6.5|i386|ami-7101cf06|aki-68a3451f
> sa-east-1|RHEL|6.5|i386|ami-e9e847f4|aki-5b53f446
> us-west-1|RHEL|6.5|i386|ami-eacfc8af|aki-8e0531cb
ack +1
So, this failed.
- fedimg had a hardcoded list of regions. eu-central-1 was missing.
- I tried adding the eu-central-1 region, but libcloud didn't have a
definition for it.
- I tried monkey patching libcloud from fedimg to support
eu-central-1, and it worked!
https://github.com/fedora-infra/fedimg/pull/39
- I then discovered an older entry from the libcloud changelog that
showed that the way that fedimg is accessessing the ec2 drivers is
deprecated. That's why there's no eu-central-1 definition there,
because they're trying to phase it out.
- I've got a new patch now that uses this new method. It works as
best as I can tell.
https://github.com/fedora-infra/fedimg/pull/40
With that reviewed, released, and deployed, we should be good to go.