| *** tosky has quit IRC | 00:12 | |
| openstackgerrit | wes hayutin proposed openstack/python-tempestconf master: add jobs for python-tempestconf only https://review.openstack.org/581084 | 01:56 |
|---|---|---|
| *** openstackgerrit has quit IRC | 05:10 | |
| *** serverascode has quit IRC | 05:16 | |
| *** davidlenwell has quit IRC | 05:16 | |
| *** beisner has quit IRC | 05:16 | |
| *** chkumar|ruck has quit IRC | 05:28 | |
| *** davidlenwell has joined #refstack | 05:35 | |
| *** serverascode has joined #refstack | 05:35 | |
| *** beisner has joined #refstack | 05:35 | |
| *** davidlenwell has quit IRC | 05:35 | |
| *** davidlenwell has joined #refstack | 05:35 | |
| *** kopecmartin has joined #refstack | 06:34 | |
| *** openstackgerrit has joined #refstack | 08:04 | |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Add etc/ to .gitignore https://review.openstack.org/583487 | 08:04 |
| *** tosky has joined #refstack | 08:06 | |
| *** markvoelker_ has quit IRC | 08:07 | |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Don't expose credentials https://review.openstack.org/583504 | 08:26 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Add User Guide documentation https://review.openstack.org/583516 | 08:58 |
| *** markvoelker has joined #refstack | 10:08 | |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Return False if healthcheck api failed https://review.openstack.org/582996 | 10:29 |
| *** markvoelker has quit IRC | 10:42 | |
| *** markvoelker has joined #refstack | 11:39 | |
| *** markvoelker has quit IRC | 12:05 | |
| *** markvoelker has joined #refstack | 12:05 | |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Fix identity URL parsing https://review.openstack.org/583570 | 13:15 |
| *** rmart04 has joined #refstack | 13:18 | |
| tosky | oh, I've seen the doc change, but it's big to review, so it will take some time | 13:23 |
| arxcruz | tosky: https://review.openstack.org/#/c/582996/ it's passing properly as you can see in http://logs.openstack.org/20/573220/14/check/tripleo-ci-centos-7-scenario004-multinode-oooq-container/29e7a7c/logs/undercloud/home/zuul/tempest.log.txt.gz#_2018-07-18_12_50_20 | 13:26 |
| tosky | arxcruz: it kind of defeats the purpose of discovery, sadly; the idea was to call an entry point | 13:27 |
| arxcruz | tosky: well... if you have a better idea, i'm listening... | 13:28 |
| arxcruz | ¯\_(ツ)_/¯ | 13:28 |
| arxcruz | tosky: however, it still using the entry point | 13:28 |
| arxcruz | it's swift implementation that is wrong | 13:28 |
| arxcruz | because entrypoint is v1/AUTH/%s | 13:28 |
| arxcruz | but for healtcheck you need to call /healthcheck | 13:29 |
| tosky | and then for ceph you need to call /somethingelse/healthcheck | 13:29 |
| arxcruz | tosky: exactly | 13:30 |
| arxcruz | ceph is swift/healthcheck | 13:30 |
| tosky | but then which swift implementation is wrong? | 13:30 |
| arxcruz | well, say that swift implementation is wrong is too strong | 13:30 |
| arxcruz | IMHO it should be v1/AUTH/healthcheck | 13:30 |
| arxcruz | because swift endpoint is v1/AUTH/%s | 13:30 |
| arxcruz | and ceph should follow that | 13:31 |
| tosky | but then the question again is: why do we need to use /healthcheck, if it does not give any special advantage compared to the old solution? | 13:31 |
| tosky | it is still not generic | 13:31 |
| arxcruz | that's a question that chandam must answer, i'm just trying to fix the code at this point, if you want to revert the patch to the previous implementation, i'm also okay | 13:32 |
| arxcruz | if not, please you and kopecmartin +2+w https://review.openstack.org/#/c/582996/ | 13:33 |
| tosky | I'm waiting for the gates | 13:33 |
| arxcruz | so we can move on, with the sprint | 13:33 |
| arxcruz | tosky: just finishes | 13:33 |
| * kopecmartin checking the review | 13:38 | |
| arxcruz | kopecmartin: https://www.youtube.com/watch?v=10DQeSk1LaY | 13:56 |
| openstackgerrit | Merged openstack/python-tempestconf master: Add etc/ to .gitignore https://review.openstack.org/583487 | 13:56 |
| kopecmartin | arxcruz, I've workflowed it already ;) | 13:58 |
| arxcruz | hehehehe | 13:58 |
| kopecmartin | arxcruz, tosky btw, with the doc https://review.openstack.org/#/c/583516/ , take your time to have a closer look, writing all of the things down was harder than i expected :D | 14:00 |
| tosky | I guessed so, reviewing takes time too | 14:02 |
| *** kopecmartin has quit IRC | 14:56 | |
| *** rmart04 has quit IRC | 15:32 | |
| *** rmart04 has joined #refstack | 18:59 | |
| *** rmart04 has quit IRC | 19:00 | |
| openstackgerrit | Merged openstack/python-tempestconf master: Return False if healthcheck api failed https://review.openstack.org/582996 | 19:28 |
| *** trevormc has joined #refstack | 20:10 | |
| trevormc | hey tosky, I'm working through python-tempestconf again and I found a failure in the configure_horizon function. It's similar to the story I reported earlier. For it to work on the cloud I'm using, identity in the hostname part of the url should be replaced with dashboard as well. https://github.com/openstack/python-tempestconf/blob/master/config_tempest/services/horizon.py#L23-L24 | 20:39 |
| tosky | trevormc: that's a bit more complex, horizon is not a service like the other proper openstack services | 20:40 |
| tosky | but at least it should not be too much a problem, as there are really no many tempest tests for horizon and they are in a separate plugin | 20:41 |
| *** trevormc_ has joined #refstack | 20:41 | |
| tosky | you can open a story with the issue, sure, but I suspect that without a good idea on how to get that information, it may be tricky | 20:41 |
| * tosky suspects that trevormc lost part of the chat | 20:42 | |
| trevormc_ | sorry, proxy issue i think. | 20:42 |
| trevormc_ | I heard its complex issue and then I can open a story with a good idea on how to get the info | 20:42 |
| trevormc_ | hmm i don't need horizon, I think we can make a change to make horizon false in this case. I'll think about it some more, thanks | 20:43 |
| *** trevormc has quit IRC | 20:43 | |
| trevormc_ | err has_horizon = False | 20:43 |
| trevormc_ | So currently the program errors out with CertificateError in my case, and thats not handled in the code so I can make a report for that and then it will mark has_horizon = False for me. | 20:45 |
| tosky | oh, ok | 20:45 |
| tosky | as I wrote: horizon is not a service like the other proper openstack services | 20:45 |
| tosky | and then I though: "but at least it should not be too much a problem, as there are really no many tempest tests for horizon and they are in a separate plugin" | 20:45 |
| tosky | but it should not fail at least | 20:45 |
| tosky | that can be fixed | 20:45 |
| trevormc_ | right, ok I'll submit something. | 20:46 |
| *** trevormc_ has quit IRC | 21:39 | |
| *** tosky has quit IRC | 23:20 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!