| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-26 | |||
| 12:20:42 | mriedem | http://logs.openstack.org/47/467447/14/check/gate-trove-scenario-dsvm-mariadb-multi-ubuntu-xenial-nv/6efec50/logs/screen-n-sch.txt.gz#_May_26_03_24_31_029544 | |
| 12:20:52 | mriedem | the anti-affinity filter kicked it out, which is why you're getting NoValidHost, | |
| 12:20:57 | mriedem | also, the scheduler is only seeing 1 host | |
| 12:21:02 | sdague | mriedem: right, that's expected | |
| 12:21:06 | mriedem | are you using devstack? | |
| 12:21:08 | sdague | that's actually the point of the test | |
| 12:21:28 | amrith | mriedem it is devstack driven through trovestack (formerly redstack) | |
| 12:21:49 | amrith | i can send you my local confs if that will help | |
| 12:22:01 | amrith | but you can see the confs in the CI as well | |
| 12:22:02 | sdague | the real question is actually why nova-conductor fails to actually error the the node | |
| 12:22:16 | sdague | not that it fails to schedule | |
| 12:22:28 | mriedem | yes i realize | |
| 12:22:33 | sdague | ok, cool | |
| 12:24:39 | mriedem | odd that i can't get hits on this in logstash | |
| 12:24:51 | sdague | is conductor indexed? | |
| 12:24:58 | sdague | or these jobs? | |
| 12:25:11 | mriedem | conductor is certainly indexed, | |
| 12:25:21 | mriedem | but i didn't think it was per job | |
| 12:25:24 | mriedem | it's per file | |
| 12:26:26 | mriedem | i think i see the problem, maybe | |
| 12:26:28 | mriedem | patch incoming | |
| 12:28:48 | sdague | cool | |
| 12:28:51 | sdague | thanks | |
| 12:28:59 | mriedem | amrith: got a bug? | |
| 12:30:23 | sdague | mriedem: I don't think he does, and he just had to head off for holiday weekend | |
| 12:31:08 | mriedem | ok, i'll put up a trove patch with depends-on to test this out | |
| 12:31:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Use targeted context when burying instances in cell0 https://review.openstack.org/468401 | |
| 12:31:38 | mriedem | dansmith: ^ | |
| 12:32:38 | openstackgerrit | Sean Dague proposed openstack/python-novaclient master: Have python-novaclient support global_request_id https://review.openstack.org/468126 | |
| 12:33:36 | mriedem | aha https://bugs.launchpad.net/nova/+bug/1693438 | |
| 12:33:36 | openstack | Launchpad bug 1693438 in OpenStack Compute (nova) "error instances remain in "Build" status and can't delete it" [Undecided,New] - Assigned to Tetsuro Nakamura (tetsuro0907) | |
| 12:33:42 | mriedem | already reported yesterday by someone doing a good job at CD | |
| 12:34:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Use targeted context when burying instances in cell0 https://review.openstack.org/468401 | |
| 12:35:51 | sdague | mriedem: nice | |
| 12:39:47 | mriedem | amrith: fyi https://review.openstack.org/#/c/468402/ | |
| 12:40:45 | mriedem | and now i'm afraid i must get cleaned up | |
| 12:42:08 | sdague | mriedem: what was the dogpile cache issue with nova? | |
| 12:44:24 | amrith | mriedem your trove change will fail because of the nova client 8.0.0. if you'd like please dep my change (which aims to fix that) on your proposed fix | |
| 12:44:28 | amrith | sdague ^^ | |
| 12:44:47 | sdague | amrith: yeh, I can update that | |
| 13:00:39 | leakypipes | morning folks | |
| 13:02:56 | sdague | leakypipes: in reading through test logs of my failing patches, I saw a lot of sqla warnings are being emitted around the traits tests | |
| 13:03:03 | sdague | because of unicode mismatch | |
| 13:03:17 | cdent | sfinucan: leakypipes put his +2 on https://review.openstack.org/#/c/461494/ so it is safe for you to raise your +1 to +2 if you like. or anyone else if they are feeling like a quick win | |
| 13:03:43 | sfinucan | cdent: Yup, done | |
| 13:03:44 | leakypipes | sdague: ya. I've been meaning to ask alex_xu to submit fixups for those... | |
| 13:03:50 | cdent | sfinucan: yay! | |
| 13:04:26 | sdague | leakypipes: ok, cool | |
| 13:04:31 | leakypipes | sdague: but since it's now the weekend for alex_xu I'll put the word out to get that cleaned up. | |
| 13:04:43 | leakypipes | efried: feel like an easy one? :) | |
| 13:04:53 | sdague | yeh, that's fine, I just wanted to make sure it was known | |
| 13:05:15 | leakypipes | sdague: yup, thank you sir :) | |
| 13:06:21 | sdague | cross service request id tracking to cinder is Jenkins +1 - https://review.openstack.org/#/c/468355/ if anyone wants to push that in | |
| 13:11:18 | openstackgerrit | Sean Dague proposed openstack/nova master: Migrate to oslo request_id middleware - mv 2.46 https://review.openstack.org/467998 | |
| 13:22:39 | mriedem | sdague: i don't think a bug was reported for the dogpile.cache test failure | |
| 13:27:17 | mriedem | sdague: tsk tsk https://review.openstack.org/#/c/468355/1 | |
| 13:28:23 | sdague | mriedem: yeh, I was going to swing around with minimums at the end | |
| 13:28:44 | sdague | because otherwise it's another couple of days per patch | |
| 13:28:55 | mriedem | we're using a new thing on a minimum version, | |
| 13:28:59 | mriedem | we should require the minimum version | |
| 13:29:01 | sdague | and if you are CDing and not using upper constraints | |
| 13:29:03 | mriedem | regardless of u-c | |
| 13:29:04 | sdague | you are crazy | |
| 13:29:16 | mriedem | if i'm packaging and picking up when minimums bump... | |
| 13:29:20 | mriedem | at the least | |
| 13:29:26 | mriedem | anyway, can we just do that? it's trivial | |
| 13:29:31 | mriedem | and it's the right thing | |
| 13:29:53 | sdague | mriedem: we have to do g-r change to and Depends-On, that's fine, but I need to swing around on it | |
| 13:30:07 | mriedem | sure | |
| 13:30:13 | mriedem | or i can quick | |
| 13:30:46 | sdague | mriedem: I'll at least state for my defense, even trivial individual patches aren't so much when you have 12 of them outstanding in 7 repos, with depends-on :) | |
| 13:31:12 | sdague | I'll do it | |
| 13:31:39 | mriedem | just pushed the reqs change | |
| 13:31:42 | sdague | ok | |
| 13:31:51 | mriedem | https://review.openstack.org/468417 | |
| 13:32:10 | sdague | oh, you want to bump cinderclient as well | |
| 13:32:14 | mriedem | sure | |
| 13:32:14 | sdague | if you want to be strict about it | |
| 13:32:17 | mriedem | i do | |
| 13:33:41 | mriedem | done | |
| 13:33:50 | mriedem | https://review.openstack.org/468419 | |
| 13:36:06 | stvnoyes | mriedem: johnthetubaguy: when you get a minute, please take a look at https://review.openstack.org/#/c/463987/ - cinder v3 migrate update. thanks. | |
| 13:36:09 | dansmith | who is responsible for the bindep stuff? is that fungi? | |
| 13:37:30 | leakypipes | figleaf: you pushing up a new patch for the send instance UUIDs in param changeset? | |
| 13:38:03 | sdague | dansmith: mostly, what's up though? | |
| 13:38:28 | dansmith | sdague: was just trying it on a fresh centos7 and it just does nothing, "file not found" or something.. I can repro here in a sec | |
| 13:38:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Send request_id on neutron calls https://review.openstack.org/468356 | |
| 13:38:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Send request_id on cinder calls https://review.openstack.org/468355 | |
| 13:38:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Send request_id on glance calls https://review.openstack.org/467242 | |
| 13:38:41 | mriedem | sdague: ^ updated with depends-on | |
| 13:38:50 | leakypipes | dansmith: don't you need to find a phone booth to change your outfit in? | |
| 13:38:51 | mriedem | marked neutron and glance ones as wips until the related clients are in g-r | |
| 13:39:07 | dansmith | leakypipes: no, I'm more of a public exposure kinda guy | |
| 13:39:13 | sdague | mriedem: ok, thanks | |
| 13:39:13 | leakypipes | lmao | |
| 13:39:41 | sdague | mriedem: can you squash those 2 g-r changes? | |
| 13:40:04 | sdague | because otherwise we're running about 20 jobs for each and they don't actually even test anything | |
| 13:40:17 | sdague | or, I'll do it | |
| 13:40:19 | mriedem | the cinderclient one could cause someting to fail | |
| 13:40:27 | mriedem | that's why i separated them | |