openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network and secgroup resources https://review.openstack.org/100169 | 00:02 |
---|---|---|
openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Replace screen_it() with run_process() throughout https://review.openstack.org/120077 | 00:07 |
cdent | jogo, dtroyer ^ I have to give up for the night | 00:11 |
*** dmellado has quit IRC | 00:11 | |
*** aimon has quit IRC | 00:13 | |
*** dmellado has joined #openstack-qa | 00:13 | |
*** dmorita has joined #openstack-qa | 00:24 | |
*** harlowja_ has quit IRC | 00:25 | |
*** cdent has quit IRC | 00:25 | |
*** harlowja has joined #openstack-qa | 00:28 | |
*** weshay has quit IRC | 00:30 | |
*** aimon has joined #openstack-qa | 00:37 | |
openstackgerrit | Miguel Lavalle proposed a change to openstack/tempest: Fix tempest.scenario.manager.ScenarioTest https://review.openstack.org/119659 | 00:38 |
*** suneelb has quit IRC | 00:46 | |
*** dimtruck is now known as zz_dimtruck | 00:49 | |
*** jpich has quit IRC | 00:51 | |
*** yamahata has joined #openstack-qa | 00:51 | |
*** jpich has joined #openstack-qa | 01:07 | |
*** zz_dimtruck is now known as dimtruck | 01:24 | |
*** dims_ has joined #openstack-qa | 01:32 | |
*** amotoki has joined #openstack-qa | 01:34 | |
*** mlavalle has quit IRC | 01:35 | |
*** oomichi has joined #openstack-qa | 01:37 | |
*** Sukhdev has joined #openstack-qa | 01:40 | |
*** Sukhdev has quit IRC | 01:42 | |
*** Sukhdev has joined #openstack-qa | 01:43 | |
*** yjiang5 has quit IRC | 01:44 | |
*** nosnos has joined #openstack-qa | 01:47 | |
*** Sukhdev has quit IRC | 01:47 | |
*** jaypipes has quit IRC | 01:58 | |
*** rodrigods_ has joined #openstack-qa | 01:59 | |
*** markmcclain has quit IRC | 02:05 | |
*** jpich has quit IRC | 02:06 | |
*** jyuso has joined #openstack-qa | 02:09 | |
*** amotoki_ has joined #openstack-qa | 02:10 | |
*** amotoki has quit IRC | 02:11 | |
*** amotoki_ is now known as amotoki | 02:11 | |
*** rodrigods_ has quit IRC | 02:14 | |
*** dkranz has joined #openstack-qa | 02:16 | |
*** jpich has joined #openstack-qa | 02:19 | |
*** markmcclain has joined #openstack-qa | 02:26 | |
*** sileht has quit IRC | 02:46 | |
*** dims_ has quit IRC | 02:49 | |
*** dims_ has joined #openstack-qa | 02:49 | |
*** dims_ has quit IRC | 02:54 | |
*** rodrigods_ has joined #openstack-qa | 03:00 | |
*** rodrigods_ has quit IRC | 03:00 | |
*** arnaud has quit IRC | 03:01 | |
*** adalbas has quit IRC | 03:05 | |
*** rfolco has joined #openstack-qa | 03:29 | |
*** Poornima has joined #openstack-qa | 03:29 | |
*** Longgeek has joined #openstack-qa | 03:29 | |
*** Longgeek_ has joined #openstack-qa | 03:32 | |
*** Longgeek has quit IRC | 03:36 | |
*** rodrigods_ has joined #openstack-qa | 03:37 | |
*** dimtruck is now known as zz_dimtruck | 03:42 | |
*** rodrigods_ has quit IRC | 03:45 | |
*** jpich has quit IRC | 03:58 | |
*** zz_dimtruck is now known as dimtruck | 04:02 | |
*** harlowja is now known as harlowja_away | 04:05 | |
*** jpich has joined #openstack-qa | 04:10 | |
*** krtaylor has joined #openstack-qa | 04:11 | |
*** dimtruck is now known as zz_dimtruck | 04:11 | |
*** chandankumar has joined #openstack-qa | 04:19 | |
openstackgerrit | A change was merged to openstack/tempest: Allow out of quota failure status code to be 413 or 403 https://review.openstack.org/118099 | 04:28 |
*** Poornima has quit IRC | 04:31 | |
*** markmcclain has quit IRC | 04:34 | |
*** HenryG is now known as HenryG_zzzz | 04:40 | |
*** Sukhdev_ has joined #openstack-qa | 04:43 | |
*** yamahata has quit IRC | 04:46 | |
*** yamahata has joined #openstack-qa | 04:47 | |
*** jaypipes has joined #openstack-qa | 04:54 | |
*** ChanServ sets mode: +o jaypipes | 04:54 | |
*** Madan has joined #openstack-qa | 04:57 | |
*** Poornima has joined #openstack-qa | 04:57 | |
*** torandu has quit IRC | 05:05 | |
*** pcrews has quit IRC | 05:07 | |
*** sabeen has quit IRC | 05:09 | |
*** Madan has quit IRC | 05:18 | |
*** neeti has joined #openstack-qa | 05:28 | |
*** Madan has joined #openstack-qa | 05:33 | |
*** nmagnezi has joined #openstack-qa | 05:44 | |
*** yfried has joined #openstack-qa | 05:44 | |
*** rdekel has joined #openstack-qa | 05:49 | |
*** flaper87|afk is now known as flaper87 | 05:58 | |
*** nmagnezi has quit IRC | 05:59 | |
*** vigneshvar has joined #openstack-qa | 05:59 | |
*** vigneshvar_ has joined #openstack-qa | 06:01 | |
*** vigneshvar_ has quit IRC | 06:03 | |
*** oomichi_ has joined #openstack-qa | 06:03 | |
*** oomichi has quit IRC | 06:05 | |
*** zz_dimtruck is now known as dimtruck | 06:10 | |
*** mpaolino has joined #openstack-qa | 06:12 | |
*** andreaf has joined #openstack-qa | 06:13 | |
*** SridharG has joined #openstack-qa | 06:17 | |
*** Longgeek_ has quit IRC | 06:17 | |
*** Longgeek has joined #openstack-qa | 06:18 | |
*** dimtruck is now known as zz_dimtruck | 06:20 | |
*** Longgeek has quit IRC | 06:20 | |
*** mauricioliima has quit IRC | 06:23 | |
*** Sukhdev_ has quit IRC | 06:23 | |
*** dkranz has quit IRC | 06:28 | |
*** dkranz has joined #openstack-qa | 06:28 | |
*** dkranz has quit IRC | 06:33 | |
*** dkranz has joined #openstack-qa | 06:34 | |
*** Madan1 has joined #openstack-qa | 06:34 | |
*** k4n0 has joined #openstack-qa | 06:34 | |
*** afazekas is now known as __afazekas | 06:34 | |
*** Madan1 has quit IRC | 06:35 | |
*** Madan1 has joined #openstack-qa | 06:35 | |
*** Madan has quit IRC | 06:36 | |
*** gszasz has joined #openstack-qa | 06:37 | |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Replace confusing member name https://review.openstack.org/120706 | 06:37 |
*** rhsu has quit IRC | 06:49 | |
*** cmyster_away is now known as cmyster | 06:49 | |
*** rfolco has quit IRC | 06:51 | |
*** Longgeek has joined #openstack-qa | 06:52 | |
*** mrodden has quit IRC | 06:56 | |
*** mrodden has joined #openstack-qa | 06:58 | |
*** jlibosva has joined #openstack-qa | 06:59 | |
*** Qarekhani has quit IRC | 07:03 | |
*** sileht has joined #openstack-qa | 07:03 | |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add a test for deleting multiple objects by POST method https://review.openstack.org/116547 | 07:06 |
*** chandankumar has quit IRC | 07:09 | |
*** ildikov has joined #openstack-qa | 07:09 | |
*** dkranz has quit IRC | 07:18 | |
*** jpich has quit IRC | 07:20 | |
*** dkranz has joined #openstack-qa | 07:20 | |
openstackgerrit | A change was merged to openstack/tempest: Migrate security_groups_basic to tempest clients https://review.openstack.org/112199 | 07:21 |
yfried | masayukig: andreaf: could you please take a look at this. it's ready for merge. https://review.openstack.org/#/c/113124/ | 07:22 |
openstackgerrit | Benny Kopilov proposed a change to openstack/tempest: glance v1 add image with duplicate id https://review.openstack.org/119610 | 07:23 |
*** sileht has quit IRC | 07:29 | |
*** sileht has joined #openstack-qa | 07:29 | |
*** e0ne has joined #openstack-qa | 07:32 | |
*** dkranz has quit IRC | 07:34 | |
*** dkranz has joined #openstack-qa | 07:34 | |
*** ashishg has joined #openstack-qa | 07:35 | |
*** chandankumar has joined #openstack-qa | 07:39 | |
*** jlanoux has joined #openstack-qa | 07:40 | |
yfried | masayukig: ping | 07:44 |
*** oomichi_ has quit IRC | 07:45 | |
*** qba73 has joined #openstack-qa | 07:46 | |
yfried | does anyone know why we expose the clients in specifically in scenario.manager.py? why can't we use "self.clients.network_clients" instead of "self.network_clients"? | 07:46 |
*** pkoniszewski has joined #openstack-qa | 07:47 | |
*** dkranz has quit IRC | 07:50 | |
*** dkranz has joined #openstack-qa | 07:52 | |
ashishg | does anybody know why i am hitting conflict issue with https://review.openstack.org/#/c/115527/ | 07:54 |
*** cnesa9 has joined #openstack-qa | 08:00 | |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add test to validate negative testing schema's https://review.openstack.org/120033 | 08:00 |
*** jordanP has joined #openstack-qa | 08:05 | |
*** afazekas_ has joined #openstack-qa | 08:05 | |
*** vigneshvar has quit IRC | 08:11 | |
*** ianw has quit IRC | 08:18 | |
*** ianw has joined #openstack-qa | 08:19 | |
*** vigneshvar has joined #openstack-qa | 08:24 | |
*** dkranz has quit IRC | 08:25 | |
*** dkranz has joined #openstack-qa | 08:26 | |
*** mpavlase has joined #openstack-qa | 08:30 | |
*** gszasz has quit IRC | 08:33 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 08:34 | |
*** dkranz has quit IRC | 08:36 | |
*** dkranz has joined #openstack-qa | 08:36 | |
*** Ch00k has joined #openstack-qa | 08:38 | |
*** gszasz has joined #openstack-qa | 08:40 | |
*** ianw has quit IRC | 08:41 | |
*** ianw has joined #openstack-qa | 08:42 | |
*** andreaf has quit IRC | 08:47 | |
*** oomichi has joined #openstack-qa | 08:52 | |
*** pkoniszewski has quit IRC | 08:54 | |
*** mkollaro has joined #openstack-qa | 08:54 | |
gmann | ashishg: may be this will help - fetch your patch - (git fetch https://review.openstack.org/openstack/tempest refs/changes/27/115527/1 && git checkout FETCH_HEAD) | 08:55 |
*** e0ne has quit IRC | 08:55 | |
gmann | ashishg: create a patch from this - git log -p -n1 > <new patch file path> | 08:56 |
gmann | ashishg: checkout the master, update it and the apply your patch on that new branch from master by - patch -p1 < <created patch file> | 08:56 |
gmann | ashishg: if there is any conflict those you need to resolve manually | 08:57 |
*** e0ne has joined #openstack-qa | 09:01 | |
*** gfidente has quit IRC | 09:04 | |
*** gfidente has joined #openstack-qa | 09:05 | |
*** oomichi has quit IRC | 09:10 | |
*** dkranz has quit IRC | 09:11 | |
*** dkranz has joined #openstack-qa | 09:11 | |
*** rdekel has quit IRC | 09:13 | |
afazekas_ | andreaf: https://review.openstack.org/#/c/120386/ | 09:14 |
*** mkollaro has quit IRC | 09:14 | |
yfried | andreaf_: a quick question about https://blueprints.launchpad.net/tempest/+spec/resource-cleanup | 09:15 |
yfried | andreaf_: to you mean that we want be able to override setUpClass, but instead we have our own "setup_class()" we can override, that will be always called form ancestor's setUpClass? | 09:18 |
ashishg | <gmann>: apply your patch on that new branch from master by - patch -p1 < <created patch file> not able to get it ? | 09:19 |
gmann | ashishg: after creating the patch file. checkout the master and then creat new branch from master (git checkout -b <new branhc name>) | 09:23 |
*** rfolco has joined #openstack-qa | 09:23 | |
gmann | ashishg: then on that new branch run patch -p1 < <created patch file> . it will apply the patch to new branch | 09:24 |
*** mkollaro has joined #openstack-qa | 09:26 | |
*** rdekel has joined #openstack-qa | 09:26 | |
*** aix has joined #openstack-qa | 09:26 | |
openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: multi-node dg setup https://review.openstack.org/106043 | 09:27 |
*** dkranz has quit IRC | 09:27 | |
*** dkranz has joined #openstack-qa | 09:30 | |
openstackgerrit | A change was merged to openstack/tempest: Move response code checking to client for v3 identity tests https://review.openstack.org/117778 | 09:31 |
openstackgerrit | Nir Magnezi proposed a change to openstack/tempest: Adds status check for router Port in scenarios https://review.openstack.org/117454 | 09:31 |
openstackgerrit | A change was merged to openstack/tempest: Add baremetal API tests about console https://review.openstack.org/117715 | 09:31 |
*** mpaolino has quit IRC | 09:34 | |
*** dkranz has quit IRC | 09:45 | |
*** dkranz has joined #openstack-qa | 09:46 | |
*** dims_ has joined #openstack-qa | 09:51 | |
*** dims_ has quit IRC | 10:00 | |
*** dims_ has joined #openstack-qa | 10:01 | |
*** mpaolino has joined #openstack-qa | 10:03 | |
ashishg | <gmann>I was able to resolve the conflicts but in the commit message when i perform git commit -a --amend i am seeing some other commits message(Merge "Migrate security_groups_basic to tempest clients") should i edit the message or i am doing something wrong | 10:03 |
gmann | ashishg: that great. for commit msg as this is new branch you need to write the commit msg again. just copy paste from gerrit | 10:05 |
gmann | do not runt git commit --amend. just run 'git commit' | 10:05 |
*** dims_ has quit IRC | 10:06 | |
ashishg | <gmann> performing tsocks git review getting error : tsocks git review Errors running git rebase -i remotes/gerrit/master error: could not apply 47100bb... Port API Tests Enhancement When you have resolved this problem run "git rebase --continue". If you would prefer to skip this patch, instead run "git rebase --skip". To check out the original branch and stop rebasing run "git rebase --abort". Could not apply 47100bb... Port | 10:14 |
ashishg | <gmann > Switching to some no branch | 10:15 |
gmann | ashishg: how you created your new branch from master only right? | 10:16 |
ashishg | <gmann> yes | 10:17 |
openstackgerrit | Rohan Kanade proposed a change to openstack/tempest: Add subnet tests for extra attributes https://review.openstack.org/106326 | 10:20 |
ashishg | <gmann> git checkout master git checkout -b Test-Port-User-Defined-Mac1 | 10:20 |
gmann | ashishg: what patch file shows? only your changes right | 10:21 |
*** rdekel has quit IRC | 10:23 | |
ashishg | <gmann>git checkout -b Test-Port-User-Defined-Mac1 patch -p1 < ashish.patch patching file tempest/api/network/test_ports.py Hunk #1 succeeded at 233 (offset -5 lines). git status # On branch Test-Port-User-Defined-Mac1 # Changes not staged for commit: # (use "git add <file>..." to update what will be committed) # (use "git checkout -- <file>..." to discard changes in working directory) # # modified: tempest/api/netw | 10:24 |
*** marun_afk has joined #openstack-qa | 10:25 | |
gmann | ashishg: git status? only your 1 file change right | 10:27 |
ashishg | <gmann> yes | 10:27 |
*** yamahata has quit IRC | 10:31 | |
gmann | ashishg: just check manually in that file for any conflict | 10:32 |
ashishg | <gmann> yes it has <<<<<<< HEAD >>>>>>> 47100bb... Port API Tests Enhancement | 10:36 |
*** e0ne has quit IRC | 10:37 | |
gmann | ashishg: :). just fix those. then it will work | 10:37 |
*** jyuso has quit IRC | 10:49 | |
*** marun_afk is now known as marun | 10:56 | |
*** rodrigods_ has joined #openstack-qa | 11:07 | |
*** e0ne has joined #openstack-qa | 11:10 | |
*** johnthetubaguy is now known as zz_johnthetubagu | 11:12 | |
*** marun has quit IRC | 11:15 | |
*** dims_ has joined #openstack-qa | 11:17 | |
*** rodrigods_ has quit IRC | 11:21 | |
mkoderer | dkranz: could you have a look to https://review.openstack.org/#/c/120033/ | 11:24 |
mkoderer | dkranz: some of the existing schemas are already broken :( | 11:24 |
*** dims_ has quit IRC | 11:30 | |
*** dims_ has joined #openstack-qa | 11:31 | |
*** ashishg has quit IRC | 11:32 | |
*** dkranz has quit IRC | 11:32 | |
*** dims__ has joined #openstack-qa | 11:34 | |
*** dims_ has quit IRC | 11:35 | |
openstackgerrit | Tomas Sedovic proposed a change to openstack/tempest: Use the AWS instance type in the cfn_init scenario https://review.openstack.org/120749 | 11:38 |
*** cnesa9 has quit IRC | 11:43 | |
afazekas_ | mkoderer, dkranz: https://review.openstack.org/#/c/120386/ | 11:43 |
*** enikanorov_ has joined #openstack-qa | 11:54 | |
*** enikanorov__ has quit IRC | 11:54 | |
*** andreaf has joined #openstack-qa | 11:54 | |
*** adalbas has joined #openstack-qa | 11:57 | |
*** mwagner_lap has quit IRC | 11:59 | |
*** bdperkin has quit IRC | 12:02 | |
*** bdperkin has joined #openstack-qa | 12:02 | |
*** weshay has joined #openstack-qa | 12:08 | |
*** markmcclain has joined #openstack-qa | 12:09 | |
*** dmorita has quit IRC | 12:11 | |
openstackgerrit | Tomas Sedovic proposed a change to openstack/tempest: Use the AWS instance type in the cfn_init scenario https://review.openstack.org/120749 | 12:12 |
*** mwagner_lap has joined #openstack-qa | 12:15 | |
*** e0ne has quit IRC | 12:16 | |
*** aysyd has joined #openstack-qa | 12:20 | |
*** zz_jgrimm is now known as jgrimm | 12:20 | |
*** nosnos has quit IRC | 12:22 | |
*** zz_dimtruck is now known as dimtruck | 12:22 | |
*** nosnos has joined #openstack-qa | 12:22 | |
*** masayukig has quit IRC | 12:24 | |
openstackgerrit | Andrea Frittoli proposed a change to openstack/qa-specs: add bp:resource-cleanup https://review.openstack.org/118352 | 12:25 |
*** nosnos has quit IRC | 12:27 | |
*** cnesa9 has joined #openstack-qa | 12:28 | |
*** masayukig has joined #openstack-qa | 12:30 | |
*** mpaolino has quit IRC | 12:31 | |
*** dkranz has joined #openstack-qa | 12:32 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 12:37 | |
*** Madan1 has quit IRC | 12:47 | |
*** Poornima has quit IRC | 12:48 | |
*** dims__ has quit IRC | 12:49 | |
*** dims_ has joined #openstack-qa | 12:50 | |
*** dims_ is now known as dims | 12:51 | |
*** marun has joined #openstack-qa | 12:53 | |
*** HenryG_zzzz is now known as HenryG | 12:55 | |
*** jgrimm is now known as zz_jgrimm | 13:03 | |
*** neeti has quit IRC | 13:10 | |
andreaf | dkranz: ping - regarding https://review.openstack.org/#/c/118352 - resource cleanup spec, I fixed two typos and added an etherpad link, if you have time for a quick re-review | 13:11 |
*** dustins has joined #openstack-qa | 13:13 | |
dkranz | andreaf: ok, done | 13:13 |
andreaf | dkranz: thank you! | 13:13 |
andreaf | mtreinish: ping - regarding resource cleanup spec (https://review.openstack.org/#/c/118352) - do you think we got enough reviews? I'd like to start working on that pretty soon if possible | 13:15 |
*** vrovachev has joined #openstack-qa | 13:22 | |
*** dimtruck is now known as zz_dimtruck | 13:24 | |
*** mriedem has joined #openstack-qa | 13:24 | |
mtreinish | andreaf: ok, I'll take a look at it now | 13:27 |
andreaf | mtreinish, thank you | 13:28 |
*** pcrews has joined #openstack-qa | 13:31 | |
*** jecarey has joined #openstack-qa | 13:32 | |
*** ylobankov has joined #openstack-qa | 13:32 | |
*** torandu has joined #openstack-qa | 13:33 | |
*** sabeen has joined #openstack-qa | 13:34 | |
*** weshay has quit IRC | 13:36 | |
*** weshay has joined #openstack-qa | 13:39 | |
*** toure has joined #openstack-qa | 13:39 | |
*** Longgeek has quit IRC | 13:41 | |
*** e0ne has joined #openstack-qa | 13:42 | |
*** chandankumar has quit IRC | 13:47 | |
*** bobj has joined #openstack-qa | 13:50 | |
bobj | Hello everybody. I'm trying run stress test on object storage module using tempest, I using @test.stresstest but these approach doesn't work. Somebody can help me? | 13:50 |
*** balles has joined #openstack-qa | 13:50 | |
mtreinish | mkoderer: ^^^ | 13:53 |
*** samuelbercovici has joined #openstack-qa | 13:58 | |
*** markmcclain has quit IRC | 13:58 | |
*** arnaud has joined #openstack-qa | 14:00 | |
mtreinish | andreaf: reviewed, if you respin it quickly we can fast path it through | 14:02 |
mtreinish | bobj: you need to provide a bit more info on why it doesn't work | 14:03 |
*** mtreinish changes topic to "All Things QA..." | 14:03 | |
*** Longgeek has joined #openstack-qa | 14:05 | |
*** mpaolino has joined #openstack-qa | 14:14 | |
*** zz_dimtruck is now known as dimtruck | 14:14 | |
*** zz_jgrimm is now known as jgrimm | 14:15 | |
*** rbak has joined #openstack-qa | 14:15 | |
*** mpavlase has quit IRC | 14:17 | |
*** mpaolino has quit IRC | 14:17 | |
*** amotoki has quit IRC | 14:21 | |
*** yamahata has joined #openstack-qa | 14:23 | |
*** dmellado_ has joined #openstack-qa | 14:26 | |
*** vigneshvar has quit IRC | 14:31 | |
*** dmellado_ has quit IRC | 14:32 | |
openstackgerrit | git-harry proposed a change to openstack/tempest: VolumeMultiBackendTest: delete error volumes https://review.openstack.org/120793 | 14:33 |
*** gordc has joined #openstack-qa | 14:36 | |
*** amotoki has joined #openstack-qa | 14:38 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: always check servers are active before rescuing https://review.openstack.org/120796 | 14:40 |
*** arnaud has quit IRC | 14:40 | |
sdague | dkranz: so that's based on a bug you triaged, where I think the tempest side is actually part of the problem (or at least not helping in the solution) | 14:41 |
hockeynut | Greetings all - trying to figure out where our Barbican tests need to live (in our Barbican repo or in tempest repo). Was under the impression that they live in Barbican repo, so that's where our devstack gated tests live for now. Do they need to move? | 14:42 |
*** Ch00k has quit IRC | 14:42 | |
*** Ch00k_ has joined #openstack-qa | 14:42 | |
*** pkoniszewski has joined #openstack-qa | 14:45 | |
mtreinish | hockeynut: it depends on what the tests are doing | 14:45 |
mtreinish | hockeynut: that being said, having some tempest coverage is a graduation requirement though | 14:47 |
mtreinish | the tests which would be best to have in tempest are ones that tickle integration between barbican and another project | 14:47 |
*** markmcclain has joined #openstack-qa | 14:47 | |
hockeynut | mtreinish - these are our functional tests - only integration is with keystone for auth | 14:48 |
*** Ch00k_ has quit IRC | 14:48 | |
mtreinish | hockeynut: do you have a link? | 14:48 |
*** markmcclain1 has joined #openstack-qa | 14:49 | |
*** markmcclain1 has quit IRC | 14:49 | |
hockeynut | mtreinish https://github.com/openstack/barbican/tree/master/functionaltests | 14:49 |
*** balles has quit IRC | 14:49 | |
*** markmcclain1 has joined #openstack-qa | 14:50 | |
*** samuelbercovici has quit IRC | 14:51 | |
hockeynut | mtreinish that is the beginning - we are adding more, but want to be sure we're on the right track | 14:51 |
mtreinish | hockeynut: ok so that kinda of testing is something we're pushing to have back in the projects (right now they're in tempest) | 14:52 |
*** dmorita has joined #openstack-qa | 14:52 | |
mtreinish | but, the way you're using tempest inside barbican like that probably isn't the best idea | 14:52 |
*** markmcclain has quit IRC | 14:52 | |
mtreinish | because tempest doesn't provide any stability on it's interfaces | 14:52 |
hockeynut | mtreinish I'm open to suggestions... | 14:52 |
mtreinish | hockeynut: what you really want is: http://specs.openstack.org/openstack/qa-specs/specs/tempest-library.html | 14:53 |
mtreinish | but that's still a WIP | 14:53 |
hockeynut | mtreinish - for our info, what kind of tests are supposed to go into tempest repo? Anything we need for Barbican? | 14:53 |
*** balles has joined #openstack-qa | 14:54 | |
hockeynut | mtreinish just integration? or other? | 14:54 |
*** cdent has joined #openstack-qa | 14:55 | |
mtreinish | hockeynut: ideally long term it'll be just integration. But for right now I'm thinking that those tests maybe should be in tempest (which I'm reluctant to say especially if we're going to ask you to move them back in a cycle or 2) | 14:55 |
*** gszasz has quit IRC | 14:55 | |
mtreinish | but it really is a project level decision to a certain degree, what goes in tempest is waht you want to be co-gating when barbican is integrated | 14:55 |
mtreinish | and you still need to setup a devstack-gate job that runs tempest with barbican enabled | 14:56 |
hockeynut | mtreinish we do have that job and its working today | 14:56 |
mtreinish | well, if it runs tempest then it should be an experimental job on tempest, and any other projects which will start using barbican then | 14:57 |
hockeynut | also I have no problem with the WIP aspect - that's business as usual :-) | 14:57 |
hockeynut | right now we don't do anything w/experimental. | 14:57 |
mtreinish | hockeynut: http://git.openstack.org/cgit/openstack/tempest-lib/ well it hasn't even existed for a week yet :) | 14:57 |
*** cbader has joined #openstack-qa | 14:57 | |
mtreinish | it's only for cli testing right now, and I haven't pushed it to pypi yet | 14:58 |
*** atiwari has joined #openstack-qa | 14:58 | |
hockeynut | a week is better than nothing :-) Howsabout we stick with our current setup and when you're happy with the tempest-lib then we migrate? | 14:59 |
dkranz | sdague: I agree with the comments Matt R made in the ticket and that was the reason I pushed it to nova. I could be wrong of course. | 14:59 |
sdague | dkranz: did you see my proposed tempest side enhancement for it? | 14:59 |
*** marun has quit IRC | 15:00 | |
dkranz | sdague: yes | 15:00 |
bobj | mtreinish I'm using @test.stresstest on test_create_object, but when I ran the tempest and checked results I get test_create_object[gate,stress] OK 0.96. However I think these time is very low to a stress test execution | 15:00 |
bobj | mtreinish: I'm using @test.stresstest on test_create_object, but when I ran the tempest and checked results I get test_create_object[gate,stress] OK 0.96. However I think these time is very low to a stress test execution | 15:00 |
*** dimtruck is now known as zz_dimtruck | 15:01 | |
mtreinish | hockeynut: that's fine, but you'll still need to have something in tempest. If there are tests from your functional test suite that you think makes sense to eventually be co-gating you should put that in tempest | 15:01 |
*** k4n0 has quit IRC | 15:01 | |
hockeynut | how do you determine if a test needs to be co-gating? | 15:02 |
dkranz | sdague: It seemed to me that setUpClass and all the addCleanUps mean that the test is always ACTIVE when it starts | 15:02 |
*** afazekas_ has quit IRC | 15:03 | |
hockeynut | would that be other components who have a dependency on Barbican? | 15:03 |
dkranz | sdague: So I don't think your change is wrong but I don't see how it fixes the problem is there was one. | 15:03 |
*** zz_dimtruck is now known as dimtruck | 15:04 | |
mtreinish | hockeynut: yeah, that's one way. Another is to have a test case which involves more than just barbican to work | 15:04 |
*** balles has quit IRC | 15:04 | |
hockeynut | ok - that makes sense | 15:04 |
*** SridharG has left #openstack-qa | 15:05 | |
hockeynut | mtreinish thanks for the info - greatly appreciated!! | 15:05 |
mtreinish | bobj: if you've decoratored the test you'll need to use the run_stress command to run it as a stress test. Doing a normal test run doesn't do that | 15:05 |
mtreinish | bobj: see the stress tox job in tox.ini for an example. I also think there is something in the tempest docs about using the command | 15:05 |
sdague | dkranz: well, it would at least make it more clear to debug | 15:05 |
*** balles has joined #openstack-qa | 15:06 | |
mtreinish | hockeynut: sure np, if you need anymore help feel free to ping me | 15:06 |
hockeynut | you can bet on it :-) | 15:07 |
*** Longgeek has quit IRC | 15:07 | |
bobj | mtreinish: I'm looking a little bit more in the tempest docs and I'll try to run using tox.ini too. Thank! | 15:11 |
andreaf | mtreinish: ok thanks for your comments. I'm fine with the approach of you proposed of first making the setupclass safe and then do the staged approach | 15:11 |
andreaf | mtreinish: for the names, is something like safe_class_setup and safe_class_teardown or did you have anything specific in mind | 15:13 |
mtreinish | andreaf: no I didn't have anything in mind, I just thought that calling it setup_class wasn't really different enough | 15:15 |
*** yamahata has quit IRC | 15:15 | |
*** marun has joined #openstack-qa | 15:17 | |
andreaf | mtreinish: well I don't know it's what they are - it could be pre_tests_hook - once we go to the staged approach it will be split in several methods anyways | 15:18 |
mtreinish | andreaf: well, it was a suggestion. I was just thinking about trying to talk about it in person, how would you know which one we were talking about | 15:19 |
mtreinish | I'll leave it up to you though | 15:19 |
*** markmcclain1 has quit IRC | 15:20 | |
*** markmcclain has joined #openstack-qa | 15:20 | |
*** balles has quit IRC | 15:24 | |
*** balles has joined #openstack-qa | 15:26 | |
*** rhsu has joined #openstack-qa | 15:34 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/tempest: Support lack of ephemeral volumes for baremetal https://review.openstack.org/120814 | 15:34 |
*** atiwari has quit IRC | 15:35 | |
*** jgrimm is now known as zz_jgrimm | 15:37 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack-infra/devstack-gate: Don't use ephemeral volumes for Ironic agent driver https://review.openstack.org/120817 | 15:39 |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/tempest: Support lack of ephemeral volumes for baremetal https://review.openstack.org/120814 | 15:43 |
openstackgerrit | Tomas Sedovic proposed a change to openstack/tempest: Use the AWS instance type in the cfn_init scenario https://review.openstack.org/120749 | 15:43 |
*** cbader_ has joined #openstack-qa | 15:43 | |
openstackgerrit | John Davidge proposed a change to openstack-dev/devstack: Add IPv6 support for tenant data network https://review.openstack.org/87987 | 15:44 |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/tempest: Support lack of ephemeral volumes for baremetal https://review.openstack.org/120814 | 15:45 |
*** zz_jgrimm is now known as jgrimm | 15:48 | |
*** balles has quit IRC | 15:50 | |
*** ZZelle has quit IRC | 15:52 | |
*** ZZelle has joined #openstack-qa | 15:53 | |
*** atiwari has joined #openstack-qa | 15:53 | |
*** balles has joined #openstack-qa | 15:54 | |
*** Ch00k has joined #openstack-qa | 15:57 | |
*** bkopilov has quit IRC | 15:58 | |
openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Change LOG.exception to LOG.info for skipped tests https://review.openstack.org/120824 | 16:01 |
openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Fix safe_setup decorator to not call tearDownClass for skipped tests https://review.openstack.org/120825 | 16:01 |
openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Add unittests for safe setup decorator https://review.openstack.org/120826 | 16:01 |
cdent | dtroyer, jogo, sdague: This change https://review.openstack.org/#/c/120077/ is now passing its tests but only because nova-compute has gone back to using screen_it to start up. When using run_process, the cells job fails. | 16:02 |
cdent | I fought with it last night to try to figure out why but the late hour and my ignorance got the better of me. Any ideas/help? | 16:02 |
sdague | cdent: it works for the non cells jobs? | 16:02 |
cdent | Yup | 16:02 |
sdague | so... honestly, I'd say propose deleting the cells job at the same time | 16:02 |
sdague | it doesn't do anything useful | 16:02 |
*** jlanoux has quit IRC | 16:03 | |
*** mlavalle has joined #openstack-qa | 16:03 | |
cdent | this is the failure from before I switched it back to screen it: http://logs.openstack.org/77/120077/7/check/check-devstack-dsvm-cells/f5054ad/ | 16:04 |
cdent | I'd be curious to know what's going on because clearly there is something different between how the process is started and that difference is meaningful | 16:05 |
cdent | and potentially a clue to other broken crap | 16:05 |
cdent | that is: we have a situation now where a minor change is making a significant difference in behavior, if we can determine what that is we'll have super powers | 16:05 |
cdent | but at the moment, I'm stuck on how to dig effectively | 16:05 |
cdent | brb | 16:06 |
cdent | (note the failure happens in a local devstack too) | 16:07 |
*** e0ne has quit IRC | 16:09 | |
*** bkopilov has joined #openstack-qa | 16:12 | |
*** qba73 has quit IRC | 16:13 | |
sdague | cdent: actually, what's the version that fails cells? | 16:14 |
*** qba73 has joined #openstack-qa | 16:14 | |
*** yjiang5 has joined #openstack-qa | 16:16 | |
cdent | sorry sdague had to run something to the neighbors, what you after? | 16:17 |
sdague | I wanted to see the patch version that worked on everything except cells | 16:17 |
openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Remove query for (fixed) bug 1338844 https://review.openstack.org/120831 | 16:18 |
cdent | https://review.openstack.org/#/c/120077/7 | 16:18 |
*** jordanP has quit IRC | 16:18 | |
*** markmcclain has quit IRC | 16:19 | |
*** qba73 has quit IRC | 16:20 | |
cdent | however you can see the crux of the biscuit at line 675 here: https://review.openstack.org/#/c/120077/9/lib/nova,cm | 16:20 |
*** yjiang5 has quit IRC | 16:20 | |
cdent | it may be as simple as the directory change | 16:20 |
*** bkopilov has quit IRC | 16:20 | |
sdague | cdent: we're still missing a set of quotes there | 16:21 |
*** danfinch has quit IRC | 16:21 | |
cdent | the implication being that nova-compute is not getting the proper config? | 16:22 |
* cdent looks in logs from local tests | 16:22 | |
openstackgerrit | Tomas Sedovic proposed a change to openstack/tempest: Use the AWS instance type in the cfn_init scenario https://review.openstack.org/120749 | 16:23 |
*** rwsu has quit IRC | 16:24 | |
*** mkollaro has quit IRC | 16:26 | |
*** rwsu has joined #openstack-qa | 16:29 | |
*** yjiang5 has joined #openstack-qa | 16:31 | |
*** pkoniszewski has quit IRC | 16:32 | |
*** rwsu has quit IRC | 16:33 | |
*** yfried has quit IRC | 16:33 | |
* cdent recalls now | 16:34 | |
*** balles has quit IRC | 16:35 | |
cdent | sdague: simple quotes at that section do not work: they get out of balance as the args are passed through run_process and _run_process | 16:35 |
*** rwsu has joined #openstack-qa | 16:35 | |
cdent | you end up with a syntax error when the command is actually called | 16:35 |
*** danfinch has joined #openstack-qa | 16:36 | |
*** balles has joined #openstack-qa | 16:38 | |
openstackgerrit | John Davidge proposed a change to openstack-dev/devstack: Add IPv6 support for tenant data network https://review.openstack.org/87987 | 16:38 |
*** suneelb has joined #openstack-qa | 16:41 | |
*** jgrimm is now known as zz_jgrimm | 16:42 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Remove query for (fixed) bug 1338844 https://review.openstack.org/120831 | 16:42 |
*** ildikov has quit IRC | 16:45 | |
*** salv-orlando has joined #openstack-qa | 16:45 | |
*** jlanoux has joined #openstack-qa | 16:55 | |
*** zz_jgrimm is now known as jgrimm | 16:55 | |
*** aix has quit IRC | 16:55 | |
*** jlibosva has quit IRC | 16:58 | |
openstackgerrit | Andrea Frittoli proposed a change to openstack/qa-specs: add bp:resource-cleanup https://review.openstack.org/118352 | 17:00 |
andreaf | mtreinish, ^^^^ | 17:01 |
*** dimtruck is now known as zz_dimtruck | 17:02 | |
*** markmcclain has joined #openstack-qa | 17:02 | |
*** suneelb has quit IRC | 17:03 | |
*** suneelb has joined #openstack-qa | 17:03 | |
*** Sukhdev has joined #openstack-qa | 17:04 | |
*** suneelb has quit IRC | 17:04 | |
*** suneelb has joined #openstack-qa | 17:04 | |
bobj | Hello, I'm trying to run stress test in the object storage module using tempest, but when I ran "run-tempest-stress" an message is showed "run-tempest-stress command not found". What am I forgetting? | 17:07 |
*** harlowja_away is now known as harlowja | 17:09 | |
*** Sukhdev has quit IRC | 17:09 | |
*** chandankumar has joined #openstack-qa | 17:10 | |
*** bkopilov has joined #openstack-qa | 17:17 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 17:21 |
sdague | lifeless: so this smells very much like a fixtures bug itself - https://bugs.launchpad.net/nova/+bug/1330763 - at least I can't see how it's raisable in nova or oslo.db code | 17:24 |
sdague | also, I can't assign it to python-fixtures, because launchpad has timed out on my twice trying to do so | 17:24 |
*** masayukig has quit IRC | 17:28 | |
*** Ch00k has quit IRC | 17:30 | |
*** masayukig has joined #openstack-qa | 17:34 | |
*** Sukhdev has joined #openstack-qa | 17:35 | |
*** balles has quit IRC | 17:35 | |
*** marun has quit IRC | 17:36 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Update query for bug 1334142 https://review.openstack.org/120851 | 17:37 |
*** balles has joined #openstack-qa | 17:38 | |
*** johnthetubaguy is now known as zz_johnthetubagu | 17:39 | |
*** chandankumar has quit IRC | 17:40 | |
*** salv-orlando has quit IRC | 17:42 | |
*** aimon has quit IRC | 17:46 | |
*** openstackgerrit has quit IRC | 17:46 | |
*** zz_dimtruck is now known as dimtruck | 17:47 | |
*** openstackgerrit has joined #openstack-qa | 17:47 | |
openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Fix MismatchError for LB scenario test https://review.openstack.org/120533 | 17:51 |
andreaf | mtreinish, non-isolated jobs in experimental pipeline: https://review.openstack.org/#/c/120861/1 | 17:55 |
notmyname | dkranz: also, (so as not to consume your meeting time)...I looked at swift's internal functional tests. we're testing for specific status codes. I want to know if we change a response code with a patch. but at the same time, eg a new possibility of 2xx-series response to a PUT request isn't something that should a priori be rejected | 18:01 |
openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Replace screen_it() with run_process() throughout https://review.openstack.org/120077 | 18:01 |
*** jlanoux has quit IRC | 18:01 | |
*** dmorita has quit IRC | 18:01 | |
mtreinish | andreaf: cool, thanks | 18:02 |
mkoderer | mtreinish: about the tempest-lib.. wouldn't it a good idea to split tempest/test.py into logical pieces? | 18:02 |
dkranz | notmyname: It is a philosophical question whether a particular 2xx value is specified as part of the contract | 18:02 |
dkranz | notmyname: all I was saying is that swift takes one view and every other project takes the other view | 18:03 |
dkranz | notmyname: I am not even sure which is right | 18:03 |
notmyname | dkranz: yeah. or at least theres some tension there over what the server is expected to send vs what the client is expected to accept | 18:03 |
mtreinish | mkoderer: yeah, there is going to be a decorators class, etc | 18:03 |
mkoderer | mtreinish: yep... and we should move all the negative testing classes out | 18:03 |
mtreinish | right now I just migrated the history for everything (which was probably a mistake), but am only using the very core of the base test class | 18:03 |
mtreinish | mkoderer: yeah, feel free to do that after we push the first release out | 18:04 |
mkoderer | mtreinish: yep, I will do that | 18:05 |
mtreinish | mkoderer: the only thing is I'm trying to avoid using oslo.config for the library is that something that you can do as a refactor for the negative testing? | 18:06 |
mtreinish | I don't remember how coupled it was to config | 18:06 |
mkoderer | mtreinish: negative testing has only one config parameter | 18:07 |
mkoderer | if I remember correctly | 18:07 |
mtreinish | mkoderer: ok cool, could that be passed in as a param instead? | 18:07 |
mkoderer | mtreinish: it's just the generator... yep we can do that in a param | 18:08 |
*** dimtruck is now known as zz_dimtruck | 18:09 | |
mtreinish | ok, that's all I was concerned about. I'm trying to avoid having a tempest-lib config file if we can | 18:09 |
mkoderer | mtreinish: +1 :) | 18:09 |
mkoderer | mtreinish: but we have more configs in the stress test area | 18:09 |
mtreinish | mkoderer: yeah, I'm thinking that might become a separate tool at some point | 18:10 |
notmyname | dkranz: so the thing I'm curious about is the comment about "every other project takes the other view". I'm certainly not trying to find gratuitous differences for swift. can you tell me more what you mean by that? | 18:10 |
mkoderer | I think we should discuss about the future of stress testing anyway though | 18:10 |
openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network and secgroup resources https://review.openstack.org/100169 | 18:10 |
mtreinish | mkoderer: yeah, that's a long term discussion :) | 18:10 |
mtreinish | bobj: to use that as a cli binary you need to pip install tempest | 18:12 |
*** e0ne has joined #openstack-qa | 18:12 | |
dkranz | notmyname: Just that if you look at the docs for every other project, they specify for each api call 200 or 201, etc. One specific code. | 18:12 |
mtreinish | bobj: so from the root of the tempest repo you can do 'pip install -e .' and then the run-tempest-stress command by itself | 18:12 |
cdent | does swift say '2xx'? | 18:12 |
dkranz | notmyname: Swift specifies 2xx | 18:12 |
* cdent is surprised | 18:12 | |
dkranz | notmyname: I think there are two ways of looking at these codes in rest apis | 18:13 |
notmyname | as I said in gerrit, we're clearly not just returning random.choice(range(100)+200) for every response | 18:13 |
dkranz | 1. The 2xx value is like a return value from a normal api in any language | 18:13 |
notmyname | there are specific things we respond with, and there are specific meaning for specific codes | 18:13 |
dkranz | 2. The client should interpret any 2xx response based on the meaning (sort of) in the http spec and act accordingly | 18:14 |
dkranz | notmyname: right, that is what I called (2) | 18:14 |
*** aimon has joined #openstack-qa | 18:15 | |
dkranz | notmyname: But the other projects are also committing to not changing the actual response except in a new api version | 18:15 |
dkranz | notmyname: that is a difference in philosophy | 18:15 |
mtreinish | bobj: you could also just call the script directly without installing it. From the tempest root dir it would be: tempest/cmd/run_stress.py | 18:15 |
*** zz_dimtruck is now known as dimtruck | 18:16 | |
dkranz | notmyname: so the other project's tempest tests look like swift's internal ones | 18:16 |
notmyname | dkranz: yeah, so the question is testing as a well-behaved client should act or as a server should act (or has acted) | 18:20 |
dkranz | notmyname: My guess is that this debate never happened early on. swift did it one way and nova the other, and then others copied nova | 18:21 |
dkranz | notmyname: But I am not sure about that. | 18:21 |
notmyname | that is very similar to other things in openstack :-) | 18:21 |
dkranz | notmyname: :) | 18:22 |
dkranz | notmyname: Well, I hope we now have a shared understanding of the issue | 18:22 |
notmyname | dkranz: what's the tempest view? what should tempest be testing? the server's expectations of the clients (as the current swift patch does) or the client's expectations of the server (as you say the other projects do)? | 18:23 |
notmyname | dkranz: and how does that translate to the future in-project testing that -infra is pushing for? | 18:24 |
dkranz | notmyname: with regard to the second, it should not change anything. The semantics should depend on a decision, not on where the repo is | 18:25 |
*** dimtruck is now known as zz_dimtruck | 18:25 | |
dkranz | notmyname: But tempest does not have a view about this. I think both arguments are valid. | 18:25 |
dkranz | notmyname: And as mtreinish said, we have just gone by what the PTL of each project says | 18:25 |
*** torgomatic has joined #openstack-qa | 18:26 | |
notmyname | ok, cool | 18:26 |
mtreinish | dkranz: heh, I said that :) | 18:26 |
notmyname | just had an interesting in-office conversation about it :-) | 18:27 |
dkranz | notmyname: I would guess though, that there is a lot of client code that would break if the server started returning 204 instead of 200 | 18:27 |
dkranz | notmyname: In other words, if a server starts acting asynchronously all of a sudden I might consider that to be an api change. | 18:27 |
* cdent pages postel | 18:28 | |
notmyname | so the http spec also says that headers are case-insensitive and can be in any order. but we aren't testing that. eg we could strip out timestamp headers and then take the md5 of the response to see that _nothing_ has changed | 18:28 |
*** vigneshvar has joined #openstack-qa | 18:28 | |
notmyname | but we clearly don't do that, even though I've come across java swift clients that do in face rely on header name case sensitivity (ie Etag != ETag, therefore the client broke) | 18:28 |
notmyname | and the http spec says that clients shoudld treat 2xx as success | 18:29 |
*** zz_dimtruck is now known as dimtruck | 18:29 | |
notmyname | dkranz: I'd expect 204 to be returned from a GET if in fact it was a zero-byte object ;-) /pedant | 18:29 |
dkranz | notmyname: I was thinking of a post that creates a resource | 18:30 |
dkranz | notmyname: It really matters if the resource is ready to use when the call returns or not | 18:30 |
dkranz | notmyname: even if the client treats both cases as success | 18:30 |
notmyname | right. does "success" mean that the request was accepted or that it was completed successfully | 18:31 |
*** amotoki has quit IRC | 18:31 | |
notmyname | (there's a code for that. 202) | 18:31 |
dkranz | notmyname: Sorry, that is what I meant when I said 204 | 18:31 |
notmyname | ah ok | 18:31 |
dkranz | notmyname: my http is a little rusty :) | 18:31 |
notmyname | heh | 18:32 |
*** amotoki has joined #openstack-qa | 18:32 | |
* notmyname needs to start memorizing the new sections of the new rfcs that replaced 2616 | 18:32 | |
*** suneelb has quit IRC | 18:32 | |
*** suneelb has joined #openstack-qa | 18:32 | |
dkranz | notmyname: so I do fall more on the way other projects are doing it only in the sense that these kinds of changes might warrant a new api version anyway due to the probability of breaking clients if you went from 200 to 202 or something like that. | 18:33 |
notmyname | dkranz: then why not go all the way and look for case sensitivity and order and new headers? (/me playing devil's advocated) | 18:33 |
dkranz | notmyname: I would not be opposed to that necessarily. It is tricky to define what is an api change. | 18:34 |
*** yfried has joined #openstack-qa | 18:34 | |
dkranz | notmyname: THis is why I don't like trying to slavishly map an http spec onto api type signatures | 18:35 |
notmyname | right. which is where my point comes in that the rfc says that clients should tread any 2xx response as a success. and yes particular 2xx codes have particular meanings :-) | 18:35 |
dkranz | notmyname: But that is kind of what we do | 18:35 |
cdent | Tests aren't for testing success are they, they are for testing the contract | 18:35 |
cdent | that's not the same thing | 18:35 |
cdent | (I'm sorry to horn in on you guys, but this is fascinating and interesting) | 18:35 |
dkranz | cdent: Yes, but it is a philosophical discussion about what the contract should actually be | 18:36 |
dkranz | cdent: and we have two different examples in OpenStack | 18:36 |
notmyname | cdent: yes, but the whole this is the questions of "what is the contract". that's just another word for "api change" | 18:36 |
cdent | I don't really see it that way: A test should test whatever the current reality is | 18:36 |
notmyname | cdent: and yes, I too think this is fascinating and interestion :-) | 18:36 |
cdent | when that reality changes, it should fail | 18:36 |
cdent | that's _separate_ from "is this an api change" | 18:36 |
*** amotoki has quit IRC | 18:36 | |
notmyname | cdent: that I completely agree with | 18:37 |
dkranz | cdent: I think the issue is that every code patch changes something, otherwise you would not do it. The question is what kind of changes should clients expect? | 18:37 |
notmyname | cdent: as shown by how the in-tree functional tests for swift are written | 18:37 |
dkranz | notmyname: IMO, an actual server change from 200 to 202 in anger would break a lot of client code, even if it should not have. | 18:38 |
*** chandankumar has joined #openstack-qa | 18:38 | |
cdent | 200 to 202 _should_ break client code | 18:39 |
dkranz | cdent: not if the code looks at the response and doesn't to use the resource until it becomes available in the 202 case | 18:39 |
dkranz | cdent: if the contract is "returns 200" then you can just use it. | 18:40 |
cdent | If the server sent 200 in response to resource creation request it's busted | 18:40 |
dkranz | cdent: If the contract is "returns 2xx" then you have to check and act accordingly. | 18:40 |
cdent | 201, 202 or 204 | 18:40 |
cdent | but not 200 | 18:40 |
cdent | bad contract | 18:41 |
dkranz | cdent: sorry I meant 201 | 18:41 |
notmyname | cdent: do you mean that in the sense that 200 shouldn't be returned for resource creation? (I agree) | 18:41 |
dkranz | notmyname: that was my mistake (again) | 18:42 |
cdent | yes notmyname | 18:42 |
notmyname | heh | 18:42 |
notmyname | dkranz: go lurn you some http! | 18:42 |
dkranz | notmyname: I'll say sync vs async instead | 18:42 |
cdent | https is hard, let's go rpc-ing | 18:42 |
notmyname | here's a different way for me to phrase things with relation to the current proposed patch | 18:43 |
notmyname | I'm actually not opposed to tempest testing for specific codes. I think that means that some patch to swift in the future might be harder, but things that change codes are (intentionally) very rare. but, I prefer tempest to check for response code classes since that's what I expect clients to use. and I _never_ want a swift patch blocked because a tempest reviewer things swift should implement a new api version | 18:45 |
notmyname | while it may be that a new api version is needed, that shoudl happen with swift contributors | 18:45 |
openstackgerrit | Matthew Treinish proposed a change to openstack/qa-specs: Move implemented bps to the implemented dir https://review.openstack.org/120879 | 18:46 |
notmyname | on the other hand, I do think that docs should talk about response code classes and not specific codes (and there's a patch to docs to do that now) | 18:46 |
cdent | It sounds like you are trying to work around a problem in tempest's responsiveness... | 18:47 |
notmyname | cdent: no, not at all | 18:47 |
notmyname | I'm not wanting to come across that way | 18:47 |
cdent | (which would be understandable) | 18:47 |
notmyname | I think I actually very much agree with what you said that the tests of reality are separate from "is this an api change" | 18:48 |
notmyname | cdent: my point is only that I think the swift devs are the ones to make the call on "is this an api change", and I think that tempest checking for the same constraints as the client is expected to have makes those conversations happen in the right place | 18:49 |
notmyname | in no way am I complaining about tempest or tempest devs. I'm actually really happy to see how much for swift has been added into tempest recently :-) | 18:50 |
cdent | I'm not familiar with the mechanics of those conversations (too new to the scene). | 18:50 |
dkranz | cdent, notmyname : The bottom line is that tempest checks for what the spec (docs) says should be returned | 18:51 |
notmyname | cdent: honestly, I'm not sure they've happened too often yet. the only example I can think of is the comment in gerrit on the current patch where a reviewer set -1 | 18:51 |
dkranz | That is determined by projects and that is not going to change | 18:52 |
notmyname | dkranz: that, in fact, is a very clear standard :-) | 18:52 |
notmyname | fun having these conversations in a text-based medium, huh? it would be much better in person :-) | 18:52 |
cdent | Just out of curiosity, not quite on topic, how would you, notmyname, expect a client developer to know about and/or deal with a creation request that responded with 202? | 18:52 |
cdent | (given that you've already said client code would think in terms of 2xx) | 18:53 |
*** vigneshvar has quit IRC | 18:53 | |
*** __afazekas is now known as afazekas | 18:53 | |
notmyname | cdent: well, to be honest, there are some creation requests in swift that _do_ respond with 202. but the difference is that in swift we don't provision things that you use. instead, we only return success when your data is durably persisted to disk | 18:53 |
afazekas | mtreinish: https://review.openstack.org/#/c/120386/ | 18:54 |
notmyname | cdent: or more specifically, you could get 201 or 202 | 18:54 |
cdent | in what situations does a 202 happen? | 18:54 |
*** suneelb has quit IRC | 18:54 | |
*** suneelb has joined #openstack-qa | 18:55 | |
notmyname | cdent: PUT request to a container that has already been created | 18:56 |
afazekas | cdent: after a 202 you usually need to do polling | 18:56 |
cdent | afazekas: yes, I know, I'm just trying imagine how 202 syncs with "we only return success when your data is durably persisted" and "need to do polling" | 18:57 |
cdent | s/imagine/understand/ | 18:57 |
*** jecarey has quit IRC | 18:58 | |
notmyname | we return 2xx because the resource (the container) is available right then. ie you can still read a container | 18:58 |
afazekas | 202 usually used with the server actions | 18:58 |
notmyname | dkranz: heh. temptest tests what the docs say. the docs patch (https://review.openstack.org/#/c/120622/) just got a -2 because the tempest code hasn't landed :-) | 18:59 |
cdent | notmyname: in the example you're thinking of what (if anything) is in the headers and body of the response that has the 202 status? | 19:00 |
cdent | (not trying to find holes here, just trying to expand my understanding) | 19:00 |
dkranz | notmyname: more evidence this discussion never happened... | 19:00 |
*** melwitt has joined #openstack-qa | 19:01 | |
notmyname | cdent: making a paste for you | 19:01 |
cdent | cool, thanks very much | 19:01 |
andreaf | mtreinish, sdague have you ever seen this before: http://logs.openstack.org/85/107685/31/check/gate-tempest-dsvm-neutron-large-ops/8157620/logs/tempest.txt.gz?level=WARNING | 19:01 |
notmyname | cdent: https://gist.github.com/notmyname/6254035176dd3deb17ae | 19:02 |
* cdent read | 19:02 | |
cdent | s | 19:02 |
*** jecarey has joined #openstack-qa | 19:03 | |
afazekas | BTW: which code is used for http://docs.openstack.org/developer/swift/overview_container_sync.html ? | 19:03 |
notmyname | afazekas: implementing that feature or generating those docs? | 19:04 |
*** vigneshvar has joined #openstack-qa | 19:06 | |
*** dtroyer has quit IRC | 19:06 | |
*** adam_g has quit IRC | 19:06 | |
*** StevenK has quit IRC | 19:06 | |
*** lbragstad has quit IRC | 19:07 | |
*** dtroyer has joined #openstack-qa | 19:08 | |
*** fungi has quit IRC | 19:09 | |
*** freyes has quit IRC | 19:09 | |
*** freyes has joined #openstack-qa | 19:10 | |
*** hockeynut has quit IRC | 19:10 | |
*** lbragstad has joined #openstack-qa | 19:10 | |
cdent | notmyname: is the second container put at line 47 effectively a no op? | 19:11 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Update query for bug 1334142 https://review.openstack.org/120851 | 19:11 |
notmyname | cdent: internally, I think it might update a timestamp | 19:12 |
*** StevenK_ has joined #openstack-qa | 19:12 | |
*** hockeynut has joined #openstack-qa | 19:12 | |
cdent | 202 feels wrong to me there. I'm not sure why. | 19:13 |
cdent | There is no future thing that you expect to happen. | 19:13 |
cdent | Would 204 be appropriate? | 19:14 |
*** Guest75250 has quit IRC | 19:14 | |
afazekas | notmyname: forget it, wrong example.. | 19:15 |
mtreinish | andreaf: personally I've never really noticed it before | 19:15 |
mtreinish | andreaf: but according to logstash it happens all the time | 19:15 |
notmyname | cdent: maybe. and we're not returning 202 for an object overwrite either. I think this is along the lines of not looking to change response codes :-) | 19:15 |
cdent | :) | 19:15 |
notmyname | and FWIW if you POSt to a container (eg with new metadata), you'll get a 201 | 19:16 |
cdent | I'd think that was a 204 too | 19:16 |
cdent | (if it already exists) | 19:16 |
*** dtroyer has quit IRC | 19:17 | |
notmyname | cdent: from rackspace cloud files docs from 2008: "No content is returned. A status code of 201 (Created) indicates that the Container was created as requested. Container PUTs are idempotent and a code of 202 (Accepted) is returned when the Container already existed." | 19:17 |
notmyname | ;-) | 19:17 |
cdent | but yeah, once people start expecting specific codes things get hard to change | 19:17 |
*** StevenK_ has quit IRC | 19:17 | |
*** rhsu has quit IRC | 19:17 | |
*** StevenK_ has joined #openstack-qa | 19:17 | |
andreaf | mtreinish, ok so probably a red herring | 19:18 |
andreaf | mtreinish, thanks | 19:18 |
notmyname | right. which comes back to the discussion around which change means it's an api change or not :-) | 19:18 |
cdent | indeed it does | 19:18 |
cdent | that's why I was trying to tease out the expectations on the 202 | 19:18 |
cdent | because if it was an async thing requring a followup poll, then 202 is part of the contract: it tells you want to do | 19:19 |
*** freyes has quit IRC | 19:19 | |
*** lbragstad has quit IRC | 19:19 | |
*** jroll has quit IRC | 19:19 | |
cdent | but at the moment it pretty much just says "you didn't break it, carry on" | 19:19 |
notmyname | that may be the only place you can get 202 from swift. not sure. maybe also if you PUT to an account that exists (if you have those permissions) | 19:19 |
*** nsaje_ has joined #openstack-qa | 19:19 | |
notmyname | cdent: yeah. the async stuff could be interpreted that if you do a read and get a response it might not be the version that you just PUT | 19:19 |
notmyname | cdent: in which case 202 makes more sense | 19:20 |
mtreinish | andreaf: are you looking at a specific failure? | 19:20 |
notmyname | cdent: whereas with a new container, if you make a read request and get data, you'll have the right version (thus the 201 response to the create) | 19:20 |
*** hockeynut has quit IRC | 19:21 | |
* notmyname goes downstairs to pick up stuff at a food truck | 19:21 | |
*** hockeynut has joined #openstack-qa | 19:21 | |
andreaf | mtreinish, large opts tests seems to be failing all the time on the last patch of test accounts | 19:21 |
* cdent needs food too | 19:22 | |
andreaf | mtreinish, https://review.openstack.org/#/c/107685/ | 19:22 |
*** StevenK_ has quit IRC | 19:22 | |
andreaf | mtreinish, but I fail to see any relationship between the change and the failures | 19:22 |
*** StevenK_ has joined #openstack-qa | 19:23 | |
andreaf | mtreinish, I'll try yet another recheck | 19:23 |
*** lbragstad has joined #openstack-qa | 19:23 | |
*** chandankumar has quit IRC | 19:23 | |
mtreinish | andreaf: yeah, it looks like it's going into an error state | 19:24 |
mtreinish | jogo: when you figure out the broken migration stuff ^^^ | 19:24 |
*** StevenK_ has quit IRC | 19:27 | |
*** lbragstad has quit IRC | 19:28 | |
*** vigneshvar has quit IRC | 19:28 | |
*** StevenK_ has joined #openstack-qa | 19:28 | |
jogo | mtreinish: ? | 19:28 |
*** markmcclain has quit IRC | 19:28 | |
mtreinish | a large ops job failure | 19:29 |
mtreinish | jogo: http://logs.openstack.org/85/107685/31/check/gate-tempest-dsvm-neutron-large-ops/8157620/console.html#_2014-09-08_02_56_22_273 | 19:30 |
*** hockeynut has quit IRC | 19:30 | |
*** hockeynut has joined #openstack-qa | 19:30 | |
jogo | mtreinish: http://logs.openstack.org/85/107685/31/check/gate-tempest-dsvm-neutron-large-ops/8157620/logs/screen-n-cpu.txt.gz?level=TRACE#_2014-09-08_02_55_52_890 | 19:31 |
jogo | mtreinish: neutron is out of fixed IPs | 19:31 |
mtreinish | oh, yeah that would do it | 19:31 |
*** StevenK_ has quit IRC | 19:33 | |
jogo | not sure how that happened | 19:33 |
*** fungi has joined #openstack-qa | 19:33 | |
*** stevebaker has quit IRC | 19:33 | |
*** StevenK_ has joined #openstack-qa | 19:33 | |
*** lbragstad has joined #openstack-qa | 19:34 | |
*** Sukhdev has quit IRC | 19:35 | |
*** sc68cal_ has joined #openstack-qa | 19:36 | |
*** lbragstad has quit IRC | 19:37 | |
*** mlavalle_ has joined #openstack-qa | 19:37 | |
*** jecarey has quit IRC | 19:38 | |
andreaf | mtreinish, jogo thanks for your help, I'll continue digging into this tomorroq | 19:38 |
*** StevenK_ has quit IRC | 19:38 | |
*** jecarey has joined #openstack-qa | 19:38 | |
andreaf | tomorrow | 19:38 |
*** yamahata__ has joined #openstack-qa | 19:39 | |
*** StevenK_ has joined #openstack-qa | 19:39 | |
*** hockeynut has quit IRC | 19:39 | |
*** hockeynut has joined #openstack-qa | 19:40 | |
*** sc68cal has quit IRC | 19:40 | |
*** mlavalle has quit IRC | 19:40 | |
*** yamahata_ has quit IRC | 19:40 | |
*** mriedem1 has joined #openstack-qa | 19:40 | |
*** mriedem has quit IRC | 19:40 | |
*** nsaje_ is now known as nsaje | 19:42 | |
jogo | andreaf: thanks | 19:42 |
jogo | can you make sure a bug is filed for that | 19:42 |
jogo | andreaf mtreinish: for e-r tracking | 19:42 |
mtreinish | jogo: yeah I'll do that now | 19:43 |
jogo | mtreinish: thanks | 19:43 |
jogo | mtreinish: just dealt with https://bugs.launchpad.net/nova/+bug/1368391 | 19:43 |
*** StevenK_ has quit IRC | 19:44 | |
*** StevenK_ has joined #openstack-qa | 19:44 | |
*** dims has quit IRC | 19:44 | |
*** vigneshvar has joined #openstack-qa | 19:45 | |
*** Dafna has quit IRC | 19:45 | |
openstackgerrit | Anderson Mesquita proposed a change to openstack/tempest: Refactor stack actions and add stack check https://review.openstack.org/106456 | 19:46 |
mtreinish | jogo: hmm, actually maybe not. It's only shown up on 2 check queue changes, andreaf's cred provider patch and https://review.openstack.org/#/c/120070 | 19:46 |
mtreinish | so I'm not sure it's a real bug | 19:47 |
jogo | mtreinish: cool glad its no agate bug | 19:47 |
jogo | we have enough of those today | 19:47 |
*** hockeynut has quit IRC | 19:48 | |
*** StevenK_ has quit IRC | 19:49 | |
*** StevenK_ has joined #openstack-qa | 19:50 | |
*** mgagne has joined #openstack-qa | 19:51 | |
*** jroll has joined #openstack-qa | 19:51 | |
*** mestery has quit IRC | 19:54 | |
*** mestery has joined #openstack-qa | 19:54 | |
*** weshay has quit IRC | 20:01 | |
*** dimtruck is now known as zz_dimtruck | 20:02 | |
*** lbragstad has joined #openstack-qa | 20:02 | |
*** dims_ has joined #openstack-qa | 20:03 | |
*** rhsu has joined #openstack-qa | 20:03 | |
*** markmcclain has joined #openstack-qa | 20:10 | |
openstackgerrit | Benny Kopilov proposed a change to openstack/tempest: compute-image : create server from snapshot image https://review.openstack.org/120899 | 20:12 |
*** yfried has quit IRC | 20:13 | |
*** weshay has joined #openstack-qa | 20:15 | |
*** masayukig has quit IRC | 20:17 | |
*** dims__ has joined #openstack-qa | 20:17 | |
*** e0ne has quit IRC | 20:19 | |
*** dims_ has quit IRC | 20:20 | |
*** masayukig has joined #openstack-qa | 20:21 | |
*** dims__ has quit IRC | 20:22 | |
*** flaper87 is now known as flaper87|afk | 20:24 | |
*** zz_dimtruck is now known as dimtruck | 20:25 | |
bobj | mtreinish: Thank for your support. | 20:25 |
mtreinish | bobj: so you got it working? | 20:26 |
bobj | mtreinish: Not yet, I'm researching how do it better. | 20:30 |
bobj | mtreinish: I'll configure one json file to test just object storage module. Thank you again, your comments were very important. | 20:33 |
*** e0ne has joined #openstack-qa | 20:45 | |
*** e0ne has quit IRC | 20:45 | |
*** dimtruck is now known as zz_dimtruck | 20:48 | |
openstackgerrit | Benny Kopilov proposed a change to openstack/tempest: compute-image : create server from saved image saved image can be reused by new servers https://review.openstack.org/120899 | 20:51 |
*** stevebaker has joined #openstack-qa | 20:55 | |
*** cdent has quit IRC | 20:57 | |
*** zz_dimtruck is now known as dimtruck | 20:59 | |
*** markmcclain has quit IRC | 20:59 | |
*** mestery has quit IRC | 21:00 | |
*** enikanorov_ has quit IRC | 21:00 | |
*** enikanorov_ has joined #openstack-qa | 21:00 | |
*** mestery has joined #openstack-qa | 21:00 | |
*** mwagner_lap has quit IRC | 21:04 | |
*** toure has quit IRC | 21:06 | |
*** bobj has quit IRC | 21:10 | |
*** vigneshvar has quit IRC | 21:11 | |
*** aysyd has quit IRC | 21:12 | |
*** dustins has quit IRC | 21:13 | |
lifeless | sdague: looking | 21:14 |
*** torgomatic has left #openstack-qa | 21:16 | |
*** adam_g has joined #openstack-qa | 21:26 | |
*** rfolco has quit IRC | 21:26 | |
*** adam_g has quit IRC | 21:27 | |
*** adam_g has joined #openstack-qa | 21:27 | |
*** nsaje has quit IRC | 21:27 | |
*** cbader_ has quit IRC | 21:29 | |
*** cbader has quit IRC | 21:29 | |
*** dkranz has quit IRC | 21:30 | |
*** catohornet1 has joined #openstack-qa | 21:32 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/tempest: Support lack of ephemeral volumes for baremetal https://review.openstack.org/120814 | 21:37 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove query for bug 1347692 https://review.openstack.org/120916 | 21:42 |
lifeless | sdague: pushing a possible patch up now | 21:45 |
lifeless | sdague: https://review.openstack.org/120919 | 21:45 |
sdague | lifeless: ok, cool, appreciated | 21:46 |
lifeless | sdague: added a little more commentary in the bug | 21:47 |
*** StevenK_ is now known as StevenK | 21:47 | |
lifeless | sdague: I need to context switch today sorry, but it should be clear, I hope :) | 21:47 |
sdague | sure, I'll early propose that to nova | 21:47 |
sdague | because I don't think oslo has any testing infrastructure to expose that | 21:48 |
lifeless | sdague: the change is in oslo.incubator | 21:48 |
lifeless | sdague: oh, you mean its not tested locally to oslo.incubator? | 21:48 |
*** jecarey has quit IRC | 21:48 | |
lifeless | sdague: so yeah, it will need some shenanigans to verify it works etc. | 21:48 |
*** atiwari has quit IRC | 21:50 | |
sdague | yeh, honestly, I'm fine violating normal oslo rules here | 21:54 |
sdague | because I don't think that will get us to a sane fix | 21:54 |
*** qba73 has joined #openstack-qa | 21:55 | |
lifeless | sdague: in case its not obvious this is actually a secondary issue, its not the cause of the failure | 21:56 |
sdague | lifeless: sure | 21:56 |
lifeless | sdague: the postgresql db is reporting available but isn't actually connectable to | 21:56 |
sdague | but we can't see the base issue | 21:56 |
lifeless | yup | 21:56 |
lifeless | I'm going to concurrently fix that aspect in testtools | 21:56 |
lifeless | which is fully tested :) | 21:56 |
sdague | appreciated | 21:56 |
lifeless | but first, context switching | 21:56 |
lifeless | since lynne just crashed the car | 21:56 |
sdague | oh, bummer | 21:57 |
sdague | take care | 21:57 |
lifeless | shes fine | 21:57 |
lifeless | but the light at the riccarton cinema parking lot, and the car bonnet... not so much | 21:57 |
*** mriedem1 has quit IRC | 21:59 | |
*** weshay has quit IRC | 22:06 | |
*** mwagner_lap has joined #openstack-qa | 22:09 | |
*** dims_ has joined #openstack-qa | 22:10 | |
*** jhenner has quit IRC | 22:11 | |
*** dims_ has quit IRC | 22:14 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove query for bug 1357677 https://review.openstack.org/120925 | 22:17 |
*** weshay has joined #openstack-qa | 22:19 | |
*** gfidente has quit IRC | 22:20 | |
*** cnesa9 has quit IRC | 22:20 | |
*** andreaf has quit IRC | 22:23 | |
*** weshay has quit IRC | 22:25 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Install PyYaml for test-matrix.py https://review.openstack.org/120929 | 22:30 |
*** gordc has quit IRC | 22:32 | |
*** jgrimm is now known as zz_jgrimm | 22:32 | |
*** qba73 has quit IRC | 22:33 | |
*** qba73 has joined #openstack-qa | 22:34 | |
*** openstackgerrit has quit IRC | 22:38 | |
*** openstackgerrit_ has joined #openstack-qa | 22:38 | |
*** qba73 has quit IRC | 22:39 | |
*** openstackgerrit_ is now known as openstackgerrit | 22:40 | |
*** dims_ has joined #openstack-qa | 22:44 | |
*** dimtruck is now known as zz_dimtruck | 22:54 | |
*** jpich has joined #openstack-qa | 22:59 | |
*** ianw has quit IRC | 23:04 | |
*** ianw has joined #openstack-qa | 23:04 | |
*** rbak has quit IRC | 23:06 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Install PyYaml for test-matrix.py https://review.openstack.org/120929 | 23:28 |
*** dims_ has quit IRC | 23:34 | |
*** oomichi has joined #openstack-qa | 23:37 | |
openstackgerrit | David Lyle proposed a change to openstack/tempest: Skipping dashboard test to unblock gate https://review.openstack.org/120938 | 23:46 |
openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Fix Ironic agent ramdisk URL https://review.openstack.org/120939 | 23:46 |
jroll | ianw, quick bugfix if you have a sec, this is my fault: https://review.openstack.org/120939 | 23:47 |
*** dims_ has joined #openstack-qa | 23:52 | |
openstackgerrit | David Lyle proposed a change to openstack-dev/devstack: Moving horizon compression offline https://review.openstack.org/120940 | 23:53 |
*** chuckC has joined #openstack-qa | 23:53 | |
*** dims__ has joined #openstack-qa | 23:55 | |
*** suneelb has quit IRC | 23:55 | |
*** suneelb has joined #openstack-qa | 23:55 | |
*** aix has joined #openstack-qa | 23:56 | |
*** dims_ has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!