Container Orchestration Engine list should not be hardcoded

[Marcos Lobo]

The Container Orchestration Engine list, in "Create Baymodel" view, is hardcoded as you can see here
https://github.com/openstack/magnum-ui/blob/1.0.0/magnum_ui/static/dashboard/containers/baymodels/create/info/baymodel.info.controller.js#L42

I would propose to mode that list to local_settings file. I this way, you can configure which Container Orchestration Engines you want to expose in Horizon.

[hongbin (hongbin034)]

@Brad, @Shu, What do you think about this proposal?

[Shu Muto (shu-mutou)]

@Hongbin, of cause, we don't want to hardcode COE list, Network driver list and Volume driver list.
I think, UI should get these lists from Magnum API. I had commented last of https://review.openstack.org/#/c/235620/ .
And @Yuanying is planning that Magnum API provide these fixed value using Swagger.
We should wait for Yuanying's plan, IMO.

[Bradley Jones (bradjones)]

I recognize hardcoding these values is an issue, I would suggest that getting the information from the Magnum API would be the best approach here as it doesn't involve any extra configuration steps. Depending on how long we are looking before @Yuanying implements this change we could look at this as an intermediary solution.

[Marcos Lobo (marcos-fermin-lobo)]

@shu-mutou, @bradjones I'm totally agree with you guys, IMO this list should be provided by Magnum API.

As @bradjones said, if @Yuanying's implementation takes a long time, probably we could start whit something intermediary.

I'd propose to mimic the future real Magnum API call, I explain. I'd code in the Magnum-UI all the method calls chain (python) and, in the last method, close to HTTP request call, just return the hardcoded list.

Example pseudo-code:
In https://github.com/openstack/magnum-ui/blob/master/magnum_ui/api/magnum.py
def get_coe_list():
    # Future API call
    # return magnumclient(request).coe.list()

    # Temp patch
    return [{name: "swarm", label: gettext("Docker Swarm")},
                 {name: "kubernetes", label: gettext("Kubernetes")},
                 {name: "mesos", label: gettext("Mesos")}];

Any comments?

[Shu Muto (shu-mutou)]

@Marcos, I basically agree with the mimic.

According to COE selection, Magnum-UI changes pull-down options for Network Driver and Volume Driver.
So not only COE list, let us care Network Driver list and Volume Driver list at the same time.

https://github.com/openstack/magnum-ui/blob/1.0.0/magnum_ui/static/dashboard/containers/baymodels/create/info/baymodel.info.controller.js#L47
https://github.com/openstack/magnum-ui/blob/1.0.0/magnum_ui/static/dashboard/containers/baymodels/create/info/baymodel.info.controller.js#L55

[Marcos Lobo (marcos-fermin-lobo)]

shu-mutou, I agree