| *** sthaha has joined #openstack-qa | 00:08 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/58191 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 00:34 |
|---|---|---|
| jamielennox | anyone with some grenade knowledge here? I can't see what's wrong here: https://launchpad.net/bugs/1262934 and it's preventing me landing reviews in keystone | 01:23 |
| morganfainberg | jamielennox, ick, granade issues? | 01:31 |
| *** pmathews has joined #openstack-qa | 01:32 | |
| jamielennox | morganfainberg: it's been the same thing for a week or so now, every now and then i get on and try to bug someone about it and nothing | 01:32 |
| morganfainberg | jamielennox, If you haven't gotten it resolved after christmas / holiday time, I'll follow up w/ folks for you during the day (US) | 01:33 |
| jamielennox | really starting to annoy me, because i have no idea how to even go about finding where it's coming from, i'm currently seeing if i can take the localrc from a grenade run and run the same thing locally | 01:33 |
| morganfainberg | jamielennox, see if we can rally some help | 01:33 |
| morganfainberg | jamielennox, yeah, :( wish i was more help on that front | 01:33 |
| jamielennox | but it appears like the branch setup is done by some sort of gate proxy and not grenade itself so i can't even figure out how to replicate it | 01:33 |
| morganfainberg | jamielennox, a lot of that work (iirc) is done in the jenkins slave scripts | 01:34 |
| morganfainberg | at least for most gate jobs | 01:34 |
| jamielennox | yea, well the keystone_branch is master so somewhere they are pointing the repo at jenkins - but not in a way that i can use it again later | 01:34 |
| morganfainberg | =/ | 01:34 |
| jamielennox | morganfainberg: it's the reason that there are no passing kds builds :( i need to get some reviews on that if we want to land it for I2 | 01:35 |
| morganfainberg | jamielennox, well, post holidays when we have more infra types back, i'll help you hunt someone down if you haven't already solved it | 01:36 |
| morganfainberg | i just don't know if there is much else i can do until people are done holiday-ing | 01:36 |
| jamielennox | morganfainberg: yea, i know - i've got the same thing, i won't be back till ike the 6th - was going for a last minute fix :) | 01:37 |
| morganfainberg | honestly, i should learn more about granade, but i have a couple patchsets i need to get up and some other things that need attention while holiday is in effect | 01:37 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: rename old config object to TempestConfigPrivate https://review.openstack.org/63529 | 01:40 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: remove last errant file parse warning https://review.openstack.org/63649 | 01:40 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: remove unneeded __init__ file https://review.openstack.org/63530 | 01:40 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: experimental skip decorators https://review.openstack.org/63540 | 01:40 |
| sdague | jamielennox: so how far have you debugged the issue? like where does it fail in the console, and what service is failing when that's the case? We keep pretty extensive logs for a grenade job, so you should have most of what you need to get to root cause there. | 01:48 |
| jamielennox | sdague: hey! so i think that the old environment is being kept around and when we upgrade keystone we are getting a collision of the services that are still running | 01:53 |
| jamielennox | sdague: https://review.openstack.org/#/c/63246/ is my test case because all it does is update the requirements to add WSME to keystone | 01:54 |
| morganfainberg | sdague, isn't it a sunday and holiday season? (not that i'm complaining that you're around to help jamielennox out ;) ) | 01:54 |
| sdague | morganfainberg: it is... so I'll have to run from the computer quickly | 01:54 |
| morganfainberg | sdague, well.. happy holidays man :) have a good one. (i'll let you back to chatting w/ jamielennox now) | 01:54 |
| jamielennox | sdague: i've been going through devstack following the install_stackforge stuff but that doesn't appear to be run from grenade | 01:55 |
| sdague | jamielennox: if you read the console log you can see we are applying the new global requirements, which should have picked that up | 01:55 |
| jamielennox | but one problem is it's failing in swift-proxy but i can't see anywhere that swift-proxy actually has a dependency on wsme | 01:55 |
| sdague | http://logs.openstack.org/46/63246/1/check/check-grenade-dsvm/23e94a6/console.html.gz#_2013-12-20_01_15_35_769 | 01:55 |
| *** harlowja has joined #openstack-qa | 01:55 | |
| *** pmathews has quit IRC | 01:56 | |
| sdague | does ceilometer have swift hooks? | 01:59 |
| jamielennox | sdague: i've no idea - but i previously identified that the old version of ceilometer had the same conflicting requirements | 02:00 |
| sdague | the old wsme comes from ceilometer dragging it in - http://logs.openstack.org/46/63246/1/check/check-grenade-dsvm/23e94a6/console.html.gz#_2013-12-20_01_07_20_137 | 02:00 |
| sdague | the reason swift is exploding is actually because wsme is upgraded | 02:00 |
| sdague | and something, probably an entry point, requires the old version | 02:00 |
| jamielennox | or just that ceilometer is still running with the older reqs when swift is being upgraded | 02:01 |
| sdague | it's not that it's running | 02:01 |
| sdague | swift is actually failing to start | 02:01 |
| sdague | so something in it's load path requires the old version | 02:01 |
| jamielennox | yep, but nothing i can see in swift in devstack is out of the swift tree other than authtoken | 02:02 |
| jamielennox | unless grenade is setting that up differently | 02:02 |
| sdague | so ceilometer works by injecting code into other projects | 02:02 |
| sdague | that's probably the issue | 02:02 |
| sdague | and it probably comes down to the fact that ceilometer is installed on the old side, but we don't seem to be upgrading it. | 02:03 |
| sdague | search for /opt/stack/old/ceilometer and /opt/stack/new/ceilometer | 02:04 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61168 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1256106 and https://bugs.launchpad.net/bugs/1249065 | 02:04 |
| jamielennox | huh, no new/ceilometer - so it's not being upgraded at all | 02:04 |
| sdague | so probably the fastest path is to stop installing ceilometer on the old side until they have an upgrade script in there | 02:04 |
| sdague | yeh | 02:04 |
| jamielennox | or at least until after swift is trying to use it | 02:04 |
| sdague | right | 02:05 |
| sdague | or create an upgrade-ceilometer piece in grenade, and hope all it needs is setup_devel | 02:06 |
| jamielennox | yea, nothing in grenade that would appear to do much for ceilometer, so it needs to not be not in enabled_services for gate jobs | 02:06 |
| jamielennox | it would have to be ordered though right - ceilometer would have to be pretty much the first piece to be upgraded | 02:07 |
| sdague | https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L256 | 02:07 |
| sdague | so I expect we're taking the global service list in the grenade jobs (and after the big config / layout refactor something might have gotten tweaked) | 02:08 |
| sdague | I thought we had the smaller service list here, I'm actually surprised the ceilometer was being added | 02:08 |
| sdague | given all of it's issues it's probably better to pull it here | 02:08 |
| sdague | https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L49 is the logic that probably needs to be fixed | 02:10 |
| sdague | anyway, I need to stop working now. I will be on tomorrow. | 02:10 |
| sdague | that's my last day for the year | 02:10 |
| jamielennox | sdague: ok, i'll have a look - i don't yet see ceilometer mentioned their in infra | 02:11 |
| sdague | the issue you'll run into of devstack or grenade locally vs. devstack-gate, is the wrapper to bring up the d-g environment | 02:12 |
| sdague | https://github.com/openstack-infra/devstack-gate/blob/master/README.rst - "Simulating Devstack Gate Tests" | 02:12 |
| sdague | good reading | 02:12 |
| jamielennox | sdague: ah, that will be useful | 02:13 |
| sdague | ok, night folks | 02:13 |
| jamielennox | that was where i was stuggling to get passed debugging as the localrc in grenade won't actually help you test the change | 02:13 |
| jamielennox | sdague: thanks for your help | 02:13 |
| sdague | no prob | 02:13 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/63540 failed tempest with an unrecognized error | 02:32 |
| *** sthaha has quit IRC | 02:34 | |
| *** sthaha has joined #openstack-qa | 02:34 | |
| openstackrecheck | openstack/glance change: https://review.openstack.org/61595 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 02:37 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/63649 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 02:46 |
| openstackgerrit | Jamie Lennox proposed a change to openstack-infra/devstack-gate: Remove ceilometer from the grenade gate https://review.openstack.org/63664 | 02:49 |
| *** pmathews has joined #openstack-qa | 02:49 | |
| openstackrecheck | openstack/python-heatclient change: https://review.openstack.org/63661 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 02:51 |
| *** harlowja is now known as harlowja_away | 02:55 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/61102 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 03:00 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/60699 failed tempest because of: https://bugs.launchpad.net/bugs/1256106 | 03:06 |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add version test for Ceilometer cli https://review.openstack.org/63621 | 03:22 |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add version test for Ceilometer cli https://review.openstack.org/63621 | 03:23 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/58191 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 03:30 |
| *** psharma has joined #openstack-qa | 03:41 | |
| openstackrecheck | openstack/python-ceilometerclient change: https://review.openstack.org/63279 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 03:45 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: experimental skip decorators https://review.openstack.org/63540 | 03:57 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63100 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 04:08 |
| openstackrecheck | openstack/python-novaclient change: https://review.openstack.org/63567 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 04:11 |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/61102 failed tempest with an unrecognized error | 04:20 |
| *** HenryG has quit IRC | 04:22 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61351 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 04:25 |
| openstackrecheck | openstack/python-heatclient change: https://review.openstack.org/63665 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 04:28 |
| *** harlowja_away is now known as harlowja | 04:31 | |
| *** harlowja has quit IRC | 04:35 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/63649 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 04:40 |
| *** HenryG has joined #openstack-qa | 04:51 | |
| *** dkorn has quit IRC | 04:58 | |
| *** dkorn has joined #openstack-qa | 05:15 | |
| *** prekarat has joined #openstack-qa | 05:26 | |
| *** yfried has quit IRC | 05:29 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/61679 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 05:31 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/63669 failed tempest with an unrecognized error | 05:39 |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/61102 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1256106 | 05:39 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63047 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 06:07 |
| *** dkorn has quit IRC | 06:14 | |
| *** yfried has joined #openstack-qa | 06:19 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/63669 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 06:25 |
| *** macjack has joined #openstack-qa | 06:34 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63047 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 06:53 |
| *** yfried has quit IRC | 06:56 | |
| *** yfried has joined #openstack-qa | 07:02 | |
| *** yfried has quit IRC | 07:05 | |
| *** yfried has joined #openstack-qa | 07:06 | |
| *** dkorn has joined #openstack-qa | 07:07 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61351 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 07:11 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/63671 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 07:16 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63056 failed tempest with an unrecognized error | 07:19 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63670 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 07:22 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/63672 failed tempest because of: https://bugs.launchpad.net/bugs/1240256 and https://bugs.launchpad.net/bugs/1253896 | 07:22 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/63666 failed tempest with an unrecognized error | 07:30 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/63533 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 07:35 |
| *** jlibosva has joined #openstack-qa | 07:39 | |
| *** prekarat has quit IRC | 07:45 | |
| yfried | enikanorov__: ping | 07:45 |
| yfried | marun: ping | 07:45 |
| marun | yfried: pong | 07:45 |
| yfried | marun: do you how many networks should be created for a test when using isolated-tenants? | 07:46 |
| yfried | marun: I'm seeing 2 extra nets even before the test creates its own network | 07:47 |
| marun | uh | 07:47 |
| marun | yfried: 2 extra nets owned by who? | 07:48 |
| yfried | marun: It seems like it's creating a network for the test's tenant | 07:48 |
| marun | yfried: that's kind of the point, yeah | 07:48 |
| marun | yfried: i'm not sure I understand what you see as a problem | 07:48 |
| yfried | so why would the test need to create a new network? | 07:48 |
| marun | yfried: what kind of network? | 07:49 |
| *** prekarat has joined #openstack-qa | 07:49 | |
| yfried | marun: network_basic, and cross_tenant, both create networks for their own use | 07:49 |
| yfried | marun: "cross" creates a network for each tenant | 07:50 |
| marun | yfried: i'm afraid I still don't understand what you're asking me | 07:50 |
| yfried | marun: bear with me please | 07:50 |
| marun | yfried: presumably creating an isolated tenant creates a private network for that tenant | 07:50 |
| yfried | marun: so "cross" is trying to hold 5 networks (I'm taking public into account) | 07:51 |
| marun | yfried: that would be more for the nova tests | 07:51 |
| marun | 'hold'? | 07:51 |
| yfried | marun: during "cross" execution - 5 networks should co-exist | 07:51 |
| *** jamielennox is now known as jamielennox|away | 07:52 | |
| yfried | marun: I'm having 3 issues: | 07:52 |
| yfried | marun: 1. you have public->tenant_network->test_created_network for each tenant. That's different than the public->test_created_network you had in mind designing the test. Could be an issue for some scenarios | 07:54 |
| yfried | marun: 2. what about quotas for more complicated tests? | 07:54 |
| marun | yfried: ah, ok | 07:55 |
| yfried | marun: 3. I'm using VLANs on my local setup and the test fails because I simply do not have enough VLANs available | 07:55 |
| marun | yfried: really? how many vlans are you testing with? | 07:55 |
| marun | yfried: 2. quotas are per-tenant | 07:56 |
| yfried | marun: 3 - 1 for public and 2 extras | 07:56 |
| marun | 1. i'd recommend detecting when you're using isolated and reusing the tenant network rather than creating a new one | 07:56 |
| marun | yfried: the limit is 4096 vlans, i'm not sure how you could possibly be running out | 07:56 |
| yfried | marun: 1. that's what I had in mind. I will open a bug for it | 07:56 |
| marun | cool | 07:57 |
| yfried | marun: VLANs are assigned to me by the office network manager | 07:57 |
| yfried | marun: how do I become a tempest member on launchpad? where do I register? | 07:58 |
| marun | yfried: apply to join here https://launchpad.net/~tempest-drivers | 07:58 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/59655 failed tempest with an unrecognized error | 08:03 |
| marun | yfried: regarding vlans, why are you using a real network? | 08:03 |
| marun | yfried: virtualization of hosts and networks is highly recommended for testing openstack. | 08:03 |
| yfried | marun: that's an internal issue - using multiple node setup, our hosts communicate using private vlans. This might not be the most efficient setup, but that's how it works | 08:05 |
| yfried | marun: we are working on "virtualizing" our testing setup | 08:05 |
| yfried | marun: new issue: | 08:06 |
| yfried | marun: I need help with configuring the gate for my external connectivity patch. | 08:07 |
| yfried | marun: I posted on the mailing list, but no response yet. | 08:07 |
| *** sthaha has quit IRC | 08:08 | |
| marun | yfried: how can I help? I'm not sure external connectivity is even allowed at present. | 08:08 |
| yfried | marun: can you help? I'm being constantly bumped between infra, neutron, and qa rooms | 08:08 |
| *** pmathews has quit IRC | 08:08 | |
| yfried | marun: do we want this patch in? https://review.openstack.org/#/c/55146/ if so - it needs some kind of network to test agains | 08:09 |
| yfried | t | 08:09 |
| yfried | marun: I was thinking using the gate's local dns | 08:10 |
| marun | yfried: that's not a bad idea. | 08:10 |
| yfried | marun: great. now, how do I get it done? | 08:12 |
| marun | yfried: I'm afraid I'm not the person to ask. Regulars in this channel and #openstack-infra are your best bet. | 08:13 |
| yfried | I'm not getting any response so far | 08:13 |
| marun | yfried: well, keep pinging them. The tz difference is probably working against you in this regard. | 08:14 |
| marun | yfried: actually... | 08:14 |
| yfried | marun: I've been trying this for a few days now. Mailing list also... | 08:15 |
| *** akamyshnikova_ has joined #openstack-qa | 08:15 | |
| marun | yfried: It's the holidays for much of NA and Europe, so it's going to be slow going for the next week or two | 08:15 |
| marun | yfried: the people in NA that are most likely to be able to help are dkranz and mlavelle | 08:16 |
| yfried | yfried: ok. I have a meeting with dkranz later today on this issue. | 08:16 |
| yfried | marun: is mlavelle redhat ? | 08:16 |
| marun | yfried: no, but he's championing tempest testing for neutron | 08:17 |
| yfried | marun: great. I'll talk to him as well | 08:18 |
| *** prekarat has quit IRC | 08:22 | |
| *** prekarat has joined #openstack-qa | 08:23 | |
| *** kashyap has joined #openstack-qa | 08:27 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61351 failed tempest with an unrecognized error | 08:29 |
| *** prekarat1 has joined #openstack-qa | 08:42 | |
| *** prekarat has quit IRC | 08:45 | |
| *** jlibosva has quit IRC | 08:50 | |
| *** jlibosva has joined #openstack-qa | 08:51 | |
| *** salv-orlando_ has joined #openstack-qa | 08:54 | |
| *** rossella_s has joined #openstack-qa | 08:56 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63047 failed tempest because of: https://bugs.launchpad.net/bugs/1251448 | 08:57 |
| *** yassine has joined #openstack-qa | 08:57 | |
| *** salv-orlando has quit IRC | 08:57 | |
| *** salv-orlando_ is now known as salv-orlando | 08:57 | |
| *** safchain has joined #openstack-qa | 08:58 | |
| *** SergeyLukjanov has joined #openstack-qa | 09:02 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/61102 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 09:03 |
| *** vponomaryov has joined #openstack-qa | 09:07 | |
| *** salv-orlando has quit IRC | 09:07 | |
| *** salv-orlando has joined #openstack-qa | 09:07 | |
| *** prekarat has joined #openstack-qa | 09:13 | |
| *** prekarat1 has quit IRC | 09:15 | |
| *** prekarat1 has joined #openstack-qa | 09:19 | |
| *** prekarat has quit IRC | 09:22 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61351 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 09:23 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61168 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 09:25 |
| *** shardy_afk is now known as shardy | 09:32 | |
| openstackrecheck | openstack/python-keystoneclient change: https://review.openstack.org/63408 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 09:35 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63492 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 09:39 |
| *** tkammer has joined #openstack-qa | 09:45 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/62028 failed tempest with an unrecognized error | 09:48 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59787 failed tempest because of: https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1244255 and https://bugs.launchpad.net/bugs/1257641 | 10:00 |
| *** wanghao has quit IRC | 10:02 | |
| *** wanghao has joined #openstack-qa | 10:02 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61067 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1253896 | 10:02 |
| *** SergeyLukjanov has quit IRC | 10:03 | |
| *** SergeyLukjanov has joined #openstack-qa | 10:04 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/62440 failed tempest with an unrecognized error | 10:04 |
| *** zul has quit IRC | 10:06 | |
| *** zul has joined #openstack-qa | 10:08 | |
| *** tkammer has quit IRC | 10:09 | |
| *** tkammer has joined #openstack-qa | 10:18 | |
| *** ruhe has joined #openstack-qa | 10:18 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63585 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 10:21 |
| *** rossella_s has quit IRC | 10:28 | |
| *** prekarat1 has quit IRC | 10:31 | |
| *** ruhe is now known as ruhe_ | 10:42 | |
| *** yassine has quit IRC | 10:45 | |
| *** ruhe_ is now known as ruhe | 10:46 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52930 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 10:46 |
| *** rossella_s has joined #openstack-qa | 10:50 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/63529 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 10:58 |
| openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/tempest: Add pluggable functionality for api and cli tests https://review.openstack.org/62429 | 11:00 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Commit test for neutron full gate tests https://review.openstack.org/63713 | 11:01 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/63666 failed tempest with an unrecognized error | 11:13 |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/63641 failed tempest with an unrecognized error | 11:15 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/63601 failed tempest with an unrecognized error | 11:16 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63585 failed tempest because of: https://bugs.launchpad.net/bugs/1256106 | 11:18 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63240 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1244255 and https://bugs.launchpad.net/bugs/1257641 | 11:26 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61168 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1253896 | 11:26 |
| *** rossella_s has quit IRC | 11:29 | |
| *** wanghao has quit IRC | 11:36 | |
| *** krtaylor has joined #openstack-qa | 11:37 | |
| *** wanghao has joined #openstack-qa | 11:37 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/tempest: Verify more information in floating ip tests https://review.openstack.org/63723 | 11:38 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 11:48 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/63132 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 11:48 |
| *** _SergeyLukjanov has quit IRC | 11:48 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/63709 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 11:57 |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/61102 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 12:02 |
| *** prekarat has joined #openstack-qa | 12:21 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/63720 failed tempest with an unrecognized error | 12:28 |
| *** SergeyLukjanov has joined #openstack-qa | 12:30 | |
| *** che-arne has quit IRC | 12:37 | |
| *** jlibosva has quit IRC | 12:38 | |
| openstackgerrit | A change was merged to openstack/tempest: rename old config object to TempestConfigPrivate https://review.openstack.org/63529 | 12:40 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/63530 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 12:42 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/53609 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1235435 | 12:43 |
| yfried | sdague: tempest gate doesn't have neutron-isolated like the neutron gate does, right? | 12:48 |
| sdague | yfried: that's correct | 12:49 |
| sdague | I think | 12:49 |
| yfried | sdague: why is that? I want to push a change to isolated neutron test and I need to make sure I don't break the gate like I did during the weekend | 12:49 |
| sdague | historical reasons | 12:50 |
| yfried | sdague: or should I just push it and let neutron guys complain? | 12:50 |
| sdague | basically the neutron isolated jobs aren't fully isolated at this point | 12:50 |
| sdague | the whole end game is to get everything over to them | 12:51 |
| *** ruhe is now known as ruhe_ | 12:51 | |
| yfried | sdague: it did cause an issue last week... | 12:51 |
| sdague | and drop the other jobs, but it moved *a lot* slower than we expected | 12:51 |
| yfried | sdague: shouldn't we at least have this as a non-voting? | 12:51 |
| sdague | yfried: I'd be fine with non-voting | 12:51 |
| sdague | you know how to add a job like that? | 12:51 |
| *** ruhe_ has quit IRC | 12:52 | |
| yfried | sdague: no idea. I really have very little knowledge on the subject of the gate | 12:52 |
| *** jlibosva has joined #openstack-qa | 12:52 | |
| sdague | something you want to learn? I can walk you through it. (need to grab a another cup of coffee first) | 12:53 |
| *** yassine has joined #openstack-qa | 12:54 | |
| yfried | sdague: I would love to | 12:54 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59743 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 12:54 |
| sdague | so the key file that describes all of the zuul jobs is this - https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/zuul/layout.yaml | 12:55 |
| sdague | so you'll want to check yourself out an openstack-infra/config repo, which is the puppet repository that runs *all* of infrastructure | 12:56 |
| yfried | sdague: would you like to have a bluejeans session? | 12:56 |
| sdague | the important part is it builds a matrix of jobs: x projects: | 12:56 |
| sdague | bluejeans? | 12:57 |
| yfried | sdague: http://bluejeans.com/ | 12:57 |
| yfried | sdague: or google hangouts | 12:57 |
| yfried | sdague: or is this a quick explanation | 12:58 |
| sdague | well, I'm still very much early morning, so probably too early for video for me | 12:58 |
| dkranz | sdague: I can explain this to Yair | 12:58 |
| sdague | I'd rather do it here, also this channel is logged, which is good for others learning | 12:58 |
| yfried | sdague: ok. tnx | 12:58 |
| yfried | I'm "listening" | 12:58 |
| sdague | so the basic model is there is a list of jobs, and then you tell which project to run them on | 12:59 |
| sdague | and which queue in that project to run them on | 12:59 |
| sdague | so actually, I think for this case, we probably want to add the isolated jobs to the tempest experimental queue | 12:59 |
| sdague | https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/zuul/layout.yaml#L1892 | 12:59 |
| sdague | which would let you run "check experimental" on any patch you think might be important for this | 13:00 |
| sdague | and it will give you all those tests as an additional post of results | 13:00 |
| *** ruhe has joined #openstack-qa | 13:01 | |
| sdague | and all you need to do to propose that is build a review with the job name in that experimental list | 13:01 |
| openstackrecheck | openstack/python-cinderclient change: https://review.openstack.org/63147 failed tempest because of: https://bugs.launchpad.net/bugs/1210483 | 13:02 |
| yfried | sdague: do you have an example of similar patch? | 13:07 |
| yfried | dkranz: We will meet in ~50min, ok? | 13:09 |
| dkranz | yfried: ok | 13:10 |
| sdague | yfried: https://review.openstack.org/#/c/53917/19/modules/openstack_project/files/zuul/layout.yaml is not exactly the same thing, but close | 13:15 |
| sdague | it's just a single line patch with the check- jobs that yuo want to add to experimental | 13:15 |
| yfried | sdague: I need to leave for a meeting shortly and it's still cloning (bad bandwidth). I will try this later. Thank you for the offer and the explanation | 13:16 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/59655 failed tempest with an unrecognized error | 13:16 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52930 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 13:20 |
| yfried | sdague: so I just need to add a line under "experimental"? | 13:23 |
| yfried | sdague: will it run for every patch? | 13:24 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: prune grenade service lists to things we can upgrade https://review.openstack.org/63751 | 13:26 |
| dkranz | yfried: No. Jobs in the experimental queue only run when you post a review comment of "check experimental" in a project that has stuff on its experimental queue. | 13:26 |
| sdague | yfried: it won't run on every patch | 13:26 |
| sdague | dkranz: if this looks good to you, could use another +2 - https://review.openstack.org/#/c/63649/ | 13:27 |
| dkranz | sdague: Looking. Will get to more reviews around 10am | 13:28 |
| sdague | yep, no worries | 13:28 |
| sdague | masauki and cyeoh put through the patch I really wanted to land, which was the TempestConfigPrivate one | 13:28 |
| *** psharma has quit IRC | 13:29 | |
| sdague | someone else had manage to land an old style config change before that went in, so it needed a fix over the weekend, but all good now | 13:29 |
| sdague | I also sent an email to the list | 13:29 |
| dkranz | sdague: ok, approved. Thanks for doing that. | 13:30 |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/59214 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 13:31 |
| dkranz | sdague: I was going to reply suggesting that no one do this low-hanging-fruit in one commit | 13:31 |
| dkranz | sdague: Global bombings don't work so well in our environment | 13:32 |
| *** prekarat has quit IRC | 13:32 | |
| sdague | dkranz: sure, be it could be a directory at a time | 13:32 |
| *** jaypipes has joined #openstack-qa | 13:36 | |
| *** ChanServ sets mode: +o jaypipes | 13:36 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63735 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 13:37 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/63722 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 13:40 |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/61102 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 13:41 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59743 failed tempest with an unrecognized error | 13:46 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/62836 failed tempest with an unrecognized error | 13:47 |
| *** ruhe is now known as ruhe_ | 13:51 | |
| *** ruhe_ has quit IRC | 13:52 | |
| *** yfried has quit IRC | 14:00 | |
| openstackgerrit | A change was merged to openstack/tempest: remove unneeded __init__ file https://review.openstack.org/63530 | 14:02 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Serialize plurals correctly in neutron xml client https://review.openstack.org/59029 | 14:05 |
| *** ruhe has joined #openstack-qa | 14:06 | |
| openstackgerrit | A change was merged to openstack/tempest: remove last errant file parse warning https://review.openstack.org/63649 | 14:11 |
| *** yfried has joined #openstack-qa | 14:13 | |
| openstackrecheck | openstack/python-novaclient change: https://review.openstack.org/62550 failed tempest with an unrecognized error | 14:19 |
| *** wanghao has quit IRC | 14:23 | |
| *** wanghao has joined #openstack-qa | 14:23 | |
| openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/63751 failed tempest with an unrecognized error | 14:25 |
| *** macjack has quit IRC | 14:28 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52930 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 14:37 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/63722 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 and https://bugs.launchpad.net/bugs/1254890 | 14:38 |
| *** rossella_s has joined #openstack-qa | 14:41 | |
| *** malini has joined #openstack-qa | 14:50 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/62696 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 14:50 |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Always dump errors to console https://review.openstack.org/61850 | 14:53 |
| *** rwsu has joined #openstack-qa | 14:59 | |
| *** rustlebee is now known as russellb | 14:59 | |
| openstackrecheck | openstack/glance change: https://review.openstack.org/59725 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 15:00 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Serialize plurals correctly in neutron xml client https://review.openstack.org/59029 | 15:02 |
| *** ruhe has quit IRC | 15:07 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59743 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 15:08 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/63655 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 15:09 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/62231 failed tempest because of: https://bugs.launchpad.net/bugs/1259907 and https://bugs.launchpad.net/bugs/1253896 | 15:11 |
| malini | hello!!! I am working on adding marconi tests to tempest. I have a jenkins failure coz it cannot find the queuing endpoint (http://logs.openstack.org/49/63449/3/check/check-tempest-dsvm-full/a525b64/ ) . My guess is its because marconi is not enabled in devstack that gating uses. Any pointers to what I need to update to make marconi service enabled ? | 15:15 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/63727 failed tempest with an unrecognized error | 15:21 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Use create_router_interface and create_port https://review.openstack.org/61527 | 15:26 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Serialize plurals correctly in neutron xml client https://review.openstack.org/59029 | 15:26 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63566 failed tempest with an unrecognized error | 15:28 |
| *** jlibosva has quit IRC | 15:30 | |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Add create_floating_ip function https://review.openstack.org/61529 | 15:33 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63769 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 15:34 |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Add 2 tests to the Floating IP test case https://review.openstack.org/56349 | 15:35 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59359 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 15:37 |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/63754 failed tempest because of: https://bugs.launchpad.net/bugs/1259907 | 15:38 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 15:42 | |
| *** _SergeyLukjanov has quit IRC | 15:43 | |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/tempest: Add negative tests for routers https://review.openstack.org/56680 | 15:48 |
| vponomaryov | malini: marconi is not installed in job you mentioned and you have enabled your tests by default. As you can see, there is no ironic and savanna too in this jobs | 15:48 |
| malini | vponomaryov: is the standard to disable the tests till the project graduates? | 15:49 |
| malini | vponomaryov: I am trying to figure out what the usual process is to add a new project to tempest | 15:50 |
| *** rossella_s_ has joined #openstack-qa | 15:51 | |
| vponomaryov | I dont have such info, I recommend you to look how is it done in other projects | 15:51 |
| malini | thanks for the pointer vponomaryov!! | 15:51 |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/63770 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 15:51 |
| *** rossella_s has quit IRC | 15:52 | |
| *** rossella_s_ is now known as rossella_s | 15:52 | |
| *** pmathews has joined #openstack-qa | 15:53 | |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/tempest: Adds basic Marconi test https://review.openstack.org/63449 | 15:56 |
| *** yfried has quit IRC | 15:56 | |
| *** yfried has joined #openstack-qa | 15:57 | |
| *** corvus is now known as jeblair | 15:58 | |
| vponomaryov | malini: if you want change value of some opt, you should change it in tempest/config.py and after it regenarate, if I am not wrong. Because if someone else change it, your value become as in tempest/config.py | 16:01 |
| vponomaryov | it is related to your patch set 4 | 16:02 |
| malini | thx vponomaryov..I updated https://review.openstack.org/#/c/63449/4/etc/tempest.conf.sample manually to set service_available.queuing to false with this patch..Apparently tht didnt help | 16:03 |
| *** SergeyLukjanov has joined #openstack-qa | 16:03 | |
| malini | I will chk the config.py | 16:04 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63775 failed tempest because of: https://bugs.launchpad.net/bugs/1210483 | 16:07 |
| *** yfried has quit IRC | 16:08 | |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/tempest: Adds basic Marconi test https://review.openstack.org/63449 | 16:08 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/61527 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 16:08 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63084 failed tempest with an unrecognized error | 16:13 |
| openstackgerrit | Malini Kamalambal proposed a change to openstack/tempest: Adds basic Marconi test https://review.openstack.org/63449 | 16:15 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59743 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 16:22 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/58954 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 16:23 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61067 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 16:27 |
| *** SergeyLukjanov has quit IRC | 16:28 | |
| *** ruhe has joined #openstack-qa | 16:33 | |
| *** ruhe has quit IRC | 16:34 | |
| *** prekarat has joined #openstack-qa | 16:44 | |
| openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/63751 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 16:44 |
| *** rossella_s_ has joined #openstack-qa | 16:48 | |
| *** rossella_s has quit IRC | 16:49 | |
| *** rossella_s_ is now known as rossella_s | 16:49 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63775 failed tempest with an unrecognized error | 16:52 |
| *** pmathews has quit IRC | 16:55 | |
| *** rossella_s has quit IRC | 16:58 | |
| *** prekarat has quit IRC | 16:59 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/63619 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 17:01 |
| *** cyeoh has quit IRC | 17:03 | |
| *** rossella_s has joined #openstack-qa | 17:05 | |
| *** mlavalle has joined #openstack-qa | 17:11 | |
| *** yassine has quit IRC | 17:16 | |
| *** pmathews has joined #openstack-qa | 17:17 | |
| *** pmathews has quit IRC | 17:22 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: Fix bug with Xen ext4-using guests https://review.openstack.org/63795 | 17:25 |
| *** akamyshnikova_ has quit IRC | 17:27 | |
| *** metabro has quit IRC | 17:27 | |
| *** vrovachev has quit IRC | 17:27 | |
| *** obondarev has quit IRC | 17:27 | |
| *** enikanorov has quit IRC | 17:27 | |
| *** akamyshnikova_ has joined #openstack-qa | 17:28 | |
| *** metabro has joined #openstack-qa | 17:28 | |
| *** enikanorov has joined #openstack-qa | 17:28 | |
| *** vrovachev has joined #openstack-qa | 17:29 | |
| *** obondarev has joined #openstack-qa | 17:30 | |
| *** yfried has joined #openstack-qa | 17:37 | |
| *** safchain has quit IRC | 17:42 | |
| *** ruhe has joined #openstack-qa | 17:47 | |
| *** harlowja has joined #openstack-qa | 17:50 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/60571 failed tempest with an unrecognized error | 17:51 |
| *** dkorn has quit IRC | 17:51 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/59334 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 17:51 |
| *** SergeyLukjanov has joined #openstack-qa | 17:52 | |
| *** rossella_s_ has joined #openstack-qa | 17:52 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Timeout devstack/tempest internally https://review.openstack.org/63512 | 17:53 |
| openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/62983 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 17:53 |
| *** rossella_s has quit IRC | 17:54 | |
| *** rossella_s has joined #openstack-qa | 17:55 | |
| *** rossella_s_ has quit IRC | 17:57 | |
| openstackrecheck | openstack/requirements change: https://review.openstack.org/63754 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 17:57 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61168 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 17:58 |
| *** tkammer has quit IRC | 18:00 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/61142 failed tempest with an unrecognized error | 18:00 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/58533 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1253896 | 18:14 |
| *** rossella_s_ has joined #openstack-qa | 18:17 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59948 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 18:18 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/58641 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 18:19 |
| *** rossella_s has quit IRC | 18:19 | |
| *** rossella_s_ is now known as rossella_s | 18:19 | |
| *** ruhe has quit IRC | 18:28 | |
| *** markmcclain has joined #openstack-qa | 18:46 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/63619 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1235435 | 18:48 |
| *** rossella_s_ has joined #openstack-qa | 18:51 | |
| *** DennyZhang has joined #openstack-qa | 18:51 | |
| *** rossella_s has quit IRC | 18:52 | |
| *** rossella_s_ is now known as rossella_s | 18:52 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Remove strange duplication of setup_host and output https://review.openstack.org/62983 | 18:53 |
| mlavalle | mtreinish: ping | 18:56 |
| *** rossella_s_ has joined #openstack-qa | 18:59 | |
| *** rossella_s has quit IRC | 19:02 | |
| *** rossella_s_ is now known as rossella_s | 19:02 | |
| sdague | mlavalle: he's off on vacation for the rest of the year | 19:03 |
| mlavalle | sdague: it seems I missed the conversation on negative api tests | 19:04 |
| mlavalle | we are not implementing those anymore? | 19:04 |
| sdague | as a policy, not new ones, because we really need to do this in a generative way | 19:06 |
| sdague | hand writing out the negative space is no good, it's too much time, and too much review time | 19:06 |
| sdague | anyway, it's basically holidays, so probably a conversation we won't really pick up until Jan | 19:06 |
| mlavalle | so' I've doing the gap analysis for neutron api tests. I've been including negative api tests | 19:07 |
| mlavalle | should I continue doing that? | 19:07 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59948 failed tempest with an unrecognized error | 19:08 |
| *** rossella_s_ has joined #openstack-qa | 19:12 | |
| *** rossella_s has quit IRC | 19:14 | |
| *** rossella_s_ is now known as rossella_s | 19:14 | |
| *** DennyZhang has quit IRC | 19:17 | |
| *** DennyZhang has joined #openstack-qa | 19:17 | |
| *** yamahata__ has joined #openstack-qa | 19:19 | |
| *** hemna_ has joined #openstack-qa | 19:19 | |
| sdague | mlavalle: so we should fix where it's disjoint to nova-network | 19:20 |
| *** vrovachev1 has joined #openstack-qa | 19:20 | |
| sdague | but I wouldn't focus on adding a lot beyond that | 19:20 |
| mlavalle | cool | 19:20 |
| mlavalle | sdague: to make sure we are in alignment, let me give you an example | 19:21 |
| mlavalle | lbaas….. that doesn't exist in nova network, right? so we should develop negative tests for that. Am I correct? | 19:22 |
| sdague | mlavalle: no, we shouldn't at this point | 19:23 |
| mlavalle | sdague: ok | 19:23 |
| sdague | negative tests for new things need to come in via the generative approach | 19:23 |
| mlavalle | I'll concentrate on positive tests | 19:23 |
| sdague | yep, that would be good | 19:23 |
| mlavalle | sdague: are we meeting on Thursday/ | 19:24 |
| mlavalle | ? | 19:24 |
| sdague | no, skipping the next 2 meetings. Did mtreinish not send that to the list? | 19:24 |
| mlavalle | probably he did and I just missed it….. happy Holidays! | 19:25 |
| sdague | you too | 19:25 |
| *** vrovachev has quit IRC | 19:26 | |
| *** openstackrecheck has quit IRC | 19:26 | |
| *** yamahata_ has quit IRC | 19:26 | |
| *** hemnafk has quit IRC | 19:26 | |
| *** openstackrecheck has joined #openstack-qa | 19:26 | |
| *** rossella_s_ has joined #openstack-qa | 19:34 | |
| *** rossella_s has quit IRC | 19:38 | |
| *** rossella_s_ is now known as rossella_s | 19:38 | |
| *** DennyZhang has quit IRC | 19:44 | |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/63560 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 19:55 |
| openstackgerrit | mouad benchchaoui proposed a change to openstack/tempest: Make testing neutron extensions optional https://review.openstack.org/55133 | 20:03 |
| *** malini is now known as malini_afk | 20:09 | |
| jog0 | sdague: I plan on adding rolling nova-compute support to grenade this week | 20:15 |
| openstackgerrit | Mauro S. M. Rodrigues proposed a change to openstack/tempest: Add aggregates scenario test https://review.openstack.org/62618 | 20:22 |
| jog0 | and when you have a minute wanted to sync on the steps needed | 20:23 |
| sdague | sure | 20:25 |
| sdague | you want to sync here, or do like a g+ thing? | 20:29 |
| *** rossella_s_ has joined #openstack-qa | 20:32 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63775 failed tempest with an unrecognized error | 20:33 |
| jog0 | here should be fine, at least as a first pass | 20:33 |
| *** rossella_s has quit IRC | 20:34 | |
| *** rossella_s_ is now known as rossella_s | 20:34 | |
| sdague | so basically the trick of it is going to be convincing grenade to start services off different trees after upgrade | 20:35 |
| sdague | so you can use the whole "old" piece | 20:35 |
| jog0 | sdague: right | 20:35 |
| jog0 | so I think we want venvs for new and old python deps | 20:35 |
| sdague | because the assumption is to simulate that this was different nodes? | 20:38 |
| jog0 | sdague: and if my memory serves me correctly we want to ran one grenade as is, and another that upgrades everything but nova-compute | 20:38 |
| jog0 | sdague: because the assumption is that $stable and $trunk may have different dependencies | 20:39 |
| sdague | sure | 20:39 |
| sdague | but if you don't actually shut down stable, does it matter? | 20:40 |
| sdague | so I'd actually start with this - https://github.com/openstack-dev/grenade/blob/master/stop-base#L36 | 20:40 |
| sdague | and make stop more granular | 20:40 |
| jog0 | I don't *think* so but that sounds very hackey | 20:40 |
| sdague | because right now it's just bouncing the whole screen session | 20:40 |
| sdague | jog0: why, it's repeatable | 20:40 |
| jog0 | are we 100% sure upgrading the deps from underneath python won't hurt something that is running? | 20:41 |
| sdague | this is going to take a while to figure out the right way to do this, so expect us to bounce around | 20:41 |
| jog0 | what happens if there is a lazy load | 20:41 |
| jog0 | sdague: but yeah I agree we can start without the venv and add it in | 20:41 |
| sdague | well, I think we then isolate it with venv after wards | 20:42 |
| sdague | so start with the stop list being something you can control | 20:42 |
| jog0 | sounds good | 20:42 |
| sdague | and the logic to start up only the listed services on the new side | 20:42 |
| sdague | that will get you about 90% of the way there | 20:42 |
| sdague | jog0: how much are you working between now an new years? | 20:43 |
| *** rossella_s_ has joined #openstack-qa | 20:43 | |
| jog0 | sdague: almost full -- taking vacation in January instead | 20:43 |
| sdague | well... good luck on reviews :) | 20:44 |
| jog0 | I don't expect to get any code reviewed ... just be able to get some work done | 20:44 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/55133 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 20:45 |
| jog0 | I think first step should be to wire up infra with an experimental job | 20:45 |
| jog0 | that has the stop list | 20:45 |
| *** rossella_s has quit IRC | 20:47 | |
| *** rossella_s_ is now known as rossella_s | 20:47 | |
| *** pmathews has joined #openstack-qa | 20:47 | |
| jog0 | https://etherpad.openstack.org/p/grenade-rolling-upgrades | 20:48 |
| sdague | cool, seems sane | 20:49 |
| jog0 | and this is explicitly constrainted in scope, as the big upgrade test will be coming from tripleo | 20:51 |
| sdague | yep | 20:54 |
| jog0 | sdague: what are your thoughts un expanding the set of tempest tests that grenade runs? | 20:54 |
| sdague | -1 | 20:54 |
| jog0 | you know a the worst kind of feedback is a -1 without an explination heh | 20:55 |
| sdague | it's coming... | 20:55 |
| *** dkorn has joined #openstack-qa | 20:55 | |
| sdague | so if we weren't going to get lynched for it, I'd actually want to run the full battery | 20:55 |
| sdague | but the time is a bit too long for that | 20:56 |
| sdague | the whole point is that it should be a working openstack cloud | 20:56 |
| sdague | so it should pass all the tests | 20:56 |
| jog0 | what about full api but no scenario or something | 20:56 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/61067 failed tempest with an unrecognized error | 20:56 |
| jog0 | or vice versa | 20:56 |
| sdague | why do you want to decrease the tests? | 20:57 |
| jog0 | I don't -- I want to get the most number of meaningful tests in the least amount of time that covers the things we expect to fail | 20:58 |
| jog0 | not sure what that is though | 20:58 |
| sdague | honestly, we never know what we expect to fail | 20:58 |
| sdague | and everything we don't tests, seems to eventually fail | 20:58 |
| sdague | see: neutron + devstack exercises | 20:59 |
| jog0 | right ... | 20:59 |
| jog0 | I don't see any good answer besides test it all | 20:59 |
| sdague | yeh, the current smoke compromise is what it is | 20:59 |
| jog0 | why is greande so slow anyway? | 20:59 |
| sdague | smoke is currently all scenario tests + tagged api tests | 21:00 |
| sdague | *so slow*? | 21:00 |
| *** amotoki_ has joined #openstack-qa | 21:00 | |
| sdague | it needs to sanity check the first cloud with a set of tests to make sure that it stood up, otherwise, debugging the fails later are terrible | 21:02 |
| sdague | feel free to comb logs and look for optimizations | 21:02 |
| jog0 | https://jenkins02.openstack.org/view/All/job/gate-grenade-dsvm/buildTimeTrend | 21:04 |
| jog0 | its about 10 min faster then https://jenkins02.openstack.org/job/gate-tempest-dsvm-postgres-full/buildTimeTrend | 21:04 |
| sdague | yep | 21:04 |
| sdague | remember, real testing takes real time :) | 21:05 |
| jog0 | interesting so grenade setup is about 10 min slower then devstack | 21:05 |
| jog0 | not too bad | 21:05 |
| sdague | so I take it you aren't coming to montreal any more? | 21:05 |
| jog0 | 431 seconds of tempest in grenade vs 1635 in full | 21:06 |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/63619 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 21:06 |
| jog0 | sdague: no, too much traveling | 21:06 |
| sdague | yeh, I can understand that | 21:06 |
| jog0 | I will see anteaya the week before though | 21:06 |
| *** amotoki is now known as __amotoki__ | 21:06 | |
| *** amotoki_ is now known as amotoki | 21:06 | |
| sdague | dkranz: if you have a few minutes, curious on thoughts on this style approach - https://review.openstack.org/#/c/63540/ | 21:08 |
| dkranz | sdague: Looking... | 21:08 |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: prune grenade service lists to things we can upgrade https://review.openstack.org/63751 | 21:08 |
| openstackgerrit | A change was merged to openstack/tempest: Add swift scenario tests https://review.openstack.org/61845 | 21:08 |
| sdague | it's in no way complete, but just a sample of some new decorators to try to get all the skip logic out of the tests themselves | 21:09 |
| dkranz | sdague: It seems reasonable to me, though I might have used "protocol" rather than "payload". | 21:11 |
| *** pmathews has quit IRC | 21:11 | |
| sdague | dkranz: sure, exact names I can easily adjust | 21:12 |
| dkranz | sdague: Right | 21:12 |
| sdague | just wanted to sanity check it before I officially stop working for the year, in case I decide I want to play with it when out | 21:12 |
| sdague | so it's not a waste of time | 21:12 |
| dkranz | sdague: Sure, though I am only one opinion :) | 21:13 |
| dkranz | sdague: Can we discuss https://review.openstack.org/#/c/61978 a bit? | 21:13 |
| dkranz | sdague: My first reaction, and second, is ick | 21:13 |
| sdague | dkranz: well, how do we expect clients to do retry? | 21:13 |
| dkranz | sdague: I get the motivation but the bug is reallyin the clients | 21:13 |
| dkranz | sdague: swiftclient actually does a kind of retry | 21:14 |
| dkranz | sdague: I guess if we have to do this I would prefer to have a separate call for each client and a function that takes arguments like the code to check for, name of methoc, etc. | 21:15 |
| dkranz | sdague: Rather than one giant mess that will break when anything changes or new clients are added. | 21:15 |
| dkranz | sdague: Oh well. | 21:15 |
| dkranz | sdague: And do we want this on by default, and in the gate? | 21:16 |
| sdague | we turn off rate limit in the gate | 21:16 |
| sdague | so we don't need it there | 21:16 |
| sdague | dkranz: well is it something where there would be reasonable unit tests for it? | 21:17 |
| sdague | so we could test that it actually works | 21:17 |
| sdague | in the unit test path | 21:17 |
| dkranz | sdague: There could be | 21:17 |
| sdague | I agree it's black magic enough that I'm twitchy on it | 21:17 |
| sdague | which is why I didn't cycle around and +2 it yet | 21:17 |
| notmyname | seems that I may want to pay attention to this conversation, butI can't seem to find the beginning to see what it's about | 21:18 |
| dkranz | sdague: Well we can wait until the new year | 21:18 |
| dkranz | notmyname: It is about working around the fact that the python clients don't let you work around rate limit errors without catching them at every call. | 21:18 |
| dkranz | notmyname: Though swiftclient had some retry logic | 21:18 |
| dkranz | notmyname: But not specific for this case. | 21:19 |
| notmyname | dkranz: https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L1114 | 21:19 |
| notmyname | dkranz: not specific to this case? how do you mean? | 21:19 |
| dkranz | notmyname: Yes, that code. I meant that it retries on all errors, not just ratelimit. | 21:20 |
| notmyname | at least with python-swiftclient, errors will result in retries (including rate limit errors) | 21:20 |
| notmyname | ah, ok | 21:20 |
| dkranz | notmyname: I think that could be improved | 21:20 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/62618 failed tempest with an unrecognized error | 21:20 |
| dkranz | notmyname: But the other clients don't do anything at all | 21:20 |
| notmyname | something we should change in the swift client? what are you looking for? | 21:20 |
| notmyname | dkranz: that seems like a bug with those clients | 21:20 |
| *** vrovachev has joined #openstack-qa | 21:21 | |
| dkranz | notmyname: Indeed https://bugs.launchpad.net/python-keystoneclient/+bug/1263069 | 21:21 |
| dkranz | notmyname: I think this needs more thought and/or discussion | 21:21 |
| dkranz | notmyname: Sometimes you want a call to just wait until it can get in. Sometimes you need to react to the rate limiting. | 21:22 |
| dkranz | notmyname: My thought was just that it would be nice if when you make the connection you could specify that. | 21:22 |
| dkranz | notmyname: It is what _retry does but just more general | 21:23 |
| dkranz | notmyname: But i'm not sure how much generality is needed. | 21:23 |
| vrovachev | Guys, please consider this patch https://review.openstack.org/#/c/61958/. It's very important to telemetry tempest tests. | 21:23 |
| notmyname | actually, _retry is part of the Connection class, so the numner of retries is settable per connection. and if you don't want anything, then you can using the direct module methods (which Connection() instances just wrap in _retry()) | 21:24 |
| dkranz | notmyname: Yes, you could do that I suppose. | 21:25 |
| dkranz | notmyname: I'll think if that solves the tempest problem. | 21:25 |
| dkranz | notmyname: But it would only help for swift, which just merged scenario tests. | 21:25 |
| dkranz | vrovachev: You need to take this to the #openstack-infra channel. Most of us do not have +2 for devstack-gate. | 21:26 |
| notmyname | dkranz: the tempest problem is that client libs either (1) don't do any retries or (2) do retries for everything, both errors and rate limits? | 21:26 |
| notmyname | dkranz: and swift is in category 2? | 21:26 |
| dkranz | notmyname: All but switfclient don't do any, swift is (2) | 21:26 |
| dkranz | notmyname: Yes | 21:26 |
| dkranz | notmyname: So this awful monkeypatch stuff was proposed and I just was trying to come up with a different way. | 21:27 |
| sdague | notmyname: right, which is not a problem in the gate. But for people that want to hit live clouds with rate limits... it can be. | 21:27 |
| vrovachev | dkranz: thx for info :) | 21:27 |
| shardy | dkranz: Thanks for the reviews, I put a response on https://review.openstack.org/#/c/61175/ | 21:28 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/57747 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 | 21:29 |
| shardy | dkranz: I'd like to keep some method of testing delete in there, but I'm happy to post a refactor patch which abstracts it into a tearDown handler, or whatever you prefer | 21:29 |
| dkranz | shardy: I would actually be ok with just a comment that it is intended to happen before cleanup. | 21:30 |
| dkranz | shardy: Just not obvious :) | 21:30 |
| shardy | dkranz: Ok, sure, should I add it to that patch, or would you like a comment added to every test? | 21:30 |
| shardy | dkranz: maybe the cleanest way would be to move the delete to a common teardown handler, and put a comment there | 21:31 |
| dkranz | shardy: To me it is only needed where there is a delete call that is not needed to actually make sure things are cleaned up | 21:31 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/57748 failed tempest with an unrecognized error | 21:31 |
| dkranz | shardy: That would be ok too. | 21:31 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63056 failed tempest with an unrecognized error | 21:32 |
| shardy | dkranz: Ok, I'll post a followup patch which does that, thanks! | 21:32 |
| *** vrovachev has left #openstack-qa | 21:37 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: allow hypervisors to be down but still pass https://review.openstack.org/61879 | 21:39 |
| notmyname | dkranz: ok, so clarification, python-swiftclient doesn't automatically retry on ratelimit errors | 21:41 |
| dkranz | notmyname: That is my reading of the code | 21:42 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: wip: use getattr to cleanup config further https://review.openstack.org/63837 | 21:47 |
| *** jamielennox|away is now known as jamielennox | 21:47 | |
| *** mestery has joined #openstack-qa | 21:50 | |
| rossella_s | dkranz: hi! thanks for reviewing my patches. I have on questions, who can approve them? | 21:51 |
| notmyname | dkranz: https://review.openstack.org/#/c/63838/ | 21:51 |
| dkranz | notmyname: That is along the lines I was thinking | 21:53 |
| openstackgerrit | Vadim Rovachev proposed a change to openstack-infra/devstack-gate: Added ceilometer-anotification to enabled services https://review.openstack.org/61958 | 21:54 |
| notmyname | dkranz: if you could, I'd appreciate you leaving a comment about why you want the change | 21:54 |
| dkranz | notmyname: Sure thing. | 21:54 |
| notmyname | thanks | 21:54 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63827 failed tempest with an unrecognized error | 21:55 |
| dkranz | rossella_s: Any one on the core tempest team. A bunch of folks are kindof out for the holidays. Hopefully one of our Japanese core reviewers can get to them. | 21:56 |
| rossella_s | dkranz: thanks! I am gonna be out soon too. I wanted to get them merged before my holidays. | 21:57 |
| *** dkorn has quit IRC | 21:57 | |
| rossella_s | dkranz: can you please tell me who I can ping to get an approved? | 21:58 |
| rossella_s | s/approved/approval | 21:58 |
| dkranz | rossella_s: Maybe sdague can get to it before he leaves. | 21:58 |
| rossella_s | thanks a lot dkranz! | 21:59 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/62441 failed tempest with an unrecognized error | 21:59 |
| dkranz | rossella_s: Ken'ichi Ohmichi is in Japan but I'm not sure of his irc handle. He is not here now. | 22:00 |
| rossella_s | dkranz: thanks again, I'll ping sdague | 22:01 |
| rossella_s | sdague: will you be so kind to have a look at my patches? It's almost xmas :) https://review.openstack.org/#/c/59029/ https://review.openstack.org/#/c/56349/ https://review.openstack.org/#/c/56680/ https://review.openstack.org/#/c/61527/ https://review.openstack.org/#/c/61529/ thanks!!! | 22:01 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: keystone trusts API test, move delete into tearDown handler https://review.openstack.org/63841 | 22:02 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: keystone OS-TRUST extension, test list operations https://review.openstack.org/61176 | 22:02 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: keystone OS-TRUST extension, test with expiry https://review.openstack.org/61175 | 22:02 |
| shardy | dkranz: ^^ | 22:02 |
| dkranz | shardy: thanks | 22:03 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/62988 failed tempest with an unrecognized error | 22:03 |
| rossella_s | dkranz: sorry to bother again. Can you have a look at https://review.openstack.org/#/c/61527/ https://review.openstack.org/#/c/61529/ ? I've realized you didn't check those too. Thanks a lot!! | 22:04 |
| dkranz | notmyname: ok, I hope my comment is sufficient. | 22:04 |
| *** shardy is now known as shardy_afk | 22:04 | |
| notmyname | dkranz: perfect, thanks | 22:05 |
| bnemec | Oh ugh. This commit https://github.com/openstack/tempest/commit/87f772c0bd59790aa43de59938d9c138963f9f4e means that a basic "testr run foo" doesn't work anymore. At least not for me. | 22:05 |
| bnemec | That was really unpleasant to figure out. | 22:05 |
| bnemec | git bisect FTW though. :-) | 22:05 |
| *** amotoki has quit IRC | 22:06 | |
| dkranz | sdague: ^^^ | 22:06 |
| *** rossella_s has quit IRC | 22:08 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/62760 failed tempest because of: https://bugs.launchpad.net/bugs/1224518 | 22:09 |
| bnemec | Now why does it work when I run 'OS_TEST_PATH=./tempest/test_discover testr run tempest.api.compute.*' but not if I change the default in .testr.conf... | 22:10 |
| sdague | dkranz: so there is this alternative approach - https://review.openstack.org/#/c/62614/ | 22:12 |
| sdague | I'm concerned they will collide | 22:12 |
| dkranz | sdague: What will collide with that? | 22:13 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/63570 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 22:13 |
| sdague | dkranz: oh, you are right | 22:14 |
| sdague | never mind, the earlier patch went through | 22:14 |
| sdague | honestly, my brain is a bit too fried to do code reviews at this point | 22:14 |
| dkranz | sdague: ok, understood | 22:14 |
| dkranz | sdague: I have spent more time doing that today than any other day in a while. I'm about ready to quit too. | 22:15 |
| dkranz | sdague: Maybe just one last one, since you made a comment that was not understood, and I'm not quite sure either https://review.openstack.org/#/c/61529/ | 22:19 |
| *** che-arne has joined #openstack-qa | 22:24 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/61879 failed tempest with an unrecognized error | 22:26 |
| *** mlavalle has quit IRC | 22:37 | |
| *** pmathews has joined #openstack-qa | 22:37 | |
| *** HenryG has quit IRC | 22:38 | |
| *** HenryG has joined #openstack-qa | 22:38 | |
| *** HenryG has quit IRC | 22:43 | |
| *** HenryG has joined #openstack-qa | 22:44 | |
| sdague | dkranz: so it's a little weird to leave those ips around for class teardown instead of just cleaning them up with an addCleanup | 22:44 |
| openstackgerrit | Ben Nemec proposed a change to openstack/tempest: Fix OS_TEST_PATH in .testr.conf https://review.openstack.org/63847 | 22:49 |
| bnemec | ^Victory! | 22:49 |
| sdague | bnemec: did you really put html in the commit message.... | 22:56 |
| sdague | ಠ_ಠ | 22:56 |
| bnemec | sdague: No, I definitely did not. I'm trying to figure out what's going on there. | 22:56 |
| bnemec | In my local git log it just show http://stackoverflow.com/questions/10938483/bash-specifying-environment-variables-for-echo-on-command-line | 22:57 |
| clarkb | lol | 22:58 |
| clarkb | its gerrit | 22:58 |
| clarkb | we have a fixed in place for that problem, but it requires restarting gerrit | 22:58 |
| clarkb | fungi: ^ maybe we should do that today? | 22:58 |
| fungi | clarkb: sure thing. looking at the graph to see if it's a good time | 22:59 |
| openstackgerrit | mouad benchchaoui proposed a change to openstack/tempest: Make testing neutron agents optional https://review.openstack.org/55133 | 23:00 |
| bnemec | Phew, so it wasn't me. :-) | 23:00 |
| clarkb | fungi: and probably double check jhesketh's change is in place and all that | 23:00 |
| fungi | looks like activity is starting to trail off for review.o.o | 23:02 |
| clarkb | fungi: I do need to step out shortly. Doing last minute holdiay stuff (woo!) and probably grabbing food | 23:02 |
| sdague | fungi: yeh, it's been a slow day | 23:03 |
| fungi | puppet is running on review, gerrit.config has <p></p> in the commentlink pattern | 23:04 |
| clarkb | fungi: and uses html instead of link? | 23:04 |
| sdague | bnemec: so why does it need to come after the python command? | 23:04 |
| fungi | clarkb: yep | 23:05 |
| sdague | ⛇ | 23:06 |
| bnemec | sdague: If you try to set it on the command-line the way it was, it doesn't take because the variables are evaluated before the environment is altered by the variables set on the command-line. | 23:07 |
| bnemec | The stackoverflow link is basically a simple example of that. | 23:07 |
| bnemec | sdague: If you run "FOO=foo echo $FOO" in a shell you'll see that it doesn't work the way you expect. Hopefully. :-) | 23:10 |
| sdague | yep, gotcha | 23:10 |
| sdague | took a second to think it through | 23:10 |
| sdague | yeh, this is actually the fix for the thing beagles and I were hitting earlier | 23:11 |
| sdague | nice | 23:11 |
| *** pmathews has quit IRC | 23:12 | |
| sdague | dkranz: so I looked a rossa's patches, +Aed 2 of them in | 23:16 |
| sdague | dkranz: https://review.openstack.org/#/c/63847/ should get approved if you are still kicking around | 23:16 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/63840 failed tempest with an unrecognized error | 23:20 |
| sdague | ok, officially done with work for the year, though I might check on some things in the early morning hours over the holiday. Happy holidays everyone. | 23:20 |
| *** markmcclain has quit IRC | 23:24 | |
| *** markmcclain has joined #openstack-qa | 23:26 | |
| *** harlowja_ has joined #openstack-qa | 23:29 | |
| *** harlowja has quit IRC | 23:29 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Neutron: create network resources when agents are started https://review.openstack.org/63641 | 23:34 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/63827 failed tempest with an unrecognized error | 23:37 |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Added ceilometer-anotification to enabled services https://review.openstack.org/61958 | 23:43 |
| *** SergeyLukjanov has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!