| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-13 | |||
| 18:27:54 | bnemec | aspiers: +2. Thanks for taking care of this! | |
| 18:28:03 | aspiers | bnemec: yw | |
| 18:28:20 | mriedem | sean-k-mooney: don't think so | |
| 18:28:27 | mriedem | unless you reset the instance state to ERROR | |
| 18:28:29 | sean-k-mooney | im fine to leave that up to the operator to clean up too, but im just wondering about the case where the reason their are pending migration or evacuation is because of a host failure | |
| 18:28:51 | sean-k-mooney | or reset it ot active | |
| 18:28:57 | sean-k-mooney | but ok | |
| 18:29:58 | sean-k-mooney | if it was in resize_verify or whatever the state is when we havent confirmed yet but was successafaly migrated we would proably want to reset it to active | |
| 18:30:04 | mriedem | if you're in the middle of a migration and the source host went offline, i don't see why you'd delete the service - just bring the service back up if possible | |
| 18:31:31 | sean-k-mooney | ya im somewhat unclear why the compute service is being deleted with outstanding migrations too | |
| 18:32:14 | sean-k-mooney | also reset state to active instead of confirming might not be right as it likely would not have updated the db correctly for the new host | |
| 18:32:46 | sean-k-mooney | so you would want to confirm. let it error out and then rest to active/hardreboot to fix it right | |
| 18:33:54 | mriedem | well the good news is a contracted google recruiter has contacted me about opportunities in the bay area of new york so if i want to avoid this problem i can just follow that lead and up-root my entire life | |
| 18:34:03 | mriedem | delete compute service problems be damned! | |
| 18:34:43 | sean-k-mooney | haha well that is one way to solve it yes then you can work on kubernetes problems | |
| 18:35:28 | sean-k-mooney | its really a seperate issue but should we allow you to confim a resize/migration if the souce host is down without erroring out? | |
| 18:35:29 | mriedem | then after 6 months you get there and they're like, "ok we have this resource leak issue when you delete a pod" | |
| 18:36:33 | mriedem | sean-k-mooney: i'm not sure what happens in that case, i haven't tried it, but i'd think the cast to the source compute would blow up and you'd get a 500 response from the confirmResize API | |
| 18:37:04 | sean-k-mooney | well if its a cast it a fire and forget if its a call then yes | |
| 18:37:37 | sean-k-mooney | do we start the confrim form the souce node so the api will jsut time out the request | |
| 18:37:47 | sean-k-mooney | e.g. because notheing handels it. | |
| 18:38:10 | melwitt | efried: do you want a follow up patch for your comments on https://review.opendev.org/651947 | |
| 18:38:23 | efried | melwitt: meh, they're so minor. | |
| 18:38:29 | melwitt | ack | |
| 18:38:31 | efried | melwitt: If you want to spin a new PS I can fast approve | |
| 18:38:39 | sean-k-mooney | i need to get a better undestand of how to create nova functional test to test this kind fo stuff without actully depolying devstack | |
| 18:38:41 | efried | or we can just ignore | |
| 18:40:03 | efried | mriedem: Was about to start composing the email whining about bug queue. Unless you want to do it... <bats eyes> | |
| 18:40:09 | melwitt | efried: ok. I can respin, that's easy enough. mostly I'm thinking of the doc rendering where literal would help | |
| 18:40:53 | efried | melwitt: There's probably opportunity to add literals throughout that doc - I didn't check. If so, maybe we could scoop it all up in a fup | |
| 18:41:45 | efried | anyway, I'm around for at least another 3-4h to push whatever | |
| 18:42:12 | melwitt | ok. yeah, I see most seem literal'd but some others aren't so could be done all together | |
| 18:43:45 | mriedem | efried: i do not, writing a reply to this compute service delete thread | |
| 18:44:06 | efried | mriedem: ack. Okay if I run it by you before sending? | |
| 18:47:01 | sean-k-mooney | mriedem: i might steal your test code form https://review.opendev.org/#/c/663737/1/nova/tests/functional/wsgi/test_services.py and try and create a similar test for that migrate confim case while im waiting for the pizza to arrive | |
| 18:48:01 | dansmith | mmm pizza | |
| 18:48:16 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: api-guide: better explain scheduler hints. https://review.opendev.org/522106 | |
| 18:48:38 | sean-k-mooney | hehe one of irelands suckiest inventions is slow "fast food" | |
| 18:49:28 | sean-k-mooney | all the quality of fast food with the slowness of going to the store and cooking it your self. | |
| 18:49:37 | mriedem | COOKIES | |
| 18:49:39 | sean-k-mooney | given i havent ordered it yet i might actully do that | |
| 18:51:27 | melwitt | did somebody say pizza | |
| 18:52:10 | mriedem | dansmith: i was just thinking about that yesterday for some reason, probably b/c i made a pizza for lunch | |
| 18:53:11 | dansmith | heh | |
| 19:03:10 | dansmith | mriedem: I don't even remember what the gag was.. trying to explain to sdague how to use punctuation or something | |
| 19:04:36 | openstackgerrit | melanie witt proposed openstack/nova master: Literalize CLI options in docs https://review.opendev.org/665217 | |
| 19:05:27 | mriedem | i think it was related to commit message titles | |
| 19:05:29 | mriedem | something like that | |
| 19:05:31 | mriedem | Something like that. | |
| 19:05:40 | mriedem | one of the annual flare ups in the ML | |
| 19:05:48 | mriedem | the good old days | |
| 19:06:17 | mriedem | i think we were shitting on his commit message which was all lower case with sparse amount of periods | |
| 19:06:31 | dansmith | sounds right | |
| 19:06:38 | mriedem | mostly to poke the bear and air grievances | |
| 19:06:43 | mriedem | it wasn't even festivus | |
| 19:07:14 | mriedem | and now look at him, in front of congress and all | |
| 19:07:17 | mriedem | we really showed him | |
| 19:07:32 | dansmith | because congress has such high standads | |
| 19:07:34 | dansmith | *standards | |
| 19:08:17 | mriedem | like garth brooks, sean has friends in low places | |
| 19:08:33 | dansmith | heh | |
| 19:09:22 | melwitt | remember chris gaines | |
| 19:10:37 | mriedem | how could i forget | |
| 19:10:50 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/657016 | |
| 19:10:50 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in delete_resource_provider https://review.opendev.org/657021 | |
| 19:10:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Optimize SchedulerReportClient.delete_resource_provider https://review.opendev.org/657070 | |
| 19:11:20 | mriedem | remember...THIS?! https://www.youtube.com/watch?v=oGpFcHTxjZs | |
| 19:12:17 | dansmith | man.. bad hair day | |
| 19:12:34 | melwitt | the intro made it look like the panpipe sound was coming from the tugboat lol | |
| 19:12:43 | mriedem | maya got her first exposure to that in the car last night, with me singing along to it | |
| 19:12:44 | melwitt | peruvian panpipes | |
| 19:13:05 | mriedem | and laura telling me to shut up | |
| 19:13:46 | melwitt | did you dance too? I remember you said that's a favorite for car trips with the family | |
| 19:14:03 | mriedem | i did the shoulder shake sure | |
| 19:14:09 | mriedem | it cannot be stopped | |
| 19:16:51 | melwitt | yeah, that's what I meant. shoulder shake was the thing they are a fan of | |
| 19:20:06 | efried | mriedem: http://paste.openstack.org/show/752899/ | |
| 19:20:09 | efried | not sure what else to say | |
| 19:21:27 | mriedem | efried: looks fine to me | |
| 19:21:33 | efried | ight, thx | |
| 19:31:53 | mriedem | lyarwood: efried: did anyone talk about case here? https://review.opendev.org/#/c/663011/17/nova/virt/libvirt/blockinfo.py@275 | |
| 19:34:04 | efried | mriedem: doesn't look like it's ever uppercase, but I really don't know. | |
| 19:35:33 | mriedem | yeah the image meta props don't define an enum | |
| 19:37:52 | mriedem | looks like libvirt / qemu uses q35 though http://logs.openstack.org/11/663011/17/check/tempest-full-py3/6a826d2/controller/logs/screen-n-cpu.txt.gz#_Jun_12_22_24_13_577256 | |
| 19:38:09 | openstackgerrit | Merged openstack/nova stable/rocky: Move get_pci_mapping_for_migration to MigrationContext https://review.opendev.org/661499 | |
| 19:38:17 | openstackgerrit | Merged openstack/nova stable/rocky: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/661500 | |
| 19:38:26 | openstackgerrit | Merged openstack/nova stable/rocky: Fixes multi-registry config in Quobyte driver https://review.opendev.org/662695 | |
| 19:38:34 | openstackgerrit | Merged openstack/nova stable/rocky: tox: Use basepython of python3.5 https://review.opendev.org/663887 | |
| 19:38:42 | openstackgerrit | Merged openstack/nova stable/rocky: docs: Don't version links to reno docs https://review.opendev.org/662178 | |
| 19:38:50 | openstackgerrit | Merged openstack/nova stable/rocky: Include all network devices in nova diagnostics https://review.opendev.org/661962 | |
| 20:01:07 | openstackgerrit | Merged openstack/nova master: Api-guide: Add Block Device Mapping https://review.opendev.org/522084 | |
| 20:07:38 | spatel | sean-k-mooney: Question | |
| 20:08:45 | spatel | building new cloud so should i go with linuxbridge or OVS ( any performance difference here? other than features) | |
| 20:15:07 | mriedem | efried: fwiw i'm going through stephen's metadata docs change now | |
| 20:15:22 | efried | mriedem: cool, thanks for that. | |
| 20:15:32 | efried | mriedem: It still hasn't rendered, right? | |
| 20:15:44 | efried | I guess I could pull it down and build it myself... | |
| 20:15:49 | mriedem | that's what i did | |
| 20:17:57 | mriedem | the docs are failing to build locally for me | |
| 20:32:52 | mriedem | i see why now | |