| *** gszasz has joined #openstack-qa | 00:08 | |
| *** gszasz has quit IRC | 00:09 | |
| *** wuhg has joined #openstack-qa | 00:09 | |
| *** gszasz has joined #openstack-qa | 00:10 | |
| *** andersonvom has quit IRC | 00:15 | |
| *** gszasz has quit IRC | 00:16 | |
| *** zz_dimtruck is now known as dimtruck | 00:21 | |
| *** gmann has quit IRC | 00:22 | |
| *** gmann has joined #openstack-qa | 00:22 | |
| *** dmorita has joined #openstack-qa | 00:26 | |
| *** openstackgerrit has quit IRC | 00:31 | |
| *** openstackgerrit has joined #openstack-qa | 00:32 | |
| *** melwitt has quit IRC | 00:33 | |
| *** gszasz has joined #openstack-qa | 00:33 | |
| *** gszasz has quit IRC | 00:33 | |
| *** rhsu has quit IRC | 00:34 | |
| *** gszasz has joined #openstack-qa | 00:38 | |
| *** andersonvom has joined #openstack-qa | 00:41 | |
| *** gszasz has joined #openstack-qa | 00:42 | |
| *** gszasz has quit IRC | 00:44 | |
| *** mwagner_ has joined #openstack-qa | 00:45 | |
| *** mwagner_ has quit IRC | 00:46 | |
| *** andersonvom has quit IRC | 00:46 | |
| *** gszasz has joined #openstack-qa | 00:46 | |
| *** gszasz has quit IRC | 00:50 | |
| *** gszasz has joined #openstack-qa | 00:52 | |
| *** liusheng has quit IRC | 00:55 | |
| *** gszasz has quit IRC | 00:56 | |
| *** yjiang5 has quit IRC | 00:57 | |
| *** JayJ____ has joined #openstack-qa | 00:58 | |
| *** Longgeek has joined #openstack-qa | 01:00 | |
| *** mkollaro1 has quit IRC | 01:04 | |
| *** Sukhdev has quit IRC | 01:11 | |
| *** rodrigods_ has quit IRC | 01:13 | |
| *** suneelb has quit IRC | 01:26 | |
| *** andreaf has quit IRC | 01:48 | |
| *** catohornet has quit IRC | 01:48 | |
| *** fungi has quit IRC | 01:48 | |
| *** cyeoh has quit IRC | 01:48 | |
| *** mitz has quit IRC | 01:48 | |
| *** zhiyan has quit IRC | 01:48 | |
| *** jamielennox|away has quit IRC | 01:48 | |
| *** jogo has quit IRC | 01:48 | |
| *** zz_johnthetubagu has quit IRC | 01:48 | |
| *** liusheng has joined #openstack-qa | 01:49 | |
| *** JayJ____ has quit IRC | 01:49 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-dev/devstack: Install tempest-lib as part of install_tempest https://review.openstack.org/119850 | 01:52 |
|---|---|---|
| *** mestery has quit IRC | 01:53 | |
| *** mestery has joined #openstack-qa | 01:53 | |
| *** dims has joined #openstack-qa | 01:53 | |
| *** andreaf has joined #openstack-qa | 01:53 | |
| *** fungi has joined #openstack-qa | 01:54 | |
| *** cyeoh has joined #openstack-qa | 01:54 | |
| *** mitz has joined #openstack-qa | 01:54 | |
| *** zhiyan has joined #openstack-qa | 01:54 | |
| *** jamielennox|away has joined #openstack-qa | 01:54 | |
| *** jogo has joined #openstack-qa | 01:54 | |
| *** zz_johnthetubagu has joined #openstack-qa | 01:54 | |
| *** mitz has quit IRC | 01:54 | |
| *** mitz has joined #openstack-qa | 01:55 | |
| *** dims is now known as Guest9788 | 01:55 | |
| *** pcrews has quit IRC | 01:55 | |
| *** sweston has quit IRC | 01:57 | |
| *** sweston has joined #openstack-qa | 01:59 | |
| *** JayJ____ has joined #openstack-qa | 02:00 | |
| *** mestery has quit IRC | 02:04 | |
| *** mestery has joined #openstack-qa | 02:04 | |
| *** yjiang5 has joined #openstack-qa | 02:13 | |
| *** Poornima_ has joined #openstack-qa | 02:16 | |
| *** pcrews has joined #openstack-qa | 02:20 | |
| *** pcrews has quit IRC | 02:20 | |
| *** JayJ____ has quit IRC | 02:22 | |
| *** yamamoto has quit IRC | 02:23 | |
| *** harlowja is now known as harlowja_away | 02:32 | |
| *** amotoki has quit IRC | 02:38 | |
| *** andersonvom has joined #openstack-qa | 02:42 | |
| *** amotoki has joined #openstack-qa | 02:45 | |
| *** liusheng has quit IRC | 02:46 | |
| *** andersonvom has quit IRC | 02:46 | |
| *** yamamoto has joined #openstack-qa | 02:47 | |
| *** Guest9788 has quit IRC | 03:12 | |
| *** amotoki has quit IRC | 03:12 | |
| *** andersonvom has joined #openstack-qa | 03:37 | |
| *** Poornima_ has quit IRC | 03:48 | |
| *** dimtruck is now known as zz_dimtruck | 03:53 | |
| *** torandu has joined #openstack-qa | 04:03 | |
| *** gmann has quit IRC | 04:14 | |
| *** Poornima_ has joined #openstack-qa | 04:14 | |
| *** amotoki has joined #openstack-qa | 04:15 | |
| *** yfried|afk has quit IRC | 04:20 | |
| *** swamireddy has joined #openstack-qa | 04:24 | |
| *** gmann has joined #openstack-qa | 04:26 | |
| *** Ch00k has joined #openstack-qa | 04:32 | |
| *** SridharG has joined #openstack-qa | 04:35 | |
| *** chandankumar has joined #openstack-qa | 04:40 | |
| *** rhsu has joined #openstack-qa | 04:58 | |
| *** chandankumar has quit IRC | 05:05 | |
| *** Longgeek has quit IRC | 05:07 | |
| *** Longgeek has joined #openstack-qa | 05:19 | |
| *** Longgeek_ has joined #openstack-qa | 05:20 | |
| *** Longgeek has quit IRC | 05:23 | |
| *** Longgeek_ has quit IRC | 05:24 | |
| *** e0ne has joined #openstack-qa | 05:28 | |
| *** vigneshvar has joined #openstack-qa | 05:28 | |
| *** Longgeek has joined #openstack-qa | 05:31 | |
| *** e0ne has quit IRC | 05:43 | |
| openstackgerrit | Ian Wienand proposed a change to openstack-infra/devstack-gate: Always set DATA_DIR https://review.openstack.org/122064 | 05:45 |
| *** yfried|afk has joined #openstack-qa | 05:46 | |
| *** ildikov has joined #openstack-qa | 05:47 | |
| *** Qarekhani has joined #openstack-qa | 05:55 | |
| *** swamireddy has quit IRC | 06:00 | |
| *** bkopilov has quit IRC | 06:03 | |
| *** swamireddy has joined #openstack-qa | 06:05 | |
| *** Ch00k has quit IRC | 06:06 | |
| *** Madan has joined #openstack-qa | 06:08 | |
| *** rhsu has quit IRC | 06:16 | |
| *** nmagnezi has joined #openstack-qa | 06:18 | |
| *** chandankumar has joined #openstack-qa | 06:19 | |
| *** yfried|afk is now known as yfried | 06:19 | |
| *** flaper87|afk is now known as flaper87 | 06:20 | |
| yfried | masayukig: https://review.openstack.org/102700 could you please take a look? | 06:22 |
| *** k4n0 has joined #openstack-qa | 06:26 | |
| *** jyuso has joined #openstack-qa | 06:27 | |
| *** andreaf has quit IRC | 06:29 | |
| yfried | salv-orlando: ping | 06:30 |
| *** rdekel has joined #openstack-qa | 06:30 | |
| yfried | k4n0: https://review.openstack.org/#/c/106326 you've got some interesting comment there. did you check this out? | 06:31 |
| *** andersonvom has quit IRC | 06:31 | |
| *** evgenyf has joined #openstack-qa | 06:31 | |
| k4n0 | yfried, fixed them in patch 20 | 06:32 |
| yfried | k4n0: why aren't you using "addCleanup"? | 06:35 |
| k4n0 | yfried, I am using the teardown of BaseNetworkTest | 06:36 |
| k4n0 | yfried, the resources are cleaned up correctly | 06:36 |
| k4n0 | yfried, but it seems the "check-tempest-dsvm-neutron-icehouse" job is failing | 06:37 |
| yfried | k4n0: I'm not sure about this, since I'm not familiar, but I prefer using addCleanup. it's a project-wide method and it's easier to test/fix/update | 06:38 |
| k4n0 | yfried, Actually, these lines cleanup any networking resources we create during tests https://github.com/openstack/tempest/blob/master/tempest/api/network/base.py#L138 | 06:38 |
| *** rdekel has quit IRC | 06:39 | |
| yfried | k4n0: I understand. but this means we have network tests implementing their own cleanup structure. this could be better or worse than other stuff, but it's unique and that's not good, IMO. | 06:40 |
| yfried | k4n0: I know you are just aligning with the rest of the network tree. I was just wondering if you knew why it's like that | 06:41 |
| *** tosky has joined #openstack-qa | 07:03 | |
| *** mpaolino has joined #openstack-qa | 07:06 | |
| swamireddy | cyeoh: Hi - could you please review the patch - https://review.openstack.org/#/c/115944/ | 07:07 |
| *** pkoniszewski has joined #openstack-qa | 07:10 | |
| *** e0ne has joined #openstack-qa | 07:12 | |
| *** gszasz has joined #openstack-qa | 07:15 | |
| *** gszasz has quit IRC | 07:15 | |
| *** gszasz has joined #openstack-qa | 07:17 | |
| *** gszasz has quit IRC | 07:20 | |
| *** gszasz has joined #openstack-qa | 07:21 | |
| *** jlibosva has joined #openstack-qa | 07:22 | |
| *** e0ne has quit IRC | 07:23 | |
| *** e0ne has joined #openstack-qa | 07:25 | |
| swamireddy | cyeoh: Thanks for review...But to combine 2 tests in one could be a problem because: If reboot_instnace fails - the test report shows that, reboot and terminate test failed, so I have separeted. Please let me know if this ok combine in this case | 07:27 |
| *** cmyster_away is now known as cmyster | 07:27 | |
| *** bkopilov has joined #openstack-qa | 07:28 | |
| cyeoh | swamireddy: that's fine we'd just have one test - test_run_reboot_terminate_instance. Can work out from the test output which part actually failed | 07:28 |
| *** luqas has joined #openstack-qa | 07:29 | |
| yfried | cyeoh: You're core tempest, right? | 07:32 |
| cyeoh | yfried: yes | 07:32 |
| *** qba73 has joined #openstack-qa | 07:33 | |
| yfried | cyeoh: https://review.openstack.org/#/c/119670/3 https://review.openstack.org/#/c/113807/7 | 07:33 |
| yfried | cyeoh: these 2 need +A. I believe they are ready to merge, and they are part of an almost-done bp | 07:33 |
| yfried | cyeoh: could you please give them the final push? | 07:34 |
| cyeoh | yfried: I'll have a look | 07:34 |
| yfried | cyeoh: tnx | 07:34 |
| yfried | cyeoh: if we have two reviewer giving it +1, do we need 2 more cores to give it +2,+A ? | 07:35 |
| yfried | cyeoh: or is a core's +A enough? | 07:35 |
| cyeoh | yfried: we need 2 +2's (doesn't matter how many +1's there are) before a core will +A a patch | 07:36 |
| *** tosky has quit IRC | 07:36 | |
| cyeoh | but the core doing the second +2 will most likely +A it at the same time unless they want someone else to look at it for some reason | 07:37 |
| yfried | cyeoh: so +1s don't mean anything? | 07:38 |
| cyeoh | yfried: no, they're certainly helpful because cores know the patches have already been reviewed | 07:39 |
| yfried | cyeoh: where is this policy written? Is this a Tempest thing, or is it the same all across openstack? | 07:39 |
| cyeoh | yfried: and speaking personally I know I get to know non core reviewers after a while and build up trust in those who pick up issues | 07:39 |
| cyeoh | yfried: but ultimately is the -1s (from anyone) that is the most valuable | 07:40 |
| cyeoh | yfried: it is standard practice across openstack | 07:40 |
| cyeoh | most projects have some exceptions - eg if a patch is +A'd at some point but ends up needing a rebase then one +2 is sufficient to +A it again | 07:40 |
| yfried | cyeoh: tnx. so were is it documented? I'm not complaining, I just figured I'm a little vague on the review process | 07:41 |
| cyeoh | it might be on wiki.openstack.org, but if it is I don't know where (it should be on there if it isn't) | 07:41 |
| cyeoh | yfried: so those two patches you referred to earlier on. I don't understand why it was split into two patches | 07:43 |
| yfried | cyeoh: me neither. I guess gmann was hoping the first would be merged quickly, and the 2nd would require some arguments | 07:45 |
| yfried | cyeoh: the 1st was due to some requests/bugs. the 2nd is part of a bp | 07:45 |
| yfried | cyeoh: does it really matter at this moment? | 07:46 |
| cyeoh | yfried: can put more load on the gate | 07:46 |
| openstackgerrit | Michele Paolino proposed a change to openstack-dev/devstack: Support for libvirt/QEMU tar releases https://review.openstack.org/108714 | 07:47 |
| cyeoh | yfried: more risk with two patches that one will get a spurious error in the gate and get rejected | 07:47 |
| yfried | cyeoh: well, squashing them now would probably take us over the weekend. | 07:48 |
| yfried | cyeoh: merging them now before the US guys wake up and start pushing is relatively low-risk. (I hope) and we could always do "reverify" | 07:49 |
| cyeoh | ok the gate queue is really short now so it shouldn't matter | 07:49 |
| yfried | cyeoh: tnx. my point | 07:50 |
| *** Longgeek has quit IRC | 07:50 | |
| yfried | cyeoh: also, since you're at it - https://review.openstack.org/#/c/102700/ | 07:50 |
| * yfried is shamelessly self-promoting | 07:50 | |
| *** Albert_13 has joined #openstack-qa | 07:51 | |
| cmyster | yfried: its OK... I'm used to you being shameless | 07:52 |
| *** gfidente has joined #openstack-qa | 07:52 | |
| cyeoh | yfried: I'll get to the other one is a bit | 07:52 |
| *** Longgeek has joined #openstack-qa | 07:56 | |
| *** Longgeek has quit IRC | 07:57 | |
| *** yamamoto has quit IRC | 07:57 | |
| yfried | cyeoh: masayukig: andreaf_: https://review.openstack.org/#/c/121149/ | 07:57 |
| gmann | cyeoh: yfried: yes, those could have been merged together but i separated bcz it can be explicitly seen to have large_ops test should be derived from scenario base class tests which can go through Nova/Nova proxy | 07:58 |
| *** Longgeek has joined #openstack-qa | 07:59 | |
| gmann | cyeoh: yfried: currently scenario network base class became for neutron only tests. So i separated those patch not to be have confusion in case anyone think to change the base class for those tests | 07:59 |
| *** Qarekhani has quit IRC | 07:59 | |
| *** yamahata has quit IRC | 07:59 | |
| gmann | otherwise those changes could have been mixed up with client migration changes :) | 08:00 |
| cyeoh | gmann: ok, thanks | 08:02 |
| yfried | gmann: it was already for neutron only. I just made it clearer | 08:07 |
| gmann | yfried: yes. but large ops test was derived from it and using very base class function not network base class. But with your change it will be clean/clear for tests which one to use as base. | 08:09 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 08:11 | |
| *** jordanP has joined #openstack-qa | 08:14 | |
| *** tnurlygayanov has quit IRC | 08:16 | |
| *** tnurlygayanov has joined #openstack-qa | 08:16 | |
| *** yamamoto has joined #openstack-qa | 08:19 | |
| *** tnurlygayanov has quit IRC | 08:23 | |
| *** tnurlygayanov has joined #openstack-qa | 08:24 | |
| yfried | jordanP: ping | 08:24 |
| jordanP | yfried, pong | 08:25 |
| *** safchain has joined #openstack-qa | 08:25 | |
| yfried | jordanP: about https://review.openstack.org/#/c/117454/ | 08:26 |
| yfried | jordanP: this is part of a series of status checks. | 08:26 |
| jordanP | yfried, sure, but I just don't know why we should wait for the port to be active after a router has been added to a subnet | 08:27 |
| yfried | jordanP: so, what do you suggest? | 08:28 |
| yfried | jordanP: we have to check this feature | 08:28 |
| jordanP | if the test needs this, then the test should call itself wait_for_port_status() | 08:28 |
| jordanP | but the wait_for_port_status() should not be done in the add_to_router() method | 08:28 |
| *** Dafna has joined #openstack-qa | 08:29 | |
| yfried | jordanP: so the alternative is for a specific test to check it, but we would miss other scenarios were it might fail. | 08:30 |
| jordanP | what might fail ? | 08:31 |
| yfried | jordanP: how a about identifying all waiters (VM, port, floating ip, vip, etc) and have a gate without them (to check for those races) | 08:31 |
| yfried | jordanP: I can't predict failures (if I could, we wouldn't need Tempest :)) | 08:31 |
| jordanP | yfried, to be honest, imo, there are already to many gates | 08:31 |
| yfried | jordanP: we have a full neutron gate, and we have many other (that use neutron as well). we can make sure that the full-neutron (which is non-voting ATM) is running without waiters | 08:33 |
| yfried | jordanP: if we make sure that network_basic_ops is waiting for status, we might miss failures in hotplug_nic | 08:33 |
| yfried | jordanP: ^ I'm talking about checking the VM ports status now | 08:34 |
| jordanP | but the main point is, why do we need to wait ? | 08:34 |
| jordanP | some tests currently does'nt need to wait the port is active, why make them wait now ? | 08:35 |
| nmagnezi | jordanP, because the port can become active only after some time (usually it happens very fast) | 08:35 |
| yfried | jordanP: what he said... :) | 08:35 |
| jordanP | nmagnezi, what's the problem with the port becoming activer *after* ? Those some tests fail because of this ? Does it make impossible to write reliable tests ? | 08:36 |
| *** yamahata has joined #openstack-qa | 08:37 | |
| *** yamamoto has quit IRC | 08:37 | |
| yfried | jordanP: when/how would you suggest that we test for the port status then? Keep in mind that we want to test all ports' status (GW, interfaces, VMs) | 08:39 |
| *** amotoki has quit IRC | 08:41 | |
| *** luqas has quit IRC | 08:41 | |
| *** jlanoux has joined #openstack-qa | 08:43 | |
| *** cdent has joined #openstack-qa | 08:43 | |
| yfried | anyone knows what's up with mulit-node gates? I know there was some work going on. I wonder where it's at now | 08:44 |
| jordanP | yfried, we can't wait for a port to be active if there's no reason to do so. If a test needs a port to be active to go further/test other stuff then this test should wait and call wait_for_port_status. That's just my opinion, i'd like to be wrong | 08:46 |
| swamireddy | cyeoh: OK...Its fine with me..I will combine the reboot_instance test in run_terminate test...and submit the patch... | 08:48 |
| yfried | jordanP: the point of this patch-series is to verify that port status is reported correctly (ports are becoming active when created) https://bugs.launchpad.net/neutron/+bug/1253634 | 08:52 |
| jordanP | yfried, this bug seems inalid to me :) but that's not the point anyway | 08:53 |
| yfried | jordanP: how would you suggest to check it, other than wait for it to become active? we could however, give it a very small timeout. | 08:53 |
| yfried | jordanP: well, if we don't care about port status, we should remove this from the API. if it's there, then we need to test it is correct | 08:54 |
| yfried | jordanP: I'm not arguing against you. I'm asking you how to solve your concern, while still test that this feature is real | 08:55 |
| *** amotoki has joined #openstack-qa | 08:57 | |
| *** ylobankov has quit IRC | 08:58 | |
| *** ylobankov has joined #openstack-qa | 08:58 | |
| jordanP | yfried, if you want a test that tests that this feature is real, that would be great, but write just one/1 test to do so. Don't force all tests to test this indirectly | 09:01 |
| *** andreaf_ is now known as andreaf | 09:01 | |
| *** bobj has quit IRC | 09:02 | |
| *** Poornima_ has quit IRC | 09:08 | |
| *** yfried is now known as yfried|afk | 09:09 | |
| openstackgerrit | A change was merged to openstack/tempest: large ops test should be derived from OfficialClientTest https://review.openstack.org/119670 | 09:09 |
| openstackgerrit | varun kumar yadav proposed a change to openstack/tempest: Test to update port with CIDR value https://review.openstack.org/113861 | 09:10 |
| *** luqas has joined #openstack-qa | 09:13 | |
| *** cnesa12 has joined #openstack-qa | 09:13 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Using credentials of users created on-the-fly https://review.openstack.org/115279 | 09:16 |
| *** yamamoto has joined #openstack-qa | 09:19 | |
| *** yamamoto has quit IRC | 09:19 | |
| *** yamamoto has joined #openstack-qa | 09:19 | |
| openstackgerrit | A change was merged to openstack/tempest: Migrate test_large_ops to tempest client https://review.openstack.org/113807 | 09:22 |
| *** tkammer has joined #openstack-qa | 09:23 | |
| *** jpich has quit IRC | 09:25 | |
| cdent | sdague, dtroyer, jogo: Is there a plan or procedure for getting this stuff happening in the right places in the gate? https://review.openstack.org/#/c/120077/ | 09:27 |
| *** swamireddy has quit IRC | 09:31 | |
| *** vigneshvar has quit IRC | 09:38 | |
| *** swamireddy has joined #openstack-qa | 09:39 | |
| *** Ch00k has joined #openstack-qa | 09:42 | |
| *** qba73_ has joined #openstack-qa | 09:42 | |
| *** qba73_ has quit IRC | 09:42 | |
| *** qba73 has quit IRC | 09:45 | |
| *** mkollaro has joined #openstack-qa | 09:56 | |
| *** aix has joined #openstack-qa | 09:57 | |
| *** yamahata has quit IRC | 09:59 | |
| *** Ch00k has quit IRC | 10:00 | |
| openstackgerrit | Gonéri Le Bouder proposed a change to openstack-dev/devstack: Debian: use libvirt-bin when needed https://review.openstack.org/122113 | 10:01 |
| *** yfried|afk is now known as yfried | 10:05 | |
| *** jyuso has quit IRC | 10:06 | |
| *** evgenyf has quit IRC | 10:14 | |
| *** psedlak has joined #openstack-qa | 10:22 | |
| *** swamireddy1 has joined #openstack-qa | 10:27 | |
| *** qba73 has joined #openstack-qa | 10:27 | |
| *** swamireddy has quit IRC | 10:30 | |
| *** dims has joined #openstack-qa | 10:37 | |
| *** dims has quit IRC | 10:39 | |
| *** dims has joined #openstack-qa | 10:39 | |
| *** mwagner_lap has quit IRC | 10:50 | |
| openstackgerrit | Roman Bogorodskiy proposed a change to openstack/tempest: Allow to customize volume type extra specs https://review.openstack.org/118619 | 10:51 |
| *** ildikov_ has joined #openstack-qa | 10:53 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: don't use screen for grenade https://review.openstack.org/100229 | 10:54 |
| sdague | cdent: that's the switch, lets see if it works | 10:54 |
| openstackgerrit | Roman Bogorodskiy proposed a change to openstack/tempest: Allow to customize volume type extra specs https://review.openstack.org/118619 | 10:54 |
| *** ildikov has quit IRC | 10:55 | |
| *** amotoki has quit IRC | 10:57 | |
| *** yamahata has joined #openstack-qa | 11:06 | |
| *** salv-orlando_ has joined #openstack-qa | 11:10 | |
| *** dmorita has quit IRC | 11:12 | |
| *** salv-orlando has quit IRC | 11:12 | |
| *** salv-orlando_ is now known as salv-orlando | 11:12 | |
| yfried | salv-orlando: could you please take a look https://review.openstack.org/#/c/102700/23 ? | 11:13 |
| *** JayJ____ has joined #openstack-qa | 11:15 | |
| *** salv-orlando has quit IRC | 11:16 | |
| *** salv-orlando has joined #openstack-qa | 11:17 | |
| *** salv-orlando has quit IRC | 11:17 | |
| *** JayJ____ has quit IRC | 11:19 | |
| *** Poornima_ has joined #openstack-qa | 11:22 | |
| *** salv-orlando has joined #openstack-qa | 11:25 | |
| *** salv-orlando has quit IRC | 11:25 | |
| *** salv-orlando has joined #openstack-qa | 11:32 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: Fix race condition waiting for VM to start https://review.openstack.org/122125 | 11:42 |
| *** Madan has quit IRC | 11:45 | |
| *** weshay has joined #openstack-qa | 11:46 | |
| *** evgenyf has joined #openstack-qa | 11:47 | |
| *** SridharG has quit IRC | 11:50 | |
| *** weshay has quit IRC | 11:50 | |
| yfried | salv-orlando: tnx for the review | 11:55 |
| yfried | salv-orlando: what's your opinion on the "return self" issue? | 11:55 |
| salv-orlando | yfried: I think it’s acceptable. Ideally you would like to have a DeletableXXX object as a wrapper around an actual model object. I’m not sure if that’s what maru is referring to, but it surely something that can’t be implemented in this patch. | 11:56 |
| yfried | salv-orlando: what do you mean "wrapper around an actual model object"? | 11:58 |
| *** Poornima_ has quit IRC | 11:58 | |
| *** mmedvede has joined #openstack-qa | 11:59 | |
| *** adalbas has joined #openstack-qa | 12:01 | |
| *** Albert_13 has quit IRC | 12:02 | |
| salv-orlando | yfried: basically that get in a DeletableFloatingIP instance would probably return a floatingIP. But nvm, that probably does not make sense anyway | 12:05 |
| yfried | salv-orlando: what do you mean? self IS a floatingIP | 12:06 |
| salv-orlando | yfried: right. That’s why I said that this problably does not make a lot sense because the DeletableFloatingIP is a floating ip object from the tempest perspective. | 12:07 |
| yfried | salv-orlando: what other perspective is there? do you mean a neutron FLIP object? | 12:08 |
| salv-orlando | perhaps marun is just thinking that a get an object which then returns the object itself is a bit strange, let’s what for him. | 12:08 |
| salv-orlando | I have absolutely no idea of how marun thinks this should be done. | 12:08 |
| yfried | salv-orlando: AFAIK, a get method should both update the instance and return it | 12:09 |
| salv-orlando | yfried: I am probably a bit old, but when I went to school they thaught me that getters are not supposed to change the state of an object (but that’s probably off topic as I think the question from marun is about the ‘return self’ bit) | 12:11 |
| yfried | salv-orlando: it's not really a "getter". it's more of an updater - it fetches the object from db. | 12:12 |
| yfried | salv-orlando: AFAIK, this methods should always return self. my issue is, that there is no cost here, is there? it's not like we have a better return value, so why is objectionable? | 12:13 |
| *** k4n0 has quit IRC | 12:14 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 12:14 |
| salv-orlando | yfried: as I said, I don’t know. I can only speculate on reasons. Frankly thing like getters that change state and object methods that return the object itself are in my opinion questionable. Point is however that this discussion cannot apply to your patch as I reckon that the way tempest’s deletable resources have been designed to be used. For more details, you should wait for marun which raised the issue. | 12:16 |
| *** JayJ____ has joined #openstack-qa | 12:16 | |
| salv-orlando | In my review I just limited myself to ask for additional feedback from him! | 12:16 |
| yfried | salv-orlando: tnx. I was just hoping you'd provide some insight, cause as you said you are "a bit old"er than me :) | 12:18 |
| salv-orlando | yfried: my insights are that if a getter changes the status of an object, might lead to unexpected behaviours. if you have a “user” resource you won’t expect user.get() to change its name, would you? Also, returning self on a get creates a situation where you can do instance.get().get().get() …. and keep returning the same object an infinite number of times. This is not wrong, but a bit strange. | 12:19 |
| salv-orlando | yfried: my other consideration is that there’s very little you can do about these insights in your patch. | 12:20 |
| salv-orlando | and that’s the reason why I did not bother question your apporach. | 12:20 |
| *** cdent_ has joined #openstack-qa | 12:23 | |
| *** cdent has quit IRC | 12:24 | |
| *** cdent_ is now known as cdent | 12:24 | |
| *** Longgeek has quit IRC | 12:27 | |
| *** marun_ has joined #openstack-qa | 12:27 | |
| *** mwagner_lap has joined #openstack-qa | 12:32 | |
| *** mriedem has joined #openstack-qa | 12:33 | |
| *** Ch00k has joined #openstack-qa | 12:34 | |
| *** aysyanne has joined #openstack-qa | 12:35 | |
| sdague | cdent: it looks like there is still something wrong with shutdown | 12:35 |
| sdague | http://logs.openstack.org/29/100229/2/check/check-grenade-dsvm/07d9624/logs/grenade.sh.txt.gz | 12:35 |
| *** e0ne has quit IRC | 12:36 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: WIP - Adds network to large scale scenario https://review.openstack.org/120070 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate network API tests to resource_* fixtures https://review.openstack.org/121541 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate identity API tests to resource_* fixtures https://review.openstack.org/121540 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate database API tests to resource_* fixtures https://review.openstack.org/121543 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate baremetal API tests to resource_* fixtures https://review.openstack.org/121542 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Framework for resource safe class level fixtures https://review.openstack.org/121537 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate computev3 API tests to resource_* fixtures https://review.openstack.org/121539 | 12:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate computev2 API tests to resource_* fixtures https://review.openstack.org/121538 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate telemetry API tests to resource_* fixtures https://review.openstack.org/121549 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate queuing API tests to resource_* fixtures https://review.openstack.org/121548 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate scenario API tests to resource_* fixtures https://review.openstack.org/121551 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate volume API tests to resource_* fixtures https://review.openstack.org/121550 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate image API tests to resource_* fixtures https://review.openstack.org/121545 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate data_processing API tests to resource_* fixtures https://review.openstack.org/121544 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate orchestration API tests to resource_* fixtures https://review.openstack.org/121547 | 12:37 |
| *** Ch00k has quit IRC | 12:37 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate object_storage API tests to resource_* fixtures https://review.openstack.org/121546 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate thirdparty API tests to resource_* fixtures https://review.openstack.org/121552 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate cli API tests to resource_* fixtures https://review.openstack.org/121553 | 12:37 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Hacking rule to forbid resource unsafe fixtures https://review.openstack.org/121554 | 12:37 |
| *** chandankumar has quit IRC | 12:41 | |
| yfried | salv-orlando: andreaf: could you please take a look, https://review.openstack.org/120070 | 12:41 |
| *** Longgeek has joined #openstack-qa | 12:41 | |
| yfried | it's WIP but I think it's important. I'd appricaite your early review. | 12:42 |
| *** markmcclain has joined #openstack-qa | 12:42 | |
| *** yfried has quit IRC | 12:42 | |
| mriedem | mtreinish: sdague: jogo: are we considering this the same as bug 1349617? the error is different: http://logs.openstack.org/01/116001/3/gate/gate-grenade-dsvm-partial-ncpu/3dce5d7/logs/tempest.txt.gz#_2014-09-16_22_32_14_888 | 12:42 |
| mriedem | no route to host vs "Error reading SSH protocol banner[Errno 104] Connection reset by peer" | 12:43 |
| cdent | is there a bug associated with that shutdown problem sdague ? | 12:43 |
| sdague | cdent: I don't think so | 12:43 |
| sdague | mriedem: that might just be a new paramiko release changing messages? | 12:44 |
| mriedem | sdague: we've been on the latest paramiko for about 3 weeks now | 12:45 |
| mriedem | so wouldn't think it's that | 12:45 |
| *** dims has quit IRC | 12:45 | |
| sdague | mriedem: ok, possibly | 12:45 |
| mriedem | trying to get a decent logstash query for it | 12:45 |
| *** dims has joined #openstack-qa | 12:45 | |
| *** weshay has joined #openstack-qa | 12:46 | |
| mriedem | no route to host shows up in successful runs too until it makes the connection, so trying to find the failures | 12:46 |
| mriedem | here we go: message:"_get_ssh_connection" AND message:"error: [Errno 113] No route to host" AND tags:"tempest.txt" | 12:48 |
| *** e0ne has joined #openstack-qa | 12:48 | |
| mriedem | 63 hits in 10 days, the other ssh one is 29 hits in 10 days, so looks like a separate bug | 12:49 |
| mriedem | maybe same root cause, not sure | 12:49 |
| *** Albert_13 has joined #openstack-qa | 12:50 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1370496 | 12:52 |
| mriedem | i'll get an e-r query up | 12:52 |
| salv-orlando | mriedem: which one is “the other one”? 1349617? | 12:54 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for ssh 'no route to host' bug 1370496 https://review.openstack.org/122140 | 12:55 |
| mriedem | salv-orlando: yup | 12:55 |
| salv-orlando | “no route to host” might be a sympton of dropbear sshd failing to start. In some neutron jobs (which print also console output) I noticed cirros starting in readonly mode, and sshd failing to start. I have a bug report for that. looking for it. | 12:57 |
| mriedem | ok, i didn't get into the console dump much, in meetings most of the morning and out this afternoon | 12:57 |
| salv-orlando | mriedem: this it the bug I filed. https://bugs.launchpad.net/neutron/+bug/1362528 - not sure if that’s happening for the grenade job too however. | 13:00 |
| *** aix has quit IRC | 13:00 | |
| swamireddy1 | cyeoh: cann´´t use the run_terminate test case to include the reboot_instance, because, reboot cann´´t be called/used if the current state is running | 13:00 |
| swamireddy1 | cyeoh: alternatively we can include the reboot_instance test in test_run_stop_terminate_instance as test_run_stop_reboot_terminate_instance | 13:01 |
| mriedem | salv-orlando: doesn't appear to be an issue here, i'm not seeing any messages about a read-only file system | 13:02 |
| swamireddy1 | Is this could be fine to include the reboot_instance test test_run_stop_terminate_instance? | 13:02 |
| swamireddy1 | cyeoh: Is this could be fine to include the reboot_instance test test_run_stop_terminate_instance? | 13:02 |
| *** qba73 has quit IRC | 13:06 | |
| salv-orlando | mriedem: yeah. This thing about readonly file system is also fairly rate. | 13:09 |
| *** amotoki has joined #openstack-qa | 13:12 | |
| *** dkranz has joined #openstack-qa | 13:12 | |
| *** sballe has joined #openstack-qa | 13:17 | |
| *** mpaolino has quit IRC | 13:19 | |
| *** mpaolino has joined #openstack-qa | 13:21 | |
| *** dustins has joined #openstack-qa | 13:22 | |
| *** dkranz has quit IRC | 13:24 | |
| *** aix has joined #openstack-qa | 13:27 | |
| *** SridharG has joined #openstack-qa | 13:28 | |
| *** pcrews has joined #openstack-qa | 13:30 | |
| mtreinish | mriedem: have you seen this before: http://logs.openstack.org/62/119862/1/check/check-tempest-dsvm-full-icehouse/e7c3e2f/logs/screen-n-cpu.txt.gz#_2014-09-16_18_09_38_140 | 13:30 |
| *** Longgeek has quit IRC | 13:31 | |
| mtreinish | from what I can tell soft reboot failed, it switches to doing a hard reboot which pushes it over the wait timeout in tempest which fails the test | 13:31 |
| mriedem | mtreinish: i thought the soft reboot tests were skipped? | 13:31 |
| mriedem | since we didn't know if the soft reboot worked or if we did a hard reboot | 13:32 |
| mriedem | i've been meaning to write up a spec in nova for that for kilo | 13:32 |
| mriedem | https://bugs.launchpad.net/tempest/+bug/1014647 | 13:32 |
| mtreinish | mriedem: hmm, it looks like it was explicitly set to soft during the tempest client migration | 13:33 |
| mriedem | PhilD also has the graceful shutdown stuff in nova now....not sure if that would play here - maybe extending the timeout | 13:33 |
| mriedem | mtreinish: ok that is probably a bug | 13:33 |
| *** marun_ has quit IRC | 13:34 | |
| mtreinish | it just called server.reboot() before the migration | 13:34 |
| mriedem | you could refer to 1014647 in the change | 13:34 |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_minimum_basic.py#n89 | 13:35 |
| *** boden has joined #openstack-qa | 13:35 | |
| mriedem | that should be HARD | 13:35 |
| mriedem | want me to push a change? | 13:35 |
| *** sballe has quit IRC | 13:35 | |
| mtreinish | mriedem: hmm, but the novaclient default was SOFT, which is what we were using before... | 13:35 |
| mtreinish | mriedem: sure you can go ahead and do that | 13:35 |
| mriedem | oh | 13:35 |
| mriedem | mtreinish: well then i'd blame the graceful shutdown somehow | 13:36 |
| mriedem | it's probably so gd graceful that it makes tempest timeout | 13:36 |
| mtreinish | mriedem: https://git.openstack.org/cgit/openstack/python-novaclient/tree/novaclient/v1_1/servers.py#n248 | 13:36 |
| mtreinish | well that failure was from an icehouse run | 13:36 |
| mriedem | oh right | 13:36 |
| mriedem | so....probably a latent issue | 13:37 |
| mriedem | ? | 13:37 |
| *** SridharG has quit IRC | 13:37 | |
| *** dkranz has joined #openstack-qa | 13:37 | |
| mtreinish | yeah, that's what I'm thinking | 13:37 |
| *** Longgeek has joined #openstack-qa | 13:37 | |
| *** jecarey has joined #openstack-qa | 13:38 | |
| mriedem | mtreinish: test_list_instance_actions also does a HARD reboot specifically | 13:39 |
| mriedem | and test_reboot_server_soft is skipped for that bug | 13:39 |
| *** marun_ has joined #openstack-qa | 13:40 | |
| *** marun_ has quit IRC | 13:40 | |
| mtreinish | mriedem: I'm fine with changing it to hard there too, just leave a note that mentions the bug number so it's clear | 13:41 |
| mriedem | sure | 13:42 |
| *** Longgeek has quit IRC | 13:43 | |
| openstackgerrit | A change was merged to openstack/tempest: Migrate test_encrypted_cinder_volumes to tempest client https://review.openstack.org/111905 | 13:43 |
| *** bknudson has joined #openstack-qa | 13:43 | |
| *** gszasz has quit IRC | 13:45 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Change test_minimum_basic nova_reboot from SOFT to HARD https://review.openstack.org/122156 | 13:45 |
| *** JayJ____ has quit IRC | 13:45 | |
| openstackgerrit | Swami Reddy proposed a change to openstack/tempest: Add "reboot_instance" EC2 API test https://review.openstack.org/115944 | 13:48 |
| *** swamireddy1 has quit IRC | 13:51 | |
| *** SridharG has joined #openstack-qa | 13:53 | |
| *** gszasz has joined #openstack-qa | 13:55 | |
| *** gszasz has quit IRC | 13:58 | |
| *** freyes has joined #openstack-qa | 14:02 | |
| *** yamahata has quit IRC | 14:03 | |
| *** yamahata has joined #openstack-qa | 14:03 | |
| *** bkopilov has quit IRC | 14:05 | |
| *** nmagnezi has quit IRC | 14:06 | |
| *** dkranz has quit IRC | 14:09 | |
| *** marun_ has joined #openstack-qa | 14:09 | |
| *** pkoniszewski has quit IRC | 14:09 | |
| *** rbak has joined #openstack-qa | 14:11 | |
| jroll | any tempest folks have a moment to look at a quick change? :) https://review.openstack.org/#/c/120814/ | 14:11 |
| jroll | (it's got 1 +2 already) | 14:11 |
| andreaf | jroll, looking | 14:15 |
| jroll | thanks :) | 14:15 |
| andreaf | jroll, +A | 14:17 |
| jroll | thanks andreaf :D | 14:17 |
| * jroll is one step closer to CI for his driver :P | 14:17 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 14:18 |
| *** dkranz has joined #openstack-qa | 14:22 | |
| *** zz_dimtruck is now known as dimtruck | 14:23 | |
| *** gszasz has joined #openstack-qa | 14:24 | |
| *** gszasz has quit IRC | 14:24 | |
| *** gszasz has joined #openstack-qa | 14:25 | |
| *** andersonvom has joined #openstack-qa | 14:27 | |
| *** andersonvom has quit IRC | 14:28 | |
| *** zz_jgrimm is now known as jgrimm | 14:28 | |
| *** wuhg has quit IRC | 14:36 | |
| *** sweston has quit IRC | 14:36 | |
| *** zhiyan has quit IRC | 14:36 | |
| *** atiwari has joined #openstack-qa | 14:41 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Using credentials of users created on-the-fly https://review.openstack.org/115279 | 14:43 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Select AuthProvider type from credentials type https://review.openstack.org/121562 | 14:43 |
| *** markmcclain has quit IRC | 14:45 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: WIP - Javelin: enable volume resources https://review.openstack.org/121648 | 14:45 |
| *** zhiyan has joined #openstack-qa | 14:48 | |
| *** sweston has joined #openstack-qa | 14:51 | |
| *** wuhg has joined #openstack-qa | 14:51 | |
| *** Poornima_ has joined #openstack-qa | 14:53 | |
| *** marun_ has quit IRC | 14:55 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate network API tests to resource_* fixtures https://review.openstack.org/121541 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate identity API tests to resource_* fixtures https://review.openstack.org/121540 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate database API tests to resource_* fixtures https://review.openstack.org/121543 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate baremetal API tests to resource_* fixtures https://review.openstack.org/121542 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Framework for resource safe class level fixtures https://review.openstack.org/121537 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate computev3 API tests to resource_* fixtures https://review.openstack.org/121539 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate computev2 API tests to resource_* fixtures https://review.openstack.org/121538 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate telemetry API tests to resource_* fixtures https://review.openstack.org/121549 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate queuing API tests to resource_* fixtures https://review.openstack.org/121548 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate scenario API tests to resource_* fixtures https://review.openstack.org/121551 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate volume API tests to resource_* fixtures https://review.openstack.org/121550 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate image API tests to resource_* fixtures https://review.openstack.org/121545 | 14:59 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate data_processing API tests to resource_* fixtures https://review.openstack.org/121544 | 15:00 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate orchestration API tests to resource_* fixtures https://review.openstack.org/121547 | 15:00 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate object_storage API tests to resource_* fixtures https://review.openstack.org/121546 | 15:00 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate thirdparty API tests to resource_* fixtures https://review.openstack.org/121552 | 15:00 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate cli API tests to resource_* fixtures https://review.openstack.org/121553 | 15:00 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Hacking rule to forbid resource unsafe fixtures https://review.openstack.org/121554 | 15:00 |
| *** ildikov_ has quit IRC | 15:00 | |
| mtreinish | andreaf: ^^^ you really like spamming that patch series :) | 15:00 |
| andreaf | :D | 15:02 |
| *** mpaolino has quit IRC | 15:03 | |
| andreaf | so that everyone will go and review them to stop seeing them :P | 15:03 |
| *** beagles has joined #openstack-qa | 15:05 | |
| mtreinish | andreaf: I pushed out a fix for your patch: https://review.openstack.org/#/c/120861/ it should work now | 15:12 |
| *** vigneshvar has joined #openstack-qa | 15:13 | |
| *** Poornima_ has quit IRC | 15:14 | |
| *** swamireddy has joined #openstack-qa | 15:18 | |
| *** e0ne has quit IRC | 15:19 | |
| *** evgenyf has quit IRC | 15:20 | |
| *** jasondotstar has joined #openstack-qa | 15:21 | |
| *** SridharG has left #openstack-qa | 15:25 | |
| andreaf | mtreinish: thanks | 15:25 |
| *** e0ne has joined #openstack-qa | 15:26 | |
| andreaf | mtreinish: if you have a sec: https://review.openstack.org/#/c/121562/2 - this fixes an issue in auth code thus enabling development of identity v3 scenarios | 15:27 |
| *** aix has quit IRC | 15:28 | |
| *** dhellmann is now known as dhellmann_ | 15:32 | |
| mtreinish | andreaf: +2, but you might want to fix the commit message if you don't want to manually update the bug status | 15:33 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Using credentials of users created on-the-fly https://review.openstack.org/115279 | 15:36 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Select AuthProvider type from credentials type https://review.openstack.org/121562 | 15:36 |
| andreaf | mtreinish: I fixed the commit message ^^^ | 15:36 |
| andreaf | mtreinish, and lost your +2 | 15:36 |
| openstackgerrit | A change was merged to openstack/tempest: Support lack of ephemeral volumes for baremetal https://review.openstack.org/120814 | 15:37 |
| *** yamahata has quit IRC | 15:40 | |
| *** aix has joined #openstack-qa | 15:40 | |
| *** vigneshvar has quit IRC | 15:41 | |
| *** gordc has joined #openstack-qa | 15:44 | |
| *** mlavalle has joined #openstack-qa | 15:46 | |
| *** vigneshvar has joined #openstack-qa | 15:54 | |
| *** Sukhdev has joined #openstack-qa | 15:55 | |
| *** markmcclain has joined #openstack-qa | 15:56 | |
| *** chandankumar has joined #openstack-qa | 15:58 | |
| *** Ch00k has joined #openstack-qa | 15:59 | |
| *** tkammer has quit IRC | 16:01 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Deprecate API_RATE_LIMIT setting for Tempest runs https://review.openstack.org/116001 | 16:04 |
| *** jlanoux has quit IRC | 16:06 | |
| *** vigneshvar has quit IRC | 16:06 | |
| *** aimon has quit IRC | 16:08 | |
| *** mestery has quit IRC | 16:09 | |
| *** mestery has joined #openstack-qa | 16:10 | |
| *** dhellmann_ is now known as dhellmann | 16:11 | |
| *** chandankumar has quit IRC | 16:15 | |
| *** swamireddy has quit IRC | 16:18 | |
| *** swamireddy has joined #openstack-qa | 16:19 | |
| *** hemna__ is now known as hemna | 16:23 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate TestStampPattern to tempest client https://review.openstack.org/122196 | 16:23 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Unskip TestStampPattern for testing purposes https://review.openstack.org/122197 | 16:23 |
| *** vigneshvar has joined #openstack-qa | 16:26 | |
| *** vigneshvar has quit IRC | 16:35 | |
| *** bnemec has quit IRC | 16:37 | |
| *** andreaf is now known as andreaf_ | 16:38 | |
| *** swamireddy has quit IRC | 16:39 | |
| *** bnemec has joined #openstack-qa | 16:40 | |
| *** aimon has joined #openstack-qa | 16:41 | |
| *** swamireddy has joined #openstack-qa | 16:45 | |
| *** safchain has quit IRC | 16:47 | |
| *** vigneshvar has joined #openstack-qa | 16:49 | |
| *** Ch00k has quit IRC | 16:52 | |
| *** luqas has quit IRC | 16:53 | |
| *** Ch00k has joined #openstack-qa | 16:53 | |
| *** Albert_13 has quit IRC | 16:53 | |
| *** rhsu has joined #openstack-qa | 16:54 | |
| *** dkranz has quit IRC | 17:00 | |
| *** mriedem has quit IRC | 17:00 | |
| *** salv-orlando has quit IRC | 17:00 | |
| *** mkollaro has quit IRC | 17:00 | |
| *** sc68cal has quit IRC | 17:00 | |
| *** ianw has quit IRC | 17:00 | |
| *** dkranz has joined #openstack-qa | 17:00 | |
| *** mriedem has joined #openstack-qa | 17:00 | |
| *** salv-orlando has joined #openstack-qa | 17:00 | |
| *** mkollaro has joined #openstack-qa | 17:00 | |
| *** sc68cal has joined #openstack-qa | 17:00 | |
| *** ianw has joined #openstack-qa | 17:00 | |
| *** rhsu has quit IRC | 17:00 | |
| *** aimon has quit IRC | 17:00 | |
| *** mestery has quit IRC | 17:00 | |
| *** gszasz has quit IRC | 17:00 | |
| *** cdent has quit IRC | 17:00 | |
| *** rwsu has quit IRC | 17:00 | |
| *** cmyster has quit IRC | 17:00 | |
| *** akuznetsova has quit IRC | 17:00 | |
| *** rhsu has joined #openstack-qa | 17:02 | |
| *** aimon has joined #openstack-qa | 17:02 | |
| *** mestery has joined #openstack-qa | 17:02 | |
| *** gszasz has joined #openstack-qa | 17:02 | |
| *** cdent has joined #openstack-qa | 17:02 | |
| *** rwsu has joined #openstack-qa | 17:02 | |
| *** cmyster has joined #openstack-qa | 17:02 | |
| *** akuznetsova has joined #openstack-qa | 17:02 | |
| *** cmyster has quit IRC | 17:02 | |
| *** cmyster has joined #openstack-qa | 17:02 | |
| *** Sukhdev has quit IRC | 17:02 | |
| *** e0ne has quit IRC | 17:02 | |
| *** Dafna has quit IRC | 17:02 | |
| *** harlowja_away has quit IRC | 17:02 | |
| *** rfolco has quit IRC | 17:02 | |
| *** coolsvap has quit IRC | 17:02 | |
| *** arnaud__ has joined #openstack-qa | 17:02 | |
| *** 7YUAATLBR has joined #openstack-qa | 17:02 | |
| *** Sukhdev has joined #openstack-qa | 17:02 | |
| *** e0ne has joined #openstack-qa | 17:02 | |
| *** Dafna has joined #openstack-qa | 17:02 | |
| *** harlowja_away has joined #openstack-qa | 17:02 | |
| *** rfolco has joined #openstack-qa | 17:02 | |
| *** coolsvap has joined #openstack-qa | 17:02 | |
| *** mpaolino has joined #openstack-qa | 17:03 | |
| *** suneelb has joined #openstack-qa | 17:03 | |
| *** yamamoto has quit IRC | 17:05 | |
| *** chandankumar has joined #openstack-qa | 17:06 | |
| *** jlibosva has quit IRC | 17:14 | |
| *** arnaud__ has quit IRC | 17:14 | |
| *** 7YUAATLBR has quit IRC | 17:14 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:17 | |
| *** mpaolino has quit IRC | 17:20 | |
| *** shardy is now known as shardy_afk | 17:22 | |
| *** dhellmann is now known as dhellmann_ | 17:23 | |
| *** chandankumar has quit IRC | 17:24 | |
| *** cdent has quit IRC | 17:26 | |
| *** harlowja_away is now known as harlowja | 17:27 | |
| *** chandankumar has joined #openstack-qa | 17:29 | |
| jogo | mtreinish: https://review.openstack.org/#/c/113771/ e-r review needs a second review | 17:30 |
| jogo | I manually tested out the code etc | 17:30 |
| *** swamireddy has quit IRC | 17:32 | |
| sdague | dkranz: so this is fun - https://bugs.launchpad.net/nova/+bug/1331215 | 17:33 |
| sdague | the user exposed failure is actually because the code to get console over xml is wrong in tempest | 17:33 |
| sdague | dkranz: so I moved it back to confirmed in Tempest | 17:34 |
| *** marun_ has joined #openstack-qa | 17:37 | |
| *** e0ne has quit IRC | 17:39 | |
| sdague | dkranz: yeh, that's actually a tempest bug, it's not actually waiting for successful attach | 17:42 |
| *** dhellmann_ is now known as dhellmann | 17:50 | |
| *** chandankumar has quit IRC | 17:51 | |
| *** chandankumar has joined #openstack-qa | 17:51 | |
| *** yamamoto has joined #openstack-qa | 17:52 | |
| *** mriedem has quit IRC | 17:52 | |
| *** Sukhdev has quit IRC | 17:55 | |
| *** luqas has joined #openstack-qa | 17:56 | |
| *** yamamoto has quit IRC | 17:57 | |
| *** markmcclain1 has joined #openstack-qa | 17:58 | |
| *** markmcclain has quit IRC | 17:58 | |
| *** markmcclain has joined #openstack-qa | 17:58 | |
| *** markmcclain1 has quit IRC | 17:58 | |
| *** jaypipes has joined #openstack-qa | 18:04 | |
| *** ChanServ sets mode: +o jaypipes | 18:04 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack-dev/grenade: javelin-resources: attach a volume to a server https://review.openstack.org/122213 | 18:06 |
| EmilienM | jogo: sdague: this one is blocking me to make volumes work in javelin ^^^^^ (if you can have a look) | 18:06 |
| sdague | EmilienM: +2 | 18:07 |
| EmilienM | sdague: thx. | 18:07 |
| *** marun has joined #openstack-qa | 18:09 | |
| *** marun_ has quit IRC | 18:12 | |
| *** e0ne has joined #openstack-qa | 18:13 | |
| *** ylobankov has quit IRC | 18:13 | |
| *** jordanP has quit IRC | 18:13 | |
| jogo | EmilienM: +Wed | 18:15 |
| EmilienM | jogo: great, I can go ahead with javelin2 blueprint. thx | 18:15 |
| *** e0ne has quit IRC | 18:26 | |
| mtreinish | jogo: +A | 18:27 |
| *** jgrimm is now known as zz_jgrimm | 18:27 | |
| dkranz | sdague: So all that crap ERROR stuff in the cpu log that precedes the failure is expected? | 18:28 |
| sdague | dkranz: it's completely unrelated to the test that failed | 18:29 |
| sdague | it's on a different tennant | 18:30 |
| sdague | the real issue that caused this failure is neutron was a little slow | 18:30 |
| sdague | and tempest raced it | 18:30 |
| *** e0ne has joined #openstack-qa | 18:30 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add crm114 logstash queries to uncategorized page https://review.openstack.org/113771 | 18:30 |
| sdague | at least that was my reading of the logs | 18:31 |
| *** luqas has quit IRC | 18:32 | |
| *** sballe has joined #openstack-qa | 18:36 | |
| *** chandankumar has quit IRC | 18:37 | |
| *** e0ne has quit IRC | 18:41 | |
| *** e0ne has joined #openstack-qa | 18:46 | |
| *** yamamoto has joined #openstack-qa | 18:49 | |
| *** markmcclain has quit IRC | 18:50 | |
| *** yamamoto has quit IRC | 18:54 | |
| *** aix has quit IRC | 18:55 | |
| openstackgerrit | gordon chung proposed a change to openstack-dev/devstack: ceilometer compute agent not started correctly https://review.openstack.org/122223 | 18:58 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Migrate cli test framework to tempest-lib https://review.openstack.org/117649 | 19:11 |
| *** chandankumar has joined #openstack-qa | 19:12 | |
| *** cnesa12 has quit IRC | 19:14 | |
| *** yfried has joined #openstack-qa | 19:14 | |
| openstackgerrit | Claudiu Belu proposed a change to openstack/tempest: Adds TestSecurityGroupsBasicOps created VM's security_groups check https://review.openstack.org/118167 | 19:17 |
| *** sballe_ has joined #openstack-qa | 19:19 | |
| *** yfried is now known as yfried|afk | 19:21 | |
| *** sballe has quit IRC | 19:22 | |
| *** safchain has joined #openstack-qa | 19:25 | |
| *** safchain has quit IRC | 19:26 | |
| *** cdent has joined #openstack-qa | 19:28 | |
| *** yamamoto has joined #openstack-qa | 19:30 | |
| *** yfried|afk is now known as yfried | 19:31 | |
| *** yamamoto has quit IRC | 19:35 | |
| *** cdent has quit IRC | 19:47 | |
| mtreinish | dkranz: if you have a sec can you take a look at my open tempest-lib reviews? I think those 3 from me should be the last 3 before I push the 0.0.1 release... | 19:50 |
| *** markmcclain has joined #openstack-qa | 19:50 | |
| dkranz | mtreinish: sure. I have not added that repo to my setup so I have not seen them yet but I should. Do you have a link to start? | 19:51 |
| mtreinish | dkranz: https://review.openstack.org/#/q/project:openstack/tempest-lib+status:open,n,z | 19:52 |
| mtreinish | dkranz: it's on my list to add them to the templates on gerrit-dash-creator (or whatever it's called) | 19:52 |
| dkranz | mtreinish: You mean Sean's dashboard? | 19:52 |
| *** wuhg has quit IRC | 19:53 | |
| *** aix has joined #openstack-qa | 19:53 | |
| mtreinish | dkranz: yeah it's a git repo, I need to update the sample qa project file and add a separate tempest one | 19:54 |
| mtreinish | unless you'd like to do it? :) | 19:54 |
| dkranz | mtreinish: where is is? | 19:54 |
| dkranz | mtreinish: it | 19:55 |
| mtreinish | dkranz: http://git.openstack.org/cgit/stackforge/gerrit-dash-creator/ | 19:55 |
| *** Ch00k has quit IRC | 19:59 | |
| dkranz | mtreinish: I'll do it | 20:01 |
| dkranz | mtreinish: You mean "tempest dash" is qaprogram - devstack + tempset-lib | 20:01 |
| dkranz | mtreinish: right? | 20:01 |
| dkranz | mtreinish: or keep devstack? | 20:02 |
| *** andreaf has joined #openstack-qa | 20:02 | |
| mtreinish | dkranz: yeah tempest dash would be tempest, tempest-lib, and maybe qa-specs | 20:04 |
| mtreinish | devstack would have it's own still | 20:04 |
| mtreinish | and the qa-program one will be everything | 20:04 |
| *** e0ne has quit IRC | 20:04 | |
| *** melwitt has joined #openstack-qa | 20:04 | |
| *** e0ne has joined #openstack-qa | 20:05 | |
| dkranz | mtreinish: make sense. I would keep qa-specs with tempest for practical reasons since we don't have "tempest-specs" | 20:06 |
| dkranz | mtreinish: I'll get on it after reviewing tempest lib | 20:07 |
| mtreinish | dkranz: ok cool, thanks. | 20:07 |
| mtreinish | dkranz: the other thing with the tempest-lib stuff, is that it turns out we're not able to migrate git history over from tempest | 20:07 |
| mtreinish | which was my mistake, so what's there now is what we're stuck with | 20:07 |
| dkranz | mtreinish: I really don't think that is the end of the world | 20:07 |
| *** e0ne has quit IRC | 20:07 | |
| mtreinish | dkranz: neither do I, especially if we link all the sha1s from tempest involved in a migration | 20:08 |
| *** chandankumar has quit IRC | 20:09 | |
| ekhugen | andreaf are you there? I wanted to ask you about a bug patch that overlaps with one of your blueprints | 20:12 |
| *** boden has quit IRC | 20:13 | |
| andreaf | ekhugen, I'm ~ here, I only have a few moments | 20:13 |
| *** boden has joined #openstack-qa | 20:13 | |
| ekhugen | I did this patch https://review.openstack.org/#/c/120245/ but matt pointed out that your patch https://review.openstack.org/#/c/107685/34 will take care of it | 20:13 |
| ekhugen | do you want me to abandon my patch? | 20:14 |
| ekhugen | andreaf ^ | 20:14 |
| andreaf | ekhugen, checking... | 20:15 |
| andreaf | ekhugen: yeah I already did that | 20:16 |
| andreaf | ekhugen: and using the deprecated options bit | 20:17 |
| ekhugen | okay, I'll just abandon, can you put Closes-Bug: #1340380 on your patch? | 20:17 |
| andreaf | ekhugen: ok sure will do | 20:17 |
| ekhugen | thanks | 20:17 |
| andreaf | ekhugen, sorry I didn't know there was a bug for that | 20:18 |
| *** boden has quit IRC | 20:18 | |
| ekhugen | np, I picked it up on bug day and should've checked some of the bps before I put a patch in | 20:19 |
| *** yamamoto has joined #openstack-qa | 20:19 | |
| dkranz | mtreinish: I am not enough of a git expert to validate https://review.openstack.org/#/c/121988/2/tools/migrate_from_tempest.sh,unified but if you need my +2 I will trust you. | 20:20 |
| *** dustins has quit IRC | 20:20 | |
| *** stevemar has joined #openstack-qa | 20:21 | |
| stevemar | looking for some QA guidance here, hoping to test out a CLI, but i noticed in the docs it mentions that the tests should not modify the cloud - http://docs.openstack.org/developer/tempest/field_guide/cli.html#example-of-a-good-test | 20:23 |
| *** aix has quit IRC | 20:23 | |
| stevemar | so i'm wondering how we go about testing that out? | 20:24 |
| *** yamamoto has quit IRC | 20:24 | |
| *** jasondotstar has quit IRC | 20:26 | |
| *** e0ne has joined #openstack-qa | 20:28 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 20:31 |
| *** jecarey has quit IRC | 20:32 | |
| andreaf | ekhugen: done ^^^ | 20:33 |
| ekhugen | thanks andreaf | 20:35 |
| mtreinish | dkranz: heh, well the filter branch code was stolen from dhellmann's oslo lib graduation script. | 20:36 |
| mtreinish | the review was more for around semantics around building a commit message with all the sha1s | 20:37 |
| dkranz | mtreinish: I am happy to give it a +2 | 20:37 |
| *** yfried is now known as yfried|afk | 20:40 | |
| *** cnesa12 has joined #openstack-qa | 20:44 | |
| *** yfried|afk is now known as yfried | 20:47 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate thirdparty API tests to resource_* fixtures https://review.openstack.org/121552 | 20:48 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate cli API tests to resource_* fixtures https://review.openstack.org/121553 | 20:48 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Hacking rule to forbid resource unsafe fixtures https://review.openstack.org/121554 | 20:48 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate scenario API tests to resource_* fixtures https://review.openstack.org/121551 | 20:48 |
| andreaf | mtreinish: only four of them this time | 20:48 |
| *** mwagner_lap has quit IRC | 20:48 | |
| *** cnesa12 has quit IRC | 20:48 | |
| mtreinish | andreaf: heh, I take that means you got the base patch working? | 20:49 |
| andreaf | mtreinish, yep | 20:49 |
| andreaf | mtreinish, I only have spurious failures now | 20:49 |
| andreaf | in some of the patches | 20:50 |
| *** cnesa12 has joined #openstack-qa | 20:50 | |
| andreaf | mtreinish, in fact I was about to ask for reviews on https://review.openstack.org/#/q/topic:bp/resource-cleanup,n,z as I think they should be prettry straight forward to review | 20:50 |
| andreaf | mtreinish, sdague, dkranz, cyeoh : ^^^ if you have time for some easy reviews: https://review.openstack.org/#/q/topic:bp/resource-cleanup,n,z | 20:51 |
| mtreinish | andreaf: ok, I'll take a look at it soon | 20:51 |
| andreaf | mtreinish, thanks | 20:51 |
| *** mriedem has joined #openstack-qa | 20:53 | |
| openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Make function for logstash query encoding https://review.openstack.org/122246 | 20:54 |
| *** sballe has joined #openstack-qa | 20:55 | |
| *** marun has quit IRC | 20:57 | |
| *** yfried has quit IRC | 20:57 | |
| *** jecarey has joined #openstack-qa | 20:57 | |
| *** sballe_ has quit IRC | 20:59 | |
| *** markmcclain has quit IRC | 21:00 | |
| *** markmcclain1 has joined #openstack-qa | 21:00 | |
| *** markmcclain1 has quit IRC | 21:00 | |
| *** markmcclain has joined #openstack-qa | 21:00 | |
| *** markmcclain has quit IRC | 21:01 | |
| *** markmcclain1 has joined #openstack-qa | 21:01 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Unskip TestStampPattern for testing purposes https://review.openstack.org/122197 | 21:05 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Migrate TestStampPattern to tempest client https://review.openstack.org/122196 | 21:05 |
| *** mmedvede has quit IRC | 21:06 | |
| *** gordc has quit IRC | 21:10 | |
| *** stevemar has quit IRC | 21:12 | |
| andreaf | masayukig: ping - do you plan to migrate the 2nd heat scenario test as well? | 21:12 |
| andreaf | masayukig: we're almost there with this bp and I'd really like to get rid of all the official client specific forks in the code | 21:13 |
| *** cnesa13 has joined #openstack-qa | 21:15 | |
| *** cnesa12 has quit IRC | 21:17 | |
| *** yamamoto has joined #openstack-qa | 21:19 | |
| *** cdent has joined #openstack-qa | 21:21 | |
| masayukig | andreaf: Do you mean about test_autoscaling.py ? | 21:22 |
| *** shardy_afk is now known as shardy | 21:23 | |
| *** yamamoto has quit IRC | 21:24 | |
| masayukig | andreaf: I'd like to do it. But I don't have enough time to do it these days.. | 21:25 |
| *** vigneshvar has quit IRC | 21:27 | |
| *** dimtruck is now known as zz_dimtruck | 21:30 | |
| *** mmedvede has joined #openstack-qa | 21:33 | |
| *** dims has quit IRC | 21:35 | |
| *** zz_dimtruck is now known as dimtruck | 21:35 | |
| *** dims has joined #openstack-qa | 21:35 | |
| andreaf | masayukig, ok - I'll see if I can help with that then | 21:35 |
| *** dkranz has quit IRC | 21:36 | |
| *** dims has quit IRC | 21:40 | |
| *** cdent has quit IRC | 21:46 | |
| *** adalbas has quit IRC | 21:47 | |
| *** torandu has quit IRC | 21:50 | |
| openstackgerrit | Eric Harney proposed a change to openstack-dev/devstack: Glance: Add RBD to known_stores when Ceph is enabled https://review.openstack.org/122258 | 21:56 |
| *** weshay has quit IRC | 21:58 | |
| *** jamielennox|away is now known as jamielennox | 21:59 | |
| *** e0ne has quit IRC | 22:00 | |
| *** bknudson has quit IRC | 22:00 | |
| masayukig | andreaf: Thanks:) | 22:00 |
| *** jhenner1 has quit IRC | 22:02 | |
| *** torandu has joined #openstack-qa | 22:03 | |
| *** markmcclain1 has quit IRC | 22:03 | |
| *** markmcclain has joined #openstack-qa | 22:03 | |
| openstackgerrit | A change was merged to openstack-infra/subunit2sql: Switch sample sqlalchemy url to use mysql https://review.openstack.org/115761 | 22:07 |
| *** jamielennox is now known as jamielennox|away | 22:09 | |
| *** gszasz has quit IRC | 22:11 | |
| *** flaper87 is now known as flaper87|afk | 22:16 | |
| *** yamamoto has joined #openstack-qa | 22:19 | |
| *** Sukhdev has joined #openstack-qa | 22:19 | |
| *** yamamoto has quit IRC | 22:24 | |
| openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Make function for logstash query encoding https://review.openstack.org/122246 | 22:25 |
| *** markmcclain has quit IRC | 22:34 | |
| *** markmcclain1 has joined #openstack-qa | 22:34 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add subunit2sql-db-manage utility https://review.openstack.org/118418 | 22:39 |
| *** amotoki has quit IRC | 22:42 | |
| *** boden has joined #openstack-qa | 22:45 | |
| *** boden has quit IRC | 22:49 | |
| *** enikanorov has quit IRC | 22:51 | |
| *** jpich has joined #openstack-qa | 22:55 | |
| *** rbak has quit IRC | 22:56 | |
| *** markmcclain1 has quit IRC | 23:00 | |
| mtreinish | salv-orlando: the other thing to try, which I think sdague suggested, is that in devstack on the stable branches we just hard code the full extension list for each project (because it shouldn't change after a release) | 23:00 |
| mtreinish | and master will always be all anyway so we don't have to worry about it until we branch | 23:01 |
| *** shardy is now known as shardy_z | 23:01 | |
| salv-orlando | mtreinish: what do you mean that master will always be all? I think the point of this blueprint is that we want to disable some extensions even for tests on master, isn’t it? | 23:02 |
| *** aysyanne has quit IRC | 23:03 | |
| mtreinish | salv-orlando: I thought it was just to disable some on the stable branches.(I could be wrong) Because tempest has to work against all versions but all the extensions aren't around on stable branches | 23:03 |
| *** dansmith has quit IRC | 23:04 | |
| mtreinish | I didn't really think there was a gate config where we'd be running master, but not every extension that tempest knows about is installed | 23:04 |
| salv-orlando | mtreinish: there are also the not-so-corner cases about cells disabling some extensions, and the fact that in neutron we want to runn the full test without the advanced services | 23:04 |
| salv-orlando | otherwise I agree hardcoding is way easier. | 23:05 |
| *** dansmith has joined #openstack-qa | 23:05 | |
| mtreinish | salv-orlando: oh, yeah I forgot about those | 23:06 |
| mtreinish | if it was just cells I'd say not to worry about it, because that is a low prio. But disabling the neutron advanced services is a good thing | 23:07 |
| salv-orlando | mtreinish: in theory even for master we can have a hardcoded list. | 23:07 |
| *** mwagner_lap has joined #openstack-qa | 23:08 | |
| salv-orlando | it’s just then we need to decide whether it’s ok to require devstack developers to maintain it. | 23:08 |
| mtreinish | salv-orlando: yeah, that would work. Except things will get interesting when a new extension is added | 23:08 |
| salv-orlando | mtreinish: people will find out their tempest tests won’t run until the extension is added to the list. | 23:08 |
| salv-orlando | so in a way we might say that’s fair enough | 23:09 |
| salv-orlando | but on the other hand if reading the extension list by querying the API endpoint is not too hard, we would spare developers this hassle | 23:09 |
| salv-orlando | this is obviously assuming every openstack API endpoint has an easy way to query enabled extensions | 23:09 |
| mtreinish | salv-orlando: it shouldn't be too hard, the only thing to factor in is that it's pretty much different for each api | 23:10 |
| mtreinish | take a look at verify_tempest_config to see how I had to work around all of that | 23:10 |
| salv-orlando | mtreinish: my goal is indeed to reuse your code. I don’t want to write anything new. | 23:11 |
| mtreinish | I think I got all the apis that have an extensions opt in tempest. Although it's been a while since I checked | 23:12 |
| salv-orlando | mtreinish: I’ll let you know if you missed something1 | 23:14 |
| *** yamamoto has joined #openstack-qa | 23:19 | |
| *** hemna is now known as hemnass | 23:21 | |
| *** yamamoto has quit IRC | 23:24 | |
| *** amotoki has joined #openstack-qa | 23:28 | |
| openstackgerrit | A change was merged to openstack-dev/grenade: javelin-resources: attach a volume to a server https://review.openstack.org/122213 | 23:34 |
| *** dimtruck is now known as zz_dimtruck | 23:37 | |
| *** markmcclain has joined #openstack-qa | 23:42 | |
| *** markmcclain has quit IRC | 23:42 | |
| *** markmcclain1 has joined #openstack-qa | 23:42 | |
| *** sballe has quit IRC | 23:43 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/devstack: Set API_WORKERS to 2 by default https://review.openstack.org/122269 | 23:45 |
| *** boden has joined #openstack-qa | 23:46 | |
| *** boden has quit IRC | 23:50 | |
| *** atiwari has quit IRC | 23:56 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: WIP - Javelin: enable volume resources https://review.openstack.org/121648 | 23:57 |
| openstackgerrit | Miguel Lavalle proposed a change to openstack/tempest: Migrate test_load_balancer_basic to tempest client https://review.openstack.org/118232 | 23:57 |
| *** mlavalle has quit IRC | 23:59 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for ssh 'no route to host' bug 1370496 https://review.openstack.org/122140 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!