| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-08 | |||
| 16:01:28 | mriedem | i could have sworn... | |
| 16:01:47 | mriedem | leakypipes: so, | |
| 16:01:57 | mriedem | are you going to do this upstream with a new microversion? | |
| 16:02:11 | leakypipes | mriedem: well, frankly, a placement aggregate doesn't have the service host coupling that a nova host-aggregate currently does, so I'm just trying to get a patch in place for Ocata Nova that kind of gets nova host-aggs looking/functioning more like placement aggs | |
| 16:02:32 | mriedem | what happens when oath upgrades to pike or queens? | |
| 16:02:39 | leakypipes | mriedem: I'll throw my code up and have you take a looksie, of course. | |
| 16:02:45 | mriedem | well, | |
| 16:03:10 | leakypipes | mriedem: when oath upgrades to pike/queens, we'll likely forward-port this thing I'm wokring on | |
| 16:03:20 | mriedem | my point is, backporting microversions opens up some bad issues unless you backport all intermediate microversions too | |
| 16:03:20 | leakypipes | mriedem: until placement aggs are fully utilized. | |
| 16:03:32 | leakypipes | mriedem: I'm not doing this as a microversion :) | |
| 16:03:38 | mriedem | ok | |
| 16:03:38 | leakypipes | mriedem: I'm not changing the API at all. | |
| 16:03:53 | mriedem | "this is to support PUT /os-aggregates/{agg_id} and passing a compute node UUID instead of a service hostname." | |
| 16:03:58 | leakypipes | mriedem: just looking to see if hostname is like a UUID and if so, looking up a compute node instead of a service host. | |
| 16:04:02 | mriedem | ok, | |
| 16:04:06 | mriedem | was just going to say that :) | |
| 16:04:08 | leakypipes | it's cheeky, I know. | |
| 16:04:09 | mriedem | tricksy | |
| 16:04:19 | leakypipes | cheeky cheeky monkey. | |
| 16:04:59 | mriedem | does oath also want to be able to cold migrate/resize baremetal instances? | |
| 16:05:02 | sean-k-mooney | superdan: i was under the impression that we required the hostID to be gloally unique across the entire cloud not jsut within a cell so if they are useing hostnames as the hostid .i.e the default then they would be required to have unique host names across all cells | |
| 16:05:26 | superdan | sean-k-mooney: correct, but they're moving a host between cells | |
| 16:06:09 | mriedem | leakypipes: because https://review.openstack.org/#/c/449155/ | |
| 16:06:42 | sean-k-mooney | superdan: yes but i was under the impression that the host id had to be unique period e.g. its invalid to have to hosts with the same hostname in different cells if you are using the hostname as the hostid | |
| 16:06:54 | leakypipes | mriedem: that hasn't been brought up, no | |
| 16:07:21 | superdan | sean-k-mooney: yes, that's true, I'm not sure what you're getting at | |
| 16:07:51 | superdan | sean-k-mooney: they are moving hosts between cells, which means the host gets a new UUID, and I think the problem is they're reporting a new uuid with the same name, and placement is kicking it out as a result | |
| 16:09:00 | finucannot | melwitt: Could you take another look at this some time today/early next week? https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security+status:open | |
| 16:09:01 | cdent | how does someone move a host between cells? Is that a command they run to which “remove the resource provider” needs to be added? | |
| 16:09:04 | sean-k-mooney | superdan: oh sorry they are moving the physical host between cells. not that they actlly have 2 phyical host with the same hostname | |
| 16:09:12 | superdan | correct | |
| 16:09:15 | dmsimard | mriedem, kashyap: would love if we could iron out https://bugs.launchpad.net/nova/+bug/1735823 soon, it's making a non-negligible amount of jobs flap | |
| 16:09:16 | openstack | Launchpad bug 1735823 in OpenStack Compute (nova) "Nova can hang when creating a VM with disk injection" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 16:09:17 | mriedem | cdent: very carefully | |
| 16:09:31 | cdent | as is so often the case | |
| 16:09:33 | superdan | cdent: that's my point | |
| 16:09:44 | melwitt | finucannot: been meaning to do that, yes I will | |
| 16:09:44 | kashyap | dmsimard: Hey, I've seen it; but a bit overwhelmed today | |
| 16:09:49 | kashyap | dmsimard: I haven't forgotten it | |
| 16:09:58 | superdan | cdent: if they're doing this thing, its going to leave residue we likely don't have a cleanup command for | |
| 16:09:58 | dmsimard | kashyap: I know the feeling, just a friendly reminder :) | |
| 16:09:58 | mriedem | dmsimard: are these only centos jobs? | |
| 16:10:12 | kashyap | Yeah, was wondering the same. | |
| 16:10:20 | cdent | superdan: ah okay, I think I’m _finally_ caught up then. I was way way back there in the weeds. | |
| 16:10:20 | mriedem | i'm assuming the problem is some difference in the guestfs package versions | |
| 16:10:43 | cdent | I blame society. Or my head cold. Or just being stupid. | |
| 16:10:46 | mriedem | dmsimard: because https://review.openstack.org/#/c/524748/ has been busted for a long time, and we know it doesn't fix the issue | |
| 16:10:46 | dmsimard | mriedem: the data that we have has problems on centos and opensuse, I couldn't find any on Ubuntu but that doesn't mean it doesn't happen there | |
| 16:11:05 | mriedem | i think the lock issue is the fact we're using eventlet.tpool | |
| 16:11:17 | dmsimard | mriedem: well, it kind of fixes the issue but then generates new ones ? right ? the patch with the depends-on fails horribly | |
| 16:11:55 | mriedem | https://review.openstack.org/#/c/524750/ ? | |
| 16:12:27 | kashyap | Yeah, that seems to be the test | |
| 16:12:29 | finucannot | sean-k-mooney: What's this about functional tests? | |
| 16:12:39 | finucannot | I assume you mean enabling ralonsoh's patches? | |
| 16:13:19 | mriedem | oh boy http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full/eb8d805/logs/screen-n-api.txt.gz?level=TRACE#_Dec_04_13_34_20_635874 | |
| 16:13:20 | sean-k-mooney | finucannot: yes im going to add a job to the experimental pipline for os-vif so we can run rodolfos tests | |
| 16:13:34 | sean-k-mooney | finucannot: once they are merged i can move it to the gate pipline | |
| 16:13:45 | finucannot | sean-k-mooney: Ideal | |
| 16:14:05 | finucannot | gate for os-vif or nova too? | |
| 16:14:18 | sean-k-mooney | check and gate for os-vif | |
| 16:14:40 | superdan | leakypipes: finucannot: melwitt: either one of you can mindlessly approve this trivial thing without worry: https://review.openstack.org/#/c/507874/1 | |
| 16:14:54 | leakypipes | superdan: BFV with BDMs? | |
| 16:14:59 | finucannot | superdan: If there's one thing I know how to do... | |
| 16:15:05 | superdan | leakypipes: not even | |
| 16:15:09 | leakypipes | :) | |
| 16:15:10 | superdan | finucannot: I knew I could count on you | |
| 16:15:35 | finucannot | sean-k-mooney: Cool. We already run oslo.versionedobjects tests in the experimental pipeline for nova. Might be worth adding os-vif there too | |
| 16:15:41 | finucannot | Eventually, anyway | |
| 16:15:55 | sean-k-mooney | i would also like to move all our os-vif jobs to zuul v3 in repo config and add a linux-bridge job for os-vif but it will be a few week before i have time to do that | |
| 16:16:11 | sean-k-mooney | finucannot: os-vif is already in the nova experimental pipline | |
| 16:16:47 | sean-k-mooney | check experimtal on a nova patch will test that patch with the head of the os-vif master branch in a seperate job | |
| 16:16:57 | finucannot | superdan: I know you asked for a mindless approval but...no bug? https://review.openstack.org/#/c/507874/ | |
| 16:17:14 | superdan | finucannot: it's not a bug yet | |
| 16:17:20 | sean-k-mooney | finucannot: the one thing it does not do is work with depend-on where the depends-on is an unmerged os-vif change | |
| 16:17:34 | superdan | finucannot: it will be after my live migrations patch merges, it's just an omission right now, the last of a few fixes ahead of time | |
| 16:17:54 | finucannot | superdan: Yup, just saw jianghuaw_'s comment now. Cool, +Wd | |
| 16:18:03 | superdan | finucannot: thanks | |
| 16:18:27 | mriedem | well it might be a bug | |
| 16:18:33 | finucannot | sean-k-mooney: All the os-vif jobs or just the unit tests? | |
| 16:18:40 | finucannot | *unit test jobs | |
| 16:18:40 | mriedem | if the driver doesn't pass migrate_data, we don't set the migration.status to 'error' | |
| 16:19:02 | sean-k-mooney | finucannot: ill try and move them all i think. we will see what infra advise | |
| 16:19:09 | sean-k-mooney | finucannot: what has nova done? | |
| 16:19:24 | mriedem | nothing | |
| 16:19:27 | mriedem | novaclient on the other hand https://review.openstack.org/#/q/I43a8435485751748ca6228f67d401945cb32652e | |
| 16:19:42 | finucannot | mriedem: Well https://review.openstack.org/#/q/status:open+branch:master+topic:zuulv3-nova | |
| 16:19:48 | mriedem | sean-k-mooney: there are very few ci jobs that nova runs which are just run against nova | |
| 16:19:51 | finucannot | We don't have to move much, thankfully | |
| 16:19:53 | mriedem | i think the nova-next job is the only one | |
| 16:19:54 | finucannot | Because of that, yes | |
| 16:20:25 | mriedem | oh lxc/lvm | |
| 16:20:39 | mriedem | umm, we can drop the wsgi-full job, unless we run that on stable | |
| 16:20:52 | mriedem | wsgi is the default in devstack since pike | |
| 16:20:59 | finucannot | We don't run any of them on stable. Drop that entirely so? | |
| 16:21:37 | sean-k-mooney | ah cool well i dont think it will be that hard to move os-vif. is there a plan to move nova eventually or just rely on the infra maintained jobs | |
| 16:21:37 | mriedem | idk, i'd have to go through this with a comb | |
| 16:21:51 | mriedem | ci job shit is always a minefield | |
| 16:21:59 | finucannot | sean-k-mooney: Those patches simply copy in the legacy ones. I haven't wrapped my head around translating a job to zuulv3 yet | |
| 16:22:04 | finucannot | despite mordred's best efforts | |