Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-11-11
21:52:32 openstackgerrit Daniel de Oliveira Pereira proposed openstack/nova master: Avoid getting transport_url from config file when updating Cell0 https://review.opendev.org/762433
22:16:04 openstackgerrit Merged openstack/nova master: Update contributor guide for Wallaby https://review.opendev.org/754427
23:26:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056
23:34:20 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add functional-py39 testing https://review.opendev.org/760884
23:36:01 gmann stephenfin: ^^ need to re-approve this. I rebased it to resolve merge conflict.
#openstack-nova - 2020-11-12
01:20:58 openstackgerrit chengsheng proposed openstack/nova master: Add hypervisor CPU feature check during live migration https://review.opendev.org/762330
02:32:20 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056
02:45:55 openstackgerrit chengsheng proposed openstack/nova master: Modify the default value of the force parameter in live migration https://review.opendev.org/762458
05:28:19 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056
05:57:57 openstackgerrit Xinran WANG proposed openstack/nova-specs master: SRIOV Enabled Nic Support Specification https://review.opendev.org/742785
06:14:33 openstackgerrit norman shen proposed openstack/nova-specs master: Adds spec for instance live resize https://review.opendev.org/141219
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 openstack Launchpad bug 1903979 in OpenStack Compute (nova) "nova-live-migration job fails during evacuate negative test" [High,Confirmed]
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: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

Earlier   Later