| openstackgerrit | Liron Kuchlani proposed openstack/python-tempestconf master: Add support for volume backend names https://review.openstack.org/594083 | 04:03 |
|---|---|---|
| openstackgerrit | Liron Kuchlani proposed openstack/python-tempestconf master: Add support for volume backend names https://review.openstack.org/594083 | 06:38 |
| *** tosky has joined #refstack | 07:13 | |
| *** tosky has quit IRC | 07:43 | |
| *** tosky has joined #refstack | 07:43 | |
| *** tosky has quit IRC | 08:59 | |
| *** tosky has joined #refstack | 09:00 | |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: [WIP] Remove static class loader https://review.openstack.org/590789 | 09:06 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Remove static class loader https://review.openstack.org/590789 | 11:33 |
| arxcruz | tosky: ^ I hope you be happy now :) | 11:33 |
| tosky | arxcruz: for the future, how do you plan to tackle the services without endpoint? By handling the "if not endpoint_data:" condition with more code? | 12:13 |
| arxcruz | tosky: maybe check if the class is not Service ? | 12:23 |
| arxcruz | if we have a specialized class | 12:23 |
| tosky | arxcruz: looping on services, what happens when a service (as from openstack service list) appears multiple times? | 12:27 |
| tosky | like cinderv2 and cinderv3, but they are handled by the same class | 12:28 |
| tosky | what I would have done is loop around the available Services classes, not around the endpoints | 12:29 |
| arxcruz | tosky: well, this requires a very deep change in how we get the endpoints for the services classes | 12:30 |
| arxcruz | i understand your point, but I don't think it was the scope of this change | 12:30 |
| arxcruz | if you disagree, i can try a different approach now | 12:31 |
| tosky | I don't think it requires a deep change | 12:31 |
| tosky | you can still get the list of available services | 12:31 |
| *** rmart04 has joined #refstack | 12:31 | |
| tosky | loop through the services class, if a service handled by that child of Services is available in the list, get the endpoint etc | 12:32 |
| tosky | arxcruz: which is what I wrote in the first comment of the story | 12:34 |
| tosky | arxcruz: https://storyboard.openstack.org/#!/story/2002787#comment-91047 | 12:34 |
| tosky | so it was totally in the scope | 12:35 |
| arxcruz | ok | 12:35 |
| arxcruz | gimme a few hours | 12:35 |
| tosky | no rush, we have a 2.0.0 out anyway | 12:36 |
| arxcruz | tosky: well, and the rest of services will be handled by Service class? | 12:36 |
| arxcruz | the problem is we have more services handled by Service class than specialized classes | 12:36 |
| tosky | uh, for example? | 12:38 |
| arxcruz | aodh, ceilometer, zaqar, barbican, heat, zaqar-websock, panko | 12:41 |
| arxcruz | gnocchi | 12:41 |
| tosky | I remember a comment about one of them having no endpoint | 12:42 |
| tosky | ceilometer; it has a non-Services file under config_tempest/services | 12:43 |
| tosky | there are references to aodh inside config_tempest/services/services.py, so it probably deserves its own classes | 12:44 |
| tosky | and probably the same for the others | 12:44 |
| tosky | and even now we use get_service_class, which uses service_dict, so those services are not handled anyway | 12:46 |
| arxcruz | tosky: all right, i'll loop for the classes instead of services | 13:12 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Remove static class loader https://review.openstack.org/590789 | 14:30 |
| *** openstackgerrit has quit IRC | 15:31 | |
| hogepodge | tosky: chandankumar: https://etherpad.openstack.org/p/InteropRefstackPTGDenver_2018 | 16:05 |
| chandankumar | arxcruz: tosky ^^ I have added some items there | 16:07 |
| chandankumar | hogepodge: We need to update the tempest hash aslo as tempest-19.0.0 is out | 16:08 |
| *** rmart04 has quit IRC | 17:01 | |
| tosky | sorry, I was double booked :/ | 17:03 |
| tosky | for the interop meeting | 17:03 |
| *** mtreinish has quit IRC | 17:25 | |
| *** markvoelker has quit IRC | 17:39 | |
| *** tosky has quit IRC | 18:47 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!