| *** Shrews_ is now known as Shrews | 00:05 | |
| Shrews | adam_g: hah. i'll adjust the review in the morning | 00:06 |
|---|---|---|
| adam_g | Shrews, cool | 00:06 |
| *** dmorita has joined #openstack-qa | 00:30 | |
| *** dims has joined #openstack-qa | 01:02 | |
| *** yamahata has joined #openstack-qa | 01:03 | |
| *** dims_ has joined #openstack-qa | 01:05 | |
| *** dims has quit IRC | 01:09 | |
| *** morganfainberg is now known as morganfainberg_Z | 01:13 | |
| *** aimon has joined #openstack-qa | 01:18 | |
| *** suneelb has quit IRC | 01:18 | |
| *** dims_ has quit IRC | 01:19 | |
| *** topshare has joined #openstack-qa | 01:29 | |
| *** mitz_ has joined #openstack-qa | 01:31 | |
| *** mitz has quit IRC | 01:33 | |
| *** morganfainberg_Z is now known as morganfainberg | 01:33 | |
| *** topshare has quit IRC | 01:33 | |
| *** Poornima has joined #openstack-qa | 01:35 | |
| *** weshay has quit IRC | 01:39 | |
| *** wuhg has joined #openstack-qa | 01:43 | |
| *** flwang has left #openstack-qa | 01:45 | |
| *** topshare has joined #openstack-qa | 01:50 | |
| *** amotoki has joined #openstack-qa | 01:50 | |
| *** nosnos has joined #openstack-qa | 01:56 | |
| *** pcrews has quit IRC | 01:57 | |
| *** topshare has quit IRC | 01:59 | |
| *** dims has joined #openstack-qa | 02:02 | |
| *** topshare has joined #openstack-qa | 02:03 | |
| *** topshare has quit IRC | 02:03 | |
| *** mriedem has joined #openstack-qa | 02:05 | |
| *** Guest61572 is now known as dansmith | 02:09 | |
| *** dims has quit IRC | 02:13 | |
| *** torandu has joined #openstack-qa | 02:15 | |
| *** torandu has quit IRC | 02:15 | |
| *** torandu has joined #openstack-qa | 02:16 | |
| *** rbak__ has joined #openstack-qa | 02:19 | |
| *** rbak_ has quit IRC | 02:21 | |
| *** mriedem has quit IRC | 02:28 | |
| *** ianw has quit IRC | 02:35 | |
| *** adalbas has joined #openstack-qa | 02:35 | |
| *** mrrrgn has joined #openstack-qa | 02:36 | |
| *** jamielennox|away has quit IRC | 02:37 | |
| *** jamielennox|away has joined #openstack-qa | 02:37 | |
| *** adalbas has quit IRC | 02:38 | |
| *** ianw has joined #openstack-qa | 02:38 | |
| *** ianw has quit IRC | 02:39 | |
| *** ianw has joined #openstack-qa | 02:40 | |
| *** ianw has quit IRC | 02:40 | |
| *** ianw has joined #openstack-qa | 02:41 | |
| *** jamielennox|away has quit IRC | 02:42 | |
| *** jamielennox|away has joined #openstack-qa | 02:42 | |
| *** harlowja is now known as harlowja_away | 02:44 | |
| *** ianw has quit IRC | 02:45 | |
| *** ianw has joined #openstack-qa | 02:46 | |
| *** dims has joined #openstack-qa | 02:54 | |
| * gmann is back (gone 17:00:02) | 02:57 | |
| *** jamielennox|away has quit IRC | 02:59 | |
| *** ianw has quit IRC | 02:59 | |
| *** ianw has joined #openstack-qa | 02:59 | |
| *** jamielennox|away has joined #openstack-qa | 03:00 | |
| *** mrrrgn has quit IRC | 03:00 | |
| *** ianw has quit IRC | 03:04 | |
| *** ianw has joined #openstack-qa | 03:05 | |
| *** dims has quit IRC | 03:16 | |
| *** nosnos has quit IRC | 03:23 | |
| *** nosnos has joined #openstack-qa | 03:24 | |
| *** nosnos has quit IRC | 03:28 | |
| *** harlowja_away has quit IRC | 03:29 | |
| *** catohornet has quit IRC | 03:35 | |
| *** ianw has quit IRC | 03:38 | |
| *** jamielennox|away has quit IRC | 03:38 | |
| *** ianw has joined #openstack-qa | 03:38 | |
| *** jamielennox|away has joined #openstack-qa | 03:39 | |
| *** nosnos has joined #openstack-qa | 03:53 | |
| *** cnesa7 has quit IRC | 03:54 | |
| *** Santosh_ has joined #openstack-qa | 03:56 | |
| Santosh_ | andreaf_: hi | 03:57 |
| *** Puneet has joined #openstack-qa | 03:58 | |
| *** Santosh has quit IRC | 03:59 | |
| Puneet | HI all | 04:00 |
| Puneet | Can anyone please review https://review.openstack.org/#/c/116870/ | 04:01 |
| *** oomichi has joined #openstack-qa | 04:01 | |
| *** chuck_ has quit IRC | 04:49 | |
| *** yfried_ has quit IRC | 04:53 | |
| *** ildikov_ has quit IRC | 04:57 | |
| *** yamahata has quit IRC | 04:58 | |
| *** yamahata has joined #openstack-qa | 04:58 | |
| *** torandu has quit IRC | 05:03 | |
| *** torandu has joined #openstack-qa | 05:03 | |
| *** neeti has joined #openstack-qa | 05:06 | |
| *** bkopilov has joined #openstack-qa | 05:06 | |
| *** rfolco has quit IRC | 05:11 | |
| *** torandu has quit IRC | 05:13 | |
| *** rfolco has joined #openstack-qa | 05:14 | |
| *** Puneet has quit IRC | 05:18 | |
| *** Puneet has joined #openstack-qa | 05:18 | |
| *** SridharG has joined #openstack-qa | 05:19 | |
| Puneet | Hi all | 05:19 |
| *** Santosh_ has quit IRC | 05:23 | |
| *** Santosh has joined #openstack-qa | 05:24 | |
| Puneet | Can anyone please review https://review.openstack.org/#/c/116870/ | 05:25 |
| Puneet | Can anyone please review https://review.openstack.org/#/c/115621/ | 05:27 |
| *** yfried_ has joined #openstack-qa | 05:43 | |
| yfried_ | anyone knows how I can push something to gerrit on behalf of a colleague? | 05:45 |
| yfried_ | Puneet: reviewing now | 05:47 |
| kashyap | yfried_, You might want to check on #openstack-infra | 05:50 |
| *** k4n0 has joined #openstack-qa | 05:59 | |
| *** ildikov_ has joined #openstack-qa | 05:59 | |
| *** ianw has quit IRC | 06:08 | |
| *** ianw has joined #openstack-qa | 06:08 | |
| *** jamielennox|away has quit IRC | 06:10 | |
| *** jamielennox|away has joined #openstack-qa | 06:11 | |
| *** chandankumar has joined #openstack-qa | 06:14 | |
| *** flaper87|afk is now known as flaper87 | 06:15 | |
| *** Longgeek has joined #openstack-qa | 06:20 | |
| *** nmagnezi__ has joined #openstack-qa | 06:27 | |
| yfried_ | kashyap: I did. no answer there... | 06:36 |
| kashyap | yfried_, Sometimes, I think responses are a bit erratic, IRC being IRC :-) | 06:37 |
| *** mpaolino has joined #openstack-qa | 06:40 | |
| *** eezhova has quit IRC | 06:44 | |
| *** Puneet_ has joined #openstack-qa | 06:49 | |
| *** tnurlygayanov has joined #openstack-qa | 06:56 | |
| *** afazekas has joined #openstack-qa | 06:59 | |
| *** nosnos has quit IRC | 07:04 | |
| *** nosnos_ has joined #openstack-qa | 07:04 | |
| *** freyes has quit IRC | 07:09 | |
| *** freyes has joined #openstack-qa | 07:10 | |
| *** gema has quit IRC | 07:16 | |
| *** e0ne has joined #openstack-qa | 07:20 | |
| Puneet_ | Can you please suggest something on this https://review.openstack.org/#/c/116870/ | 07:24 |
| Puneet_ | i was reading router_name from conf file | 07:25 |
| *** jordanP has joined #openstack-qa | 07:26 | |
| *** e0ne has quit IRC | 07:31 | |
| *** e0ne has joined #openstack-qa | 07:36 | |
| *** e0ne has quit IRC | 07:38 | |
| *** jamielennox|away has quit IRC | 07:38 | |
| *** ianw has quit IRC | 07:38 | |
| *** ianw has joined #openstack-qa | 07:38 | |
| *** jamielennox|away has joined #openstack-qa | 07:39 | |
| *** markus_z has joined #openstack-qa | 07:39 | |
| *** qba73 has joined #openstack-qa | 07:41 | |
| *** jlanoux has joined #openstack-qa | 07:42 | |
| *** e0ne has joined #openstack-qa | 07:42 | |
| *** tkammer has joined #openstack-qa | 07:45 | |
| *** fbo has joined #openstack-qa | 07:52 | |
| *** fbo has quit IRC | 07:55 | |
| *** e0ne has quit IRC | 07:55 | |
| *** andreaf_ has quit IRC | 07:56 | |
| *** e0ne has joined #openstack-qa | 07:59 | |
| *** jpich has joined #openstack-qa | 07:59 | |
| yfried_ | Puneet_: as I wrote - Tempest isolated_creds creates routers per test class. the module should also provide a method for fetching that router's name/id. I'm not too familiar with show this works or even IF said network resources are created for CLI tests. you will need to look into that your self or ask some more experienced members than me | 07:59 |
| *** salv-orlando_ has joined #openstack-qa | 08:00 | |
| yfried_ | salv-orlando: ^ you worked on isolated_creds. am I right? | 08:00 |
| *** jlibosva has joined #openstack-qa | 08:02 | |
| yfried_ | Puneet_: that's the module you are looking for: tempest/common/isolated_creds.py | 08:02 |
| *** dmorita_ has joined #openstack-qa | 08:03 | |
| yfried_ | Puneet_: do "git blame" to find out the people who should know best how to work with its utils | 08:03 |
| *** e0ne_ has joined #openstack-qa | 08:05 | |
| *** dhellmann_ has joined #openstack-qa | 08:08 | |
| *** SergeyLukjanov2 has joined #openstack-qa | 08:09 | |
| *** e0ne has quit IRC | 08:10 | |
| *** catohornet1 has quit IRC | 08:10 | |
| *** jgriffith has quit IRC | 08:10 | |
| *** dmorita has quit IRC | 08:10 | |
| *** salv-orlando has quit IRC | 08:10 | |
| *** dhellmann has quit IRC | 08:10 | |
| *** SergeyLukjanov has quit IRC | 08:10 | |
| *** SergeyLukjanov2 is now known as SergeyLukjanov | 08:10 | |
| *** salv-orlando_ is now known as salv-orlando | 08:10 | |
| *** dhellmann_ has quit IRC | 08:12 | |
| *** dhellmann has joined #openstack-qa | 08:12 | |
| *** jgriffith has joined #openstack-qa | 08:12 | |
| *** e0ne_ has quit IRC | 08:14 | |
| *** flaper87 is now known as flaper87|afk | 08:17 | |
| *** aix has joined #openstack-qa | 08:20 | |
| *** Longgeek has quit IRC | 08:20 | |
| *** Longgeek has joined #openstack-qa | 08:21 | |
| *** Longgeek has quit IRC | 08:25 | |
| *** Poornima has quit IRC | 08:42 | |
| *** yfried__ has joined #openstack-qa | 08:43 | |
| *** Longgeek has joined #openstack-qa | 08:43 | |
| *** yfried_ has quit IRC | 08:44 | |
| *** nosnos_ has quit IRC | 08:45 | |
| *** bdperkin has quit IRC | 08:46 | |
| *** nosnos has joined #openstack-qa | 08:46 | |
| *** markus_z has quit IRC | 08:47 | |
| *** bdperkin has joined #openstack-qa | 08:48 | |
| *** bdperkin has joined #openstack-qa | 08:48 | |
| *** markus_z has joined #openstack-qa | 08:48 | |
| *** markus_z has quit IRC | 08:48 | |
| *** nosnos has quit IRC | 08:50 | |
| *** markus_z has joined #openstack-qa | 08:53 | |
| *** e0ne has joined #openstack-qa | 09:04 | |
| *** yamahata has quit IRC | 09:06 | |
| *** oomichi has quit IRC | 09:09 | |
| *** nosnos has joined #openstack-qa | 09:11 | |
| *** luqas has joined #openstack-qa | 09:12 | |
| *** yamahata has joined #openstack-qa | 09:16 | |
| *** luqas has quit IRC | 09:17 | |
| *** luqas has joined #openstack-qa | 09:19 | |
| *** marun has joined #openstack-qa | 09:22 | |
| *** Puneet_ has quit IRC | 09:25 | |
| * gmann is away: | 09:25 | |
| *** Poornima has joined #openstack-qa | 09:27 | |
| *** Poornima has quit IRC | 09:27 | |
| *** aix has quit IRC | 09:29 | |
| *** chandankumar has quit IRC | 09:32 | |
| *** krtaylor has quit IRC | 09:32 | |
| *** jamielenz has joined #openstack-qa | 09:33 | |
| *** bdperkin has quit IRC | 09:33 | |
| *** jamielennox|away has quit IRC | 09:36 | |
| *** ianw has quit IRC | 09:37 | |
| *** ianw has joined #openstack-qa | 09:37 | |
| *** yfried has joined #openstack-qa | 09:37 | |
| *** yfried__ has quit IRC | 09:39 | |
| *** chandankumar has joined #openstack-qa | 09:40 | |
| *** cnesa7 has joined #openstack-qa | 09:42 | |
| *** luqas has quit IRC | 09:43 | |
| *** bdperkin has joined #openstack-qa | 09:43 | |
| *** e0ne has quit IRC | 09:44 | |
| *** e0ne has joined #openstack-qa | 09:44 | |
| *** cnesa8 has joined #openstack-qa | 09:45 | |
| *** cnesa7 has quit IRC | 09:47 | |
| *** luqas has joined #openstack-qa | 09:48 | |
| *** tosky has joined #openstack-qa | 09:48 | |
| *** Longgeek has quit IRC | 09:50 | |
| *** xianghuihui has joined #openstack-qa | 10:01 | |
| *** luqas has quit IRC | 10:03 | |
| *** aix has joined #openstack-qa | 10:05 | |
| yfried | masayukig: are you around? | 10:07 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 10:07 | |
| *** jamielennox|away has joined #openstack-qa | 10:07 | |
| *** ianw has quit IRC | 10:09 | |
| *** ianw has joined #openstack-qa | 10:09 | |
| *** jamielenz has quit IRC | 10:09 | |
| *** Longgeek has joined #openstack-qa | 10:16 | |
| *** marun is now known as marun_afk | 10:25 | |
| *** chandankumar has quit IRC | 10:26 | |
| *** chandankumar has joined #openstack-qa | 10:27 | |
| *** Longgeek has quit IRC | 10:32 | |
| *** zul has joined #openstack-qa | 10:43 | |
| *** marun_afk has quit IRC | 10:46 | |
| *** flaper87|afk is now known as flaper87 | 10:47 | |
| *** mpaolino has quit IRC | 10:51 | |
| *** zul has quit IRC | 10:51 | |
| *** jhenner has joined #openstack-qa | 10:55 | |
| *** krtaylor has joined #openstack-qa | 10:57 | |
| *** Santosh has quit IRC | 10:59 | |
| *** Santosh has joined #openstack-qa | 10:59 | |
| *** Longgeek has joined #openstack-qa | 11:00 | |
| *** yamahata has quit IRC | 11:04 | |
| *** zul has joined #openstack-qa | 11:04 | |
| *** sergeysh has quit IRC | 11:09 | |
| *** dhellmann has quit IRC | 11:10 | |
| *** zul has quit IRC | 11:11 | |
| *** dhellmann has joined #openstack-qa | 11:13 | |
| *** flaper87 is now known as flaper87|afk | 11:14 | |
| *** nosnos has quit IRC | 11:20 | |
| *** nosnos has joined #openstack-qa | 11:20 | |
| *** cdent has joined #openstack-qa | 11:23 | |
| *** flaper87|afk is now known as flaper87 | 11:23 | |
| cdent | Could someone with the proper powers in tempest trigger a reverify on https://review.openstack.org/#/c/116885 please? | 11:24 |
| *** sergeysh has joined #openstack-qa | 11:24 | |
| *** marun_afk has joined #openstack-qa | 11:25 | |
| *** nosnos has quit IRC | 11:25 | |
| *** dims has joined #openstack-qa | 11:36 | |
| *** dims has quit IRC | 11:40 | |
| *** dims has joined #openstack-qa | 11:41 | |
| *** dims has quit IRC | 11:41 | |
| *** dims has joined #openstack-qa | 11:42 | |
| *** flaper87 is now known as flaper87|afk | 11:42 | |
| *** dims_ has joined #openstack-qa | 11:44 | |
| *** flaper87|afk is now known as flaper87 | 11:44 | |
| *** weshay has joined #openstack-qa | 11:45 | |
| flaper87 | sdague: chmouel hey guys, you around? Can I get your thoughts on this one? https://review.openstack.org/#/c/115265/ I'm sorry for all these pings I've been doing lately, I really need to push this glance.store thing forward | 11:45 |
| *** dims has quit IRC | 11:47 | |
| *** dmorita_ has quit IRC | 11:49 | |
| *** dkranz has joined #openstack-qa | 11:52 | |
| *** gmann_ has joined #openstack-qa | 11:52 | |
| yfried | cdent: it's already in queue: http://status.openstack.org/zuul/ | 11:56 |
| sdague | flaper87: is glance.store actually a namespace package? | 11:57 |
| yfried | cdent: and it's failing again: https://jenkins02.openstack.org/job/check-grenade-dsvm-icehouse/697/ | 11:57 |
| sdague | because if so, this is going to be an issue | 11:57 |
| cdent | meh | 11:58 |
| cdent | yfried: How come the review hasn't been updated with a "I'm trying again"? | 11:58 |
| cdent | Or does it not do that now with the new interface? | 11:58 |
| yfried | cdent: seems like jenkins comments (build start, build results) are gone. all you have now is latest build. no idea why. maybe old interface retains jenkins comments. but if you are looking for you patch in the pipeline - check zuul | 12:00 |
| yfried | mtreinish: here? | 12:00 |
| *** marun_afk is now known as marun | 12:00 | |
| cdent | Thanks. | 12:00 |
| *** tosky_ has joined #openstack-qa | 12:00 | |
| yfried | masayukig: you -1ed my patch without reasons. care to explain? or did you do it on behalf of andreaf? | 12:01 |
| yfried | https://review.openstack.org/#/c/111719/ | 12:01 |
| *** tosky has quit IRC | 12:02 | |
| *** dims_ has quit IRC | 12:02 | |
| *** mwagner_lap has quit IRC | 12:02 | |
| *** dkranz has quit IRC | 12:03 | |
| *** jhenner has quit IRC | 12:03 | |
| *** jhenner has joined #openstack-qa | 12:03 | |
| *** mkollaro has joined #openstack-qa | 12:03 | |
| *** mkollaro1 has quit IRC | 12:06 | |
| *** flaper87 is now known as flaper87|afk | 12:14 | |
| *** dkranz has joined #openstack-qa | 12:16 | |
| *** flaper87|afk is now known as flaper87 | 12:18 | |
| *** mpaolino has joined #openstack-qa | 12:18 | |
| *** aysyd has joined #openstack-qa | 12:20 | |
| *** chandankumar has quit IRC | 12:21 | |
| *** gmann_ has quit IRC | 12:25 | |
| *** cdent has quit IRC | 12:28 | |
| *** yamahata has joined #openstack-qa | 12:44 | |
| *** adalbas has joined #openstack-qa | 12:46 | |
| *** sballe has joined #openstack-qa | 12:47 | |
| *** mkollaro has quit IRC | 12:55 | |
| *** mkollaro has joined #openstack-qa | 12:55 | |
| *** topshare has joined #openstack-qa | 12:57 | |
| *** tosky_ has quit IRC | 13:00 | |
| *** mkollaro1 has joined #openstack-qa | 13:00 | |
| *** tosky_ has joined #openstack-qa | 13:00 | |
| mpaolino | dtroyer, hi | 13:01 |
| *** mkollaro has quit IRC | 13:02 | |
| *** marun has quit IRC | 13:03 | |
| *** marun has joined #openstack-qa | 13:03 | |
| mtreinish | yfried: yes | 13:07 |
| dtroyer | mpaolino: hi | 13:10 |
| *** gordc has joined #openstack-qa | 13:11 | |
| *** cdent has joined #openstack-qa | 13:11 | |
| gordc | sdague: when you have the time, can we get your opinion on: https://review.openstack.org/#/c/102354/ ... | 13:11 |
| gordc | i think comments on patchset 17 are a good summary of discussion | 13:12 |
| mpaolino | dtroyer, can you have a look at https://review.openstack.org/#/c/108714/4 ? | 13:14 |
| sdague | gordc: it's still not passing right? | 13:14 |
| mpaolino | in particular, I received a comment related to the folder in which libvirt is downloaded | 13:15 |
| *** mwagner_lap has joined #openstack-qa | 13:15 | |
| mpaolino | (build_libvirt.sh, line 32) | 13:15 |
| gordc | sdague: cdent will split the patch in two... just wanted your opinion on where the 2nd patch should land. | 13:15 |
| mpaolino | I would be interested to know what is the best solution from the qa point of view | 13:16 |
| cdent | I'm just getting started on that change now gordc | 13:16 |
| gordc | sdague: from what i've read, jogo mentioned it should be somewhere... just not sure if it should be in javelin | 13:16 |
| gordc | cdent: awesome! | 13:16 |
| sdague | gordc: so honestly.... I don't know at this point. I think our goal was to get basic checking in here. And honestly, that's probably something we want to talk about more in a summit context I think for more complicated upgrade testing. | 13:17 |
| sdague | because there are implications for really deep inspection that I think we need to think about, and realistically from an overall OpenStack perspective things like compute and networks are probably more in need of the deep inspection at this point | 13:18 |
| sdague | kind of like the heat functional test conversation in the project meeting, I'm happy the team wants to tackle those things, but it's a weird starting point to figure out a rhythm because it's so far up the layers in openstackness. | 13:19 |
| gordc | sdague: i see... so i guess for the deeper inspection piece we have, we should hold on to that for now? | 13:20 |
| sdague | gordc: yeh, that's probably best | 13:20 |
| *** mpaolino has quit IRC | 13:20 | |
| sdague | the rest of it will be a real step forward | 13:20 |
| gordc | sdague: cool cool. we'll add some basic stuff for now... i'll let eglynn know when he gets back. | 13:21 |
| sdague | and lets give it a chance to breath to figure out what deep inspection things make sense from openstack as a whole perspective | 13:21 |
| cdent | Just for reference, I went down the road of deeper inspection explicitly to avoid introducing yet more ambiguity in the test infrastructure. | 13:21 |
| cdent | But I think there's probably a way to make the "part 1" useful without adding too much ambiguity. | 13:21 |
| sdague | cdent: sure, that's fair | 13:21 |
| *** e0ne is now known as e0ne_ | 13:21 | |
| cdent | I think with just part1 we'll be able make fewer conclusions about the health of the upgrade. | 13:22 |
| cdent | That's okay, and certainly better than what we've got now. | 13:22 |
| gordc | cdent: do you know what the original scope of bp was? should we open a second one to contain your deeper expansion work?> | 13:22 |
| cdent | The scope of the bp changed a lot as it was being written, in large part because there wasn't clarity on how javelin was going to behave | 13:22 |
| cdent | For example when it was written the create side of javelin only did create without any check | 13:23 |
| cdent | Now create is create + check and check is just check. | 13:23 |
| cdent | Much of the fiddling that people have done to get stuff into javelin2 have impacted it's evolution | 13:23 |
| cdent | There were a lot of questions during the spec writing about "what is it that we are really testing" | 13:24 |
| cdent | and it came out to do things: | 13:24 |
| cdent | do metrics survive the upgrade | 13:24 |
| cdent | does metered resoures survive the upgrade in a sane way | 13:24 |
| cdent | s/does/do/ | 13:24 |
| *** mriedem has joined #openstack-qa | 13:25 | |
| *** balles has quit IRC | 13:25 | |
| gordc | cdent: gotcha... yeah it's something we should look into more... unfortunately javelin is one of those things where there's very few eyes on it right now. | 13:26 |
| *** jecarey has joined #openstack-qa | 13:26 | |
| cdent | I'm sure it will all come out fine in the end. | 13:26 |
| yfried | mtreinish: rdekeil (my colleague) will be back tomorrow to push the tcp-ping patch | 13:26 |
| mtreinish | yfried: ok cool, thanks | 13:27 |
| yfried | mtreinish: but I had a discussion arround the office about your patch | 13:27 |
| yfried | mtreinish: we think it's better if you use the tcp-ping instead of regular ping in your patch to verify connectivity | 13:27 |
| yfried | mtreinish: you will be checking tcp:22 connectivity (which is exactly what secgroup is ecpecting) without opening extra rules on secgroups | 13:28 |
| mtreinish | yfried: ok, that should be simple enough to change | 13:29 |
| yfried | mtreinish: and the secgroup scenario won't be "violated" by messing about with its secgroup while its trying to test them | 13:29 |
| mtreinish | sdague: ^^^ ping before ssh was your idea, are you ok with that change? | 13:30 |
| yfried | mtreinish: also, cirros images are listening to tcp:22 only (no udp or other ports) so I wonder if we could somehow change that without sshing to the VM | 13:30 |
| yfried | mtreinish: ^ assuming we would like to check secgroup isolating ports/protocols (a test we plan to push) | 13:30 |
| *** salv-orlando has quit IRC | 13:31 | |
| *** rbak__ has quit IRC | 13:31 | |
| sdague | mtreinish: sure, I'm fine with that | 13:32 |
| mtreinish | yfried: probably not, short of changing the image we're using to start another service | 13:32 |
| yfried | mtreinish: whose incharge of cirros? could we ask for a change in future versions? | 13:33 |
| yfried | who's | 13:33 |
| mtreinish | sdague: ok, cool | 13:33 |
| *** mpaolino has joined #openstack-qa | 13:34 | |
| mtreinish | yfried: I really don't think we want to do that. It was just a hypothetical, because we could also just build the custom image as part of devstack. | 13:34 |
| mtreinish | yfried: but fyi, cirros dev is tracked here: https://launchpad.net/cirros | 13:34 |
| *** mpaolino_ has joined #openstack-qa | 13:35 | |
| yfried | mtreinish: ok. I was expecting this, so we'll just check ssh first... | 13:35 |
| yfried | mtreinish: and after ssh we can edit the image on runtime | 13:35 |
| yfried | mtreinish: can I ask for your opinion as tempest core? | 13:36 |
| yfried | mtreinish: about the scenario client migration | 13:36 |
| mtreinish | what about it? | 13:36 |
| *** malini has joined #openstack-qa | 13:37 | |
| *** mpaolino has quit IRC | 13:38 | |
| yfried | mtreinish: moving to tempest client we are loosing the Object Oriented design that most native clients give. that's no big deal with api tests which are usually small and short op | 13:39 |
| mtreinish | yfried: so the oo model from the official clients is actually part of the problem. They abstract away too much, which makes debugging failures far more difficult | 13:40 |
| yfried | mtreinish: but in scenarios it calls for more complicated/long/ugly code and sometimes we loose functionality (such as using a server object as dict key) | 13:40 |
| mtreinish | it does mean that the scenario tests will become larger and slightly more complex, but the tradeoff is worth IMO | 13:40 |
| yfried | mtreinish: could you give an example? | 13:41 |
| mtreinish | yfried: look at any of the gate failures in the scenario tests, trying to figure out exactly what failed is more difficult | 13:41 |
| *** mpaolino_ is now known as mpaolino | 13:41 | |
| mtreinish | yfried: this effort comes directly from debugging when things fail | 13:42 |
| yfried | mtreinish: I know the native clients are problematic, and I'm not trying to reverse that decision | 13:42 |
| *** e0ne_ has quit IRC | 13:43 | |
| yfried | mtreinish: what I'm asking is, assuming I'm doing all the work, could I extended the OO wrap created by marun for neutron dicts (net_common.DeletableResources) to all/most other resources | 13:44 |
| yfried | mtreinish: I have this as WIP and I would like to push this. it would still use the native clients, but would "tie" a client instance to a resource and allow better resource handling. we could also use this for a unified resource api/interface to use for cleanups/waits/searches | 13:45 |
| mtreinish | yfried: so there is no need for that if we're not using the client python apis. That was only added because the neturon client behaved differently to the other clients | 13:46 |
| flaper87 | sdague: Do you have a min to talk about https://review.openstack.org/#/c/115265/ ? | 13:46 |
| yfried | mtreinish: but I'm saying that there IS need/benefit in doing it | 13:46 |
| yfried | it's much easier/cleaner to do server.reboot() or subnet.add_router() | 13:47 |
| yfried | mtreinish: and you are still using tempest clients. | 13:47 |
| mtreinish | yfried: yes, but one of the big advantages of the tempest clients for testing is that it is very raw. While an resource model is easier to read it also abstracts away information | 13:48 |
| mtreinish | which for testing we don't want to necessarily ignore | 13:49 |
| *** chandankumar has joined #openstack-qa | 13:50 | |
| *** e0ne has joined #openstack-qa | 13:50 | |
| yfried | mtreinish: I'm sorry. I fail to understand what information would be abstracted | 13:51 |
| yfried | mtreinish: when you are using, for example server.reboot() | 13:52 |
| *** pcrews has joined #openstack-qa | 13:52 | |
| yfried | mtreinish: instead of self.servers_client.reboot_server(server['id]) | 13:52 |
| dtroyer | mpaolino: I left some comments, hopefully clarifying what I think was meant. | 13:52 |
| *** amotoki has quit IRC | 13:52 | |
| yfried | mtreinish: the same client would be use and you will get the same debug info you wanted | 13:53 |
| mtreinish | yfried: it's about being explicit in the test code. Functionally there isn't necessarily a difference, but look at the 2 lines, it says which client your using, where the id for the request is coming from, etc. | 13:54 |
| mtreinish | now if this fails and someone looks at the traceback they see that line they know exactly how what to trace | 13:55 |
| mpaolino | dtroyer, perfect, thanks | 13:55 |
| mtreinish | with server.reboot() it's not as straightforward | 13:55 |
| yfried | mtreinish: check this out please: https://review.openstack.org/#/c/102700/ | 13:56 |
| *** Dafna is now known as Dafna_away | 13:57 | |
| yfried | mtreinish: other than the waiter func, this was a 20 lines patch. if I drop the OO wrapper for floating ips, it becomes a very big and complicated code | 13:57 |
| yfried | mtreinish: I really think the added functionality would make it easier to right and read the code, as we are headed towards more complex scenarios in the future and we would start seeing a lot of duplicate code unless we manage scenario library better | 14:01 |
| mtreinish | yfried: I actually don't see that, we have plenty of waiters in code that don't need an object oriented abstraction to work. | 14:02 |
| yfried | mtreinish: "we have plenty of waiters in code" | 14:03 |
| mtreinish | there are lots of other methods that wait for status | 14:03 |
| yfried | mtreinish: OO abstraction would make it so you only need 1 waiter | 14:03 |
| *** BobBall has joined #openstack-qa | 14:04 | |
| yfried | mtreinish: would you prefer this patch https://review.openstack.org/#/c/111719/10 to delete OOwrapper and use dicts on ALL resources? | 14:05 |
| *** topshare has quit IRC | 14:05 | |
| mtreinish | yfried: no, it would just look that way in the test code. That's actually the crux of our difference in opinion | 14:06 |
| *** rbak has joined #openstack-qa | 14:06 | |
| marun | mtreinish: oo is still awesome :) | 14:06 |
| marun | mtreinish: having resources be bags of dicts really sucks | 14:06 |
| mtreinish | marun: but it's that way by design, to couple it closer the raw http. It's definitely a pain to deal with | 14:07 |
| BobBall | Hey - I'm seeing a large number of one-off failures with identical symptoms in the XenServer CI. They all seem to occur when deleting tenants at the end of a test and get "Lock Timeout occurred for key, os-revoke-events"... Has anyone seen this before, or can provide suggestions on where to start debugging? | 14:07 |
| mtreinish | but the explicitness is useful | 14:07 |
| marun | mtreinish: explicitness *can* be useful | 14:07 |
| BobBall | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/78/95278/9/23521/results.html is an example failure | 14:07 |
| marun | mtreinish: but it has a cost | 14:07 |
| marun | mtreinish: if you think it's worth bearing, well, I guess we can differ on that point | 14:07 |
| *** topshare has joined #openstack-qa | 14:07 | |
| mtreinish | marun: yeah I think a difference of opinion here is fine. | 14:08 |
| marun | mtreinish: so long as you don't want me writing code for tempest, that is true :) | 14:09 |
| *** ildikov_ has quit IRC | 14:09 | |
| yfried | mtreinish: I fail to see the terrible cost of server.reboot(). I really do. I would understand if you demanded proper unitttests for the OOwrappers, but just to save 2 lines of stack trace while adding a lot more lines of code duplication seems wrong to me | 14:09 |
| yfried | mtreinish: so to sum it up, you guys would -2 a patch/series of OOwrappers for resources. even if I provide proper unittest for it? | 14:11 |
| mtreinish | yfried: I'd probably just be -1 | 14:12 |
| cdent | sdague, gordc new version of https://review.openstack.org/#/c/102354 is up. Much smaller change. Could probably be even smaller but the current state gives a good level of confirmation that the data is mostly sane. | 14:12 |
| mtreinish | BobBall: that error msg is coming from keystone, the message there is the resp body of the tenant delete request | 14:12 |
| yfried | mtreinish: AFAIK if you feel like this is a bad idea, than -2 is in order. | 14:13 |
| mtreinish | yfried: I can't speak for everyone, but that would probably be what I'd vote | 14:13 |
| *** dpaterson has joined #openstack-qa | 14:13 | |
| yfried | mtreinish: thank you for your time. I might send this question to ML to get more opinions. | 14:15 |
| BobBall | mtreinish: I did check but there are no errors / traces in keystone at all... nor references to the lock timeout | 14:16 |
| mtreinish | BobBall: yeah I was looking at the keystone logs there, nothing stuck out at a quick glance | 14:16 |
| *** markmcclain has joined #openstack-qa | 14:17 | |
| BobBall | it seems to be failing around 30% of my tests at the moment - all of the individual test failures that I've looked at have the same failure | 14:17 |
| mtreinish | morganfainberg: ^^^ have you seen a failure like that before, where keystone returns a 500(or 501) because of a timeout waiting for the os-revoke-events lock | 14:19 |
| mtreinish | BobBall: is there a way to get timestamps on the run_tests.log file? | 14:20 |
| mtreinish | I'm trying to see what keystone is logging around the time the failure is hit | 14:20 |
| *** zz_jgrimm is now known as jgrimm | 14:22 | |
| *** yfried has quit IRC | 14:23 | |
| *** chandankumar has quit IRC | 14:24 | |
| *** Dafna_away is now known as Dafna | 14:27 | |
| *** dustins has joined #openstack-qa | 14:35 | |
| *** chandankumar has joined #openstack-qa | 14:41 | |
| cdent | sdague, gordc, jogo: I preserved the undesirable javelin+telemetry functionality in a dependent patch: https://review.openstack.org/#/c/117259/ | 14:42 |
| *** wuhg has quit IRC | 14:42 | |
| *** jecarey has quit IRC | 14:43 | |
| *** afazekas has quit IRC | 14:43 | |
| *** malini|afk has joined #openstack-qa | 14:49 | |
| *** HenryG has joined #openstack-qa | 14:51 | |
| *** mlavalle has joined #openstack-qa | 14:51 | |
| *** SridharG has left #openstack-qa | 14:53 | |
| *** hemna_ is now known as hemna | 14:53 | |
| *** neeti has quit IRC | 14:53 | |
| *** openstackgerrit has joined #openstack-qa | 14:54 | |
| mtreinish | dkranz: https://review.openstack.org/117049 should fix the coverage post jobs and would explain why it didn't work locally for some people | 14:54 |
| *** neeti has joined #openstack-qa | 14:54 | |
| dkranz | mtreinish: ok, thanks. I'll look | 14:55 |
| *** xianghuihui has quit IRC | 14:55 | |
| *** flaper87 is now known as flaper87|afk | 14:56 | |
| *** flaper87|afk is now known as flaper87 | 14:56 | |
| *** Longgeek has quit IRC | 14:56 | |
| BobBall | mtreinish: good call; I've added timestamps now but clearly that'll take a while to come through on the test results | 14:58 |
| mtreinish | BobBall: cool, thanks. | 14:58 |
| mtreinish | BobBall: I do have to say looking through those logs it makes me realize how dependent I've become on os-loganalyze | 14:59 |
| *** neeti has quit IRC | 15:00 | |
| BobBall | I know that feeling! | 15:00 |
| *** neeti has joined #openstack-qa | 15:01 | |
| BobBall | there are all sorts of niceities that the -infra jobs have that 3rd party jobs don't :/ | 15:01 |
| *** mikal has quit IRC | 15:05 | |
| *** topshare has quit IRC | 15:05 | |
| *** mikal has joined #openstack-qa | 15:06 | |
| *** topshare has joined #openstack-qa | 15:07 | |
| *** topshare has quit IRC | 15:07 | |
| *** tkammer has quit IRC | 15:08 | |
| *** topshare has joined #openstack-qa | 15:08 | |
| BobBall | mtreinish: [Wed Aug 27 14:35:07.591400 2014] in http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/86/114286/12/23562/screen-key.txt.gz | 15:18 |
| BobBall | mtreinish: seems that we got the lock OK but didn't release it within the 6 seconds so it was released for us | 15:18 |
| *** oomichi has joined #openstack-qa | 15:18 | |
| BobBall | mtreinish: so keystone was trying to delete a tenant but for some reason couldn't delete it fast enough (which we already knew I guess) | 15:18 |
| *** mpaolino has quit IRC | 15:22 | |
| *** dansmith has quit IRC | 15:22 | |
| *** ekhugen_alt has quit IRC | 15:22 | |
| *** tteggel has quit IRC | 15:22 | |
| *** akuznetsova has quit IRC | 15:22 | |
| *** jogo has quit IRC | 15:22 | |
| mtreinish | BobBall: yeah, well it could have been something else holding the lock too long instead of the tenant delete | 15:22 |
| mtreinish | but this confirms it was the tenant delete itself | 15:22 |
| *** Longgeek has joined #openstack-qa | 15:22 | |
| BobBall | found it mtreinish | 15:22 |
| BobBall | our devstack VM is swapping | 15:22 |
| *** mpaolino has joined #openstack-qa | 15:22 | |
| *** dansmith has joined #openstack-qa | 15:22 | |
| *** ekhugen_alt has joined #openstack-qa | 15:22 | |
| *** tteggel has joined #openstack-qa | 15:22 | |
| *** akuznetsova has joined #openstack-qa | 15:22 | |
| *** jogo has joined #openstack-qa | 15:22 | |
| *** Longgeek has quit IRC | 15:22 | |
| BobBall | devstack is now using > 6G of RAM | 15:23 |
| BobBall | so we'll blame ourselves for it :) | 15:23 |
| mtreinish | ahh, yeah that would probably cause things to be slow | 15:23 |
| mtreinish | how much ram do you give them? | 15:23 |
| mtreinish | BobBall: well, dhellmann found yesterday that on neutron jobs in the gate we are exceeding 8GB triggering oom-killer to blow away mysqld | 15:24 |
| BobBall | 4G atm, but I can increase that a bit... | 15:24 |
| *** markmcclain has quit IRC | 15:24 | |
| BobBall | looking at reducing the number of workers to ease up on the RAM requirement a bit | 15:24 |
| mtreinish | yeah, that should help a bit | 15:25 |
| *** atiwari has joined #openstack-qa | 15:28 | |
| *** ildikov has joined #openstack-qa | 15:28 | |
| *** oomichi has quit IRC | 15:31 | |
| *** fungi has quit IRC | 15:34 | |
| *** openstack has joined #openstack-qa | 16:19 | |
| *** dkranz has joined #openstack-qa | 16:20 | |
| *** dustins has quit IRC | 16:21 | |
| *** yamahata has quit IRC | 16:28 | |
| *** ildikov_ has joined #openstack-qa | 16:30 | |
| *** dims has joined #openstack-qa | 16:30 | |
| *** ildikov has quit IRC | 16:33 | |
| *** jpich has quit IRC | 16:33 | |
| *** aimon has joined #openstack-qa | 16:33 | |
| BobBall | Any way to restrict the number of nova-api workers in the gate? I want to set osapi_compute_workers=$TEMPEST_CONCURRENCY but we seem to only have a localrc rather than a local.conf so I don't know how I can get that value through to nova from devstack? | 16:33 |
| *** tosky_ has joined #openstack-qa | 16:38 | |
| *** bkopilov has quit IRC | 16:42 | |
| openstackgerrit | A change was merged to openstack/qa-specs: Add the qa spec for bp:more-selectable-swift-tests https://review.openstack.org/105115 | 16:46 |
| *** krtaylor has quit IRC | 16:47 | |
| jogo | cdent: I wouldn't call it undesirable per se, but thanks | 16:52 |
| cdent | undesirable in that context? | 16:52 |
| *** BobBall is now known as BobBall_Away | 16:54 | |
| jogo | cdent: yeah in that context, not in general | 16:54 |
| jogo | cdent: I will review the patch later today | 16:54 |
| cdent | that's what I meant, perhaps should have been more explicit, but being too explicit is what got me down this rathole ;) | 16:54 |
| jogo | cdent: we have all been there ^_^ | 16:54 |
| cdent | :) | 16:54 |
| jogo | cdent: anyway thank you for your patience on this one, and sorry for taking so long to respond I know this is a priority for your team | 16:55 |
| *** bkopilov has joined #openstack-qa | 16:55 | |
| *** marun has quit IRC | 16:55 | |
| cdent | It would have only become a problem if we had pushed up closer to the juno3 milestone. I know everybody has a ton to do. | 16:55 |
| cdent | It's been a very good learning experience for me. | 16:55 |
| *** mpaolino has quit IRC | 16:56 | |
| *** markmcclain has joined #openstack-qa | 16:59 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-dev/devstack: Add oslo.concurrency and oslo.middleware https://review.openstack.org/117304 | 16:59 |
| *** dpaterson has joined #openstack-qa | 17:01 | |
| *** suneelb has joined #openstack-qa | 17:01 | |
| *** amotoki has quit IRC | 17:02 | |
| *** torandu has quit IRC | 17:02 | |
| *** e0ne has quit IRC | 17:03 | |
| *** harlowja has joined #openstack-qa | 17:04 | |
| *** torandu has joined #openstack-qa | 17:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-infra/devstack-gate: Add oslo.utils and oslo.concurrency https://review.openstack.org/117307 | 17:12 |
| *** sdmonov has joined #openstack-qa | 17:15 | |
| *** qba73 has quit IRC | 17:19 | |
| anteaya | anyone working on classifying the failures for gate-tempest-dsvm-neutron-full? I can but don't want to replicate | 17:28 |
| kashyap | Does this merit a new bug, I wonder, current gate failure from "gate-tempest-dsvm-full" job) -- http://logs.openstack.org/07/116707/1/gate/gate-tempest-dsvm-full/958f55e/logs/screen-n-cond.txt.gz | 17:31 |
| * kashyap files one anyway for now | 17:33 | |
| kashyap | Ah, there's already one that's recently filed - https://bugs.launchpad.net/nova/+bug/1353131 | 17:35 |
| *** Sukhdev has quit IRC | 17:36 | |
| *** aix has quit IRC | 17:37 | |
| anteaya | so help me understand what is happening when the testr tests commands succeeded and the log checker identifies that various log files have errors | 17:39 |
| openstackgerrit | Rohan Kanade proposed a change to openstack/tempest: Add subnet tests for extra attributes https://review.openstack.org/106326 | 17:40 |
| *** zz_dimtruck is now known as dimtruck | 17:44 | |
| *** hockeynut has joined #openstack-qa | 17:58 | |
| anteaya | this fingerprint went in 13 hours ago: http://git.openstack.org/cgit/openstack-infra/elastic-recheck/log/queries/1361924.yaml and should have caught this: http://logs.openstack.org/89/116889/1/gate/gate-tempest-dsvm-neutron-full/02de4a8/logs/devstacklog.txt.gz but the fingerprint is for console not devstacklog | 18:00 |
| *** mikal has quit IRC | 18:00 | |
| anteaya | I'll offer a patch to amend the fingerprint | 18:01 |
| *** mkollaro1 has quit IRC | 18:04 | |
| *** mikal has joined #openstack-qa | 18:05 | |
| *** rbak has quit IRC | 18:07 | |
| *** mkollaro has joined #openstack-qa | 18:10 | |
| openstackgerrit | Anita Kuno proposed a change to openstack-infra/elastic-recheck: Expanded fingerprint 1361924 https://review.openstack.org/117320 | 18:12 |
| *** cnesa8 has quit IRC | 18:14 | |
| *** e0ne has joined #openstack-qa | 18:14 | |
| *** annegent- has joined #openstack-qa | 18:15 | |
| *** Sukhdev has joined #openstack-qa | 18:23 | |
| *** yfried has joined #openstack-qa | 18:24 | |
| *** rbak has joined #openstack-qa | 18:27 | |
| *** dims has quit IRC | 18:28 | |
| *** dims has joined #openstack-qa | 18:28 | |
| *** annegent- is now known as annegentle | 18:38 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:38 | |
| *** zigo has quit IRC | 18:43 | |
| openstackgerrit | Anita Kuno proposed a change to openstack-infra/elastic-recheck: Expanded fingerprint 1361924 https://review.openstack.org/117320 | 18:43 |
| *** harlowja has quit IRC | 18:46 | |
| *** zigo has joined #openstack-qa | 18:46 | |
| *** harlowja has joined #openstack-qa | 18:46 | |
| *** torandu_ has joined #openstack-qa | 18:48 | |
| *** tosky_ has quit IRC | 18:49 | |
| mtreinish | dhellmann: did you open a bug for that failure yesteday? If not I can do it | 18:54 |
| mkoderer | mtreinish: can I get a quick review on https://review.openstack.org/#/c/113899/3 | 18:58 |
| *** atiwari has quit IRC | 18:59 | |
| *** torandu_ has quit IRC | 19:01 | |
| *** torandu has quit IRC | 19:01 | |
| *** torandu has joined #openstack-qa | 19:05 | |
| mtreinish | mkoderer: LGTM, do you think it's worth updating the ones with args to use it too? (in a separate patch) | 19:05 |
| mkoderer | mtreinish: yep I can do that.. decorators are sometimes tricky though ;) | 19:06 |
| mkoderer | mtreinish: btw the schema unification is done... I just have a clean-up patch https://review.openstack.org/#/c/115918/ | 19:07 |
| mkoderer | mtreinish: and I guess I have to update the documentation ;) | 19:07 |
| *** krtaylor has joined #openstack-qa | 19:07 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack-dev/devstack: Add rootwrap daemon mode support for Neutron https://review.openstack.org/107386 | 19:14 |
| *** cdent_ has joined #openstack-qa | 19:34 | |
| sdague | mrodden: have you looked into doing a bashateignore yet? | 19:34 |
| *** cdent has quit IRC | 19:36 | |
| *** cdent_ is now known as cdent | 19:36 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Take a swing at processes without screen https://review.openstack.org/117339 | 19:36 |
| dtroyer | sdague: (seeing the bashate comment) I was playing with doing a gitignore-stype thing with it, starting with using .gitignore itself… | 19:37 |
| dtroyer | sdague: anyway, this review ^^^ is the current state of the process execution art | 19:37 |
| sdague | dtroyer: ok, I'll take a look in a minute | 19:38 |
| sdague | mrodden: https://review.openstack.org/#/c/113892/ probably could use landing, that would solve the biggest part of the ignore issue | 19:38 |
| sdague | dtroyer: do you have +2 on bashate? | 19:39 |
| dtroyer | sdague: I think so | 19:39 |
| sdague | ok, so I think that link above is probably going to solve most of the issues with integrating it in devstack | 19:39 |
| sdague | it hides all the . dirs | 19:40 |
| *** Santosh has quit IRC | 19:42 | |
| *** malini has quit IRC | 19:44 | |
| *** malini|afk is now known as malini | 19:44 | |
| *** cdent has quit IRC | 19:47 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/tempest: Fix network/secgroup scenario tests for ironic https://review.openstack.org/112660 | 19:47 |
| openstackgerrit | David Shrewsbury proposed a change to openstack/tempest: Fix network/secgroup scenario tests for ironic https://review.openstack.org/112660 | 19:49 |
| sdague | adam_g: so there was another patch in flight that did the db stuff that you had in save-state in your patch | 19:49 |
| *** cdent has joined #openstack-qa | 19:49 | |
| *** cdent has quit IRC | 19:49 | |
| adam_g | sdague, https://review.openstack.org/#/c/115001/ ? | 19:49 |
| sdague | yeh | 19:49 |
| adam_g | sdague, so that will still fail if any of the databases MYSQL_SERVICES do not exist | 19:51 |
| adam_g | at least in grenade.sh (i think save-state runs without set -e?) | 19:51 |
| *** cdent has joined #openstack-qa | 19:57 | |
| *** aimon has quit IRC | 19:57 | |
| *** e0ne has quit IRC | 19:59 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Fix check-sanity + save-state /w disabled services https://review.openstack.org/113405 | 20:00 |
| adam_g | sdague, ^ rebased on 115001 | 20:01 |
| sdague | adam_g: so one question.... this is because ironic is enabled but has no old db? | 20:03 |
| sdague | I want to understand context for the existance check | 20:04 |
| adam_g | sdague, cinder does not currently get deployed /w ironic | 20:05 |
| sdague | it feels like it would be better to just populate MYSQL_SERVICES correctly | 20:05 |
| sdague | so populate it from actual enabled services | 20:05 |
| sdague | instead of a hardcoded list | 20:05 |
| adam_g | sdague, then we're in the business of mapping service names to assumed database names. we're already stuck doing that with processes -> devstack services | 20:06 |
| sdague | yes, but this has a failure mode of cinder db manage breaks and doesn't create a db, and we skip it as fine. | 20:07 |
| sdague | maybe it's fine | 20:08 |
| sdague | seems odd | 20:08 |
| sdague | I'll +A based on the fact that we'd probably catch the failure in a different mechanism | 20:08 |
| adam_g | agreed, tho i think we'd see more obvious failures in that case elsewhere | 20:08 |
| *** suneelb has quit IRC | 20:10 | |
| adam_g | sdague, thanks | 20:12 |
| sdague | so I've been trying to hit all the right pieces related to the sideways upgrade, but I'm sure I missed chunks. So if there is stuff you need my eyes on please share urls here | 20:12 |
| adam_g | sdague, appreciated. we're down to just a handful: https://review.openstack.org/#/q/topic:ironic_tempest,n,z + https://review.openstack.org/#/q/topic:ironic_grenade,n,z | 20:14 |
| adam_g | im hoping the migration script stuff lands in ironic today or tomorrow and to have the experimental check-grenade-dsvm-ironic-sideways job doing something useful by EOW | 20:15 |
| *** qba73 has joined #openstack-qa | 20:17 | |
| *** samuelbercovici has joined #openstack-qa | 20:17 | |
| *** nsaje has quit IRC | 20:17 | |
| *** adalbas has quit IRC | 20:22 | |
| *** Sukhdev has quit IRC | 20:23 | |
| *** nmagnezi__ has joined #openstack-qa | 20:24 | |
| adam_g | hmm. what am i missing? shouldn't this show an +A? https://review.openstack.org/#/c/112731/ | 20:24 |
| adam_g | it even went through gate checks | 20:24 |
| *** aimon has joined #openstack-qa | 20:32 | |
| *** ildikov has joined #openstack-qa | 20:37 | |
| *** malini1 has joined #openstack-qa | 20:40 | |
| *** malini1 has quit IRC | 20:41 | |
| *** ildikov_ has quit IRC | 20:41 | |
| *** malini1 has joined #openstack-qa | 20:41 | |
| sdague | I think it was trivial rebase detection | 20:42 |
| sdague | readded the +A | 20:43 |
| sdague | ok, so we need a 2nd +2 on this - https://review.openstack.org/#/c/114087/ - jogo / dtroyer ? | 20:43 |
| mtreinish | sdague: +A | 20:47 |
| sdague | oh, right, you have grenade +2? | 20:48 |
| *** stevebaker has quit IRC | 20:49 | |
| *** stevebaker has joined #openstack-qa | 20:49 | |
| *** stevebaker has quit IRC | 20:49 | |
| dtroyer | I did add an answer regarding the ordering question anyway… | 20:49 |
| *** stevebaker has joined #openstack-qa | 20:49 | |
| *** stevebaker has quit IRC | 20:49 | |
| mtreinish | yeah, I cleaned up the acls while you were out: https://review.openstack.org/#/c/114024/ | 20:49 |
| *** stevebaker has joined #openstack-qa | 20:49 | |
| *** stevebaker has quit IRC | 20:49 | |
| mtreinish | as part of the devstack adoption | 20:49 |
| *** stevebaker has joined #openstack-qa | 20:49 | |
| *** stevebaker has quit IRC | 20:49 | |
| *** stevebaker has joined #openstack-qa | 20:49 | |
| *** stevebaker has quit IRC | 20:50 | |
| *** stevebaker has joined #openstack-qa | 20:50 | |
| *** stevebaker has quit IRC | 20:50 | |
| *** stevebaker has joined #openstack-qa | 20:50 | |
| *** stevebaker has quit IRC | 20:50 | |
| *** stevebaker has joined #openstack-qa | 20:50 | |
| *** stevebaker has quit IRC | 20:50 | |
| *** stevebaker has joined #openstack-qa | 20:50 | |
| gordc | random question, is there a reason tempest doesn't use import_opt import_group when using oslo.config options not defined in module? | 20:50 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for horizon not registered bug 1362325 https://review.openstack.org/117350 | 20:51 |
| mriedem | horizon is exploding ^ | 20:51 |
| *** dkranz has quit IRC | 20:52 | |
| openstackgerrit | yunhong-jiang proposed a change to openstack/tempest: Test boot from volume checking https://review.openstack.org/116739 | 20:52 |
| openstackgerrit | yunhong-jiang proposed a change to openstack/tempest: Passing parameter to create image https://review.openstack.org/116738 | 20:52 |
| mtreinish | gordc: most likely because tempest is run within a test runner and not as it's own process. That puts some unique constraints on how we have to deal with the CONF object | 21:01 |
| *** samuelbercovici has quit IRC | 21:01 | |
| mriedem | mtreinish: is that why using CONF.foo as a default arg in a method blows up? | 21:02 |
| mtreinish | gordc: but, I don't think anyone has tried to use those methods in tempest as alternative to what's there | 21:02 |
| gordc | mtreinish: i see... wanted to ask before i started to pour through files to make changes.. | 21:02 |
| gordc | mtreinish: hmm... maybe i'll give it a shot. see if it breaks anything... i assume it can't hurt. | 21:03 |
| mtreinish | mriedem: well that might have been a bug that was fixed by: http://git.openstack.org/cgit/openstack/tempest/commit/?id=a96ee210f20145372ce9430dceca6700c7ef2aa5 | 21:03 |
| mriedem | mtreinish: jogo: sdague: i'm gonna start going through this list of old -1'ed e-r changes and abandon some https://review.openstack.org/#/q/status:open+project:openstack-infra/elastic-recheck,n,z | 21:03 |
| mriedem | didn't realize there were so many | 21:03 |
| *** krtaylor has quit IRC | 21:04 | |
| mtreinish | gordc: no it probably won't hurt, but it would mean a large refactor. So the bigger question is it worth it? :) | 21:05 |
| gordc | mtreinish: i feel like something is being hinted at... | 21:07 |
| *** dpaterson has quit IRC | 21:07 | |
| sdague | mriedem: good call | 21:07 |
| gordc | mtreinish: i'll take a quick look. i'm pretty sure ADHD will kick in before i finish first few files | 21:07 |
| *** catohornet has joined #openstack-qa | 21:08 | |
| *** stevebaker has quit IRC | 21:08 | |
| *** stevebaker has joined #openstack-qa | 21:08 | |
| mtreinish | gordc: heh, no just words from my experience, I've been down this particular config object rabbit hole a couple of times before. :) | 21:08 |
| gordc | mtreinish: lol i assume it ended up with tables being flipped... | 21:09 |
| gordc | i'll save it for another day... the grep was enough to put it off. | 21:10 |
| mtreinish | heh, yeah pretty much... | 21:10 |
| *** markmcclain has quit IRC | 21:12 | |
| *** nmagnezi__ has quit IRC | 21:13 | |
| *** catohornet has quit IRC | 21:14 | |
| *** e0ne has joined #openstack-qa | 21:15 | |
| *** aysyd has quit IRC | 21:15 | |
| *** mwagner_lap has quit IRC | 21:15 | |
| *** markmcclain has joined #openstack-qa | 21:18 | |
| *** e0ne has quit IRC | 21:18 | |
| *** e0ne has joined #openstack-qa | 21:22 | |
| jogo | mriedem: yeah without auto abandon ... | 21:23 |
| mrodden | sdague: was waiting for someone to test that patch out a bit more, which it looks like dean got to doing | 21:24 |
| jogo | mriedem mtreinish sdague: re gate debugging | 21:24 |
| jogo | I was going to help pleia2 write a e-r query today and document how to do it a a first pass at some docs | 21:24 |
| jogo | but your schedules didn't work out, so I am looking for someone else to help | 21:25 |
| jogo | any ideas | 21:25 |
| mriedem | jogo: mtreinish has a blog post on the topic | 21:25 |
| mriedem | and the e-r readme is pretty comprehensive on writing the queries | 21:25 |
| jogo | mriedem: he does? link | 21:26 |
| jogo | mriedem: well I was thinking we may need more docs around what the logs look like | 21:26 |
| jogo | but not sure until a fresh pair of eyes look | 21:26 |
| mtreinish | jogo: it's still a draft | 21:27 |
| anteaya | mriedem: I'm stumped then | 21:27 |
| anteaya | mriedem: http://logs.openstack.org/89/116889/1/gate/gate-tempest-dsvm-neutron-full/02de4a8/logs/devstacklog.txt.gz | 21:27 |
| sdague | sure, honestly I think often it's better to get the fresh eyes to write the docs, because it comes with what the new person feels is important vs. the things we think are (which might be important, but require too much context to understand) | 21:27 |
| anteaya | mriedem: why is that log on the unclassified page? | 21:27 |
| mtreinish | jogo: I still need to do things like open the bug and push the e-r change before I publish it | 21:27 |
| anteaya | jogo: I don't think the problem is not enough docs | 21:28 |
| mriedem | anteaya: i don't see devstacklog.txt.gz on the uncategorized bugs page | 21:28 |
| anteaya | the problem is people don't see the point | 21:28 |
| anteaya | mriedem: under gate-tempest-dsvm-neutron-full : 13 Uncategorized Fails. 84.7% Classification Rate (85 Total Fails) | 21:28 |
| anteaya | 2014-08-27T00:48: http://logs.openstack.org/89/116889/1/gate/gate-tempest-dsvm-neutron-full/02de4a8/ | 21:29 |
| anteaya | is that not there for you? | 21:29 |
| jogo | anteaya: maybe just not the write docs. Or maybe we have them but they are hard to find. I think a fresh pair of eyes would help | 21:29 |
| mriedem | anteaya: i see it | 21:29 |
| jogo | as I am too close to really say | 21:29 |
| anteaya | jogo: I had a session for people at neutron mid-cycle (2 people showed up, one a collegue of mriedem) | 21:29 |
| anteaya | I couldn't convince the other person of the need to classify and submit fingerprints | 21:30 |
| mriedem | i work with people?! | 21:30 |
| mriedem | anteaya: i see what you mean, subunit2cvs is in devstacklog.txt.gz in there but not console.html | 21:30 |
| anteaya | so I'm told | 21:30 |
| mriedem | that's weird, anyway, couldn't hurt to have your change, let me find it | 21:30 |
| anteaya | mriedem: I'll make the fingerprint be what you want | 21:30 |
| anteaya | not sure what it should be though | 21:30 |
| SergeyLukjanov | mriedem, sdague, I think I know the reason for https://bugs.launchpad.net/horizon/+bug/1362325 | 21:30 |
| SergeyLukjanov | mriedem, sdague, we've deprecated today the sahara-dashboard repo's code | 21:31 |
| anteaya | jogo: having pleia2's input on e-r docs is great | 21:31 |
| SergeyLukjanov | sdague, https://review.openstack.org/#/c/112000 and https://review.openstack.org/#/c/112001 | 21:31 |
| mriedem | anteaya: +W | 21:31 |
| SergeyLukjanov | sdague, will fix the issue | 21:32 |
| anteaya | but I don't know if it will move the needle for getting more than the 8 already creating fingerprints to create fingerprints | 21:32 |
| jogo | anteaya: sadly it will have to wait till next week | 21:32 |
| anteaya | jogo: kk | 21:32 |
| jogo | anteaya: for pleia2's thoughts | 21:32 |
| anteaya | k | 21:32 |
| anteaya | mriedem: not sure what +W means, hope that it is good for me | 21:32 |
| mriedem | SergeyLukjanov: those 2 changes aren't merged though | 21:32 |
| *** tkammer has joined #openstack-qa | 21:32 | |
| mriedem | SergeyLukjanov: that horizon failure is in the gate | 21:32 |
| cdent | Has there been any thought put into making a tool for presenting the logs that shows them in some kind of folded format? | 21:32 |
| mriedem | anteaya: workflow + 1 | 21:32 |
| mriedem | +Walrus | 21:33 |
| anteaya | mriedem: ah thanks | 21:33 |
| anteaya | and +Walrus | 21:33 |
| SergeyLukjanov | mriedem, this two changes will fix the issue | 21:33 |
| cdent | At the moment you look at the logs and kind of do a "ow" | 21:33 |
| anteaya | ko ko kachew | 21:33 |
| *** e0ne has quit IRC | 21:33 | |
| mriedem | SergeyLukjanov: ah ok | 21:33 |
| mriedem | i'll update the bug | 21:33 |
| cdent | If the logs were folded that might make them a little more diggable | 21:33 |
| mriedem | cdent: folded how? you can filter the logs by log level for starters | 21:33 |
| mriedem | that's what i do when looking for test failures, i filter by trace | 21:33 |
| cdent | that's only some of the los | 21:34 |
| cdent | logs | 21:34 |
| mriedem | since trace is what i'm usually going to use for e-r queries | 21:34 |
| mriedem | cdent: what did you have in mind? | 21:34 |
| cdent | I'm not sure if my experience is common, but the grenade.sh log is where I usually find myself at a first pass | 21:35 |
| mriedem | cdent: the grenade log is the most fun thing ever when it fails | 21:35 |
| cdent | It will lead me the right place once I find the problem in it | 21:35 |
| SergeyLukjanov | mriedem, sdague, the reason for the https://bugs.launchpad.net/horizon/+bug/1362325 is that we removed deprecated code from the sahara-dashboard repo that is still used in the devstack code | 21:35 |
| mriedem | <sarcasm/> | 21:35 |
| cdent | but initially it's just a bunch of noise about install stuff | 21:35 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Expanded fingerprint 1361924 https://review.openstack.org/117320 | 21:35 |
| cdent | So I don't really have an idea, just wonder if there's an idea somewhere in there. | 21:36 |
| cdent | I don't generally like to say "maybe we just need better tools" but they might help some. | 21:36 |
| cdent | It might be the case that the best way to learn how to do it is just do it, but that doesn't help the first few goes for someone new. | 21:37 |
| cdent | When I first looked at the logs I really thought someone was playing a joke on me. | 21:37 |
| mriedem | yeah idk, there are a few common failures in there, (1) setup/install of the services (old/new), (2) tempest smoke test failures (old/new), and (3) sanity checks at the end | 21:38 |
| mriedem | with any test failure bug log dump you should start at the end and work up | 21:38 |
| mriedem | the 'failed to start service' in grenade.sh.log has thrown me off when that's not the failure | 21:39 |
| anteaya | okay mriedem since you are here and after you are finished chatting with cdent | 21:39 |
| anteaya | what do I do with testr tests that pass and log checker that finds errors in the log files? http://logs.openstack.org/51/105351/5/gate/gate-tempest-dsvm-neutron-full/d9b1be5/console.html | 21:40 |
| mriedem | jogo: found a problem with the 'open reviews' change on the e-r status page, | 21:40 |
| mriedem | jogo: it's showing open reviews in e-r itself :) | 21:40 |
| mriedem | i don't think that was the intent | 21:40 |
| mriedem | anteaya: depends on which log files are allowed to have failures, but for sure n-cond can't have errors in it | 21:40 |
| mriedem | anteaya: the summary for the fail is at the end http://logs.openstack.org/51/105351/5/gate/gate-tempest-dsvm-neutron-full/d9b1be5/console.html#_2014-08-26_16_08_29_349 | 21:41 |
| anteaya | so what does it mean if n-cond has errors and testr has succeeded? | 21:41 |
| mriedem | cdent: ^ | 21:41 |
| * cdent is watching | 21:41 | |
| mriedem | anteaya: so this is bug 1353131 where we have a retry on deadlock in the db api code, | 21:42 |
| mriedem | anteaya: so we can have the error on the first pass and then it works on the 2nd | 21:42 |
| mriedem | so tests don't fail, but the n-cond log has the error | 21:42 |
| mriedem | maybe we need to whitelist that guy now | 21:42 |
| mtreinish | mriedem: heh, that's probably because it searches gerrit for the bug id in the message | 21:42 |
| mtreinish | which e-r patches will hit | 21:42 |
| mriedem | i've looked for ways to fingerprint that (so has dolphm) which filters out successful hits and so far nothing great | 21:42 |
| mriedem | mtreinish: yup | 21:43 |
| mriedem | so we just need to filter out openstack-infra/elastic-recheck projects | 21:43 |
| anteaya | ick | 21:43 |
| mriedem | anteaya: e.g. https://review.openstack.org/#/c/114566/ | 21:43 |
| jogo | mriedem: yeah I noticed that ;) | 21:44 |
| SergeyLukjanov | chmouel, dtroyer, ianw, jeblair, sdague, folks, could you please take a look at https://review.openstack.org/#/c/112000 and https://review.openstack.org/#/c/112001 ? We need to land them to stop installing sahara-dashboard in the devstack code (we removed the deprecated code from the sahara-dashboard today), it's needed to unbreak the gate - https://bugs.launchpad.net/horizon/+bug/1362325 | 21:44 |
| mriedem | anteaya: the weird thing is i've run queries against logstash for the log file error thing in console.html and it shows successful runs, i'm not sure why | 21:44 |
| mriedem | jogo: i'm opening a bug to track | 21:45 |
| anteaya | :( | 21:45 |
| anteaya | the fingerprint you linked me to of dolph's has a link to this bug in the commit message: https://bugs.launchpad.net/nova/+bug/1297590 | 21:46 |
| mriedem | anteaya: yeah, there are 2, arosen's was from icehouse i think? | 21:46 |
| anteaya | 2014-03-26 | 21:46 |
| anteaya | so yeah, we can say icehouse | 21:46 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add query for bug 1362347 https://review.openstack.org/117358 | 21:47 |
| mtreinish | jogo, mriedem: oom-killer fun ^^^ | 21:47 |
| openstackgerrit | Chris Dent proposed a change to openstack/tempest: Add scenario test for swift-proxy ceilometer middleware https://review.openstack.org/115971 | 21:48 |
| mriedem | looking | 21:48 |
| anteaya | okay so I'll be honest, right now everything you said makes sense to you mriedem and gives me the deer in the headlights feeling | 21:49 |
| anteaya | as in, I have no idea what to do with what you told me | 21:49 |
| mriedem | muwahahaha | 21:49 |
| mriedem | anteaya: the short answer on that one is it's complicated | 21:49 |
| anteaya | that I did understand | 21:50 |
| anteaya | I can wander away quietly and try to find a safter unclassified failure | 21:50 |
| anteaya | s/safter/safer | 21:50 |
| mriedem | so i'm searching on something like this: | 21:50 |
| mriedem | message:"Not Whitelisted" AND message:"ERROR nova.quota" AND message:"Failed to commit reservations" AND tags:"console" | 21:50 |
| anteaya | great | 21:50 |
| mriedem | that gives me 323 hits in 7 days, multiple queues but it's not 100% fail | 21:51 |
| dtroyer | SergeyLukjanov: both of those reviews appear to be failing on Xenserver multiple times, can you characterize what that is about? | 21:51 |
| mriedem | dtroyer: that's the neutron oom killer | 21:51 |
| mriedem | BobBall_Away: was talking about that in nova this morning | 21:52 |
| mriedem | dtroyer: xenserver ci has been blowing up due to that all week i think | 21:52 |
| dtroyer | mriedem: ah, ok, thanks. I hadn't looked at those jobs for a while | 21:52 |
| mriedem | anteaya: so getting back, i'd add that to the known bug fingerprint but the success rate on that query is too high | 21:53 |
| mriedem | anteaya: if i could figure out why it's successful in some runs we could filter those out, e.g. maybe there are jobs that aren't failing on errors in n-cond? | 21:53 |
| mriedem | anteaya: http://logs.openstack.org/40/111840/10/check/check-tempest-dsvm-full/34262e3/console.html#_2014-08-27_18_14_56_284 | 21:54 |
| SergeyLukjanov | dtroyer, ++ for mriedem comment, I don't see any patch specific things in their logs, additionally there was +1 on the both patches from the xen ci | 21:54 |
| chmouel | SergeyLukjanov: look good to me but i haven't tested, i assume it works for you guys | 21:54 |
| mriedem | anteaya: if we could figure out when it's OK for log files to have errors (n-cond in this case), we can filter those out, but i don't have the answeratm | 21:55 |
| mriedem | i'm sure others in here would know | 21:55 |
| *** dims has quit IRC | 21:55 | |
| mriedem | jogo: https://bugs.launchpad.net/openstack-ci/+bug/1362348 i'll try to take a crack at that sometime when i find the time | 21:55 |
| SergeyLukjanov | chmouel, all sahara dashboard code has been merged to horizon and so we've started removing all old code related to the separated repo... | 21:56 |
| dtroyer | SergeyLukjanov: +2A | 21:56 |
| chmouel | ok | 21:57 |
| *** tkammer has quit IRC | 21:58 | |
| *** mriedem has quit IRC | 22:00 | |
| SergeyLukjanov | dtroyer, thank you | 22:00 |
| *** ianw has quit IRC | 22:04 | |
| *** malini1 has quit IRC | 22:04 | |
| *** ianw has joined #openstack-qa | 22:04 | |
| adam_g | mtreinish, does the TODO added here sufficiently address your concerns? id like to improve that soonish but dont want to block the current ironic patch queue | 22:05 |
| openstackgerrit | A change was merged to openstack-dev/grenade: Save base and target release state correctly https://review.openstack.org/115001 | 22:06 |
| cdent | go me | 22:07 |
| *** cdent has quit IRC | 22:08 | |
| *** gordc has quit IRC | 22:08 | |
| mtreinish | adam_g: added where? Sorry it's just a context switch for me, I know we were talking about it yesterday... | 22:09 |
| *** bnemec has quit IRC | 22:11 | |
| *** bnemec has joined #openstack-qa | 22:12 | |
| adam_g | mtreinish, jeez, my bad https://review.openstack.org/#/c/113707/ | 22:12 |
| *** harlowja_ has joined #openstack-qa | 22:13 | |
| *** harlowja_ has quit IRC | 22:15 | |
| *** harlowja_ has joined #openstack-qa | 22:15 | |
| *** harlowja has quit IRC | 22:16 | |
| anteaya | I missed him, I was on the phone | 22:26 |
| *** mkollaro has quit IRC | 22:29 | |
| *** weshay has quit IRC | 22:34 | |
| *** sdmonov has quit IRC | 22:34 | |
| *** sballe_ has joined #openstack-qa | 22:35 | |
| *** bkopilov has quit IRC | 22:36 | |
| *** jgrimm is now known as jgrimm_zz | 22:37 | |
| *** sballe has quit IRC | 22:37 | |
| *** markmcclain has quit IRC | 22:38 | |
| *** krtaylor has joined #openstack-qa | 22:45 | |
| mtreinish | adam_g: yeah, that was TODO covered it | 22:46 |
| sdague | mtreinish: your -2 here can probably come off - https://review.openstack.org/#/c/96661/ | 22:47 |
| *** bkopilov has joined #openstack-qa | 22:48 | |
| *** ZackaryDoyle has joined #openstack-qa | 22:56 | |
| *** dimtruck is now known as zz_dimtruck | 22:57 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: clean up readme https://review.openstack.org/115011 | 23:00 |
| jogo | sdague mtreinish: the fixed readme from dolph ^ | 23:00 |
| jogo | for e-r | 23:01 |
| sdague | +2 | 23:02 |
| sdague | ok, dinner time. Enjoy | 23:02 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for horizon not registered bug 1362325 https://review.openstack.org/117350 | 23:08 |
| *** jamielennox|away is now known as jamielennox | 23:10 | |
| salv-orlando | mtreinish, sdague: is anybody already working on this? http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiTm90UmVnaXN0ZXJlZDogRGFzaGJvYXJkIHdpdGggc2x1ZyBcXFwic2FoYXJhXFxcIiBpcyBub3QgcmVnaXN0ZXJlZFwiIEFORCB0YWdzOlwiaG9yaXpvbl9lcnJvci50eHRcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiODY0MDAiLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxNDA5MTgxMDQxNjAxLCJtb2RlIjoiIiwiYW5hbHl6ZV9maWVsZCI6IiJ9 | 23:12 |
| salv-orlando | sdague, mtreinish: here it is, mriedem filed it https://bugs.launchpad.net/horizon/+bug/1362325 | 23:14 |
| morganfainberg | mtreinish, sdague, is there a way to log in devstack from within a function that's echo will be captured as the output? | 23:21 |
| morganfainberg | mtreinish, sdague, not seeing a clear way to do that | 23:21 |
| morganfainberg | i guess echo $msg 1>&2; would work. | 23:23 |
| dtroyer | morganfainberg: warn() is the closest thing… yeah… | 23:23 |
| morganfainberg | dtroyer, i was asked to log what apache version was detected in https://review.openstack.org/#/c/102425 | 23:23 |
| morganfainberg | dtroyer, should i just explicitly echo, make a "info()" use "warn()"? | 23:24 |
| dtroyer | morganfainberg: ah, found it…echo_nolog() is defined in stack.sh. that should be moved to functions-common, that's what you want if yo don't want it logged | 23:26 |
| morganfainberg | ah. ok | 23:26 |
| morganfainberg | dtroyer, no uhm here look at Robbie's comment arounjd 109 https://review.openstack.org/#/c/102425/5/lib/apache | 23:27 |
| morganfainberg | erm | 23:27 |
| morganfainberg | dtroyer, line 70 that is | 23:27 |
| morganfainberg | dtroyer, is what i'm trying to address | 23:27 |
| dtroyer | so this is all run with xtrace on, right? where does he want it logged? | 23:28 |
| morganfainberg | yeah i dunno | 23:29 |
| morganfainberg | i think i'm going to respond with 'but xtrace is on' | 23:29 |
| morganfainberg | now that i'm re-looking at it | 23:29 |
| dtroyer | and if VERBOSE=False, you don't care…oh, that's why…write only to log and not stdout then | 23:29 |
| morganfainberg | so, do i need to do anything? | 23:30 |
| dtroyer | I don't think so? | 23:30 |
| morganfainberg | dtroyer, yeah that was my view. | 23:30 |
| morganfainberg | dtroyer, ok making sure i didn't miss something | 23:30 |
| morganfainberg | dtroyer, thanks! | 23:30 |
| dtroyer | He needs to clarify if xtrace isn't enough | 23:30 |
| dtroyer | np | 23:31 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Use the apache 2.4 ErrorLogFormat directive https://review.openstack.org/102425 | 23:33 |
| *** oomichi has joined #openstack-qa | 23:39 | |
| * gmann is back (gone 14:16:04) | 23:41 | |
| morganfainberg | gmann, not sure about this channel but typically autoaway/autoreturn messages are frowned upon (you may want to disable that) | 23:42 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Use the apache 2.4 ErrorLogFormat directive https://review.openstack.org/102425 | 23:51 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Use the apache 2.4 ErrorLogFormat directive https://review.openstack.org/102425 | 23:53 |
| *** bkopilov has quit IRC | 23:57 | |
| openstackgerrit | James E. Blair proposed a change to openstack-dev/devstack: Move sahara client installation to lib/sahara https://review.openstack.org/112000 | 23:58 |
| *** dims has joined #openstack-qa | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!