| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-21 | |||
| 17:19:02 | tssurya | dansmith: not needed | |
| 17:19:22 | dansmith | tssurya: especially if it means I spend the time reviewing your stuff yeah? :) | |
| 17:19:35 | tssurya | yeaaaaa :D | |
| 17:20:24 | melwitt | +1 to skipping today | |
| 17:20:46 | cdent | the whole day | |
| 17:20:48 | dansmith | woot, even if mriedem really wants a meeting, he's outnumbered | |
| 17:21:44 | ildikov | jgriffith: you need to repeat those that thousands of times so they stick in people's heads :) | |
| 17:22:32 | cfriesen | tssurya: you pinged earlier? | |
| 17:25:21 | mriedem | ildikov: or push a patch to nova to add something to the docstring for the attachment_update method so we don't forget | |
| 17:25:26 | mriedem | i'd +2 that | |
| 17:26:21 | ildikov | mriedem: I can when I'm off booth duty and got to the next hotel today | |
| 17:26:22 | mriedem | dansmith: we've changed 'features' to 'fixes' in release notes in backports for other nova-manage changes for optional things, in order to fix bugs, so i think that's fine | |
| 17:27:07 | tssurya | cfriesen: yea just to ask about the third point in the bug description, since you explicitly stated delete service/compute node, I wanted to just confirm we are still doing a soft delete | |
| 17:27:49 | tssurya | cfriesen: saw you review btw, will change it to cascade, working on some tests and then will update the patch | |
| 17:27:51 | ildikov | mriedem: It won't replace the need of repeating the phrase a thousand times everywhere else | |
| 17:28:48 | cfriesen | tssurya: I think the service/node portion can remain as it is currently. what you have looks reasonable to me with the cascade change | |
| 17:29:54 | tssurya | cfriesen: ack, however I think maybe we need to provide a new command at some point to allow compute node deletions ? like we have osc placement allowing resource provider deletions ? | |
| 17:29:57 | cfriesen | tssurya: dansmith is the go-to guy in this area. :) | |
| 17:30:18 | cfriesen | tssurya: you mean compute node deletions without service deletion? | |
| 17:30:27 | dansmith | mriedem: fine, fine | |
| 17:30:28 | tssurya | cfriesen: yes he is, he just promised he will review ^^ ;) | |
| 17:30:42 | tssurya | cfriesen: yes | |
| 17:31:06 | sean-k-mooney | mriedem: just looked at https://review.openstack.org/#/c/554917 and yes i think that is fine for stable/queens | |
| 17:31:17 | tssurya | I mean deletion of cn, along with its dependencies | |
| 17:32:08 | tssurya | the record as such does not get removed right ? | |
| 17:32:26 | tssurya | or I don't know when its moved to shadow tables | |
| 17:34:08 | dansmith | in the nova database, | |
| 17:34:09 | dansmith | deleting a thing marks it as deleted (deleted=row['id']) | |
| 17:34:11 | mdbooth | dansmith: Thanks for the ping, caught up. | |
| 17:34:20 | dansmith | "nova-manage db archive-deleted rows" moves it to shadow tables | |
| 17:34:26 | dansmith | "nova-manage db purge" removes it from shadow tables | |
| 17:34:27 | dansmith | tssurya: ^ | |
| 17:34:42 | dansmith | mdbooth: ack, I assume we're to expect a review from mriedem forthwith | |
| 17:34:54 | tssurya | ah yes was just checking this, I don't know why for some reason I though only the instance* tables were archived | |
| 17:35:00 | mdbooth | dansmith: Fortunately the above is consistent with my understanding having spoken to Gorka the other week. | |
| 17:35:00 | dansmith | \o/ | |
| 17:35:09 | dansmith | mdbooth: ack, that's good :) | |
| 17:35:14 | mdbooth | dansmith: Heh, I do that, except I tend to google first :) | |
| 17:35:30 | dansmith | mdbooth: this is 'merica.. fire from the hip and ask questions later | |
| 17:35:52 | cfriesen | tssurya: under what scenario would we want to delete only a compute node and not a service? | |
| 17:36:08 | dansmith | cfriesen: ironic nodes outnumber the service(s) they're owned by | |
| 17:36:20 | dansmith | cfriesen: and, ironic nodes balance between services, and could potentially be orphaned | |
| 17:36:41 | tssurya | cfriesen: nope, I was talking about the delete records in the compute node table | |
| 17:36:42 | dansmith | cfriesen: I would guess maybe the same could happen if you change the hostname on a virt host | |
| 17:36:50 | tssurya | but that is done by archive_deleted_rows | |
| 17:36:56 | tssurya | already so taken care of :) | |
| 17:37:01 | tssurya | sorry for the confusion | |
| 17:37:04 | mdbooth | mriedem: Incidentally lyarwood pointed out that we're not running volume tests in the live migration job by default, so on his suggestion I hacked a run based on another patch he pointed me at. Link is in a review comment. | |
| 17:38:44 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Update spec to reflect reality https://review.openstack.org/555000 | |
| 17:39:28 | stephenfin | jaypipes, gibi: Could you stick this on your review queue. Just ensure the spec for a Queens feature matches the actual implementation https://review.openstack.org/555000 | |
| 17:39:31 | stephenfin | *? | |
| 17:40:49 | sean-k-mooney | stephenfin: is that the pci numa policy stuff? | |
| 17:41:26 | sean-k-mooney | stephenfin: i taught we did not want to put the policy in the alias? | |
| 17:43:07 | sean-k-mooney | stephenfin: hum well its in the code here https://github.com/openstack/nova/blob/master/nova/pci/request.py#L90-L92 so i guess we did | |
| 17:43:15 | cfriesen | sean-k-mooney: ah, but we did...that way you can be flexible for one device and strict for another | |
| 17:43:28 | openstackgerrit | Dan Smith proposed openstack/nova stable/pike: Add --by-service to discover_hosts https://review.openstack.org/554603 | |
| 17:45:01 | sean-k-mooney | cfriesen: yes i see that i guess it works but eventually i would hope we can get this kind of stuff out of configs and into openstack server create --device vendor-id=xyz ... | |
| 17:45:58 | sean-k-mooney | cfriesen: with cyborg becoming a thing that seams like the correct route long term | |
| 17:46:07 | cfriesen | sean-k-mooney: I'd be fine with that, but it might affect the instance packing (which dansmith has smacked me with before) | |
| 17:46:30 | sean-k-mooney | instance packing? | |
| 17:46:33 | sean-k-mooney | in placement | |
| 17:46:44 | cfriesen | defining flavors to pack instances neatly onto a compute node with minimal waste | |
| 17:47:15 | dansmith | cfriesen: I don't believe I smacked you, I just made a big farting noise and gave it a thumbs down | |
| 17:47:19 | dansmith | you know, to be respectfyul | |
| 17:47:21 | sean-k-mooney | cfriesen: ah ok. well currently passh through devices are not first class resouces. if they are it changes the usage model and billing model | |
| 17:49:02 | sean-k-mooney | but ya in any case having the numa policy in the alias kind of works but its not discoverable by the tenant so it wont work for everyone. espcilally the people that dont like setting things in the nova conf | |
| 17:50:42 | openstackgerrit | melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | |
| 17:50:43 | openstackgerrit | melanie witt proposed openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414 | |
| 17:50:43 | openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | |
| 17:51:59 | cfriesen | sean-k-mooney: are you proposing that we add an HTTP API for PCI aliases? | |
| 17:53:16 | sean-k-mooney | cfriesen: well no i dont think we should keep adding features to the aliases but maybe in the flavour details show some of the alais fields | |
| 17:53:57 | sean-k-mooney | cfriesen: long term however i would much prefer this handeled with resouce requests and traits in the flavor then using aliases | |
| 17:55:02 | sean-k-mooney | cfriesen: the mater of how that resource request gets there(set by admin directly or via boot option e.g. --device) is a seperate disction | |
| 17:55:17 | sean-k-mooney | /disction/topic/ | |
| 17:58:47 | sean-k-mooney | cfriesen: these https://github.com/openstack/nova/blob/master/nova/pci/request.py#L74-L89 can all be modeled as traits on the RP in placement and the numa_policy can be modeled as a flavor extra spec key that is consumed by the numa_topology/pci passhtrough filters. | |
| 18:04:19 | openstackgerrit | Mark Goddard proposed openstack/nova master: Request only instance_uuid in ironic node list https://review.openstack.org/539509 | |
| 18:18:49 | cfriesen | once nova-compute spawns a privsep daemon, does it stay running after that? | |
| 18:21:13 | sean-k-mooney | cfriesen: yes and a seperate priv sep deamon is spawned for each context | |
| 18:22:23 | sean-k-mooney | cfriesen: so if you have two contexts with 2 different permission sets you will have 2 privsep deamons running for nova-compute | |
| 18:22:58 | sean-k-mooney | when nova compute exits the deamons also exits when the unix socket gets closed | |
| 18:26:15 | melwitt | dansmith: patch for removing the useless non-periodic task line has acks from efried_rollin and gibi https://review.openstack.org/#/c/554381 | |
| 18:27:00 | dansmith | melwitt: ah yeah thanks for the reminder | |
| 18:27:08 | dansmith | melwitt: so confirmed what I was saying about that test yeah? | |
| 18:27:22 | melwitt | yes, I tried it out and indeed it was only covering that one line : | |
| 18:27:24 | dansmith | melwitt: and I also wonder if that was the only source of the overwriting-context problem initially | |
| 18:27:25 | melwitt | :\ | |
| 18:27:29 | dansmith | ack | |
| 18:27:35 | melwitt | yeah, I think it was -_- | |
| 18:38:11 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Snapshot https://review.openstack.org/543023 | |
| 18:40:08 | edmondsw | https://review.openstack.org/#/c/547169/ should be a quick review if someone has a few minutes | |
| 18:40:44 | edmondsw | prereq to starting to flip some of those capabilities to True as we introduce functions in later commits | |
| 18:42:25 | cfriesen | sean-k-mooney: cool, thanks | |
| 18:46:08 | mriedem | dansmith: commented on mdbooth's change https://review.openstack.org/#/c/551302/ | |
| 18:46:15 | mriedem | dansmith: overall it's ok, but i think we also have a problem in rollback | |
| 18:48:02 | dansmith | mriedem: cool thanks | |
| 18:53:56 | mriedem | i looked at his live migration test patch that enabled the volume-backed live migration tests, | |
| 18:54:04 | mriedem | the grenade live migration job passed, which uses mixed computes | |
| 18:54:16 | mriedem | and goes back and forth (rocky->queens->rocky) | |
| 18:54:20 | mriedem | and vice-versa | |
| 18:54:34 | mriedem | the other non-grenade live migration job failed, looks like all rpc messaging timeouts | |