*** felipemonteiro__ has joined #openstack-qa | 00:07 | |
oomichi | gmann_: masayukig: can you take a look at https://review.openstack.org/#/c/553705 and https://review.openstack.org/#/c/554661 ? | 00:16 |
---|---|---|
gmann_ | oomichi: checking | 00:20 |
*** salv-orlando has joined #openstack-qa | 00:27 | |
*** mvk has joined #openstack-qa | 00:27 | |
*** salv-orlando has quit IRC | 00:32 | |
*** fnaval has quit IRC | 00:36 | |
*** felipemonteiro__ has quit IRC | 00:40 | |
*** fnaval has joined #openstack-qa | 00:43 | |
*** zhurong has joined #openstack-qa | 00:46 | |
gmann_ | oomichi: done, both lgtm | 00:48 |
oomichi | gmann_: thanks :) | 00:48 |
*** yamamoto has joined #openstack-qa | 00:52 | |
*** yamamoto has quit IRC | 00:57 | |
*** salv-orlando has joined #openstack-qa | 01:27 | |
*** harlowja has quit IRC | 01:31 | |
*** salv-orlando has quit IRC | 01:32 | |
*** oomichi has quit IRC | 01:35 | |
*** yamamoto has joined #openstack-qa | 01:53 | |
*** ramishra has joined #openstack-qa | 01:55 | |
*** yamamoto has quit IRC | 01:59 | |
*** zhurong has quit IRC | 02:02 | |
*** mriedem has quit IRC | 02:07 | |
openstackgerrit | zhufl proposed openstack/tempest master: Make microversioned keypair schema full one https://review.openstack.org/555112 | 02:27 |
*** salv-orlando has joined #openstack-qa | 02:28 | |
*** salv-orlando has quit IRC | 02:34 | |
*** yamamoto has joined #openstack-qa | 02:37 | |
*** edmondsw has joined #openstack-qa | 02:42 | |
*** yamamoto has quit IRC | 02:46 | |
*** edmondsw has quit IRC | 02:49 | |
vivsoni_ | Hi Team, smcginnis: from https://github.com/openstack/cinder-specs/tree/master/specs, whether all specs related tempest test case will be implemented or only few ? | 02:53 |
*** yamamoto has joined #openstack-qa | 02:56 | |
*** zhurong has joined #openstack-qa | 02:57 | |
gmann_ | vivsoni_: did not get the query completely, do you have particular spec example ? basically it depends on proposal if it need tempest tests or not | 02:58 |
vivsoni_ | gmann_: i just wanted to know which are the feature/specs from the link https://github.com/openstack/cinder-specs/tree/master/specs | 03:02 |
vivsoni_ | are considered while creating or including tempest test cases | 03:02 |
vivsoni_ | do we have all the specs related tempest test cases | 03:03 |
vivsoni_ | mean what are the criteria, whihc state this specs tempest should be added and so ? | 03:04 |
gmann_ | vivsoni_: ok, that should be proposed during spec in 'Testing' section | 03:04 |
gmann_ | vivsoni_: like this - https://github.com/openstack/cinder-specs/blob/master/specs/queens/transfer-snapshots-with-volumes.rst | 03:04 |
gmann_ | https://github.com/openstack/cinder-specs/blob/master/specs/queens/transfer-snapshots-with-volumes.rst#testing | 03:04 |
*** bkopilov has quit IRC | 03:06 | |
*** dave-mccowan has quit IRC | 03:11 | |
*** salv-orlando has joined #openstack-qa | 03:30 | |
*** harlowja has joined #openstack-qa | 03:32 | |
*** salv-orlando has quit IRC | 03:35 | |
gmann_ | felipemonteiro_: seems difficult to me to accept. is there any projects/cinder mentioned that they do not or do not want to maintain the policy compatibility? | 03:35 |
gmann_ | felipemonteiro_: because if we skip the test on renamed policy then i can pass any cloud with patrole where RBAC can be changed any time | 03:36 |
gmann_ | felipemonteiro_: because backward compatibility check also one of patrole goal | 03:37 |
*** d0ugal_ has joined #openstack-qa | 03:39 | |
*** d0ugal has quit IRC | 03:39 | |
*** nikeshm has joined #openstack-qa | 03:45 | |
*** harlowja has quit IRC | 03:54 | |
*** felipemonteiro_ has quit IRC | 04:04 | |
nikeshm | what is the correct way to use driverfixes branch in layout.yaml | 04:05 |
*** edmondsw has joined #openstack-qa | 04:05 | |
*** edmondsw has quit IRC | 04:10 | |
vivsoni_ | gmann_: Thanks | 04:19 |
*** Nakul has quit IRC | 04:21 | |
*** bkopilov has joined #openstack-qa | 04:23 | |
*** salv-orlando has joined #openstack-qa | 04:31 | |
*** salv-orlando has quit IRC | 04:36 | |
*** zhurong has quit IRC | 04:45 | |
*** felipemonteiro__ has joined #openstack-qa | 04:45 | |
*** pgadiya has joined #openstack-qa | 04:46 | |
felipemonteiro__ | gmann_: here is example where it fails on pike for neutron/cinder: https://review.openstack.org/#/c/547851/ | 04:47 |
felipemonteiro__ | or here: http://logs.openstack.org/51/547851/3/check/patrole-member-pike/3d370c0/job-output.txt.gz#_2018-02-25_20_38_52_911018 | 04:47 |
*** felipemonteiro__ has quit IRC | 04:57 | |
*** Sana has joined #openstack-qa | 05:03 | |
*** Sana has quit IRC | 05:03 | |
openstackgerrit | zhufl proposed openstack/tempest master: Replacing deprecated list_hosts with list_hypervisors https://review.openstack.org/554760 | 05:09 |
*** sridharg has joined #openstack-qa | 05:14 | |
*** rosmaita has quit IRC | 05:18 | |
*** lkuchlan_ has joined #openstack-qa | 05:19 | |
*** janzian has quit IRC | 05:22 | |
*** salv-orlando has joined #openstack-qa | 05:29 | |
openstackgerrit | Ghanshyam Mann proposed openstack/patrole master: Update docs building https://review.openstack.org/552726 | 05:30 |
*** rcernin has quit IRC | 05:34 | |
*** zhurong has joined #openstack-qa | 05:36 | |
*** rcernin has joined #openstack-qa | 05:46 | |
*** edmondsw has joined #openstack-qa | 05:54 | |
*** edmondsw has quit IRC | 05:59 | |
*** rcernin has quit IRC | 06:06 | |
*** rcernin has joined #openstack-qa | 06:08 | |
*** gcheresh_ has joined #openstack-qa | 06:15 | |
*** e0ne has joined #openstack-qa | 06:15 | |
*** lucas-afk has quit IRC | 06:20 | |
*** lucasagomes has joined #openstack-qa | 06:23 | |
*** agurenko has joined #openstack-qa | 06:27 | |
*** e0ne has quit IRC | 06:33 | |
*** itzikb has joined #openstack-qa | 06:33 | |
*** zhurong has quit IRC | 06:34 | |
*** armaan has joined #openstack-qa | 06:46 | |
*** e0ne has joined #openstack-qa | 06:46 | |
*** e0ne has quit IRC | 06:52 | |
*** nikeshm has quit IRC | 07:07 | |
openstackgerrit | Lenny Verkhovsky proposed openstack-dev/devstack master: Install/upgraded pip only if needed. https://review.openstack.org/555174 | 07:10 |
*** d0ugal_ has quit IRC | 07:14 | |
*** zhurong has joined #openstack-qa | 07:14 | |
*** d0ugal has joined #openstack-qa | 07:14 | |
*** d0ugal has joined #openstack-qa | 07:14 | |
*** alexchadin has joined #openstack-qa | 07:19 | |
*** agurenko has quit IRC | 07:25 | |
*** vaidy has quit IRC | 07:25 | |
*** slaweq has quit IRC | 07:25 | |
*** bdperkin has quit IRC | 07:25 | |
*** rajinir has quit IRC | 07:25 | |
*** rajinir_ has joined #openstack-qa | 07:26 | |
*** rajinir_ has quit IRC | 07:26 | |
*** rajinir_ has joined #openstack-qa | 07:26 | |
*** rajinir_ is now known as rajinir | 07:26 | |
*** slaweq has joined #openstack-qa | 07:26 | |
*** agurenko has joined #openstack-qa | 07:27 | |
*** wznoinsk has quit IRC | 07:28 | |
*** vaidy has joined #openstack-qa | 07:28 | |
alexchadin | hi | 07:30 |
alexchadin | live migration doesn't work in devstack-tempest job | 07:31 |
*** rcernin has quit IRC | 07:31 | |
*** wznoinsk has joined #openstack-qa | 07:31 | |
*** salv-orlando has quit IRC | 07:33 | |
*** markvoelker has quit IRC | 07:35 | |
*** kopecmartin has joined #openstack-qa | 07:39 | |
*** edmondsw has joined #openstack-qa | 07:42 | |
*** pcaruana has joined #openstack-qa | 07:42 | |
*** pcaruana has quit IRC | 07:44 | |
*** pcaruana has joined #openstack-qa | 07:44 | |
*** pcaruana has quit IRC | 07:45 | |
*** pcaruana has joined #openstack-qa | 07:45 | |
*** edmondsw has quit IRC | 07:46 | |
*** pcaruana has quit IRC | 07:47 | |
*** armaan has quit IRC | 07:47 | |
*** pcaruana has joined #openstack-qa | 07:47 | |
*** cloud-liang has joined #openstack-qa | 07:47 | |
*** armaan has joined #openstack-qa | 07:47 | |
*** pcaruana has quit IRC | 07:48 | |
*** pcaruana has joined #openstack-qa | 07:48 | |
*** pcaruana has quit IRC | 07:50 | |
*** pcaruana has joined #openstack-qa | 07:50 | |
*** pcaruana has quit IRC | 07:51 | |
*** pcaruana has joined #openstack-qa | 07:51 | |
*** armaan has quit IRC | 07:52 | |
*** pcaruana has quit IRC | 07:53 | |
*** pcaruana has joined #openstack-qa | 07:53 | |
*** pcaruana has quit IRC | 07:54 | |
*** pcaruana has joined #openstack-qa | 07:55 | |
*** pcaruana has quit IRC | 07:56 | |
*** Roamer` has quit IRC | 07:59 | |
*** Roamer` has joined #openstack-qa | 08:01 | |
*** pcaruana has joined #openstack-qa | 08:06 | |
*** pcaruana has quit IRC | 08:07 | |
*** pcaruana has joined #openstack-qa | 08:08 | |
*** pcaruana has quit IRC | 08:09 | |
*** armaan has joined #openstack-qa | 08:09 | |
*** pcaruana has joined #openstack-qa | 08:09 | |
*** pcaruana has quit IRC | 08:10 | |
*** pcaruana has joined #openstack-qa | 08:11 | |
*** tesseract has joined #openstack-qa | 08:11 | |
*** pcaruana has quit IRC | 08:12 | |
*** pcaruana has joined #openstack-qa | 08:13 | |
*** pcaruana has quit IRC | 08:15 | |
*** pcaruana has joined #openstack-qa | 08:15 | |
*** pcaruana has quit IRC | 08:16 | |
*** pcaruana has joined #openstack-qa | 08:18 | |
*** tosky has joined #openstack-qa | 08:19 | |
*** pcaruana has quit IRC | 08:20 | |
*** pcaruana has joined #openstack-qa | 08:20 | |
*** pcaruana has quit IRC | 08:21 | |
*** pcaruana has joined #openstack-qa | 08:21 | |
*** pcaruana has quit IRC | 08:22 | |
*** ragiman has joined #openstack-qa | 08:23 | |
openstackgerrit | Merged openstack/tempest master: Fix expected response code of show_image_file https://review.openstack.org/553705 | 08:24 |
*** snapiri has joined #openstack-qa | 08:25 | |
*** alexchadin has quit IRC | 08:27 | |
*** pcaruana has joined #openstack-qa | 08:29 | |
*** pcaruana has quit IRC | 08:30 | |
*** salv-orlando has joined #openstack-qa | 08:34 | |
*** markvoelker has joined #openstack-qa | 08:34 | |
*** pcaruana has joined #openstack-qa | 08:36 | |
*** pcaruana has quit IRC | 08:37 | |
*** pcaruana has joined #openstack-qa | 08:38 | |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 08:38 |
*** pcaruana has quit IRC | 08:39 | |
*** pcaruana has joined #openstack-qa | 08:40 | |
*** salv-orlando has quit IRC | 08:40 | |
*** pcaruana has quit IRC | 08:40 | |
*** pcaruana has joined #openstack-qa | 08:41 | |
*** pcaruana has quit IRC | 08:41 | |
*** amoralej|off is now known as amoralej | 08:44 | |
*** jpena|off is now known as jpena | 08:48 | |
*** e0ne has joined #openstack-qa | 08:56 | |
gmann_ | time for office hours | 09:00 |
gmann_ | #startmeeting qa | 09:00 |
openstack | Meeting started Thu Mar 22 09:00:25 2018 UTC and is due to finish in 60 minutes. The chair is gmann_. Information about MeetBot at http://wiki.debian.org/MeetBot. | 09:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 09:00 |
*** openstack changes topic to " (Meeting topic: qa)" | 09:00 | |
openstack | The meeting name has been set to 'qa' | 09:00 |
gmann_ | who all here today? | 09:00 |
*** pcaruana has joined #openstack-qa | 09:02 | |
gmann_ | seems no one, ll wait for couple of min | 09:02 |
gmann_ | andreaf: chandankumar ping | 09:02 |
wznoinsk | hi there | 09:03 |
*** pcaruana has quit IRC | 09:03 | |
gmann_ | wznoinsk: hi | 09:03 |
*** pcaruana has joined #openstack-qa | 09:03 | |
wznoinsk | gmann_, I have one item but not sure whether we have enough ppl to discuss | 09:04 |
*** pcaruana has quit IRC | 09:05 | |
gmann_ | wznoinsk: sure | 09:05 |
gmann_ | let's wait for 1 more min if more people join otherwise we will start | 09:05 |
wznoinsk | gmann_, ack | 09:05 |
*** e0ne has quit IRC | 09:07 | |
gmann_ | let's start | 09:07 |
gmann_ | #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting | 09:07 |
gmann_ | agenda for today ^^ | 09:07 |
gmann_ | i will try to cover minimum as not much people today | 09:08 |
gmann_ | #topic Announcement and Action Item (Optional) | 09:08 |
*** openstack changes topic to "Announcement and Action Item (Optional) (Meeting topic: qa)" | 09:08 | |
gmann_ | i had 1 action item from last meeting | 09:08 |
gmann_ | gmann_ to have a priority tracking review topic link also somewhere. | 09:08 |
gmann_ | i created the etherpad and we can start adding the gerrit link there for review | 09:09 |
gmann_ | #link https://etherpad.openstack.org/p/qa-rocky-priorities-tracking | 09:09 |
gmann_ | I will skip other agenda and directly jump to open | 09:10 |
gmann_ | #topic Open Discussion | 09:10 |
*** openstack changes topic to "Open Discussion (Meeting topic: qa)" | 09:10 | |
gmann_ | wznoinsk: you have something , please go ahead | 09:10 |
wznoinsk | gmann_, right, I found this feature to be very useful in debugging SDN controller environments https://review.openstack.org/#/c/553896/ | 09:11 |
wznoinsk | gmann_, hence I'm proposing it | 09:11 |
*** e0ne has joined #openstack-qa | 09:11 | |
wznoinsk | gmann_, I see a few typos to fix there but I hope the idea is clear | 09:12 |
wznoinsk | gmann_, I coupled it with a script to dump ovs flows etc as well as sosreport, result: https://s2dzmds67jf70fjqjwhpjw-on.drv.tw/failing_tempest_28.html | 09:13 |
wznoinsk | ups, ^ link is not working anymore | 09:13 |
gmann_ | wznoinsk: i got the idea | 09:14 |
gmann_ | wznoinsk: you want on teardown also? | 09:14 |
*** gfidente has joined #openstack-qa | 09:14 | |
*** gfidente has quit IRC | 09:14 | |
*** gfidente has joined #openstack-qa | 09:14 | |
gmann_ | wznoinsk: i got it for failure where you want to dump the state of cloud or resources of cloud etc | 09:15 |
wznoinsk | gmann_, originally it was only on_failure but I realized, when I had to troubleshoot resource leaks, the test were actually passing but leaving a lot of rubbish behind, hence run_on_teardwon was useful | 09:15 |
*** rossella_s has quit IRC | 09:15 | |
*** e0ne has quit IRC | 09:15 | |
*** zhurong has quit IRC | 09:16 | |
wznoinsk | gmann_, I'm struggling to share the html results in public, give me a sec | 09:17 |
gmann_ | wznoinsk: we have cleanup CLI to cleanup the leaked resources which can be run as dry run before deleting test leaked resources | 09:17 |
gmann_ | wznoinsk: sure | 09:17 |
gmann_ | wznoinsk: its office hour so take your time :) | 09:18 |
*** rossella_s has joined #openstack-qa | 09:18 | |
*** gkadam has joined #openstack-qa | 09:19 | |
*** jesusaur has quit IRC | 09:20 | |
gmann_ | wznoinsk: while you get the results link, does cleanup CLI fulfill the teardown case ? | 09:27 |
*** gfidente has quit IRC | 09:28 | |
*** jesusaur has joined #openstack-qa | 09:29 | |
gmann_ | wznoinsk: because i was thinking, after teardown either we have leaked resource or whatever cloud state is so we do have things for both, cleanup CLI can clean leaked resources and cloud state can be checked by openstack CLI | 09:29 |
*** gfidente has joined #openstack-qa | 09:29 | |
gmann_ | wznoinsk: but anyways those kind of discussion we can do on patch also, it does not need to be here | 09:29 |
wznoinsk | gmann_, not in 100% as one of the reasons I have the leftover resources is there were 2 or 3 same routers created and I wouldn't see that straight away if I'd do it at cleanup CLI only | 09:29 |
wznoinsk | gmann_, the results are https://drive.google.com/file/d/1cnsbolA4n0uhBXZH_pvDcEDCSg26rdGI/view?usp=sharing but you have to download and open it from local machine , I'll try to figure out a better place | 09:30 |
*** edmondsw has joined #openstack-qa | 09:30 | |
gmann_ | wznoinsk: you can run cleanup with dry run and check all resources created by test | 09:30 |
gmann_ | wznoinsk: sure thanks, failure case looks good for me. till script is ready only and not destructive :) | 09:31 |
wznoinsk | gmann_, ok, I need to get famliar with the cleanup then but it sounds like yes it would be good for my leftovers case | 09:31 |
gmann_ | wznoinsk: ok, and further discussion we can do on gerrit. idea looks good to me | 09:32 |
wznoinsk | gmann_, yes, I feel like I'm crossing these tempest boundaries by proposing something like that, although proven to be very useful - especially with tests like east_west where you have 4 combinations of them running in one go (there's no way to --pdb them) | 09:32 |
wznoinsk | gmann_, good to hear!... i'l put more use cases in https://blueprints.launchpad.net/tempest/+spec/run-on-teardown-failure then | 09:33 |
gmann_ | i understand that, as long as it is configurable it does not harm and people setting script know what they are doing | 09:33 |
gmann_ | wznoinsk: cool | 09:34 |
wznoinsk | gmann_, good discussion, thanks | 09:34 |
gmann_ | wznoinsk: we do have process of spec if there is BP | 09:34 |
*** alexchadin has joined #openstack-qa | 09:34 | |
*** edmondsw has quit IRC | 09:35 | |
gmann_ | wznoinsk: but i do not think we need in this case until it need more discussion on gerrit patch. | 09:35 |
*** salv-orlando has joined #openstack-qa | 09:35 | |
wznoinsk | gmann_, sure, do I fill all pros/cons in the BP ^ tho?... I have a feeling it would convince ppl a bit faster ;-) | 09:36 |
gmann_ | wznoinsk: ok then you can add spec for that and all details can go there | 09:36 |
gmann_ | wznoinsk: here #link https://github.com/openstack/qa-specs/tree/master/specs/tempest | 09:37 |
gmann_ | and this is the template which is small one i think #link https://github.com/openstack/qa-specs/blob/master/template.rst | 09:38 |
wznoinsk | gmann_, could I leave the 'more info' in a form of LP blueprint and spawn a spec only if there's really more discussion? | 09:38 |
*** salv-orlando has quit IRC | 09:38 | |
*** salv-orlando has joined #openstack-qa | 09:38 | |
tosky | (or better: http://git.openstack.org/cgit/openstack/qa-specs/tree/template.rst ) | 09:38 |
gmann_ | wznoinsk: ok, i am ok with that. if we need more design discussion then we can propose spec | 09:39 |
wznoinsk | gmann_, thanks | 09:39 |
tosky | gmann_: wouldn't it be better to limit LP blueprint? Because they will lost after (because it will happen ^_^) the storyboard migration | 09:39 |
gmann_ | tosky: thanks. my browser typo of git* give me that but yea git. better | 09:39 |
gmann_ | tosky: yea but it is just specless BP till now nothing much | 09:40 |
openstackgerrit | Waldemar Znoinski proposed openstack/tempest master: run a shell command on a test failure (at teardown stage) https://review.openstack.org/553896 | 09:40 |
gmann_ | tosky: we will have lot other history data lost for that migration | 09:40 |
gmann_ | tosky: but yea we mostly need spec if need discussions or debates | 09:41 |
slaweq | andreaf: hi, I have a question about https://review.openstack.org/#/c/552846/13/.zuul.yaml | 09:42 |
gmann_ | anything else for discussion ? | 09:42 |
slaweq | andreaf: I switched from "all-plugin" to "all" as You adviced but now tests fails: http://logs.openstack.org/46/552846/14/check/neutron-tempest-plugin-scenario-linuxbridge/060407d/job-output.txt.gz#_2018-03-22_09_05_39_855262 | 09:43 |
slaweq | did I something wrong there? should I maybe use different regex with "all"? | 09:43 |
tosky | gmann_: I have few questions about zuulv3 and multinode, but I guess I would need andreaf too | 09:44 |
tosky | my multinode job is working but I see few potential issues | 09:44 |
gmann_ | tosky: sure, we can discuss later too once he is up. | 09:44 |
gmann_ | tosky: please put link, i will check if get time tomorrow | 09:45 |
*** gcheresh_ has quit IRC | 09:45 | |
*** itzikb has quit IRC | 09:46 | |
*** bkopilov has quit IRC | 09:46 | |
*** ragiman has quit IRC | 09:46 | |
tosky | gmann_: I explained the issues on Monday, so I have the chat logs! | 09:47 |
tosky | it starts here http://eavesdrop.openstack.org/irclogs/%23openstack-qa/%23openstack-qa.2018-03-19.log.html#t2018-03-19T11:25:26 | 09:47 |
tosky | and then http://eavesdrop.openstack.org/irclogs/%23openstack-qa/%23openstack-qa.2018-03-19.log.html#t2018-03-19T15:18:43 | 09:47 |
gmann_ | tosky: thanks, ll check | 09:47 |
gmann_ | anything else or we close the office hour. | 09:48 |
gmann_ | #endmeeting | 09:49 |
*** openstack changes topic to "Discussion of OpenStack Quality Assurance Projects (tempest, devstack, grenade, and friends) | docs https://wiki.openstack.org/wiki/QA | specs https://review.openstack.org/#/q/project:openstack/qa-specs | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-qa/" | 09:49 | |
openstack | Meeting ended Thu Mar 22 09:49:05 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:49 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/qa/2018/qa.2018-03-22-09.00.html | 09:49 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/qa/2018/qa.2018-03-22-09.00.txt | 09:49 |
openstack | Log: http://eavesdrop.openstack.org/meetings/qa/2018/qa.2018-03-22-09.00.log.html | 09:49 |
*** itzikb has joined #openstack-qa | 09:51 | |
*** gcheresh_ has joined #openstack-qa | 09:52 | |
*** cloud-liang has quit IRC | 09:54 | |
*** itzikb_ has joined #openstack-qa | 09:57 | |
*** gcheresh has joined #openstack-qa | 09:57 | |
*** itzikb has quit IRC | 09:57 | |
*** itzikb_ has quit IRC | 09:57 | |
*** gcheresh_ has quit IRC | 09:57 | |
*** itzikb has joined #openstack-qa | 09:59 | |
*** ragiman has joined #openstack-qa | 10:02 | |
*** lkuchlan_ has quit IRC | 10:03 | |
*** lkuchlan has joined #openstack-qa | 10:03 | |
*** e0ne has joined #openstack-qa | 10:10 | |
andreaf | slaweq hi | 10:11 |
openstackgerrit | afazekas proposed openstack/tempest master: Run all scenario in parallel https://review.openstack.org/549002 | 10:11 |
andreaf | slaweq sorry I wasn't clear on my comments - I do recommend using all instead of all-plugins | 10:11 |
afazekas | gmann_, ack | 10:12 |
andreaf | slaweq however to do so you also need to make sure that the plugin is installed in the tempest venv - which you can do by using the TEMPEST_PLUGINS setting in localrc | 10:12 |
andreaf | currently your tempest plugins is being installed as a devstack plugin and it does not live in the tempest venv | 10:13 |
*** shardy has joined #openstack-qa | 10:15 | |
andreaf | tosky: variables for the controller should go in job.vars, that's correct - not in hosts.vars | 10:15 |
andreaf | tosky I did that on purpose so you can share a base job for single and multi node | 10:15 |
tosky | andreaf: right, so then also devstack_plugins? Wouldn't that enable the plugins on the sub-nodes? | 10:18 |
andreaf | tosky: for devstack plugins there's nothing defined in the base job so you can decide which vars to use | 10:19 |
andreaf | tosky you could still define plugin in job.vars in your base job | 10:20 |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 10:20 |
andreaf | tosky and in the multinode one explicitly set devstack-plugins to an empty dict in group vars | 10:20 |
andreaf | tosky so that the plugin is not installed on subnodes - if that's what you need | 10:20 |
tosky | andreaf: yes, that's the idea - otherwise I would probably find some services installed where they should not | 10:21 |
andreaf | tosky: same as for plugin configuration, if your local_conf settings for the controller do not apply for the subnode, you can override them in group vars | 10:21 |
tosky | andreaf: but then there is the second issue: regardless of where I set devstack_plugins (jobs.vars, overridden for job.group-vars.subnode, or directly under job.host-vars.controller), I have a problem | 10:24 |
tosky | this one: http://eavesdrop.openstack.org/irclogs/%23openstack-qa/%23openstack-qa.2018-03-19.log.html#t2018-03-19T15:18:43 | 10:24 |
andreaf | yeah that's what I meant with "same as for plugin configuration, if your local_conf settings for the controller do not apply for the subnode, you can override them in group vars" | 10:25 |
andreaf | wouldn't that solve your issue? | 10:25 |
tosky | if I override it, wouldn't I lose the original values from job.vars, until I copy them? | 10:26 |
*** alexchadin has quit IRC | 10:26 | |
tosky | if I redefine job.group-vars.subnode.devstack_local_conf.post-config, wouldn't that override (so not merge with) the value from vars.devstack_local_conf defined in the devstack job? | 10:28 |
tosky | I'm not sure if $NEUTRON_CONF.DEFAULT.global_physnet_mtu is useful also on the subnodes; if it's not, than I don't care | 10:28 |
tosky | if it is, well | 10:28 |
*** alexchadin has joined #openstack-qa | 10:31 | |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 10:32 |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 10:34 |
wznoinsk | gmann_, hey, I have the results at http://znoinski.com/tempest-results-failing.28.html ... where should I this review so I can spawn a discussion? (or who should I poke on gerrit review instead maybe?) | 10:34 |
*** dtantsur|afk is now known as dtantsur | 10:36 | |
slaweq | andreaf: ok, thx for the tip. I will do that | 10:41 |
andreaf | tosky: I think you need it on subnodes as well, so yeah you would have to copy that | 10:43 |
andreaf | tosky: but it's not a huge amount of config to copy so hopefully it's a reasonable compromise | 10:43 |
tosky | andreaf: maybe this specific setting (and in the future all settings that are potentially useful on the subnodes too) could be defined for subnodes also in the base devstack job? | 10:45 |
andreaf | tosky I think they are | 10:50 |
andreaf | uhm no, they are not | 10:51 |
andreaf | I need to check what d-g gate does one sec | 10:51 |
tosky | sure :) | 10:52 |
andreaf | yeah it's only set on the controller today, which is probably why I didn't add it to subnodes | 10:55 |
andreaf | tosky so it's safe to override devstack_localconf on subnodes since nothing is set there today | 10:55 |
tosky | andreaf: ah, ok, so not needed; and in the future, if some options from the base devstack will be needed on subnodes, it will be added there | 10:57 |
tosky | okidoki | 10:57 |
tosky | I can remove the WIP from my job then | 10:57 |
tosky | thanks | 10:57 |
tosky | oh, different question: some people (I think from kuryr) were asking about a base devstack-tempest job without services enabled | 10:57 |
tosky | so that they don't need to disable each and every service | 10:58 |
tosky | do you think it would make sense? | 10:58 |
* tosky tries to summon dmellado | 10:58 | |
andreaf | tosky dmellado: yeah there is a devstack base one for that, we could make a tempest abstract one that inherits from it | 11:00 |
*** sshnaidm has quit IRC | 11:00 | |
*** dsariel has joined #openstack-qa | 11:01 | |
andreaf | I think it should be pretty straight forward to do - dmellado would like to propose a patch to tempest for that? | 11:01 |
*** jesusaur has quit IRC | 11:02 | |
tosky | oh, it was related to this patch https://review.openstack.org/#/c/553813/, so networking | 11:04 |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 11:05 |
*** jesusaur has joined #openstack-qa | 11:12 | |
*** gcheresh has quit IRC | 11:16 | |
dmellado | tosky: whats up | 11:22 |
dmellado | xD | 11:22 |
* dmellado reading backlog | 11:22 | |
dmellado | andreaf: yeah, that'd be helpful | 11:22 |
dmellado | as we were broken due to the fact that base:false no longer worked | 11:23 |
dmellado | so, tosky going multinode? | 11:23 |
dmellado | I need to check about the vars on the multinode case too, heh | 11:23 |
*** chichi2 has joined #openstack-qa | 11:24 | |
chichi2 | salve a tutti | 11:24 |
chichi2 | ciao | 11:24 |
*** chichi2 has left #openstack-qa | 11:25 | |
*** gcheresh has joined #openstack-qa | 11:26 | |
dmellado | andreaf: yeah, I'll submit a patch, no worries | 11:26 |
tosky | dmellado: my multinode job is working already: https://review.openstack.org/#/c/547669/ :) | 11:26 |
dmellado | tosky: heh, I'll need to catch up! xD | 11:27 |
gmann_ | wznoinsk: thanks, you can add those info and link on commit msg, and we will review it there | 11:34 |
*** dustins has joined #openstack-qa | 11:36 | |
wznoinsk | gmann_, ack | 11:37 |
*** zhurong has joined #openstack-qa | 11:39 | |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 11:46 |
*** manjeets has quit IRC | 11:49 | |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 11:50 |
*** armaan has quit IRC | 11:51 | |
*** armaan has joined #openstack-qa | 11:52 | |
*** rfolco|off is now known as rfolco|ruck | 12:02 | |
*** dustins_ has joined #openstack-qa | 12:02 | |
frickler | tempest-full-py3 seems completely broken since some hours ago, failing with some utf-8 related error in stackviz processing, see http://zuul.openstack.org/builds.html?job_name=tempest-full-py3 | 12:04 |
*** lkuchlan has quit IRC | 12:04 | |
*** amoralej is now known as amoralej|lunch | 12:05 | |
*** dustins has quit IRC | 12:05 | |
*** ragiman has quit IRC | 12:06 | |
*** rosmaita has joined #openstack-qa | 12:08 | |
*** alexchadin has quit IRC | 12:12 | |
*** edmondsw has joined #openstack-qa | 12:15 | |
*** alexchadin has joined #openstack-qa | 12:15 | |
*** alexchadin has quit IRC | 12:18 | |
*** aojea has joined #openstack-qa | 12:18 | |
*** alexchadin has joined #openstack-qa | 12:18 | |
*** alexchadin has quit IRC | 12:18 | |
*** ragiman has joined #openstack-qa | 12:19 | |
*** alexchadin has joined #openstack-qa | 12:19 | |
*** alexchadin has quit IRC | 12:19 | |
*** alexchadin has joined #openstack-qa | 12:20 | |
*** alexchadin has quit IRC | 12:20 | |
*** alexchadin has joined #openstack-qa | 12:21 | |
*** alexchadin has quit IRC | 12:21 | |
openstackgerrit | Artom Lifshitz proposed openstack/tempest master: Integration tests for tagged attach https://review.openstack.org/391947 | 12:21 |
*** alexchadin has joined #openstack-qa | 12:21 | |
*** alexchadin has quit IRC | 12:22 | |
*** alexchadin has joined #openstack-qa | 12:22 | |
*** alexchadin has quit IRC | 12:22 | |
*** alexchadin has joined #openstack-qa | 12:23 | |
*** lucasagomes is now known as lucas-hungry | 12:23 | |
*** sshnaidm has joined #openstack-qa | 12:25 | |
frickler | infra-root: tempest-full-py3 seems to be failing hard in stackviz collection since about 6h ago, I did some digging but didn't find anything useful yet, help is appreciated | 12:26 |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 12:26 |
*** e0ne has quit IRC | 12:31 | |
*** alexchad_ has joined #openstack-qa | 12:33 | |
*** alexchadin has quit IRC | 12:33 | |
*** markvoelker has quit IRC | 12:34 | |
*** markvoelker has joined #openstack-qa | 12:34 | |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 12:41 |
*** e0ne has joined #openstack-qa | 12:45 | |
*** mriedem has joined #openstack-qa | 12:50 | |
*** yamamoto has quit IRC | 12:54 | |
*** zhurong has quit IRC | 13:00 | |
*** dkranz has joined #openstack-qa | 13:02 | |
*** pgadiya has quit IRC | 13:02 | |
openstackgerrit | Doug Hellmann proposed openstack-dev/hacking master: add lower-constraints job https://review.openstack.org/555296 | 13:06 |
*** shardy_ has joined #openstack-qa | 13:14 | |
*** shardy has quit IRC | 13:16 | |
*** sshnaidm has quit IRC | 13:19 | |
*** sshnaidm has joined #openstack-qa | 13:20 | |
*** dtantsur is now known as dtantsur|brb | 13:21 | |
*** yamamoto has joined #openstack-qa | 13:21 | |
*** lucas-hungry is now known as lucasagomes | 13:24 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack-infra/elastic-recheck master: add query for tempest-full-py3 post failure bug 1758054 https://review.openstack.org/555302 | 13:26 |
openstack | bug 1758054 in OpenStack-Gate "stackviz failing on tempest-full-py3" [Critical,New] https://launchpad.net/bugs/1758054 | 13:26 |
*** fnaval has quit IRC | 13:29 | |
*** awaugama has joined #openstack-qa | 13:30 | |
*** armaan has quit IRC | 13:35 | |
*** aojea has quit IRC | 13:36 | |
*** armaan has joined #openstack-qa | 13:36 | |
*** amoralej|lunch is now known as amoralej | 13:37 | |
*** grafuls has quit IRC | 13:39 | |
fungi | i'm pretty out of my depth, but looks like the stackviz exporter is tripping over a non-utf-8 sequence somewhere in the subunit stream, or maybe the subunit stream isn't formatted the way it expects any longer? | 13:40 |
*** armaan has quit IRC | 13:40 | |
fungi | so 1. stackviz or at least the process-stackviz task should probably be made more resilient so that we don't generate POST_FAILURE results on malformed subunit, and 2. we probably need to figure out what changed in the data a few hours ago | 13:41 |
frickler | fungi: I'm wondering whether something in our image setup may have changed so that stackviz might be running with py3 now, maybe it did run with py2 earlier even on -py3 jobs | 13:42 |
fungi | also a possibility | 13:43 |
frickler | fungi: agree on 1., will propose something | 13:43 |
fungi | perhaps something changed with the way we preinstall stackviz in the images (do we?) | 13:44 |
*** fnaval has joined #openstack-qa | 13:44 | |
frickler | fungi: there seems to be a tar from which it gets installed: http://logs.openstack.org/16/555016/2/check/tempest-full-py3/f45fe1f/ara/result/aaffd9c2-c389-43ab-a6aa-ac333893f832/ | 13:46 |
*** aojea has joined #openstack-qa | 13:46 | |
fungi | the cache-devstack element saves a copy of http://tarballs.openstack.org/package-stackviz-element/stackviz-latest.tar.gz on the images | 13:46 |
fungi | which is the only reference to "stackviz" in the nodepool dib elements in project-config | 13:47 |
*** esberglu has joined #openstack-qa | 13:48 | |
frickler | fungi: indeed the older passing jobs say "New python executable in /tmp/stackviz/bin/python2" where it says "python3" in http://logs.openstack.org/16/555016/2/check/tempest-full-py3/f45fe1f/ara/result/aaffd9c2-c389-43ab-a6aa-ac333893f832/ | 13:49 |
fungi | yeah, so for some reason the pip installed to /tmp/stackviz/bin/pip defaults to python3 now | 13:50 |
frickler | fungi: I think there was a change somewhere that stopped installing py2 by default, let me search | 13:50 |
*** aojea has quit IRC | 13:50 | |
fungi | if so, i suppose this was the desired outcome (find unexpected bugs where we were relying on python 2 in py3k compat jobs) | 13:50 |
*** janzian has joined #openstack-qa | 13:51 | |
*** yamamoto has quit IRC | 13:53 | |
*** grafuls has joined #openstack-qa | 13:54 | |
*** felipemonteiro__ has joined #openstack-qa | 13:55 | |
frickler | fungi: oh, we did build/release new images at the time things broke. so maybe that is still related to the new dib release | 13:57 |
frickler | fungi: I'm thinking to revert at least xenial to the previous images once more | 13:58 |
*** bkopilov has joined #openstack-qa | 13:59 | |
fungi | interesting theory, but why would that cause only tempest-full-python3 to switch to using python3 to install stackviz? | 13:59 |
frickler | fungi: because all other jobs would install py2 during devstack runs | 13:59 |
frickler | fungi: and that task is only used in tempest jobs | 14:00 |
fungi | i'm not strictly opposed to deleting our latest images, but would want to know what about the new images is causing this and not just delete them on a hunch | 14:00 |
frickler | fungi: sure, I got an autohold pending that will hopefully catch the next neutron job. or is there a way to hold an already running job? | 14:01 |
fungi | not easily i'm afraid | 14:01 |
fungi | there was a bit of a jump forward, time-wise, with today's images (which did start getting uploaded ~7 hours ago, looks like) | 14:03 |
fungi | ianw had previously deleted yesterday's images pending the dib 2.12.1 release | 14:03 |
*** shardy_ has quit IRC | 14:03 | |
*** felipemonteiro_ has joined #openstack-qa | 14:03 | |
fungi | so whatever changed could have been anywhere in the last ~54 hours | 14:04 |
frickler | fungi: can we keep a copy of the new image for testing and still have nodepool revert to the old one? | 14:04 |
fungi | judging from the build time of the image we were running on before this | 14:04 |
*** shardy has joined #openstack-qa | 14:05 | |
fungi | frickler: well, we can copy it somewhere before deleting it, but can't really put it back again easily (we'd have to do new image builds, really) | 14:05 |
fungi | so if we do pause image builds and roll back today's, then we need to unpause them again and wait for new images to be built and uploaded everywhere before we can move forward again | 14:06 |
fungi | which is why i'd rather not do that unless we know it's a problem with the image | 14:06 |
fungi | and we'd be rolling back to a >2.5-day-old image in the meantime | 14:07 |
frickler | fungi: o.k., so I'll wait for a node to be held so we can check in more detail | 14:07 |
*** felipemonteiro__ has quit IRC | 14:07 | |
*** yamamoto has joined #openstack-qa | 14:08 | |
fungi | also, if it turns out to be a bug in the job which is merely getting exposed by something we fixed in the images, then it may make more sense to just fix the job (assuming we can work out how fairly quickly once we know the underlying cause) | 14:09 |
*** gkadam has quit IRC | 14:09 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/tempest master: Do not fail on stackviz issues https://review.openstack.org/555314 | 14:11 |
frickler | tempest cores: andreaf: ^^ workaround for current gate issues with tempest-full-py3 | 14:11 |
frickler | fungi: right, maybe this will be enough for now ^^ | 14:12 |
*** yamamoto has quit IRC | 14:13 | |
*** hongbin has joined #openstack-qa | 14:13 | |
fungi | i mean, using python 3 to install and run stackviz in the python 3 jobs seems like a reasonable improvement to me. stadkviz itself has universal builds and so purports to support python 3 | 14:14 |
fungi | s/stadkviz/stackviz/ | 14:15 |
*** kiennt26_ has joined #openstack-qa | 14:15 | |
*** dtantsur|brb is now known as dtantsur | 14:15 | |
corvus | frickler: the autohold will act on a currently running job, as long as it matches -- it checks when the job is complete. | 14:18 |
*** alexchad_ has quit IRC | 14:22 | |
*** agurenko has quit IRC | 14:23 | |
*** yamamoto has joined #openstack-qa | 14:24 | |
frickler | corvus: hmm, can please please take a look then why my current autohold didn't catch this job? it should have been in place early enough http://logs.openstack.org/16/555016/2/check/tempest-full-py3/197b817/ | 14:24 |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 14:24 |
*** yamamoto has quit IRC | 14:28 | |
*** alexchadin has joined #openstack-qa | 14:31 | |
*** yamamoto has joined #openstack-qa | 14:39 | |
andreaf | frickler is there any difference between "failed_when: false" and "ignore_errors: true" ? | 14:40 |
*** yamamoto has quit IRC | 14:43 | |
fungi | corvus: frickler: oh, i didn't realize autohold was working on already-running jobs. noted for future, thanks! | 14:45 |
frickler | andreaf: yes, but I'm not sure what it was. I think "failed_when: false" is more generic | 14:45 |
andreaf | heh good to know | 14:46 |
*** yamamoto has joined #openstack-qa | 14:46 | |
*** yamamoto has quit IRC | 14:46 | |
chandankumar | gmann_: I am in foss ASia, so less active on irc | 14:47 |
*** felipemonteiro__ has joined #openstack-qa | 14:53 | |
*** felipemonteiro_ has quit IRC | 14:57 | |
*** ramishra has quit IRC | 14:57 | |
*** alexchadin has quit IRC | 14:58 | |
*** dustins_ is now known as dustins | 14:59 | |
mriedem | can i get a review on this cells v1 job change? the dependent patch is approved https://review.openstack.org/#/c/549783/ | 15:02 |
mriedem | nova is moving the legacy cells v1 job in-tree and renaming it | 15:02 |
mriedem | so i need to move it around for devstack and tempest before i can drop the legacy job | 15:02 |
*** itzikb has quit IRC | 15:05 | |
frickler | mriedem: if you mention that it's a tempest patch and not devstack that might be helpful ;) also note that tempest-full-py3 is currently failing, needs https://review.openstack.org/555314 as workaround | 15:08 |
*** aojea_ has joined #openstack-qa | 15:09 | |
openstackgerrit | Federico Ressi proposed openstack/tempest master: Take proxy URL from http(s)_proxy or HTTP(S)_PROXY env variables https://review.openstack.org/554944 | 15:14 |
*** aojea_ has quit IRC | 15:14 | |
*** dave-mccowan has joined #openstack-qa | 15:15 | |
*** felipemonteiro__ has quit IRC | 15:21 | |
*** felipemonteiro__ has joined #openstack-qa | 15:21 | |
*** armaan has joined #openstack-qa | 15:23 | |
*** kiennt26_ has quit IRC | 15:27 | |
*** gcheresh has quit IRC | 15:28 | |
mriedem | gmann_: mtreinish: can you check out https://review.openstack.org/555314 ? | 15:29 |
mriedem | everything is fubar right now | 15:29 |
*** pcaruana has joined #openstack-qa | 15:42 | |
*** pcaruana has quit IRC | 15:44 | |
*** yamamoto has joined #openstack-qa | 15:47 | |
*** pcaruana has joined #openstack-qa | 15:48 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: add query for tempest-full-py3 post failure bug 1758054 https://review.openstack.org/555302 | 15:50 |
openstack | bug 1758054 in OpenStack-Gate "stackviz failing on py3 causing tempest-full-py3 POST_FAILURES" [Critical,New] https://launchpad.net/bugs/1758054 | 15:50 |
*** yamamoto has quit IRC | 15:53 | |
openstackgerrit | Matt Riedemann proposed openstack-dev/devstack master: WIP: Don't set database connection in controller in superconductor mode https://review.openstack.org/555346 | 15:55 |
*** grafuls has quit IRC | 15:55 | |
openstackgerrit | Merged openstack-infra/elastic-recheck master: grenade: wait for service logs before matching against patterns https://review.openstack.org/493987 | 16:00 |
openstackgerrit | Merged openstack-infra/elastic-recheck master: Add query for OVB testenv failures https://review.openstack.org/502117 | 16:02 |
*** armaan has quit IRC | 16:03 | |
*** armaan has joined #openstack-qa | 16:04 | |
*** harlowja has joined #openstack-qa | 16:04 | |
*** armaan has quit IRC | 16:04 | |
*** felipemonteiro_ has joined #openstack-qa | 16:05 | |
*** bdperkin has joined #openstack-qa | 16:08 | |
*** bdperkin has quit IRC | 16:08 | |
*** bdperkin has joined #openstack-qa | 16:08 | |
*** felipemonteiro__ has quit IRC | 16:09 | |
*** harlowja has quit IRC | 16:09 | |
*** yamamoto has joined #openstack-qa | 16:17 | |
*** ragiman has quit IRC | 16:17 | |
openstackgerrit | Merged openstack-infra/elastic-recheck master: add query for tempest-full-py3 post failure bug 1758054 https://review.openstack.org/555302 | 16:18 |
openstack | bug 1758054 in OpenStack-Gate "stackviz failing on py3 causing tempest-full-py3 POST_FAILURES" [Critical,New] https://launchpad.net/bugs/1758054 | 16:18 |
*** yamamoto has quit IRC | 16:22 | |
*** openstackrecheck has quit IRC | 16:28 | |
openstackgerrit | Artom Lifshitz proposed openstack/tempest master: Integration tests for tagged attach https://review.openstack.org/391947 | 16:28 |
openstackgerrit | Artom Lifshitz proposed openstack/tempest master: Add wait_for_interface_detach method https://review.openstack.org/555367 | 16:28 |
*** masayukig has quit IRC | 16:29 | |
*** openstackrecheck has joined #openstack-qa | 16:29 | |
*** manjeets has joined #openstack-qa | 16:29 | |
*** yamamoto has joined #openstack-qa | 16:32 | |
*** yamamoto has quit IRC | 16:36 | |
*** sshnaidm is now known as sshnaidm|off | 16:38 | |
*** yamamoto has joined #openstack-qa | 16:47 | |
*** yamamoto has quit IRC | 16:52 | |
*** gcheresh has joined #openstack-qa | 16:57 | |
*** aojea_ has joined #openstack-qa | 16:58 | |
*** aojea_ has quit IRC | 17:02 | |
*** yamamoto has joined #openstack-qa | 17:02 | |
*** yamamoto has quit IRC | 17:06 | |
*** kopecmartin has quit IRC | 17:11 | |
*** yamamoto has joined #openstack-qa | 17:17 | |
*** gcheresh has quit IRC | 17:18 | |
*** yamamoto has quit IRC | 17:22 | |
*** dtantsur is now known as dtantsur|afk | 17:23 | |
*** sridharg has quit IRC | 17:24 | |
*** yamamoto has joined #openstack-qa | 17:32 | |
*** yamamoto has quit IRC | 17:37 | |
*** e0ne has quit IRC | 17:39 | |
*** esberglu_ has joined #openstack-qa | 17:39 | |
*** lucasagomes is now known as lucas-afk | 17:40 | |
*** esberglu has quit IRC | 17:41 | |
*** tosky has quit IRC | 17:44 | |
*** yamamoto has joined #openstack-qa | 17:47 | |
*** tosky has joined #openstack-qa | 17:48 | |
openstackgerrit | Tim Buckley proposed openstack/stackviz master: Read from sys.stdin.buffer if available https://review.openstack.org/555388 | 17:48 |
mordred | mtreinish: stestr prints captured stdout/stderr for tests - is there any mechanism to disable that? | 17:50 |
smcginnis | mtreinish: Our favorite stestr issue? https://review.openstack.org/#/c/555388/1 | 17:50 |
mordred | mtreinish: I ask because the pattern I generally do in base test classes is to have a method which attaches captured logs to the stream on errors - and otherwise discards them | 17:51 |
mordred | this works with stestr great for logs - but if anything decides to print to stderr or stdout, it prints to console even for successful tests | 17:52 |
smcginnis | mordred: I've handled that elsewhere by having a fixture capture that output. | 17:52 |
*** yamamoto has quit IRC | 17:52 | |
smcginnis | mordred: But then occasionally hit what I had just pinged him about. | 17:52 |
smcginnis | mordred: A la https://review.openstack.org/#/c/523530/ | 17:52 |
mordred | smcginnis: yah - that's what we have currently - but I think the issue I'm hitting now is that the stdout fixture is attaching to the test case out of my contorl | 17:52 |
smcginnis | mordred: Ah. I agree it would be nice to have an option in stestr. I think he didn't want to default it because he felt different people would want different things, but a knob for it would be good. | 17:53 |
smcginnis | And might avoid some of the problems I've seen. | 17:53 |
mordred | ah - there's a trick you may be interested | 17:54 |
mordred | in | 17:54 |
smcginnis | Yeah? | 17:54 |
mordred | http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/tests/base.py#n50 through 57 | 17:55 |
mordred | smcginnis: basically, self.addOnException(self.attachLogs) is the fun bit | 17:55 |
mordred | but it only attaches the log output to the test stream if the test fails, otherwise it's completely discarded | 17:55 |
smcginnis | mordred: Oh, nifty! | 17:56 |
mordred | (unless you set the flag for always logging, which can be useful in debugging) | 17:56 |
mordred | I keep meaning to port that over into the oslo.test base class | 17:56 |
smcginnis | mordred: That would be ideal. | 17:56 |
mordred | so - the thing I'm trying to accomplish now is to do the same thing but for any non-logger output | 17:56 |
smcginnis | One less thing to copy all over the place. | 17:56 |
*** felipemonteiro has joined #openstack-qa | 17:56 | |
mordred | like, aiohttp, for intsance, decides to just print some strings to stderr sometimes | 17:57 |
mordred | and while that's neat and all - I don't want to see it :) | 17:57 |
smcginnis | Those kinds of things annoy me. ;) | 17:57 |
mordred | yup | 17:57 |
*** felipemonteiro_ has quit IRC | 17:59 | |
*** jpena is now known as jpena|off | 18:00 | |
*** yamamoto has joined #openstack-qa | 18:02 | |
*** yamamoto has quit IRC | 18:06 | |
*** yamamoto has joined #openstack-qa | 18:06 | |
*** yamamoto has quit IRC | 18:06 | |
*** jose-phillips has joined #openstack-qa | 18:10 | |
*** boris_42_ has joined #openstack-qa | 18:13 | |
*** harlowja has joined #openstack-qa | 18:15 | |
*** harlowja_ has joined #openstack-qa | 18:17 | |
*** oomichi has joined #openstack-qa | 18:18 | |
*** harlowja has quit IRC | 18:19 | |
*** dave-mccowan has quit IRC | 18:20 | |
*** tesseract has quit IRC | 18:32 | |
felipemonteiro | gmann_: thanks for feature flag comment! i have one pesky implementation question though. | 18:40 |
felipemonteiro | gmann_: for feature flag in tempest you add to devstack/lib/tempest which is branched. but patrole devstack plugin is in tree and patrole repo is branchless. | 18:40 |
felipemonteiro | gmann_: so curious if that means we have to put feature flag for policies in devstack/lib/tempest too, but add the patrole-side conf option to patrole's conf. | 18:41 |
*** felipemonteiro has quit IRC | 18:42 | |
*** felipemonteiro has joined #openstack-qa | 18:43 | |
*** aojea has joined #openstack-qa | 18:46 | |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Add functools.wraps decorator to rbac_rule_validation https://review.openstack.org/554719 | 18:49 |
*** aojea has quit IRC | 18:50 | |
openstackgerrit | James E. Blair proposed openstack-dev/devstack master: Zuul: support plugin dependencies https://review.openstack.org/522054 | 18:51 |
openstackgerrit | James E. Blair proposed openstack-dev/devstack master: Automatically set LIBS_FROM_GIT based on required projects https://review.openstack.org/549252 | 18:51 |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Add functools.wraps decorator to rbac_rule_validation https://review.openstack.org/554719 | 18:51 |
corvus | andreaf: updated https://review.openstack.org/549252 again, can you please re-review? | 18:51 |
corvus | andreaf: sorry, meant https://review.openstack.org/522054 (but also the other one if you have a sec) | 18:52 |
*** amoralej is now known as amoralej|off | 18:53 | |
*** aojea has joined #openstack-qa | 19:01 | |
*** yamamoto has joined #openstack-qa | 19:07 | |
*** yamamoto has quit IRC | 19:13 | |
*** lkuchlan has joined #openstack-qa | 19:17 | |
*** aojea has quit IRC | 19:23 | |
*** lkuchlan has quit IRC | 19:29 | |
*** awaugama has quit IRC | 19:36 | |
*** dustins has quit IRC | 19:37 | |
*** patriciadomin has quit IRC | 19:38 | |
*** aojea has joined #openstack-qa | 19:43 | |
*** esberglu_ is now known as esberglu | 19:44 | |
*** aojea has quit IRC | 19:45 | |
*** aojea has joined #openstack-qa | 19:45 | |
*** aojea has quit IRC | 19:46 | |
*** aojea has joined #openstack-qa | 19:49 | |
*** gfidente has quit IRC | 20:03 | |
*** yamamoto has joined #openstack-qa | 20:09 | |
*** yamamoto has quit IRC | 20:14 | |
*** sshwarts has quit IRC | 20:17 | |
*** aojea has quit IRC | 20:17 | |
*** aojea has joined #openstack-qa | 20:18 | |
*** gcheresh has joined #openstack-qa | 20:25 | |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Update docs building https://review.openstack.org/552726 | 20:26 |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Update docs building https://review.openstack.org/552726 | 20:31 |
*** aojea has quit IRC | 20:33 | |
*** syjulian has quit IRC | 20:33 | |
*** jamemcc has quit IRC | 20:33 | |
*** awaugama has joined #openstack-qa | 20:33 | |
*** aojea has joined #openstack-qa | 20:34 | |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Make patrole-member job run for queens and pike branches https://review.openstack.org/547851 | 20:40 |
*** boris_42_ has quit IRC | 20:41 | |
openstackgerrit | Felipe Monteiro proposed openstack-dev/devstack stable/queens: Add project_tags identity feature flag https://review.openstack.org/555450 | 20:42 |
*** edmondsw has quit IRC | 20:56 | |
*** edmondsw has joined #openstack-qa | 20:56 | |
*** aojea has quit IRC | 20:57 | |
*** edmondsw has quit IRC | 21:01 | |
*** aojea has joined #openstack-qa | 21:07 | |
*** yamamoto has joined #openstack-qa | 21:10 | |
*** yamamoto has quit IRC | 21:16 | |
*** salv-orlando has quit IRC | 21:19 | |
*** salv-orlando has joined #openstack-qa | 21:19 | |
*** rfolco|ruck is now known as rfolco|off | 21:20 | |
*** raildo has quit IRC | 21:24 | |
*** salv-orlando has quit IRC | 21:24 | |
*** salv-orlando has joined #openstack-qa | 21:26 | |
andreaf | corvus looking... | 21:31 |
*** artom has quit IRC | 21:40 | |
*** artom has joined #openstack-qa | 21:40 | |
*** gcheresh has quit IRC | 21:43 | |
*** esberglu has quit IRC | 21:44 | |
*** esberglu has joined #openstack-qa | 21:44 | |
*** esberglu has quit IRC | 21:49 | |
*** pcaruana has quit IRC | 21:53 | |
*** yamamoto has joined #openstack-qa | 21:55 | |
*** awaugama has quit IRC | 22:04 | |
*** rossella_s has quit IRC | 22:19 | |
openstackgerrit | Merged openstack/tempest master: Do not fail on stackviz issues https://review.openstack.org/555314 | 22:21 |
*** rcernin has joined #openstack-qa | 22:34 | |
*** fnaval has quit IRC | 22:46 | |
*** edmondsw has joined #openstack-qa | 22:46 | |
*** aojea has quit IRC | 22:49 | |
*** hongbin has quit IRC | 23:01 | |
openstackgerrit | Matt Riedemann proposed openstack/tempest master: Cleanup test_create_ebs_image_and_check_boot https://review.openstack.org/555495 | 23:03 |
openstackgerrit | Matt Riedemann proposed openstack/tempest master: DNM: debug test_create_ebs_image_and_check_boot https://review.openstack.org/555496 | 23:03 |
*** felipemonteiro has quit IRC | 23:03 | |
*** e0ne has joined #openstack-qa | 23:22 | |
mtreinish | mordred: I don't think there's an option about it, it should be pretty easy to add though | 23:34 |
mtreinish | mordred: can you file an issue about it, I probably won't be able to take a look at it until next week though | 23:35 |
mtreinish | mordred: you probably just want an if block around: https://github.com/mtreinish/stestr/blob/master/stestr/subunit_trace.py#L208 and add a flag on the cli (and user config file) about it | 23:36 |
mtreinish | smcginnis: you're thinking the issue we're hitting in stestr is a dual of that? | 23:38 |
mtreinish | it might be, I'd have to trace through the code to see if we're doing anything like that | 23:38 |
mtreinish | but that definitely feels like the kind of thing I could/would screw up in a refactor during the fork | 23:39 |
*** fnaval has joined #openstack-qa | 23:41 | |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Remove already-deprecated strict_policy_check option https://review.openstack.org/551765 | 23:43 |
*** tosky has quit IRC | 23:43 | |
openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Remove already-deprecated strict_policy_check option https://review.openstack.org/551765 | 23:44 |
mtreinish | mordred: fwiw, the stdout printing like that was a somewhat conscious choice I made at some point. My thoughts were if you're printing to stdout or stderr during a test run that's likely something leaking (or a bug) and you'd always want to see that | 23:47 |
mtreinish | but I can see how that would be a bad assumption to make, so I agree with having it configurable | 23:47 |
*** e0ne_ has joined #openstack-qa | 23:54 | |
*** esberglu has joined #openstack-qa | 23:54 | |
*** e0ne has quit IRC | 23:57 | |
*** esberglu has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!