| *** tzn has joined #openstack-bareon | 00:26 | |
| *** tzn has quit IRC | 01:01 | |
| *** tzn has joined #openstack-bareon | 02:58 | |
| *** tzn has quit IRC | 03:03 | |
| *** tzn has joined #openstack-bareon | 03:59 | |
| *** tzn has quit IRC | 04:04 | |
| *** tzn has joined #openstack-bareon | 05:00 | |
| *** tzn has quit IRC | 05:04 | |
| *** tzn has joined #openstack-bareon | 06:00 | |
| *** tzn has quit IRC | 06:05 | |
| *** tzn has joined #openstack-bareon | 07:01 | |
| *** tzn has quit IRC | 07:06 | |
| *** tzn has joined #openstack-bareon | 08:02 | |
| *** tzn has quit IRC | 08:07 | |
| *** sylwesterB has joined #openstack-bareon | 08:08 | |
| *** sylwesterB has quit IRC | 08:10 | |
| *** sylwesterB has joined #openstack-bareon | 08:10 | |
| sylwesterB | morning ;) | 08:16 |
|---|---|---|
| evgenyl | sylwesterB: morning :) | 08:38 |
| *** tzn has joined #openstack-bareon | 09:03 | |
| *** tzn has quit IRC | 09:08 | |
| sylwesterB | in ideal world, all direct calls to volume_manager should be removed from nailgun | 09:13 |
| evgenyl | sylwesterB: yes :D | 09:13 |
| sylwesterB | it's to much work for just recording a Demo | 09:13 |
| evgenyl | sylwesterB: what is the plan do you have on problem with get_volume_blha method which is not implemented in bareon extension? | 09:14 |
| sylwesterB | there are IMO 3 ways of resolving the issue | 09:15 |
| sylwesterB | 1. put this specific data into bareon-api, create method get_node_volumes and then volume_manager can be safely removed. | 09:16 |
| sylwesterB | 2. make more direct calls to volume_manager instead of invoking 'node_extension_call' | 09:17 |
| sylwesterB | 3. do not remove volume_manager from extensions for the demo (which is the worst I think) | 09:17 |
| sylwesterB | evgenyl: do you see any other solutions? | 09:17 |
| *** tzn has joined #openstack-bareon | 09:18 | |
| evgenyl | sylwesterB: I think 1st is the best. But it somehow should implement interface, so serializer won't fail. | 09:19 |
| *** tzn has quit IRC | 09:20 | |
| evgenyl | sylwesterB: I mean it should return a list of volumes. | 09:20 |
| *** tzn has joined #openstack-bareon | 09:21 | |
| evgenyl | sylwesterB: But ideally I think we will have to implement this logic in a part, where (in the future) you introduce ability for extensions to change deployment/provisioning data. | 09:22 |
| evgenyl | sylwesterB: so for now make it not to fail | 09:23 |
| sylwesterB | evgenyl: look https://review.openstack.org/#/c/250864/5/nailgun/nailgun/extensions/bareon/extension.py here in :37 I also use volume_manager but as a part of 'Node' model | 09:26 |
| sylwesterB | I can do the same with 'volumes'. Put this data to bareon-api, create new method in extension "get_node_volumes" and it will work for sure | 09:28 |
| sylwesterB | is that ok? | 09:28 |
| evgenyl | Regarding to your link, we will need to implement it in node adaptor of bareon-extension, copy-paste if will be required. | 09:29 |
| evgenyl | Hm, let me think. | 09:29 |
| evgenyl | sylwesterB: It's not going to be good even for the demo. Can you try to make a list out of data from bareon? How hard is it to make simple convertation? | 09:32 |
| sylwesterB | let me check | 09:33 |
| sylwesterB | ok | 09:50 |
| sylwesterB | there are 2 ways I think: | 09:50 |
| *** tzn has quit IRC | 09:50 | |
| sylwesterB | 1. https://review.openstack.org/#/c/250864/5/nailgun/nailgun/extensions/bareon/extension.py here I do "disk.pop('volumes')". Without it 'get_node_volumes' can be done easily and it should work... The only place I can have some doubts is that one: https://github.com/openstack/fuel-web/blob/master/nailgun/nailgun/orchestrator/deployment_serializers.py#L545-L552 because I don't what is that data used for later... | 09:55 |
| sylwesterB | 2. convert the hardcoded partitioning data. For example to count number of glance number i would need to parse somehow the 'device' field "/dev/mapper/image-glance" | 09:57 |
| evgenyl | sylwesterB: they are trying to find volume group with specific name | 10:01 |
| evgenyl | https://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/lib/puppet/parser/functions/get_disks_list_by_role.rb#L16 | 10:01 |
| evgenyl | So lets try 1st. | 10:02 |
| sylwesterB | ok, let's try | 10:03 |
| evgenyl | sylwesterB: basically what you are doing now is not just "a demo", you are finding the problems which we will face to very early, so the entire thing will not look like "a simple task with no problems". | 10:05 |
| sylwesterB | yep, you're right | 10:06 |
| sylwesterB | there should be a task to remove all direct volume_manager calls in one of the specs | 10:07 |
| sylwesterB | what do you think? | 10:07 |
| evgenyl | sylwesterB: yes, correct, basically there is going to be a spec on changing of serialized data, where we will be able to remove glance_blha calculation, similar call will be for ceph. | 10:11 |
| sylwesterB | you're talking about the one with so called 'data pipelines' ? | 10:12 |
| evgenyl | sylwesterB: Yeah, with this thing, extension will be able to do additional searilization, which is specific for volumes. | 10:14 |
| sylwesterB | ok | 10:15 |
| *** tzn has joined #openstack-bareon | 11:19 | |
| sylwesterB | evgenyl: deployment succeed | 11:24 |
| sylwesterB | with the option 1. | 11:24 |
| sylwesterB | with disabled volume_manager extension for cluster | 11:25 |
| evgenyl | sylwesterB: nice | 11:25 |
| sylwesterB | so I will now build custom iso and finally record it ;) | 11:25 |
| *** openstackgerrit has quit IRC | 12:50 | |
| *** openstackgerrit has joined #openstack-bareon | 12:50 | |
| *** sylwesterB has quit IRC | 15:39 | |
| *** tzn has quit IRC | 17:12 | |
| *** tzn has joined #openstack-bareon | 19:07 | |
| *** tzn has quit IRC | 19:13 | |
| *** tzn has joined #openstack-bareon | 19:28 | |
| *** tzn has quit IRC | 20:18 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!