Environment's region parameter is set to None in ui

[SFilatov]

Murano core-library's Environment class has parameter "region" and it could be set with cli. Though working in muranodashboard there's no way to set it explicitly and it is set to None. This causes issues in multiregion openstack setups.
I suggest we set region parameter to user's current region to resolve this.

[Victor Ryzhenkin (vryzhenkin)]

We are tested this feature in mitaka devstack. Did your environments configured correctly? You shouldn't be able to set environment parameter 'region' directly. This setting applied automatically if you choosing requested region in Horizon (in the top of dashboard near tenant). So, if you are not able to select region, then your environment have to be reconfigured.

[SFilatov (sergeyfilatov)]

I'm able to select region. Though it's set to None.
Besides, I don't see which part of muranodashboard code is setting region.
In my setup:
/usr/lib/python2.7/dist-packages/muranodashboard/environments/api.py:

def environment_create(request, parameters):
    # name is required param
    body = {'name': parameters['name']}
    if 'defaultNetworks' in parameters:
        body['defaultNetworks'] = parameters['defaultNetworks']
    env = api.muranoclient(request).environments.create(body)
    LOG.debug('Environment::Create {0}'.format(env))
    return env

Which I suppose should have region as one of parameters

[Valerii Kovalchuk (vakovalchuk)]

@SFilatov, are you saying about something like this? https://review.openstack.org/#/c/305747/

[SFilatov (sergeyfilatov)]

Yes, that'd solve this

[Valerii Kovalchuk (vakovalchuk)]

As you can see, this proposal was declined. You can see the discussion of that matter here: http://eavesdrop.openstack.org/meetings/murano/2016/murano.2016-04-19-17.01.log.html#l-31

[Valerii Kovalchuk (vakovalchuk)]

By the way, what issues did you mean this could cause?

[SFilatov (sergeyfilatov)]

I see the point. Though as @KZaitsev pointed out we can allow users to choose region as set default with selected one. Currently in my setup(fuel 9.0) I don't have home_region in murano.conf and was not aware if it.
Issue comes with using murano plugins: I have multidomain setup and there's no convenient way to extract current region inside the plugin, so there I'm getting endpoints for all the domains. I would really benefit from getting region value from environment class, it seems reasonable

[Valerii Kovalchuk (vakovalchuk)]

I can suggest you to add this topic to the next murano meeting: https://wiki.openstack.org/wiki/Meetings/MuranoAgenda
and to discuss it one more time

[Kirill Zaitsev (kzaitsev)]

Should be fixed by https://review.openstack.org/#/c/362316/