| *** lifeless has quit IRC | 00:38 | |
| *** lifeless has joined #refstack | 00:45 | |
| *** xinliang_ has joined #refstack | 01:28 | |
| *** noama has joined #refstack | 03:06 | |
| *** noama has quit IRC | 03:50 | |
| *** noama has joined #refstack | 04:44 | |
| *** kopecmartin has joined #refstack | 06:42 | |
| *** tosky has joined #refstack | 07:40 | |
| *** noama has quit IRC | 07:53 | |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: DNM - Testing collect logs https://review.openstack.org/576444 | 08:11 |
|---|---|---|
| openstackgerrit | Merged openstack/python-tempestconf master: More compatibility with Python 3 https://review.openstack.org/574773 | 08:18 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: DNM - Testing collect logs https://review.openstack.org/576444 | 10:33 |
| *** xinliang_ has quit IRC | 11:38 | |
| arxcruz | it's amazing, the directory is there, the files are there, and I can't access from one playbook to another :@ | 12:34 |
| openstackgerrit | Merged openstack/python-tempestconf master: Replace tenant by project, part 2 https://review.openstack.org/575233 | 12:44 |
| tosky | arxcruz: are you 100% sure that the two playbooks are executed against the same machine? | 12:50 |
| arxcruz | tosky: yes because i do a ls in the second playbook and i see the files | 12:57 |
| tosky | is the error visible in the last run? | 12:57 |
| tosky | arxcruz: ^^ | 13:08 |
| arxcruz | tosky: yes, i think it's permission, i'm running again | 13:09 |
| tosky | oh, that may be | 13:10 |
| arxcruz | tosky: http://logs.openstack.org/44/576444/16/check/python-tempestconf-tempest-packstack-admin/f42ea51/job-output.txt.gz#_2018-06-27_11_57_17_247285 | 13:10 |
| arxcruz | tosky: still, i add become and become_user tempest, still fails | 13:10 |
| tosky | arxcruz: user tempest? | 13:11 |
| arxcruz | tosky: yes, it's the user that tempest uses to execute tempest, according tempest playbook | 13:11 |
| tosky | uhm, is that the user that we use to run tempest? | 13:11 |
| tosky | oki | 13:11 |
| tosky | arxcruz: who is the owner of the destination directory? I know, the error message is different | 13:13 |
| arxcruz | tosky: it's on /tmp, should not have owner issues | 13:14 |
| tosky | arxcruz: you just created /tmp/logs | 13:14 |
| tosky | it may have owner issues | 13:14 |
| tosky | as you are not copying inside /tmp directly | 13:14 |
| arxcruz | tosky: but the error is canot find /opt/stack/tempest/etc/tempest.conf | 13:15 |
| tosky | arxcruz: right | 13:15 |
| tosky | but still | 13:15 |
| tosky | do you always trust error messages? :D | 13:15 |
| tosky | looking at the ansible source code, my guess seems to be wrong; the error is really thrown when the file can't be accessed | 13:17 |
| tosky | arxcruz: at least try, instead of ls -lah /opt/stack/tempest, an ls -lahR /opt/stack/tempest | 13:17 |
| tosky | recursive | 13:17 |
| tosky | so we can see if also /opt/stack/tempest/etc/tempest.conf has the same permissions | 13:17 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: DNM - Testing collect logs https://review.openstack.org/576444 | 13:18 |
| arxcruz | tosky: fair enough | 13:18 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: DNM - Testing collect logs https://review.openstack.org/576444 | 13:19 |
| arxcruz | tosky: i'm also setting permission to 777 for /tmp/logs/tempest just in case | 13:19 |
| tosky | arxcruz: related: about the removal of default-overrides in https://review.openstack.org/#/c/572418/, you put +2; didn't you mention some values that should be still added, or are we good? | 13:19 |
| arxcruz | tosky: the disable ssl certification is set there https://review.openstack.org/#/c/572418/11/config_tempest/main.py | 13:20 |
| arxcruz | tosky: i'm also setting it on tripleo side, once it get merged, we can remove it from tempestconf | 13:20 |
| tosky | arxcruz: ok, not a regression compared to the current value set in default-overrides | 13:23 |
| tosky | I think I can approve it then | 13:23 |
| arxcruz | tosky: don't worry, once the tripleo patch get merged, i'll remove it from tempestconf | 13:24 |
| tosky | arxcruz: I think we can abandon https://review.openstack.org/576356, right? | 13:24 |
| tosky | kopecmartin: uh, sorry for the conflict in https://review.openstack.org/577802 | 13:24 |
| tosky | and thanks for the stories about documentation | 13:24 |
| arxcruz | tosky: fine by me | 13:24 |
| tosky | arxcruz: so the last relevant commit is https://review.openstack.org/575874, which fixes the py3 support for unit tests | 13:24 |
| arxcruz | tosky: done | 13:25 |
| tosky | thanks | 13:26 |
| arxcruz | tosky: wait, the py35 test fails lol | 13:26 |
| tosky | wut | 13:26 |
| tosky | it was passing | 13:26 |
| tosky | what did it change | 13:26 |
| tosky | octavia | 13:27 |
| tosky | grrr | 13:27 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Fix check_service_status https://review.openstack.org/577802 | 13:27 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More Python 3 porting: filter and map, one import https://review.openstack.org/575874 | 13:29 |
| tosky | fixed | 13:29 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More Python 3 porting: filter and map, one import https://review.openstack.org/575874 | 13:33 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More Python 3 porting: filter and map, one import https://review.openstack.org/575874 | 13:35 |
| kopecmartin | :D | 13:35 |
| tosky | no more relative imports! | 13:37 |
| tosky | ok, now it should work | 13:47 |
| tosky | the new py35 test is passing | 13:47 |
| tosky | I defined a new job because otherwise I can't add it to project-config | 13:48 |
| tosky | zuul would have found the same job name twice (in project-config and inside python-tempestconf) and it would have rejected the change | 13:48 |
| tosky | (thanks fungi for the explanation) | 13:48 |
| tosky | so the plan, after merging 575874, is to add the default voting py35 template to project-config, and then remove the local job | 13:49 |
| fungi | glad to help | 13:49 |
| *** kopecmartin has quit IRC | 15:24 | |
| openstackgerrit | Merged openstack/python-tempestconf master: Remove the content of default-overrides.conf https://review.openstack.org/572418 | 15:42 |
| tosky | yay | 15:42 |
| *** tosky has quit IRC | 16:29 | |
| *** tosky has joined #refstack | 16:29 | |
| *** tosky has quit IRC | 23:35 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!