| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-11-12 | |||
| 06:50:56 | openstackgerrit | Jorhson Deng proposed openstack/nova master: To deal instance with soft-deleting in _init_instance https://review.opendev.org/761264 | |
| 07:54:21 | openstackgerrit | Jorhson Deng proposed openstack/nova master: To deal instance with soft-deleting in _init_instance https://review.opendev.org/761264 | |
| 08:07:18 | f0o | Hi, not sure if this is an issue in nova/kvm or the guest-os: Sometimes (really random) when a linux guest, predominantly CoreOS/Flatcar, attempts to reboot the instance is placed into Puased state instead. However the pause state is erroneously because there's no resume action possible so something must've gone bonkers in the state sync. Only issueing Hard-Reboot via cli will actually | |
| 08:07:20 | f0o | bring the guest back alive | |
| 08:23:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Improve error handling during service level check https://review.opendev.org/762471 | |
| 08:45:00 | bauzas | good and very cold morning Nova | |
| 09:19:22 | lyarwood | has anyone written up a bug for functional failing (due to an eventlet bug) on 3.9 yet? | |
| 09:19:25 | lyarwood | AttributeError: 'Semaphore' object has no attribute '_at_fork_reinit' | |
| 09:19:51 | lyarwood | https://github.com/eventlet/eventlet/issues/646 fixed it a few days ago | |
| 09:26:35 | lyarwood | kay weird now it's working | |
| 09:26:39 | lyarwood | okay* | |
| 09:26:55 | lyarwood | nvm | |
| 09:29:21 | openstackgerrit | Wenping Song proposed openstack/nova master: Remove redundant check https://review.opendev.org/762482 | |
| 09:48:38 | openstackgerrit | chengsheng proposed openstack/nova master: Modify the default value of the force parameter in live migration https://review.opendev.org/762458 | |
| 09:53:54 | stephenfin | gibi: sean-k-mooney1: FYI https://review.opendev.org/762396 | |
| 09:54:04 | stephenfin | that's the OSC change to support shelve offload | |
| 10:05:53 | gibi | stephenfin: ack, thanks, will look | |
| 10:12:30 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563 | |
| 10:31:51 | openstackgerrit | Jorhson Deng proposed openstack/nova master: To deal instance with soft-deleting in _init_instance https://review.opendev.org/761264 | |
| 10:33:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Revert "Temporarily disable parts of heal port allocation test" https://review.opendev.org/755995 | |
| 10:55:09 | openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] Rename host to hostname in conductor manager https://review.opendev.org/762499 | |
| 10:56:31 | brinzhang_ | gibi: I would like to rename the variable 'host' express 'hostname' in conductor manager, is it ok? | |
| 10:59:31 | gibi | brinzhang_: just quickly looked at it it seems OK, you might consider renaming node to nodename in the same context | |
| 11:02:09 | brinzhang_ | gibi: ok, I will do it later, thanks | |
| 11:06:35 | gibi | fyi folks, there is a new type of gate failure (at least it is new for me) https://bugs.launchpad.net/nova/+bug/1903979 | |
| 11:06:35 | openstack | Launchpad bug 1903979 in OpenStack Compute (nova) "nova-live-migration job fails during evacuate negative test" [High,Confirmed] | |
| 11:12:16 | lyarwood | gibi: I can look at this today | |
| 11:12:45 | gibi | lyarwood: thanks, I'm pushing a e-r signature for it | |
| 11:14:26 | gibi | here is the e-r patch https://review.opendev.org/762502 | |
| 11:16:03 | lyarwood | urgh zuul's webui isn't responding for me | |
| 11:16:09 | gibi | it is slow | |
| 11:16:12 | gibi | for me | |
| 11:16:16 | gibi | but eventually opens | |
| 11:16:33 | lyarwood | gibi: do you have the raw console log link? | |
| 11:19:04 | gibi | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_ca4/762319/1/check/nova-live-migration/ca4ccb9/job-output.txt | |
| 11:20:28 | gibi | lyarwood: ^^ | |
| 11:23:19 | sean-k-mooney1 | stephenfin: it more or less looks ok but i left a few comments | |
| 11:23:39 | sean-k-mooney1 | stephenfin: i think you are failing out to early | |
| 11:23:49 | openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] Rename host/node to hostname/nodename in conductor manager https://review.opendev.org/762499 | |
| 11:35:28 | stephenfin | sean-k-mooney1: replied | |
| 11:36:35 | sean-k-mooney1 | oh my nick is wrong on sec | |
| 11:43:19 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563 | |
| 11:58:24 | sean-k-mooney | stephenfin: i have removed the -1 and reviewd the other i can accept adressing the inefficnies in a follow up to keep the inital patch less complex espically since you left the todo | |
| 11:58:35 | stephenfin | great, thanks :) | |
| 11:59:00 | sean-k-mooney | stephenfin: i dont think you need a wait_for_deletes that shoudl just call a more generialsed wait_for_statuses | |
| 11:59:34 | stephenfin | I could probably do that, yeah | |
| 11:59:41 | stephenfin | I'd have to keep wait_for_delete around though | |
| 12:00:11 | sean-k-mooney | is that not just calling wait_for_statuses with a list of 1 | |
| 12:00:24 | stephenfin | no, you need exception handling | |
| 12:00:40 | sean-k-mooney | right but you can add that to wait_for_statuses | |
| 12:00:41 | stephenfin | since a 404 is actually success (the resource has been deleted) | |
| 12:00:53 | sean-k-mooney | just default it to an empty list of allowed excptionts | |
| 12:01:07 | stephenfin | Right, but people are using 'wait_for_delete' in the wild | |
| 12:01:16 | stephenfin | so I'd have to keep the API for some time at least | |
| 12:01:24 | sean-k-mooney | oh ya keep the function | |
| 12:01:29 | sean-k-mooney | just rip out the logic | |
| 12:01:32 | sean-k-mooney | and delegate | |
| 12:01:40 | stephenfin | I think it's clearer as a separate function, tbh | |
| 12:01:44 | stephenfin | deleted isn't a status | |
| 12:01:58 | stephenfin | it's a binary condition | |
| 12:02:08 | sean-k-mooney | it is if you pass the flag to list deleted servers | |
| 12:02:42 | sean-k-mooney | stephenfin: no if you have not done archive and purge | |
| 12:02:51 | sean-k-mooney | you can see deleted servers form the api | |
| 12:03:32 | sean-k-mooney | you just do "openstack server list --deleted" | |
| 12:03:36 | stephenfin | for nova, yes | |
| 12:03:49 | stephenfin | it's racy though | |
| 12:03:52 | stephenfin | even for nova | |
| 12:04:04 | sean-k-mooney | you mean with the chron job | |
| 12:04:08 | stephenfin | yes | |
| 12:04:28 | sean-k-mooney | for osp at least that only deletes things older then 30 days | |
| 12:04:42 | stephenfin | Right, but its config specific. HTTP 404 isn't, by comparison | |
| 12:05:14 | stephenfin | and it works for multiple different services, so I think that's still the correct approach to use | |
| 12:05:18 | sean-k-mooney | ya anyway if you want to keep but its fine it just really felt like needless code duplication | |
| 12:05:48 | stephenfin | It probably is, but I think it's worth the cost compared to the complexity de-duplication would bring | |
| 12:06:07 | stephenfin | We could probably have an inner function not exposed to users. I'll take a look at that later | |
| 12:18:54 | openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] Rename host/node to hostname/nodename in conductor manager https://review.opendev.org/762499 | |
| 12:41:19 | openstackgerrit | Daniel de Oliveira Pereira proposed openstack/nova master: [WIP] Avoid getting transport_url from config file when updating Cell0 https://review.opendev.org/762433 | |
| 12:55:18 | openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563 | |
| 13:18:53 | zigo | sean-k-mooney: This will need a fix in puppet-openstack and OSA, that's my concern... | |
| 13:19:15 | openstackgerrit | Wenping Song proposed openstack/nova master: Change host to hostname https://review.opendev.org/762518 | |
| 13:19:22 | zigo | Which is why I'm insisting to say we should all go on the same direction (ie: all distro and config management), otherwise it's going to be a mess. | |
| 13:20:35 | sean-k-mooney | zigo: OSA does not need a fix | |
| 13:20:48 | sean-k-mooney | it already does the right thing and does not include the db creds | |
| 13:20:58 | zigo | sean-k-mooney: It depends how you see it, the version that uses packages does. | |
| 13:21:20 | sean-k-mooney | the version that uses package does not use the config form the packages | |
| 13:21:26 | sean-k-mooney | it still generate its own | |
| 13:21:27 | zigo | And anyways, it'd be nice if it was doing the same thing. | |
| 13:21:35 | zigo | Oh ... Ok. | |
| 13:22:08 | sean-k-mooney | i think that is fairly common practice | |
| 13:22:24 | sean-k-mooney | not to use the disto config if you are using a intall tool | |
| 13:22:42 | zigo | So, I should generate the db config file with only --namespace oslo.db --namespace oslo.db.concurrency, right? | |
| 13:22:46 | sean-k-mooney | they all tend to generat there own based on the layer tehy provide in ther own tuning files | |
| 13:22:55 | zigo | Or should I just extract it from the existing nova.conf stuff? | |
| 13:23:52 | sean-k-mooney | zigo: what we were suggeting to do for the rpms was have an update script that copied the nova.conf to nova-compute.conf adn then use sed or curidini to remove the db sections | |
| 13:24:14 | sean-k-mooney | but if we go with the seperate files for db | |
| 13:24:31 | sean-k-mooney | then both would work generating only really works for new installs | |
| 13:24:46 | zigo | sean-k-mooney: I'm generating the files when building the package... | |
| 13:24:49 | sean-k-mooney | for upgrades of the package you kind of need to extract | |
| 13:25:11 | sean-k-mooney | generating the files is fine but only on a fresh install right | |
| 13:25:21 | sean-k-mooney | you dont want an apt udpate to nuke your files | |