| *** mlavalle has quit IRC | 00:02 | |
| *** andreaf has quit IRC | 00:02 | |
| *** shwetaap has quit IRC | 00:12 | |
| sdague | dtroyer: https://review.openstack.org/#/c/84146/ - thoughts? | 00:24 |
|---|---|---|
| dtroyer | is the only difference between setup_develop and setup_lib the value of flags? | 00:26 |
| sdague | yeh | 00:27 |
| sdague | basically, don't pip -e | 00:27 |
| dtroyer | given the history of the requirements bits, I'd rather not have two sets of that, but I'm fine with the idea | 00:27 |
| *** matsuhashi has joined #openstack-qa | 00:27 | |
| sdague | but the rollback is in such an odd place | 00:27 |
| sdague | so I just copy / pasted | 00:28 |
| sdague | I hear you, if yuo have a better thought, go for it :) | 00:28 |
| sdague | basically we have discovered that namespace packages vs. pip are a barely working feature | 00:28 |
| sdague | so you can get into lots of situations where pip believes the package is installed, but you can't import it | 00:29 |
| dtroyer | just another layer of nesting…setup_develop and setup_lib just call a common func with flags as an arg…which in turn calls setup_develop_no_requirements_update... | 00:29 |
| sdague | sure | 00:29 |
| dtroyer | this is all begging for a re-think but it feels to fragile to do quickly | 00:29 |
| dtroyer | so ok, if that makes it work | 00:29 |
| sdague | mostly, the pip install -e doesn't seem to make sense for libraries anyway | 00:30 |
| sdague | because you aren't going to change a library and expect it to be openstack wide without the unstack / stack anyway | 00:30 |
| sdague | unlike doing rapid changes on nova | 00:30 |
| sdague | at least that's my thinking | 00:30 |
| sdague | it's probably worth an ML thread, or at least mordred deciding if it's insane | 00:31 |
| sdague | dhellmann seemed mostly onboard | 00:31 |
| dtroyer | namespace packages and pip -e are at the edge of my packaging understanding here | 00:31 |
| sdague | yeh, mine too | 00:31 |
| dtroyer | I'll go with whatever those two want to do, I'd just liek to try to clean up the implementation at some point | 00:32 |
| sdague | sure | 00:32 |
| sdague | mostly I wanted to see if it would work :) | 00:32 |
| dtroyer | and just adding an optional -e to setup_develop() isn't foolproof enough | 00:33 |
| sdague | yeh, I wanted the top level to be really clear | 00:34 |
| dtroyer | maybe a +e and only add that in the setup_lib() places | 00:34 |
| dtroyer | yeah, not using options is probably better here | 00:34 |
| sdague | this is python, there is no +e ;) | 00:35 |
| sdague | honestly, we should probably call it setup_install | 00:35 |
| sdague | because that's what it actually is | 00:35 |
| *** rhsu has quit IRC | 00:36 | |
| mordred | sdague: reading | 00:36 |
| dtroyer | rather than setup_lib? | 00:36 |
| sdague | dtroyer: yeh | 00:36 |
| mordred | yeah. setup_install | 00:37 |
| mordred | OR | 00:37 |
| mordred | pip_install_e and pip_install ;) | 00:37 |
| sdague | ug, no :) | 00:37 |
| sdague | we already have the other naming convention in place | 00:37 |
| sdague | mordred: anyway, mostly wanted your vote on this not being completely insane, because I think it will immediately make a ton of oslo edge cases go away | 00:38 |
| sdague | however, my wife just got home, so time to really call it a night. I'll check back on things in the morning, and maybe refactor that patch | 00:39 |
| mordred | sdague: I'm fine with it - I'm VERY unhappy that pip doesn't work with namespace packages for some reason | 00:39 |
| mordred | I don't think that's a reasonable thing to not work | 00:39 |
| *** markmcclain has quit IRC | 00:45 | |
| *** zhikunliu has joined #openstack-qa | 00:45 | |
| *** giulivo has quit IRC | 00:47 | |
| *** torandu has quit IRC | 00:48 | |
| *** torandu has joined #openstack-qa | 00:49 | |
| *** markmcclain has joined #openstack-qa | 00:53 | |
| *** Sukhdev has quit IRC | 00:58 | |
| *** zigo has quit IRC | 00:59 | |
| *** zigo has joined #openstack-qa | 01:01 | |
| *** jlibosva has joined #openstack-qa | 01:06 | |
| *** rhsu has joined #openstack-qa | 01:07 | |
| *** jlibosva has quit IRC | 01:10 | |
| *** mrodden has joined #openstack-qa | 01:13 | |
| *** nosnos has joined #openstack-qa | 01:26 | |
| *** shwetaap has joined #openstack-qa | 01:35 | |
| openstackrecheck | openstack/oslo.messaging change: https://review.openstack.org/81193 failed because of: gate-tempest-dsvm-neutron-large-ops: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1224001, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 01:48 |
| *** shwetaap has quit IRC | 02:03 | |
| *** adalbas has quit IRC | 02:03 | |
| *** weshay has quit IRC | 02:06 | |
| *** jlibosva has joined #openstack-qa | 02:07 | |
| *** jlibosva has quit IRC | 02:13 | |
| *** dims has quit IRC | 02:23 | |
| *** GMann has joined #openstack-qa | 02:24 | |
| *** dims has joined #openstack-qa | 02:24 | |
| *** amotoki_ has quit IRC | 02:30 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/80383 failed because of: gate-tempest-dsvm-neutron-large-ops: unrecognized error, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 02:32 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Enable large ops if flag is >= 1. https://review.openstack.org/84319 | 02:34 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Enable large ops if flag is >= 1. https://review.openstack.org/84319 | 02:37 |
| *** shwetaap has joined #openstack-qa | 02:38 | |
| *** shwetaap has quit IRC | 02:45 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/75721 failed because of: gate-tempest-dsvm-neutron-large-ops: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1224001, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 02:53 |
| *** markmcclain has quit IRC | 02:59 | |
| *** harlowja is now known as harlowja_away | 02:59 | |
| *** shwetaap has joined #openstack-qa | 03:03 | |
| *** shwetaap has quit IRC | 03:04 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: enable nova v3 scenario tests for server basic ops and advanced ops https://review.openstack.org/73270 | 03:09 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: enable some nova v3 scenario tests https://review.openstack.org/72858 | 03:09 |
| *** jlibosva has joined #openstack-qa | 03:09 | |
| *** harlowja_away is now known as harlowja | 03:09 | |
| *** chandan_kumar has joined #openstack-qa | 03:11 | |
| *** shwetaap has joined #openstack-qa | 03:12 | |
| *** yamahata has joined #openstack-qa | 03:13 | |
| *** jlibosva has quit IRC | 03:13 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: enable nova v3 scenario tests for server basic ops and advanced ops https://review.openstack.org/73270 | 03:13 |
| *** catohornet1 has quit IRC | 03:21 | |
| *** matsuhashi has quit IRC | 03:22 | |
| *** chandan_kumar has quit IRC | 03:35 | |
| *** nosnos has quit IRC | 03:37 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check attributes of create/delete sec groups rule" https://review.openstack.org/84333 | 03:39 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/79829 failed because of: gate-tempest-dsvm-neutron-large-ops: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1224001 https://bugs.launchpad.net/bugs/1283522, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 03:43 |
| *** shwetaap has quit IRC | 03:53 | |
| *** chandan_kumar has joined #openstack-qa | 03:59 | |
| *** chandan_kumar has quit IRC | 04:01 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify create/get flavor attributes of Nova APIs https://review.openstack.org/80543 | 04:04 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/75816 failed because of: gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896, gate-tempest-dsvm-neutron-large-ops: unrecognized error, gate-tempest-dsvm-full: unrecognized error | 04:05 |
| *** chandan_kumar has joined #openstack-qa | 04:06 | |
| *** jlibosva has joined #openstack-qa | 04:10 | |
| *** matsuhashi has joined #openstack-qa | 04:14 | |
| *** jlibosva has quit IRC | 04:15 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify detail_list flavor attributes of V2/V3 APIs https://review.openstack.org/80548 | 04:23 |
| *** nosnos has joined #openstack-qa | 04:23 | |
| *** yfried__ has joined #openstack-qa | 04:36 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Missing baremetal API tests https://review.openstack.org/83345 | 04:44 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/84208 failed because of: gate-tempest-dsvm-neutron-large-ops: unrecognized error, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 04:45 |
| *** Madan has joined #openstack-qa | 04:45 | |
| *** zul has quit IRC | 05:00 | |
| *** matsuhas_ has joined #openstack-qa | 05:02 | |
| *** matsuhas_ has quit IRC | 05:02 | |
| *** zul has joined #openstack-qa | 05:04 | |
| *** matsuhashi has quit IRC | 05:05 | |
| *** matsuhashi has joined #openstack-qa | 05:05 | |
| *** psharma has joined #openstack-qa | 05:09 | |
| *** harlowja is now known as harlowja_away | 05:11 | |
| *** jlibosva has joined #openstack-qa | 05:11 | |
| *** jlibosva has quit IRC | 05:16 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify create agent attributes of V2/V3 APIs https://review.openstack.org/84344 | 05:18 |
| *** zhikunliu has quit IRC | 05:21 | |
| *** zhikunliu has joined #openstack-qa | 05:23 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/82438 failed because of: gate-tempest-dsvm-neutron-large-ops: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1224001, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 05:28 |
| *** otherwiseguy has quit IRC | 05:31 | |
| openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'startup_host' API https://review.openstack.org/83012 | 05:39 |
| openstackgerrit | Abhijeet Jain proposed a change to openstack/tempest: Nova Service API Negative Test https://review.openstack.org/84353 | 05:41 |
| openstackrecheck | openstack/oslo.messaging change: https://review.openstack.org/84238 failed because of: gate-tempest-dsvm-neutron-large-ops: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1224001, gate-tempest-dsvm-large-ops: https://bugs.launchpad.net/bugs/1253896 | 05:43 |
| *** rohitk has joined #openstack-qa | 05:43 | |
| *** ashishsnigam has joined #openstack-qa | 05:45 | |
| openstackgerrit | Abhijeet Jain proposed a change to openstack/tempest: Nova Service API Negative Test https://review.openstack.org/84353 | 05:53 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Enable large ops if flag is >= 1 https://review.openstack.org/84319 | 05:54 |
| marun | lifeless: clean install of testr gives this error, any idea wtf? KeyError: 'Could not import command module testrepository.commands.stat' | 05:58 |
| marun | lifeless: running 'testr stat' | 05:58 |
| lifeless | marun: testr commands | 06:00 |
| marun | lifeless: ah, commands have changed | 06:00 |
| marun | lifeless: sorry, I was looking at outdated docs | 06:00 |
| lifeless | marun: I'm not sure we had a 'stat' command ever :) IIRC it was aspirational but not implemented. | 06:00 |
| marun | lifeless: no, worse. pebkac :( | 06:00 |
| marun | lifeless: sorry for the bother | 06:01 |
| marun | lifeless: suggestions on using testr to read the entire test run out? I need execution time of every test. | 06:02 |
| marun | lifeless: I guess I should just look at the code? | 06:02 |
| lifeless | marun: testr last --subunit | subunit-ls --times | 06:02 |
| lifeless | marun: that might barf; if so you'll need | 06:03 |
| lifeless | marun: testr last --subunit | subunit-1to2 | subunit-ls --times | 06:03 |
| marun | lifeless: the latter works, danke | 06:05 |
| *** psharma has quit IRC | 06:06 | |
| marun | lifeless: erg, that doesn't look right | 06:07 |
| marun | lifeless: 99% of 10k tests take 0.000, and only 15 take at most 0.002 | 06:08 |
| marun | lifeless: what is the units for the time/ | 06:08 |
| lifeless | marun: seconds, but its possible there's a bug, of course | 06:09 |
| lifeless | marun: we had some issue with tempest and thats why we wrote the custom filter which (I believe) is now in-tree | 06:10 |
| marun | lifeless: testr slowest returns accurate results at least, so I think the data is good | 06:11 |
| marun | lifeless: which custom filter do you mean? | 06:11 |
| *** jlibosva has joined #openstack-qa | 06:12 | |
| marun | lifeless: subunit-1to2? | 06:12 |
| lifeless | marun: no, theres a reporting filter thats shows the overlapped nature of parallel tests etc | 06:12 |
| marun | lifeless: oh, maybe that's it. I ran the tests across 12 cores | 06:12 |
| marun | where do I find this filter? | 06:13 |
| lifeless | utils/ or something | 06:13 |
| lifeless | I forget | 06:13 |
| marun | lifeless: ok | 06:13 |
| *** rohitk has quit IRC | 06:13 | |
| marun | lifeless: I'm assuming it separates reporting by worker before computing stats? | 06:13 |
| lifeless | marun: change I1b529546e005f47aba56b451e1c0d8b0da09fca3 | 06:14 |
| lifeless | marun: is where I wrote it, I dunno if it landed or not | 06:14 |
| marun | looks like it was abandoned | 06:16 |
| *** psharma has joined #openstack-qa | 06:17 | |
| *** jlibosva has quit IRC | 06:17 | |
| openstackgerrit | Qiu Hua Qiao proposed a change to openstack/tempest: Change to use absolute path in load_template() https://review.openstack.org/84363 | 06:20 |
| *** saju_m has joined #openstack-qa | 06:21 | |
| *** beisner has quit IRC | 06:22 | |
| marun | lifeless: actually, 'testr slowest --all' works nicely :) | 06:23 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Adds unit test for negative class decorator https://review.openstack.org/84086 | 06:28 |
| *** rhsu1 has joined #openstack-qa | 06:31 | |
| *** rhsu has quit IRC | 06:31 | |
| openstackrecheck | Console logs not available after 13:23s for gate-nova-python26 83256,4,40bcb01 | 06:35 |
| *** tkammer has joined #openstack-qa | 06:36 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Enable one flavor tests https://review.openstack.org/84370 | 06:44 |
| *** yrabl has joined #openstack-qa | 06:45 | |
| *** nosnos has quit IRC | 06:45 | |
| *** nosnos has joined #openstack-qa | 06:46 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Rename basic generator test https://review.openstack.org/84371 | 06:48 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add unit tests for all generators https://review.openstack.org/84127 | 06:48 |
| lifeless | sdague: marun: unabandoned https://review.openstack.org/#/c/70021/ | 06:50 |
| lifeless | marun: heh I'd forgotten about slowest --all | 06:51 |
| *** mkoderer has joined #openstack-qa | 06:51 | |
| *** psharma has quit IRC | 06:52 | |
| *** matsuhashi has quit IRC | 06:58 | |
| *** matsuhashi has joined #openstack-qa | 06:58 | |
| *** Abhijeet has joined #openstack-qa | 07:02 | |
| *** matsuhashi has quit IRC | 07:03 | |
| *** evgenyf has joined #openstack-qa | 07:03 | |
| *** giulivo has joined #openstack-qa | 07:05 | |
| openstackgerrit | Abhijeet Jain proposed a change to openstack/tempest: Nova Service API Negative Test https://review.openstack.org/84353 | 07:05 |
| *** matsuhashi has joined #openstack-qa | 07:06 | |
| *** jlibosva has joined #openstack-qa | 07:12 | |
| *** rhsu has joined #openstack-qa | 07:16 | |
| *** jlibosva has quit IRC | 07:17 | |
| *** rhsu1 has quit IRC | 07:20 | |
| *** jamielennox is now known as jamielennox|away | 07:27 | |
| *** salv-orlando has quit IRC | 07:30 | |
| *** salv-orlando has joined #openstack-qa | 07:30 | |
| *** marun is now known as maru_afk | 07:36 | |
| openstackrecheck | Console logs not available after 13:23s for gate-python-novaclient-pypy 81688,2,78fad70 | 07:46 |
| *** rhsu has quit IRC | 07:47 | |
| *** mmaglana has joined #openstack-qa | 07:50 | |
| *** yfried__ has quit IRC | 07:52 | |
| *** yfried__ has joined #openstack-qa | 07:52 | |
| *** rohitk has joined #openstack-qa | 07:56 | |
| openstackrecheck | Console logs not available after 13:26s for check-tempest-dsvm-neutron-full 83434,1,f08ef49 | 08:01 |
| *** jpich has joined #openstack-qa | 08:02 | |
| *** matsuhashi has quit IRC | 08:03 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Enable large ops if flag is >= 1 https://review.openstack.org/84319 | 08:04 |
| *** matsuhas_ has joined #openstack-qa | 08:05 | |
| *** jlibosva has joined #openstack-qa | 08:13 | |
| *** psharma has joined #openstack-qa | 08:17 | |
| *** jlibosva has quit IRC | 08:19 | |
| *** zhikunliu has quit IRC | 08:25 | |
| *** mmaglana has quit IRC | 08:28 | |
| *** mmaglana has joined #openstack-qa | 08:28 | |
| *** mmaglana has quit IRC | 08:29 | |
| rossella_s | ro171082*S | 08:29 |
| rossella_s | sorry wrong window | 08:29 |
| *** jordanP has joined #openstack-qa | 08:32 | |
| *** matsuhas_ has quit IRC | 08:33 | |
| *** jlibosva has joined #openstack-qa | 08:35 | |
| *** andreaf has joined #openstack-qa | 08:37 | |
| *** matsuhashi has joined #openstack-qa | 08:38 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Adding resp code schema as a common parameter type https://review.openstack.org/84380 | 08:42 |
| *** saju_m has quit IRC | 08:43 | |
| *** jpich has quit IRC | 08:55 | |
| *** sth has quit IRC | 08:55 | |
| *** saju_m has joined #openstack-qa | 08:56 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-neutron-large-ops 62581,13,974b866 | 08:57 |
| *** afazekas has joined #openstack-qa | 08:57 | |
| openstackgerrit | Bob Ball proposed a change to openstack-infra/devstack-gate: Only install libvirt if the driver is libvirt(!) https://review.openstack.org/84381 | 09:01 |
| *** vrovachev has joined #openstack-qa | 09:02 | |
| *** jpich has joined #openstack-qa | 09:07 | |
| *** sth has joined #openstack-qa | 09:12 | |
| *** jlibosva1 has joined #openstack-qa | 09:14 | |
| openstackgerrit | Bob Ball proposed a change to openstack-infra/devstack-gate: Only install libvirt if the driver is libvirt(!) https://review.openstack.org/84381 | 09:14 |
| *** jlibosva1 has quit IRC | 09:18 | |
| *** chandankumar_ has joined #openstack-qa | 09:25 | |
| *** jlibosva1 has joined #openstack-qa | 09:36 | |
| *** jlibosva1 has quit IRC | 09:41 | |
| *** morganfainberg is now known as morganfainberg_Z | 09:46 | |
| *** tosky has joined #openstack-qa | 09:52 | |
| *** chandankumar_ has quit IRC | 09:54 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Remove singleton pattern in base_generator https://review.openstack.org/84395 | 10:04 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron https://review.openstack.org/58695 | 10:04 |
| *** safchain has joined #openstack-qa | 10:10 | |
| *** matsuhashi has quit IRC | 10:14 | |
| *** yamahata has quit IRC | 10:24 | |
| *** rohitk has quit IRC | 10:32 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: normalize the services for a post grizzly world https://review.openstack.org/84287 | 10:35 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: turn back on log enforcement https://review.openstack.org/81216 | 10:35 |
| *** jlibosva1 has joined #openstack-qa | 10:36 | |
| *** evgenyf has quit IRC | 10:37 | |
| *** nosnos has quit IRC | 10:40 | |
| *** jlibosva1 has quit IRC | 10:41 | |
| *** evgenyf has joined #openstack-qa | 10:44 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check attributes of change/get server password API https://review.openstack.org/84403 | 10:45 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check attributes of create/delete sec groups rule https://review.openstack.org/84333 | 10:46 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron https://review.openstack.org/58695 | 11:06 |
| *** GMann has quit IRC | 11:10 | |
| openstackgerrit | Amit Ugol proposed a change to openstack/tempest: orchestration - add sanity test to scenarios (WIP) https://review.openstack.org/84410 | 11:23 |
| *** sth has quit IRC | 11:25 | |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: don't use pip -e install for libraries https://review.openstack.org/84146 | 11:26 |
| *** jlibosva1 has joined #openstack-qa | 11:37 | |
| *** rohitk has joined #openstack-qa | 11:38 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add unit tests for all generators https://review.openstack.org/84414 | 11:40 |
| *** ams0 has joined #openstack-qa | 11:40 | |
| *** weshay has joined #openstack-qa | 11:41 | |
| *** jlibosva1 has quit IRC | 11:42 | |
| *** torandu has quit IRC | 11:43 | |
| *** saju_m has quit IRC | 11:44 | |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/tempest: Improve the extra routes test on router https://review.openstack.org/66454 | 11:46 |
| *** ams0 has quit IRC | 11:46 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add unit tests for all generators https://review.openstack.org/84414 | 11:47 |
| openstackgerrit | A change was merged to openstack/tempest: orchestration api tests remove duplicate client assignment https://review.openstack.org/83756 | 11:47 |
| *** torandu has joined #openstack-qa | 11:51 | |
| *** tosky has quit IRC | 11:56 | |
| *** saju_m has joined #openstack-qa | 11:57 | |
| *** tkammer has quit IRC | 12:01 | |
| *** saju_m has quit IRC | 12:02 | |
| *** jhenner has quit IRC | 12:02 | |
| *** rfolco has joined #openstack-qa | 12:03 | |
| *** rfolco has quit IRC | 12:03 | |
| *** afazekas has quit IRC | 12:04 | |
| *** rfolco has joined #openstack-qa | 12:04 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron https://review.openstack.org/58695 | 12:05 |
| *** jhenner has joined #openstack-qa | 12:05 | |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: don't use pip -e install for libraries https://review.openstack.org/84146 | 12:06 |
| *** che-arne has joined #openstack-qa | 12:08 | |
| *** dkranz has joined #openstack-qa | 12:09 | |
| *** amotoki has quit IRC | 12:11 | |
| *** afazekas has joined #openstack-qa | 12:18 | |
| *** tkammer has joined #openstack-qa | 12:18 | |
| *** saju_m has joined #openstack-qa | 12:21 | |
| *** dims has quit IRC | 12:22 | |
| *** saju_m has quit IRC | 12:23 | |
| *** markmcclain has joined #openstack-qa | 12:28 | |
| *** aysyd has joined #openstack-qa | 12:30 | |
| *** dims_ has joined #openstack-qa | 12:34 | |
| *** ashishsnigam has quit IRC | 12:34 | |
| mkoderer | mtreinish: around? | 12:35 |
| mkoderer | folks, instead of doing this https://review.openstack.org/#/c/79902/7/tempest/api/identity/base.py everywhere.. wouldn't it better to simply overload the _error_checker in the client? | 12:38 |
| mkoderer | dkranz: ^ | 12:38 |
| *** jlibosva1 has joined #openstack-qa | 12:38 | |
| *** rohitk has quit IRC | 12:38 | |
| *** saju_m has joined #openstack-qa | 12:40 | |
| *** jlibosva1 has quit IRC | 12:42 | |
| *** jgrimm has joined #openstack-qa | 12:43 | |
| openstackgerrit | Hemanth N proposed a change to openstack/tempest: Added Tempest_Installation_and_configuration.rst. It will fix the Bug #1201201 https://review.openstack.org/82076 | 12:44 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: don't use pip -e install for libraries https://review.openstack.org/84146 | 12:48 |
| *** rohitk has joined #openstack-qa | 12:54 | |
| vrovachev | sdague: ping :) | 13:00 |
| *** Madan has quit IRC | 13:01 | |
| *** shwetaap has joined #openstack-qa | 13:01 | |
| sdague | vrovachev: what's up/ | 13:02 |
| sdague | ? | 13:02 |
| vrovachev | sdague: hello :) I have added my questions in Agenda for 3 April: https://wiki.openstack.org/wiki/Meetings/QATeamMeeting. I can do that? | 13:04 |
| *** sphoorti has joined #openstack-qa | 13:05 | |
| sphoorti | anteaya: hello | 13:05 |
| anteaya | sphoorti: there you are | 13:05 |
| anteaya | dtroyer: when you are around, any objection to using the git.o.o urls for repo clones for the documentation at http://devstack.org/ ? | 13:06 |
| anteaya | sphoorti wrote a great blog post about using neutron with devstack: http://sphoorti.github.io/blog/devstack/devstack_setup.html | 13:07 |
| anteaya | when we discussed using a git.o.o url for the devstack clone, she questioned why devstack.org didn't do the same | 13:07 |
| anteaya | and I figured it was because noone had discussed it with you previously | 13:08 |
| anteaya | so here we are | 13:09 |
| *** dkranz has quit IRC | 13:10 | |
| *** beisner has joined #openstack-qa | 13:10 | |
| sdague | anteaya: it does use git.o.o for clones by default, the docs probably lag though | 13:13 |
| anteaya | yes was just asking about the docs | 13:13 |
| sdague | vrovachev: yes, definitely feel free to add there | 13:13 |
| *** afazekas has quit IRC | 13:13 | |
| anteaya | any objection to considering a patch for them? | 13:13 |
| sdague | anteaya: I don't think so. I actually don't quite understand the doc building process for that | 13:13 |
| anteaya | ah okay | 13:13 |
| vrovachev | sdague: thank you very much :) | 13:14 |
| anteaya | well I will take sphoorti with me and see how far we can get | 13:14 |
| *** tkammer has quit IRC | 13:14 | |
| sphoorti | anteaya: I will be around :) | 13:14 |
| sdague | vrovachev: please put your irc nick next to the items you added, so we know who to ping on it | 13:14 |
| anteaya | sphoorti: are you up for the challenge of offering a docs patch to devstack? | 13:14 |
| sphoorti | sure anteaya :) | 13:14 |
| anteaya | sphoorti: you will be doing the work and I will talk you through it | 13:14 |
| sdague | sphoorti / anteaya there is definitely a todo to get the devstack doc building built into an infra standard process | 13:15 |
| sdague | which it very much is no | 13:15 |
| sdague | is not | 13:15 |
| anteaya | first of all we need to look at the repo: http://git.openstack.org/cgit/openstack-dev/devstack/tree/ | 13:15 |
| *** saju_m has quit IRC | 13:15 | |
| *** mriedem has joined #openstack-qa | 13:15 | |
| *** tosky has joined #openstack-qa | 13:15 | |
| anteaya | the docs don't seem to live there | 13:16 |
| anteaya | sdague: which repo has the devstack.org docs? | 13:16 |
| vrovachev | sdague: done. thx :) | 13:16 |
| sphoorti | anteaya: maybe annegentle might help us | 13:17 |
| sdague | anteaya: I thought it was in a branch | 13:17 |
| sdague | but I coudl be wrong | 13:17 |
| anteaya | ah let's look at branches sphoorti | 13:17 |
| sdague | honestly, I don't know. I don't actually know that part of hte project | 13:17 |
| anteaya | sdague: fair enough | 13:17 |
| anteaya | sphoorti: I doubt annegentle knows the docs for devstack.org | 13:17 |
| anteaya | if all else fails dtroyer will know | 13:18 |
| sphoorti | okay. I ll try to search | 13:18 |
| *** jhenner has quit IRC | 13:23 | |
| anteaya | sphoorti: http://git.openstack.org/cgit/openstack-dev/devstack/tree/tools/build_docs.sh | 13:23 |
| sphoorti | anteaya: http://devstack.org/contributing.html . Here it is mentioned that git.o.o has the repository | 13:24 |
| *** dkranz has joined #openstack-qa | 13:24 | |
| sphoorti | i ll check that one | 13:24 |
| *** rohitk has quit IRC | 13:25 | |
| *** rohitk has joined #openstack-qa | 13:26 | |
| *** tkammer has joined #openstack-qa | 13:27 | |
| *** saju_m has joined #openstack-qa | 13:28 | |
| sphoorti | anteaya: so basically in this build_docs script a probably change could be changing the url to git.o.o , right ? | 13:29 |
| annegentle | anteaya: dtroyer knows, I have it somewhere and can dig if need bee | 13:32 |
| anteaya | sphoorti: yes | 13:32 |
| anteaya | annegentle: I think we found it | 13:32 |
| annegentle | anteaya: cool | 13:32 |
| anteaya | in the tools dir there is a build_docs script | 13:32 |
| anteaya | annegentle: thanks though | 13:32 |
| anteaya | sphoorti: do you feel comfortable offering a patch with that change? | 13:32 |
| sphoorti | yep anteaya . I would be able to learn | 13:33 |
| anteaya | sphoorti: okay you make a start | 13:33 |
| anteaya | don't stay stuck | 13:33 |
| anteaya | ask questions, I will check in with you in a while to see how you are doing | 13:34 |
| sphoorti | anteaya: file a bug on launchpad ? | 13:34 |
| anteaya | no, I don't think this need a launchpad bug | 13:34 |
| *** jgrimm has quit IRC | 13:34 | |
| sphoorti | the how does one go about a docs bug? | 13:34 |
| anteaya | neutron likes them I don't think -qa needs them | 13:34 |
| anteaya | just offer the patch | 13:34 |
| anteaya | it doesn't need to be a bug | 13:34 |
| anteaya | clone the repo, create a branch for this patch, edit the file, ensure git config has your name and email (and editor) | 13:35 |
| *** psharma has quit IRC | 13:36 | |
| anteaya | git add, git commit (write the message), git review | 13:36 |
| sphoorti | anteaya: I have the devstack repo | 13:36 |
| anteaya | great | 13:36 |
| anteaya | so ensure it is up to date | 13:36 |
| sphoorti | I ll just have to make changes to the file | 13:36 |
| sphoorti | gerrit is setup | 13:36 |
| anteaya | git status, git checkout master | 13:36 |
| anteaya | git remote update | 13:36 |
| sphoorti | I ll create another branch | 13:36 |
| anteaya | git pull --ff-only origin master | 13:36 |
| anteaya | is your repo up to date? | 13:37 |
| *** jhenner has joined #openstack-qa | 13:37 | |
| anteaya | sphoorti: run git log on your devstack repo | 13:37 |
| anteaya | does it match: http://git.openstack.org/cgit/openstack-dev/devstack/log/ | 13:37 |
| anteaya | if not | 13:37 |
| sphoorti | anteaya: i ll run it | 13:37 |
| anteaya | git status, git chekout master, git remote update, git pull --ff-only origin master | 13:38 |
| *** jlibosva1 has joined #openstack-qa | 13:39 | |
| sphoorti | anteaya: the logs show commits of 10 march | 13:39 |
| sphoorti | so I ll run the pull request | 13:39 |
| anteaya | with the --ff-only flag, right? | 13:39 |
| sphoorti | git pull --ff-only origin master | 13:39 |
| sphoorti | i ll run this | 13:40 |
| anteaya | awesome | 13:40 |
| anteaya | okay | 13:40 |
| anteaya | thanks for telling me what you are doing | 13:40 |
| *** mwagner_zzz has quit IRC | 13:40 | |
| anteaya | sphoorti: did you run git remote update first? | 13:40 |
| anteaya | and ensure you are on the master branch? | 13:41 |
| sphoorti | yes | 13:41 |
| sphoorti | now the pull request | 13:42 |
| sphoorti | checked | 13:42 |
| anteaya | great | 13:42 |
| sphoorti | done :) | 13:42 |
| anteaya | great | 13:43 |
| anteaya | so now do git config -l | 13:43 |
| anteaya | and if it doesn't have your name and email listed add them | 13:43 |
| anteaya | git config user.name "your name here" | 13:43 |
| anteaya | git config user.email "your email here" | 13:43 |
| sphoorti | anteaya: wont it have already? | 13:43 |
| anteaya | and I like to specifiy my editor | 13:43 |
| anteaya | sphoorti: look | 13:43 |
| anteaya | git config -l | 13:44 |
| sphoorti | since i did setup gerrit remote on neutron branch | 13:44 |
| anteaya | does it? | 13:44 |
| *** jlibosva1 has quit IRC | 13:44 | |
| anteaya | this is the devstack repo | 13:44 |
| anteaya | not the neutron repo | 13:44 |
| sphoorti | anteaya: the email and username is global so its visible | 13:44 |
| sphoorti | so is the gitreviewname | 13:45 |
| anteaya | okay great | 13:45 |
| *** markmcclain has quit IRC | 13:45 | |
| anteaya | I don't set globals so I am glad you checked | 13:45 |
| anteaya | so git config -l picks up the globals? | 13:45 |
| sphoorti | yes | 13:45 |
| anteaya | great | 13:45 |
| sphoorti | git config --global user.name "your name here" | 13:46 |
| anteaya | so branch, edit, git add, git commit, git review | 13:46 |
| sphoorti | that does it :) | 13:46 |
| anteaya | great | 13:46 |
| *** afazekas has joined #openstack-qa | 13:46 | |
| sphoorti | anteaya: I just want to confirm once. http://git.openstack.org/cgit/openstack-dev/devstack/tree/tools/build_docs.sh Here the Line 25 could be changed to git://git.openstack.org/openstack-dev/devstack.git | 13:48 |
| dkranz | mkoderer: how about making a comment in the bug ticket? | 13:48 |
| dkranz | mkoderer: I need to look again. | 13:48 |
| anteaya | sphoorti: yes | 13:48 |
| anteaya | sphoorti: that is what I would do | 13:48 |
| sphoorti | I ll do that then | 13:48 |
| Anju | dkranz: hiii , | 13:48 |
| anteaya | sphoorti: then you will get reviews and find out how many people agree with us | 13:48 |
| dkranz | Anju: hi | 13:49 |
| dtroyer | anteaya: g'morning….devstack.org docs are still in the pre-history stone age, meaning they're hosted in the old cloudbuilders github repo in the gh-pages branch. | 13:49 |
| Anju | I want to know the about comment you gave | 13:49 |
| anteaya | dtroyer: morning | 13:49 |
| anteaya | dtroyer: so do I have sphoorti steered in the right direction? | 13:49 |
| Anju | dkranz: for reminding you giving you the link :) | 13:49 |
| anteaya | dtroyer: or do you have better suggestions? | 13:49 |
| anteaya | the cloudbuiders github repo | 13:50 |
| dtroyer | the plain html files are exactly that, and a pull request against https://github.com/cloudbuilders/devstack/tree/gh-pages works for changes there | 13:50 |
| dtroyer | sorry, I haven't read all of the scrollback yet | 13:50 |
| anteaya | so outside our normal workflow | 13:50 |
| anteaya | dtroyer: np there is a lot | 13:50 |
| dtroyer | the shocco-generated stuff is rather funky and basically I have to do that, and I'm waaay overdue... | 13:50 |
| anteaya | sphoorti: so offering that review can't hurt | 13:50 |
| Anju | dkranz: https://review.openstack.org/#/c/82281/ | 13:51 |
| anteaya | dtroyer: so how coffee'd up are you | 13:51 |
| anteaya | dtroyer: should we have a conversation about how to pull that into gerrit? | 13:51 |
| dtroyer | I just woke up…I'm in CA this week so my body is all fouled up | 13:51 |
| dtroyer | ;) | 13:51 |
| anteaya | or are you wanting to put that conversation off for a bit? | 13:51 |
| anteaya | ha ha ha | 13:51 |
| dtroyer | at some point, yes, we should talk about moving everything... | 13:52 |
| dtroyer | this may not be the moment to do that though | 13:52 |
| anteaya | what point are you envisioning that conversation taking place | 13:52 |
| * anteaya nods | 13:52 | |
| dtroyer | about a year ago? | 13:52 |
| anteaya | just wanting to know when a good time to come back is | 13:52 |
| * anteaya hops into her handy time machine | 13:53 | |
| anteaya | remember that time we chatted | 13:53 |
| anteaya | about a year ago | 13:53 |
| anteaya | :D | 13:53 |
| dtroyer | in portland iirc | 13:53 |
| Anju | dkranz: should I use a e different image set here .?or I am understanding it wrong | 13:53 |
| anteaya | yes | 13:53 |
| anteaya | so the way forward ... | 13:54 |
| dtroyer | it used to be done in the RCB jenkins server (long gone now) so it shouldn't be hard, IIRC the sticky point was continuing to use github gh-pages site to host it. | 13:54 |
| anteaya | yes | 13:55 |
| anteaya | taht is not going to get a pass | 13:55 |
| anteaya | so we need to find some real estate for it | 13:55 |
| anteaya | that is a jeblair discussion | 13:55 |
| dtroyer | plus, I need to talk anotherjesse in to turning over the domain to the foundation…that should take all of one beer | 13:55 |
| anteaya | beer up | 13:56 |
| anteaya | when do you expect to see him/her | 13:56 |
| dtroyer | Thursday | 13:56 |
| anteaya | do your good work sir | 13:56 |
| anteaya | we can continue the conversation after that time | 13:56 |
| dtroyer | np | 13:56 |
| anteaya | in the meantime if sphoorti's patch is in any way not unhelpful, a review and a merge would help her opw applicaton | 13:57 |
| anteaya | if you would be so kind | 13:57 |
| *** afazekas has quit IRC | 13:57 | |
| sphoorti | anteaya: I made the changes | 13:57 |
| anteaya | once she offers it, of course | 13:57 |
| anteaya | sphoorti: great | 13:57 |
| anteaya | so git status | 13:57 |
| anteaya | git add, git commit, git review | 13:57 |
| sphoorti | I ll send it for review then | 13:57 |
| anteaya | let's see them | 13:57 |
| anteaya | great | 13:58 |
| sphoorti | anteaya: I am having a slight confusion | 14:03 |
| sphoorti | here http://git.openstack.org/cgit/openstack-dev/devstack.git it says repo not found | 14:04 |
| sphoorti | I found in tree | 14:05 |
| anteaya | git://git.openstack.org/openstack-dev/devstack | 14:05 |
| anteaya | or | 14:05 |
| anteaya | https://git.openstack.org/openstack-dev/devstack | 14:05 |
| anteaya | so https | 14:05 |
| anteaya | anad remove the cgit | 14:05 |
| anteaya | the cgit is required for rendering in the gui | 14:05 |
| *** tkammer has quit IRC | 14:05 | |
| anteaya | but should be removed for cloning | 14:06 |
| sphoorti | anteaya: I ll try that | 14:06 |
| sphoorti | I just got confused with repo not found error | 14:06 |
| anteaya | if you go to the summary page of every repo, at the bottom are the urls for cloning | 14:06 |
| anteaya | I'm glad you checked | 14:06 |
| anteaya | well done | 14:06 |
| dkranz | Anju: Sorry had to step away. | 14:06 |
| Anju | dkranz: np . will ask next time | 14:07 |
| dkranz | Anju: I was just saying that the set of images containing "raw" and "bare" need to be different. | 14:07 |
| sphoorti | thanks anteaya for telling me the cloning links are at bottom | 14:07 |
| dkranz | Anju: The way it is now, if you filter by bare and by raw you expect to get the same set of images back | 14:07 |
| sphoorti | forgot to check that | 14:07 |
| anteaya | sphoorti: np, it is a new interface | 14:08 |
| anteaya | people coming from github expect the interface to be the same and it isn't | 14:08 |
| dkranz | Anju: Just change one use of 'bare' or 'raw' to something else | 14:08 |
| *** yassine has joined #openstack-qa | 14:09 | |
| Anju | dkranz: did you ccheck the first patch | 14:09 |
| dkranz | Anju: What do you mean? | 14:10 |
| Anju | dkranz: https://review.openstack.org/#/c/82281/3/tempest/api/image/v1/test_images.py | 14:11 |
| mkoderer | dkranz: I had a closer look. So it's possible to do it in the error_checker, I will propose a patch tomorrow | 14:12 |
| dkranz | Anju: That is the patch where I made my comment. | 14:12 |
| dkranz | mkoderer: Great, thanks. | 14:12 |
| mkoderer | dkranz: but anyway https://review.openstack.org/#/c/79902 doesn't hurt | 14:12 |
| *** saju_m has quit IRC | 14:14 | |
| sphoorti | anteaya: what could be a better title for the commit message ? | 14:15 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for nova ec2 bug 1280532 https://review.openstack.org/84449 | 14:15 |
| mriedem | mtreinish: jogo: sdague: ^ really started spiking in the last day or so | 14:15 |
| mriedem | not sure why since it's a nova ec2 test and that code hasn't changed recently | 14:16 |
| sdague | mriedem: is that actually the same issue as the 2 top bugs in er? | 14:16 |
| anteaya | sphoorti: I don't know yet, I need to see the patch first | 14:16 |
| anteaya | sphoorti: submit it to gerrit and I can take a look and make a suggestion | 14:17 |
| sphoorti | I just changed that one line | 14:17 |
| sphoorti | okay I ll do that | 14:17 |
| anteaya | sphoorti: yes | 14:17 |
| anteaya | thanks | 14:17 |
| mriedem | sdague: doesn't look like it | 14:17 |
| sphoorti | I wrote the commit message | 14:17 |
| anteaya | sphoorti: good | 14:17 |
| sphoorti | was thinking if a better title would help | 14:17 |
| anteaya | sphoorti: submit to gerrit and then we can discuss it | 14:17 |
| mriedem | sdague: 1602 hits since 3/31 though | 14:17 |
| *** otherwiseguy has joined #openstack-qa | 14:17 | |
| sdague | mriedem: sure? because if this is fundamentally a network issue, my guess is it would show up there as well | 14:18 |
| mriedem | sdague: so the ssh one? | 14:18 |
| sdague | yeh | 14:19 |
| sdague | I'm just suspicious on how close the shapes of those top 2 bugs match | 14:20 |
| *** jgrimm has joined #openstack-qa | 14:20 | |
| *** tkammer has joined #openstack-qa | 14:20 | |
| *** markmcclain has joined #openstack-qa | 14:21 | |
| *** adalbas has joined #openstack-qa | 14:21 | |
| mriedem | well i can go poke through logs | 14:23 |
| openstackgerrit | Sphoorti proposed a change to openstack-dev/devstack: Change master repo to git.openstack.org https://review.openstack.org/84452 | 14:25 |
| mriedem | sdague: i'm not finding correlations, i'm not sure why the ec2 bug would be related to networking issues? | 14:26 |
| mriedem | in the ec2 bug it's a race with detaching volumes | 14:26 |
| sdague | mriedem: can you point me at a log? | 14:26 |
| mriedem | http://logs.openstack.org/37/84037/5/check/check-grenade-dsvm-partial-ncpu/db32161/logs/new/screen-n-api.txt.gz | 14:26 |
| mriedem | http://logs.openstack.org/37/84037/5/check/check-grenade-dsvm-partial-ncpu/db32161/console.html#_2014-03-31_21_20_26_739 | 14:27 |
| *** shardy is now known as shardy_afk | 14:28 | |
| openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Remove python25 workaround from glance_http https://review.openstack.org/84453 | 14:28 |
| openstackgerrit | Mauro Rodrigues proposed a change to openstack/tempest: Unit Tests for glance_http https://review.openstack.org/84454 | 14:28 |
| *** tkammer has quit IRC | 14:29 | |
| anteaya | sphoorti: okay my time I alloted for this has expired | 14:34 |
| anteaya | sphoorti: you are doing great | 14:34 |
| anteaya | sphoorti: I will be around tomorrow or ping if you need anything, and you are always welcome to ask a question in channel | 14:35 |
| sphoorti | thank you anteaya | 14:35 |
| *** yrabl has quit IRC | 14:35 | |
| anteaya | np | 14:35 |
| sphoorti | i have submitted the patch | 14:35 |
| anteaya | sphoorti: did you | 14:35 |
| sphoorti | yes | 14:35 |
| sphoorti | https://review.openstack.org/84452 | 14:35 |
| anteaya | so you did, 844452 | 14:35 |
| sphoorti | yep | 14:35 |
| sphoorti | and added you as the reviewer | 14:36 |
| *** afazekas has joined #openstack-qa | 14:37 | |
| sphoorti | anteaya: | 14:37 |
| anteaya | sphoorti: I have commented | 14:38 |
| sdague | mriedem: oh... new boto! | 14:39 |
| sdague | yay, it's pycon | 14:39 |
| *** jlibosva1 has joined #openstack-qa | 14:40 | |
| mriedem | sdague: i did look at boto releases and say there was a new one on 3/7 | 14:40 |
| mriedem | sdague: and this spiked a couple weeks after that - or is it a transitive dependency? | 14:40 |
| sdague | hmmm | 14:41 |
| anteaya | sphoorti: you know how to edit and use git commit --amend to work with the same change id for the patch, right? | 14:41 |
| *** tkammer has joined #openstack-qa | 14:41 | |
| sphoorti | yes anteaya | 14:43 |
| anteaya | k | 14:43 |
| sphoorti | anteaya: the period was actually intentional | 14:43 |
| sphoorti | I ll make those changes right away | 14:43 |
| sphoorti | thank you :) | 14:44 |
| *** jlibosva1 has quit IRC | 14:44 | |
| anteaya | np | 14:44 |
| anteaya | sphoorti: can you tell me why you wanted a whitespace before the period? | 14:45 |
| sdague | mriedem: ok, so this is a different error for sure | 14:45 |
| sdague | +A | 14:45 |
| mriedem | \o/ i'm not crazy | 14:45 |
| sphoorti | better visual clarity? | 14:45 |
| sphoorti | but now when I see it, the whitespace is not necessary | 14:46 |
| sphoorti | anteaya: | 14:46 |
| *** kgriffs|afk is now known as kgriffs | 14:46 | |
| anteaya | sphoorti: well it isn't proper english to leave the punctuation hanging lonely off the end of the sentence | 14:46 |
| anteaya | I don't know where people picked it up but I see it in things coming from europe as well | 14:47 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for nova ec2 bug 1280532 https://review.openstack.org/84449 | 14:47 |
| anteaya | the only time I think it is okay is if the punctiation will render a url as broken | 14:47 |
| mriedem | sdague: https://review.openstack.org/81762 | 14:47 |
| openstackgerrit | Sphoorti proposed a change to openstack-dev/devstack: Edit docs to include git.openstack.org url https://review.openstack.org/84452 | 14:47 |
| sphoorti | I was mentally thinking it to be a url and so thought would give the perios | 14:48 |
| sphoorti | *period | 14:48 |
| anteaya | ah | 14:48 |
| sphoorti | anyway I will keep that in mind the next time anteaya . | 14:48 |
| anteaya | ha ha ha | 14:48 |
| sphoorti | submitted a fresh patch | 14:48 |
| sphoorti | :P | 14:48 |
| sdague | mriedem: is it always failing thigns? | 14:49 |
| sdague | so I actually think that's not the root cause | 14:49 |
| *** rohitk has quit IRC | 14:49 | |
| sdague | because it completely passed | 14:49 |
| mriedem | sdague: it doesn't always fail | 14:49 |
| mriedem | but see the laundry list of things there that needed to not be flaky for that test to work | 14:49 |
| sdague | to land | 14:49 |
| sdague | this is why I was wondering if this is actually the network bug | 14:50 |
| mriedem | sdague: yeah, and it was 12 days ago | 14:50 |
| mriedem | so before the 3/31 spike | 14:50 |
| anteaya | sphoorti: looks good to me | 14:50 |
| sdague | mriedem: no, it just merged | 14:50 |
| sphoorti | anteaya: would I have to rebase? | 14:50 |
| sdague | and it passed all the tests to merge | 14:50 |
| sphoorti | I think there seems a problem | 14:50 |
| anteaya | sphoorti: no | 14:50 |
| mriedem | sdague: oh touche | 14:50 |
| sdague | twice actually | 14:50 |
| sdague | because of clean check | 14:50 |
| anteaya | sphoorti: what is the problem? | 14:50 |
| sdague | so it made it through 20 tempest runs without failing | 14:51 |
| sphoorti | New patchset patch-id matches previous patchset, but commit message has changed. | 14:51 |
| sdague | to merge | 14:51 |
| sphoorti | I forgot the rebase -i master | 14:51 |
| sphoorti | anteaya: | 14:51 |
| anteaya | sphoorti: no | 14:51 |
| anteaya | sphoorti: that is jenkins telling us that the code hasn't changed, just the commit message | 14:51 |
| sdague | which isn't to say it's not a real bug, because it is. But it's not a gate failing bug | 14:51 |
| anteaya | sphoorti: no action is required from you as a consequence of that message | 14:51 |
| sphoorti | so jenkins wont run the tests again anteaya ? | 14:52 |
| mriedem | sdague: right, it's like 99.8% check queue | 14:52 |
| sphoorti | okay great! | 14:52 |
| sphoorti | thank you anteaya for the +1 | 14:52 |
| anteaya | sphoorti: right, since there is no reason to do so | 14:52 |
| mriedem | sdague: this is the thing i'd think is related: "Attaching volume to a booted instance is not flaky." | 14:52 |
| anteaya | sphoorti: thanks for the patch | 14:52 |
| mriedem | sdague: i don't know what that's referring to | 14:52 |
| sphoorti | anteaya: you encouraged me :) | 14:52 |
| mriedem | afazekas: ^ ? | 14:52 |
| anteaya | sphoorti: so email an ammendment to your application and include the url of this patch | 14:52 |
| sphoorti | should I add dtroyer for review ? | 14:53 |
| anteaya | sphoorti: don't wait for it to be merged, since the url will stay the same | 14:53 |
| anteaya | sphoorti: if you want, he will get to it anyway | 14:53 |
| anteaya | up to you | 14:53 |
| *** shwetaap has quit IRC | 14:53 | |
| *** mlavalle has joined #openstack-qa | 14:54 | |
| *** shwetaap has joined #openstack-qa | 14:54 | |
| sphoorti | anteaya: the application process is different this time. So I ll have to make changes to existing one from my account | 14:54 |
| anteaya | sphoorti: okay, just ensure the review committee knows about this patch | 14:54 |
| sphoorti | yes. I ll ask marina about the same too | 14:55 |
| sphoorti | thanks a lot anteaya ! | 14:55 |
| anteaya | welcome | 14:55 |
| anteaya | thanks for your hard work | 14:55 |
| sphoorti | :) | 14:55 |
| *** sphoorti_ has joined #openstack-qa | 15:00 | |
| *** sphoorti has quit IRC | 15:00 | |
| mtreinish | mriedem: what happened here?: https://review.openstack.org/#/c/82537/ | 15:01 |
| mriedem | mtreinish: i don't know... | 15:02 |
| mriedem | looking | 15:02 |
| mtreinish | mriedem: you probably just need to manually rebase | 15:02 |
| mtreinish | I think that happened to me once too | 15:02 |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: unskip load balancer basic scenario test https://review.openstack.org/82480 | 15:03 |
| *** erw has joined #openstack-qa | 15:03 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Honor suspend/pause config switches in scenario tests https://review.openstack.org/82566 | 15:03 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Move run_ssh class variable into skip decorators https://review.openstack.org/82537 | 15:04 |
| mriedem | mtreinish: just pushed the rebase button in gerrit | 15:04 |
| *** TravT has joined #openstack-qa | 15:04 | |
| erw | am I wrong in saying that project-requirements-change.py is checking out the current master and resetting the branch so it isn’t actually testing the submitted patch? | 15:04 |
| erw | http://logs.openstack.org/15/82115/6/check/gate-nova-docker-requirements/638ea10/console.html | 15:04 |
| erw | (or worse, checking both… the current patch and master) | 15:05 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: normalize the services for a post grizzly world https://review.openstack.org/84287 | 15:05 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: turn back on log enforcement https://review.openstack.org/81216 | 15:05 |
| mtreinish | mriedem: ok approved again | 15:05 |
| mriedem | mtreinish: cool, thanks - the change looked the same after the rebase | 15:06 |
| *** bdperkin has quit IRC | 15:06 | |
| erw | basically, my requirements files don’t have newlines at the end… and I can’t fix it because the gate is checking the OLD files. | 15:06 |
| *** bdperkin has joined #openstack-qa | 15:08 | |
| *** bdperkin has quit IRC | 15:08 | |
| *** bdperkin has joined #openstack-qa | 15:08 | |
| *** sphoorti__ has joined #openstack-qa | 15:11 | |
| *** sphoorti_ has quit IRC | 15:12 | |
| *** chandankumar_ has joined #openstack-qa | 15:12 | |
| *** tkammer has quit IRC | 15:14 | |
| sdague | erw: that's probably an infra question | 15:17 |
| erw | sdague: yeah, I ended up submitting the bug on infra | 15:18 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Default RABBIT_HOST to be the SERVICE_HOST. https://review.openstack.org/84475 | 15:31 |
| dtroyer | erw: is that a cookiecutter-based project? I've seen that happen before and had to disable the requirements check job to merge the fix | 15:33 |
| *** shwetaap has quit IRC | 15:33 | |
| erw | dtroyer: possibly. Russell actually did the initial commit, so I’m uncertain. | 15:34 |
| *** shwetaap has joined #openstack-qa | 15:34 | |
| russellb | it was cookiecutter based, yes | 15:35 |
| erw | dtroyer: yeah, I think that’s what I’ll have to do as well.. or some fix to the script (like, “if the gate is already broken, allow a fix through”) | 15:35 |
| *** mrodden has quit IRC | 15:35 | |
| *** yassine has quit IRC | 15:40 | |
| *** jlibosva1 has joined #openstack-qa | 15:40 | |
| *** torandu has left #openstack-qa | 15:43 | |
| *** jlibosva1 has quit IRC | 15:45 | |
| *** vrovachev has quit IRC | 15:47 | |
| jlibosva | sdague: Hi, would you find a minute today to look on grenade patch? | 15:50 |
| *** mrodden has joined #openstack-qa | 15:51 | |
| openstackgerrit | Victor Stinner proposed a change to openstack-dev/devstack: Make stack.sh failing if SELinux is enforcing https://review.openstack.org/84482 | 15:53 |
| *** daudet has joined #openstack-qa | 15:54 | |
| *** otherwiseguy has quit IRC | 15:55 | |
| *** shwetaap has quit IRC | 15:56 | |
| *** jlibosva has quit IRC | 15:57 | |
| *** mmaglana has joined #openstack-qa | 16:03 | |
| *** safchain has quit IRC | 16:04 | |
| *** jlibosva has joined #openstack-qa | 16:05 | |
| *** amotoki has joined #openstack-qa | 16:10 | |
| *** Sukhdev has joined #openstack-qa | 16:10 | |
| *** mwagner_zzz has joined #openstack-qa | 16:16 | |
| *** rhsu has joined #openstack-qa | 16:17 | |
| *** Aimon has joined #openstack-qa | 16:19 | |
| openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: preparations for new tests https://review.openstack.org/84487 | 16:21 |
| openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Sahara: preparations for new tests https://review.openstack.org/84487 | 16:22 |
| *** rhsu has quit IRC | 16:25 | |
| *** shwetaap has joined #openstack-qa | 16:27 | |
| ylobankov | SergeyLukjanov: sdague: ping | 16:27 |
| *** Abhijeet has quit IRC | 16:28 | |
| *** dtroyer has quit IRC | 16:29 | |
| *** dtroyer has joined #openstack-qa | 16:30 | |
| *** maru_afk is now known as marun | 16:32 | |
| *** Sukhdev has quit IRC | 16:33 | |
| *** daudet has quit IRC | 16:34 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix Neutron plugin XTRACE handling https://review.openstack.org/83839 | 16:36 |
| *** shwetaap has quit IRC | 16:36 | |
| *** kgriffs is now known as kgriffs|afk | 16:39 | |
| *** jlibosva1 has joined #openstack-qa | 16:42 | |
| *** jlibosva1 has quit IRC | 16:46 | |
| *** daudet_ has joined #openstack-qa | 16:47 | |
| *** otherwiseguy has joined #openstack-qa | 16:50 | |
| *** rhsu has joined #openstack-qa | 16:51 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/84146 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1254872 | 16:52 |
| *** mriedem has quit IRC | 16:56 | |
| *** shwetaap has joined #openstack-qa | 16:57 | |
| *** jordanP has quit IRC | 17:00 | |
| *** arosen has quit IRC | 17:02 | |
| *** rohitk has joined #openstack-qa | 17:02 | |
| afazekas | mriedem: instable volume attache https://review.openstack.org/#/c/52740/, stable volume attache: https://review.openstack.org/#/c/62886/ (the ec2 test does this style), I was also know about https://bugs.launchpad.net/nova/+bug/1291835 | 17:03 |
| *** afazekas is now known as afazekas_noteboo | 17:05 | |
| *** amotoki has quit IRC | 17:05 | |
| *** Sukhdev has joined #openstack-qa | 17:05 | |
| *** afazekas has joined #openstack-qa | 17:08 | |
| *** Sukhdev has quit IRC | 17:09 | |
| *** mmaglana has quit IRC | 17:12 | |
| *** mlavalle has quit IRC | 17:13 | |
| *** harlowja_away is now known as harlowja | 17:14 | |
| *** mwagner_zzz is now known as mwagner_ | 17:14 | |
| afazekas_noteboo | mriedem: this was the https://bugs.launchpad.net/nova/+bug/1270608 volume bug, which could effect the ec2 test while it was originally skiped because of a neutron bug | 17:16 |
| *** Aimon has quit IRC | 17:16 | |
| *** rohitk has quit IRC | 17:16 | |
| *** rohitk has joined #openstack-qa | 17:17 | |
| *** jpich has quit IRC | 17:18 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/qa-specs: Mirror tox and sphinx fixes from nova-specs repo https://review.openstack.org/84502 | 17:18 |
| *** mriedem has joined #openstack-qa | 17:19 | |
| *** jlibosva has quit IRC | 17:20 | |
| *** shwetaap has quit IRC | 17:22 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: add trace_requests option to debug section https://review.openstack.org/82517 | 17:25 |
| *** afazekas_noteboo is now known as _afazekas | 17:25 | |
| mtreinish | sdague: ^^ so a potential point of confusion in that patch. The regex option will work if debug enabled option is false. | 17:28 |
| mtreinish | not that there's anything wrong with it being that way. But, it's a little weird | 17:28 |
| sdague | mtreinish: yes, that's vaguely intentional the debug group is weird to have that enabled flag | 17:29 |
| sdague | which should be cleared up | 17:29 |
| mtreinish | sdague: it's not actually the group enabled, but that network info barf on neutron failure enabled | 17:30 |
| sdague | yes | 17:30 |
| sdague | which needs to be cleaned up | 17:30 |
| mtreinish | do you think that we still need that option. Because I'm not a fan of having that in tree | 17:31 |
| *** morganfainberg_Z is now known as morganfainberg | 17:31 | |
| mtreinish | it was mostly supposed to be a gate debug thing | 17:31 |
| sdague | honestly, I think we need to discuss and rethink that one | 17:33 |
| sdague | it is useful in some situations | 17:33 |
| sdague | but it's not generic enough | 17:33 |
| *** ashishsnigam has joined #openstack-qa | 17:33 | |
| ashishsnigam | Can anyone help me running tempest suite for MySQL? | 17:35 |
| *** rohitk has quit IRC | 17:36 | |
| *** che-arne has quit IRC | 17:36 | |
| mtreinish | sdague: I'm fine with leaving network debug but only if the assumption about being a single node environment isn't there anymore. (or local sudo access) | 17:37 |
| sdague | yeh, I think the reality is we need to think about how we want this to work long term | 17:38 |
| sdague | even if it's a couple cycles out | 17:38 |
| sdague | because we could really push the idea of state dump api calls back to the core services | 17:38 |
| mtreinish | sdague: yeah that's what I'd rather see. That seems more tempesty (for lack of a better word) and it could also probably be broadly useful. | 17:39 |
| sdague | honestly, that is probably worth a session at summit | 17:40 |
| *** TravT has quit IRC | 17:40 | |
| mtreinish | which track? neutron, cross project, or qa? | 17:40 |
| mtreinish | I'm fine with doing it in qa but I'm not sure the right people would be there | 17:41 |
| sdague | that's a good question. Honestly, it's probably qa on this cycle, because I don't think it's well formed enough of an idea for cross project | 17:41 |
| *** jlibosva has joined #openstack-qa | 17:43 | |
| mtreinish | well I'll add it to my list for sessions then. I'm going to probably start writing them up later this week. | 17:43 |
| *** TravT has joined #openstack-qa | 17:44 | |
| sdague | want to build out an etherpad that can be discussed at the QA meeting this week? | 17:45 |
| mtreinish | sdague: yeah that's a good idea | 17:45 |
| *** erw has quit IRC | 17:45 | |
| mtreinish | I'll start that tonight (the list is on my workstation which is shutdown at home) | 17:45 |
| mtreinish | it's a good place for it... | 17:46 |
| *** chandankumar_ has quit IRC | 17:46 | |
| tosky | question about Tempest (still new around): a (scenario) test which creates some instances, logs into one of them and git clone an external test (specific for the generated environment), compiles and run it, and report back according its status, would it be acceptable for Tempest? | 17:48 |
| *** jlibosva has quit IRC | 17:48 | |
| openstackgerrit | A change was merged to openstack/tempest: Move run_ssh class variable into skip decorators https://review.openstack.org/82537 | 17:49 |
| mtreinish | tosky: probably not because it depends on the external network which is something we don't want to have in tempest. | 17:51 |
| mtreinish | tosky: but the spirit is there, that's the right way to think about scenario testing | 17:51 |
| tosky | mtreinish: interesting, "no external network" policy on the instance, on the node where tempest is executed, or both? | 17:52 |
| mtreinish | tosky: it just introduces a failure point outside of our control when running the tests | 17:52 |
| mtreinish | the policy is that it has to be runnable in the gate to be merged | 17:52 |
| mtreinish | and having a dependency on an external git repo is something that we won't have in the gate | 17:53 |
| *** kgriffs|afk is now known as kgriffs | 17:53 | |
| tosky | but this cuts out the possibility of using a set of existing domain-specific tests, unless you want to synchronize them from time to time | 17:54 |
| tosky | which is going to be quite painful | 17:54 |
| *** shwetaap has joined #openstack-qa | 17:55 | |
| tosky | having the "run this existing test" as an optional part of the test, with a variable which specifies the path to a tarball or a (git) url, would it be acceptable? | 17:55 |
| tosky | also, in the hypothesis you control the gate, because this test, if I understand the current structure, it's likely to require some specific resources (prebuilt images) | 17:55 |
| tosky | as I am here: there is a time limit for tests running on gates, right? | 17:57 |
| *** mtreinish has quit IRC | 18:00 | |
| *** mwagner_ is now known as mwagner_afk | 18:00 | |
| *** mtreinish has joined #openstack-qa | 18:01 | |
| *** jamielennox|away is now known as jamielennox | 18:02 | |
| *** otherwiseguy has quit IRC | 18:05 | |
| tosky | mtreinish: sorry for bothering again, did you see my last questions before you disconnected? | 18:05 |
| *** rockyg has joined #openstack-qa | 18:08 | |
| *** jerryz has joined #openstack-qa | 18:11 | |
| *** dkranz has quit IRC | 18:12 | |
| *** Sukhdev has joined #openstack-qa | 18:14 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/78861 failed gate-neutron-python26, gate-neutron-python27 in the gate queue with an unrecognized error | 18:14 |
| *** jlibosva has joined #openstack-qa | 18:15 | |
| *** dkranz has joined #openstack-qa | 18:26 | |
| *** erw has joined #openstack-qa | 18:34 | |
| *** sphoorti__ has quit IRC | 18:38 | |
| mtreinish | tosky: which one? it looks like I got netsplit | 18:40 |
| tosky | tosky: from "but this cuts out the possibility of using a set of existing domain-specific tests, unless you want to synchronize them from time to time" | 18:41 |
| * tosky is sleepy | 18:41 | |
| tosky | mtreinish: ^ | 18:41 |
| tosky | and then: having the "run this existing test" as an optional part of the test, with a variable which specifies the path to a tarball or a (git) url, would it be acceptable? | 18:42 |
| mtreinish | tosky: so optional is fine but to get the code merged it needs to run in a ci system. (whether that's part of infra's or an external one) | 18:43 |
| mtreinish | adding things that pull from external locations over the network will just introduce flakiness to the tests because networks are unreliable at the scale of openstack's ci | 18:43 |
| *** catohornet1 has joined #openstack-qa | 18:44 | |
| tosky | mtreinish: right, but if the codepath is optional? At least it gives the possibility of running it manually | 18:44 |
| tosky | mtreinish: and what if you control the gate and enable the optional part with a locally-available tarball of the external test? | 18:44 |
| mtreinish | tosky: if you need something external for a test. The best way to do it would be to preload it with devstack/devstack-gate so it's already there | 18:45 |
| mtreinish | look at how we grab the cirros images for example | 18:45 |
| mtreinish | tosky: if the codepath is optional and not exercised in a ci system then the patch won't be approved | 18:45 |
| mtreinish | we need to verify it's functionality continually over time | 18:45 |
| mtreinish | otherwise it'll just bitrot | 18:45 |
| tosky | I see - so basically it could go in, following a similar approach of other "mutable" resources | 18:46 |
| tosky | like cirros images, and I guess others | 18:46 |
| tosky | so I need to check how it works in those cases | 18:46 |
| mtreinish | tosky: you should also look at the neutron lbaas scenario test. It does an interesting thing where it starts a fake web server to test load balancing. (I'm not exactly sure what you're looking to test but this seems similar) | 18:49 |
| tosky | mtreinish: sahara | 18:49 |
| tosky | mtreinish: the existing test is for the hadoop pool "inside" | 18:49 |
| tosky | the idea is to recycle the existing upstream tests to stress it | 18:50 |
| *** kgriffs is now known as kgriffs|afk | 18:50 | |
| tosky | mtreinish: there is a time limit for tests running on gate, if I understand it correctly | 18:51 |
| *** mmaglana has joined #openstack-qa | 18:51 | |
| mtreinish | tosky: yeah we try to limit the runtime for tests, an hour total is pushing the limit. But for things that take a really long time we can split off a separate job to run those tests | 18:52 |
| mtreinish | for example there is a heat slow job | 18:52 |
| tosky | yes, I totally understand the need for a time limit; but if you can run a specific test separately, then it's fine | 18:53 |
| tosky | do you plan to extend somehow Tempest functionalities to have also long-term running tests? Like "non-approval" gates? | 18:54 |
| tosky | maybe it's already there and I didn't read too much the documentation | 18:54 |
| mtreinish | tosky: I'm not following. The timelimit isn't actually hardcoded into tempest (well actually I think it's 500sec per test method). But the total runtime is enforced by the ci system | 18:55 |
| tosky | mtreinish: so, if Tempests tests are meant to be running on CI system, with an hardcoded time limit, is there some official place (apart from devs and qes systems) where long-term tests are executed? | 18:56 |
| tosky | s/CI systems/gates/ | 18:56 |
| *** evgenyf has quit IRC | 18:58 | |
| mtreinish | tosky: no tempest tests can be run anywhere by anyone. The timelimit requirement isn't inherent to tempest, but the infra ci which we gate on. | 18:59 |
| mtreinish | tempest itself doesn't have a concept of a total runtime limit | 18:59 |
| *** jerryz has quit IRC | 18:59 | |
| SergeyLukjanov | tosky, we're planning to re-write current integration tests as tempest scenarios in J | 18:59 |
| SergeyLukjanov | tosky, sahara integration tests* | 19:00 |
| tosky | SergeyLukjanov: oh, oki, I will try to add some bits then | 19:00 |
| tosky | mtreinish: I was thinking about an official place (CI), different from the gates, with longer time limits (because it does not block gerrit) where a failed result would raise some alarm | 19:01 |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/tempest: Add test for OS::Neutron::* Heat resource https://review.openstack.org/81909 | 19:05 |
| tosky | mtreinish: and before I forget, thanks for answers | 19:05 |
| mtreinish | tosky: that would be a nonvoting or an experimental job | 19:05 |
| mtreinish | tosky: you should take a look at the job definitions and zuul layouts | 19:06 |
| mtreinish | SergeyLukjanov: ^^^ is there a good wiki page on that stuff? | 19:06 |
| *** mmaglana has quit IRC | 19:06 | |
| SergeyLukjanov | mtreinish, i'm afraid that we have no such page, I'll ask ylobankov to make one (he's already investigating it) | 19:07 |
| *** markmcclain has quit IRC | 19:08 | |
| *** jlibosva has quit IRC | 19:09 | |
| *** dkranz has quit IRC | 19:11 | |
| *** ashishsnigam has quit IRC | 19:14 | |
| *** dkranz has joined #openstack-qa | 19:25 | |
| *** otherwiseguy has joined #openstack-qa | 19:29 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: unskip test_list_servers_filtered_by_ip https://review.openstack.org/84533 | 19:41 |
| *** tosky has quit IRC | 19:41 | |
| *** TravT has quit IRC | 19:45 | |
| *** TravT has joined #openstack-qa | 19:54 | |
| *** yamahata has joined #openstack-qa | 19:55 | |
| *** shwetaap has quit IRC | 20:00 | |
| *** markmcclain has joined #openstack-qa | 20:03 | |
| *** mwagner_afk has quit IRC | 20:03 | |
| *** Sukhdev has quit IRC | 20:14 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: unskip test_list_non_public_flavor https://review.openstack.org/84541 | 20:17 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Unskip test_load_balancer_basic scenario https://review.openstack.org/84542 | 20:21 |
| *** shwetaap has joined #openstack-qa | 20:25 | |
| *** kgriffs|afk is now known as kgriffs | 20:34 | |
| *** kgriffs is now known as kgriffs|afk | 20:36 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: turn back on log enforcement https://review.openstack.org/81216 | 20:39 |
| openstackgerrit | A change was merged to openstack-dev/devstack: don't use pip -e install for libraries https://review.openstack.org/84146 | 20:39 |
| openstackgerrit | A change was merged to openstack/tempest: add trace_requests option to debug section https://review.openstack.org/82517 | 20:39 |
| openstackrecheck | openstack/python-swiftclient change: https://review.openstack.org/84023 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 20:40 |
| anteaya | mtreinish: https://review.openstack.org/#/c/84258/ hit the same bug kevinbenton was asking about in -infra not long ago | 20:42 |
| anteaya | it seems there is a neutron unit test bug during a directory creation | 20:42 |
| *** dims_ has quit IRC | 20:44 | |
| mtreinish | anteaya: oh I hadn't even checked it since the last recheck | 20:50 |
| mtreinish | has that been fixed or should I start digging into it? | 20:51 |
| anteaya | not sure | 20:51 |
| anteaya | kevinbenton might know more | 20:51 |
| anteaya | I just know he hit it as well | 20:51 |
| * anteaya heads out for a walk | 20:53 | |
| openstackgerrit | Paul Van Eck proposed a change to openstack/tempest: Add tests for nova floating IP Heat resources https://review.openstack.org/82282 | 20:57 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/83855 failed gate-neutron-python26, gate-neutron-python27 in the gate queue with an unrecognized error | 21:00 |
| *** aysyd has quit IRC | 21:01 | |
| *** rfolco has quit IRC | 21:02 | |
| *** rockyg has quit IRC | 21:03 | |
| *** cyeoh has quit IRC | 21:04 | |
| *** cyeoh has joined #openstack-qa | 21:05 | |
| sdague | dtroyer: opinions on - https://review.openstack.org/#/c/58695/ ? | 21:10 |
| sdague | I think it's good to go now | 21:10 |
| *** mlavalle has joined #openstack-qa | 21:16 | |
| BobBall | sdague: Introduction of trove into devstack seems to have broken xenserver CI because they don't customise based on devstack's localrc but instead have hardcoded values in the config files | 21:19 |
| SlickNik | BobBall: The guest config is not used for anything other than the trove guest; I'm trying to understand why this would break xenserver CI? | 21:20 |
| BobBall | hmmm - perhaps I've not understood right yet then - carry on talking on #openstack-trove? | 21:21 |
| SlickNik | BobBall: Installing trove doesn't shouldn't alter the network devstack creates in any way. | 21:21 |
| SlickNik | BobBall: Sure, one sec. Let me switch there | 21:21 |
| BobBall | the network isn't altered, correct, but there are hardcoded values in places that seem to be used | 21:21 |
| *** adalbas has quit IRC | 21:29 | |
| *** catohornet has quit IRC | 21:30 | |
| *** wramthun has quit IRC | 21:30 | |
| *** wramthun has joined #openstack-qa | 21:31 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Trove: Set appropriate service URLs https://review.openstack.org/84557 | 21:31 |
| sdague | BobBall: if there is a fix for devstack trove support to make it more generic, point me at the review. I'll fast track it if it's straight forward | 21:31 |
| BobBall | sdague: perhaps https://review.openstack.org/84557 | 21:32 |
| BobBall | sdague: but waiting on test results ;) | 21:32 |
| sdague | BobBall: so, actually, you should pull in lib/nova and use the vars it sets | 21:33 |
| sdague | because we let you override them | 21:33 |
| BobBall | hmmm - of course... *looks* | 21:33 |
| sdague | you can see where we build the urls for the service catalog | 21:33 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/elastic-recheck: Add a fingerprint for bug 1301027 https://review.openstack.org/84558 | 21:34 |
| *** dkranz has quit IRC | 21:34 | |
| BobBall | yes - seen that... question - what's the best way to do this in devstack? You said import lib/nova, but it's already imported by devstack, right? therefore we can just use the NOVA vars? | 21:35 |
| sdague | yes... I think so | 21:36 |
| sdague | I'm not sure if it would hurt to source it again | 21:36 |
| sdague | to handle possible future refactors | 21:37 |
| BobBall | I'll try without sourcing first - I don't like resourcing; it can change the world | 21:37 |
| sdague | sure | 21:37 |
| *** wramthun has quit IRC | 21:37 | |
| sdague | I suppose it will break horribly if the vars aren't defined | 21:37 |
| sdague | as long as it's horrible break, and not silent break, we're good | 21:38 |
| catohornet1 | Is there an easier way to import an unbuntu iso to openstack? | 21:38 |
| clarkb | catohornet1: should be able to grab ubuntu's qcow image and upload that into glance | 21:39 |
| SlickNik | sdague: FYI https://review.openstack.org/#/c/82888/ finally merged. (Bye bye mockito!) | 21:42 |
| sdague | SlickNik: awesome | 21:42 |
| *** wramthun has joined #openstack-qa | 21:42 | |
| sdague | that means we can land req sync to trove now, right? | 21:42 |
| sdague | jogo you had that change somewhere right | 21:42 |
| jogo | sdague: oh I didn't but I can do that now | 21:43 |
| SlickNik | sdague: Yes! Finally the Jenkins' req sync's are running green. (https://review.openstack.org/#/c/79648/) | 21:44 |
| sdague | nice | 21:44 |
| *** mriedem has quit IRC | 21:44 | |
| jogo | sdague: trove is already there | 21:44 |
| sdague | jogo: was it your "explode if something is left over" patch? | 21:45 |
| jogo | were you talking about https://review.openstack.org/#/c/80690/ | 21:45 |
| jogo | rerunning that now | 21:45 |
| sdague | yeh, that one | 21:45 |
| sdague | ok, cool | 21:45 |
| sdague | if that passes, we should get that in | 21:45 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Trove: Set appropriate service URLs https://review.openstack.org/84557 | 21:45 |
| SlickNik | Also submitted this to remove the non-voting trove gate now that trove is part of the integrated gate. https://review.openstack.org/#/c/84369/ | 21:46 |
| *** wramthun has quit IRC | 21:49 | |
| *** salv-orlando_ has joined #openstack-qa | 21:49 | |
| *** enikanorov has joined #openstack-qa | 21:49 | |
| *** wramthun has joined #openstack-qa | 21:49 | |
| *** erw_ has joined #openstack-qa | 21:51 | |
| *** mtaylor has joined #openstack-qa | 21:53 | |
| *** rhsu has quit IRC | 21:54 | |
| *** ruhe has quit IRC | 21:54 | |
| *** openstackgerrit has quit IRC | 21:54 | |
| *** salv-orlando has quit IRC | 21:54 | |
| *** mordred has quit IRC | 21:54 | |
| *** erw has quit IRC | 21:54 | |
| *** enikanorov_ has quit IRC | 21:54 | |
| *** salv-orlando_ is now known as salv-orlando | 21:54 | |
| *** erw_ is now known as erw | 21:54 | |
| *** ruhe has joined #openstack-qa | 21:55 | |
| sdague | BobBall: off to dinner now, I'll check on the patch later | 21:56 |
| *** rhsu has joined #openstack-qa | 21:56 | |
| catohornet1 | Thanks clarkb! | 21:59 |
| *** weshay has quit IRC | 22:04 | |
| *** dims_ has joined #openstack-qa | 22:06 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/83895 failed gate-heat-pep8 in the gate queue with an unrecognized error | 22:07 |
| *** shwetaap has quit IRC | 22:08 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/82486 failed gate-heat-pep8 in the gate queue with an unrecognized error | 22:14 |
| *** dims_ has quit IRC | 22:18 | |
| *** jgrimm has quit IRC | 22:23 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/84367 failed gate-neutron-python26, gate-neutron-python27 in the gate queue with an unrecognized error | 22:23 |
| *** jaypipes has joined #openstack-qa | 22:24 | |
| *** ChanServ sets mode: +o jaypipes | 22:24 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/83739 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 22:25 |
| *** giulivo has quit IRC | 22:27 | |
| *** shwetaap has joined #openstack-qa | 22:28 | |
| *** giulivo has joined #openstack-qa | 22:29 | |
| *** TravT has quit IRC | 22:36 | |
| *** dims_ has joined #openstack-qa | 22:36 | |
| *** BobBall_ has joined #openstack-qa | 22:38 | |
| *** BobBall has quit IRC | 22:38 | |
| *** BobBall_ has quit IRC | 22:41 | |
| *** BobBall has joined #openstack-qa | 22:43 | |
| *** BobBall is now known as Guest86798 | 22:44 | |
| *** yfried__ has quit IRC | 22:44 | |
| *** otherwiseguy has quit IRC | 22:44 | |
| *** shwetaap has quit IRC | 22:51 | |
| *** markmcclain has quit IRC | 22:52 | |
| *** mlavalle has quit IRC | 22:53 | |
| *** catohornet1 has quit IRC | 23:07 | |
| *** yamahata has quit IRC | 23:08 | |
| *** mriedem has joined #openstack-qa | 23:09 | |
| *** sthaha has joined #openstack-qa | 23:21 | |
| *** _afazekas has quit IRC | 23:21 | |
| *** mrodden has quit IRC | 23:30 | |
| *** mwagner_afk has joined #openstack-qa | 23:32 | |
| *** daudet_ has left #openstack-qa | 23:32 | |
| *** otherwiseguy has joined #openstack-qa | 23:36 | |
| *** mrodden has joined #openstack-qa | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!