Friday, 2019-08-09

*** altlogbot_3 has quit IRC02:44
*** altlogbot_0 has joined #refstack02:46
*** kopecmartin|off is now known as kopecmartin07:04
*** tosky has joined #refstack07:23
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Replace os-client-config module by openstacksdk  https://review.opendev.org/67463507:30
kopecmartinhogepodge: Hi, please, when you have a moment, have a look at the chain of reviews for refstack-client starting by this one: https://review.opendev.org/#/c/673222/07:33
openstackgerritMerged openstack/python-tempestconf master: Add Queens, Rocky and Stein job  https://review.opendev.org/67510511:52
openstackgerritMerged openstack/python-tempestconf master: Replace os-client-config module by openstacksdk  https://review.opendev.org/67463513:17
tosky... and done13:48
kopecmartin\o/14:01
toskykopecmartin: I just noticed two references to os_client_config inside config_tempest/tests/base.py; I wonder how that code can work15:20
kopecmartin:o15:21
kopecmartinlet me check15:21
kopecmartin:D the decorator is not used in the code and the other reference just mocks that in the real code, it doesn't have to be really there but if was it would be mocked15:24
kopecmartinah, i should have noticed .. let me send a follow-up patch15:25
kopecmartini was just about to create a review for a new release :/15:25
toskybut how is it working then? :)15:25
kopecmartinthat part of the code is old, ~2 years, seems the mock is useless there, the part of the code which should be mocked is not there anymore15:37
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Remove leftover os_client_config references  https://review.opendev.org/67564915:47
*** kopecmartin is now known as kopecmartin|off16:29
*** irclogbot_2 has quit IRC16:47
*** irclogbot_3 has joined #refstack16:51
*** tosky has quit IRC23:32

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!