FBR: Fix the broken fedimg hub config key

[Sayan Chowdhury]

Hi,

After the last change for updating fedimg on stg. The config key in
the prod also broke. To fix this issue I would like to apply the
following patch so the it can listen to pungi messages.

---
 roles/fedimg/templates/fedmsg.d/fedimg.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/roles/fedimg/templates/fedmsg.d/fedimg.py
b/roles/fedimg/templates/fedmsg.d/fedimg.py
index 25401cd..f3aa4da 100644
--- a/roles/fedimg/templates/fedmsg.d/fedimg.py
+++ b/roles/fedimg/templates/fedmsg.d/fedimg.py
@@ -29,8 +29,6 @@ config = {
 }
 {% else %}
 config = {
-    'fedimgconsumer.dev.enabled': False,
-    'fedimgconsumer.prod.enabled': True,
-    'fedimgconsumer.stg.enabled': False,
+    'fedimgconsumer': True,
 }
 {% endif %}

[Kevin Fenzi]

On 03/12/2018 05:04 AM, Sayan Chowdhury wrote:
> Hi,
>
> After the last change for updating fedimg on stg. The config key in
> the prod also broke. To fix this issue I would like to apply the
> following patch so the it can listen to pungi messages.
>
> roles/fedimg/templates/fedmsg.d/fedimg.py | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/roles/fedimg/templates/fedmsg.d/fedimg.py
> b/roles/fedimg/templates/fedmsg.d/fedimg.py
> index 25401cd..f3aa4da 100644
> --- a/roles/fedimg/templates/fedmsg.d/fedimg.py
> +++ b/roles/fedimg/templates/fedmsg.d/fedimg.py
> @@ -29,8 +29,6 @@ config = {
>  }
>  {% else %}
>  config = {
> 'fedimgconsumer.dev.enabled': False,
> 'fedimgconsumer.prod.enabled': True,
> 'fedimgconsumer.stg.enabled': False,
> 'fedimgconsumer': True,
>
> }
>  {% endif %}

+1. Would be good to get this working again.

kevin

[Patrick マルタインアンドレアス Uiterwijk]

> Hi,
>
> After the last change for updating fedimg on stg. The config key in
> the prod also broke. To fix this issue I would like to apply the
> following patch so the it can listen to pungi messages.
>
> roles/fedimg/templates/fedmsg.d/fedimg.py | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/roles/fedimg/templates/fedmsg.d/fedimg.py
> b/roles/fedimg/templates/fedmsg.d/fedimg.py
> index 25401cd..f3aa4da 100644
> --- a/roles/fedimg/templates/fedmsg.d/fedimg.py
> +++ b/roles/fedimg/templates/fedmsg.d/fedimg.py
> @@ -29,8 +29,6 @@ config = {
>  }
>  {% else %}
>  config = {
> 'fedimgconsumer.dev.enabled': False,
> 'fedimgconsumer.prod.enabled': True,
> 'fedimgconsumer.stg.enabled': False,
> 'fedimgconsumer': True,
>
> }
>  {% endif %}

I guess +1, though I'm curious why the change was made on Feb 19?