Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-08
15:57:22 leakypipes s/get_all()/get_by_uuid()
15:57:44 mriedem leakypipes: yeah correct - only the compute_nodes.host field is in there, and if you're ironic that doesn't help you
15:57:47 mriedem because 1:M
15:58:09 superdan cdent: well, I imagine it's a real issue, but I'm not sure exactly what to tell them.. I guess you could provide them the curl commands to grab a token and delete the old provider record to confirm that it heals up properly afterwards right?
15:58:17 leakypipes mriedem: FYI, this is to support PUT /os-aggregates/{agg_id} and passing a compute node UUID instead of a service hostname.
15:58:41 leakypipes mriedem: since our friends in Ironic can't use host aggregates due to the service host coupling.
15:59:24 leakypipes mriedem: and our Ironic friends can't use any aggregate or affinity filter that relies on host aggregates being associated with an Ironic resource node.
16:00:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove confusing comment in compute_node_get API method https://review.openstack.org/526718
16:00:10 mriedem leakypipes: see if this floats your boat ^
16:00:18 leakypipes mriedem: of course, I am going to need to backport all of this to Ocata, when none of the context.scatter_gather stuff was in. so that should be super-fun.
16:00:23 mriedem leakypipes: you can't add aggregates support for ironic in nova because mogan
16:00:33 sean-k-mooney leakypipes: FYI i talked to infra and at least for now they are ok with enableing legacy jobs so ill submit a patch for os-vif fuctional jobs on monday. ill see if i can migrate os-vif to in repo zuul v3 jobs before the PTG
16:00:35 leakypipes mriedem: hmm?
16:00:35 mriedem leakypipes: you mean backport internally...
16:00:41 leakypipes mriedem: correcto.
16:00:44 mriedem fun
16:00:47 mriedem oath?
16:00:50 leakypipes yuppers.
16:00:53 mriedem great
16:00:53 cdent superdan: when I say “ I haven’t had much of a chance to think about it” I really mean that. I’ve not processed what they are doing at all. If you’re trying to encourage me to take some responsibility for flagging something up, that’s great, and I can, but it’s not on my mental stack, despite the presumption of trying to stick ti on yours… :)
16:01:00 leakypipes I blame the penick.
16:01:05 mriedem leakypipes: oh right you weren't at the mogen session in sydney
16:01:18 mriedem aggregates support in mogan was one of the differentiators
16:01:18 leakypipes mriedem: I wasn't in Sydney. period. :)
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

Earlier   Later