Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-04-08
09:47:23 lyarwood gibi: avoiding the call
09:48:15 gibi thanks
09:48:36 gibi it feels a bit backward that we cannot disable a compute without the compute being up :)
10:40:10 openstackgerrit Wenping Song proposed openstack/nova-specs master: Repropose vGPU support spec https://review.opendev.org/c/openstack/nova-specs/+/780452
10:49:33 lyarwood elod: https://review.opendev.org/c/openstack/nova/+/751364/ would you mind hitting these again
10:49:49 lyarwood elod: sean-k-mooney has some backports for train that would be made easier with these landed
10:50:45 elod lyarwood: sure, will do
10:50:46 sean-k-mooney they would yes though i woudl prefer not to have to respine it again to pic them up since i have already got it working without them
10:51:38 sean-k-mooney a week ago i would have said yes hold hartedly but i already spent the day and a half getting everything to work
10:51:59 lyarwood sean-k-mooney: well either way we will need to respin something
10:52:32 sean-k-mooney why did you request changes?
10:53:05 sean-k-mooney i dont see any comments on the review
10:53:25 lyarwood sean-k-mooney: well if https://review.opendev.org/c/openstack/nova/+/751364/ lands you don't need all the inline changes
10:53:30 lyarwood sean-k-mooney: no not yet
10:54:09 sean-k-mooney true thoug i was hoping they but they also dont hurt
10:54:27 lyarwood yeah I guess everything is inlined so there's no conflicts
13:32:28 openstackgerrit Lucas Alvares Gomes proposed openstack/nova master: [OVN] Explicitly set grenade job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776934
13:33:24 openstackgerrit Lucas Alvares Gomes proposed openstack/nova master: [OVN] Explicitly set nova-next job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776944
13:34:19 openstackgerrit Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/c/openstack/nova/+/776945
15:50:15 gibi nova weekly meeting starts in 10 minutes in #openstack-meeting-3
15:56:03 gmann gibi: did you update the ptg etherpad for nova+keystone sessions timing? https://etherpad.opendev.org/p/nova-xena-ptg
15:58:28 gibi gmann: I did but I guess I was disconnected and it wasnt saved I fixed it now
15:58:59 gmann gibi: i see now, thanks
15:59:09 gibi thanks for the headsup
20:04:13 openstackgerrit Merged openstack/nova master: docs: Change 'add_stylesheet' to 'add_css_file' https://review.opendev.org/c/openstack/nova/+/781529
20:04:43 openstackgerrit Merged openstack/nova master: docs: Add CPU models guide https://review.opendev.org/c/openstack/nova/+/781438
22:04:45 openstackgerrit Merged openstack/nova master: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/781138
22:33:32 openstackgerrit Ghanshyam proposed openstack/nova master: Switch to new rolevar for run-temepst role https://review.opendev.org/c/openstack/nova/+/775040
#openstack-nova - 2021-04-09
01:16:14 openstackgerrit Wenping Song proposed openstack/nova-specs master: Repropose vGPU support spec https://review.opendev.org/c/openstack/nova-specs/+/780452
01:32:18 openstackgerrit Wenping Song proposed openstack/nova-specs master: Repropose vGPU support spec https://review.opendev.org/c/openstack/nova-specs/+/780452
01:55:21 openstackgerrit Merged openstack/nova master: libvirt: Simplify device_path check in _detach_encryptor https://review.opendev.org/c/openstack/nova/+/778463
01:55:46 openstackgerrit Merged openstack/nova master: trivial: fix word duplication in api ref https://review.opendev.org/c/openstack/nova/+/782028
07:03:09 openstackgerrit Qiu Fossen proposed openstack/nova-specs master: Repropose copying pmem devices data spec https://review.opendev.org/c/openstack/nova-specs/+/785563
08:18:53 openstackgerrit Xinran WANG proposed openstack/nova-specs master: Repropose smartnic support spec https://review.opendev.org/c/openstack/nova-specs/+/783632
08:21:04 xinranwang gibi: Hi gibi, I have reproposed the smartnic spec, please review it when you got time. https://review.opendev.org/c/openstack/nova-specs/+/783632
08:49:26 openstackgerrit Qiu Fossen proposed openstack/nova-specs master: Repropose copying pmem devices data spec https://review.opendev.org/c/openstack/nova-specs/+/785563
09:04:22 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/778885
09:04:27 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Remove nova-dsvm-multinode-base https://review.opendev.org/c/openstack/nova/+/778908
09:06:22 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/785576
09:06:57 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: libvirt: Simplify device_path check in _detach_encryptor https://review.opendev.org/c/openstack/nova/+/785577
09:07:35 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/785579
09:16:36 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/785582
09:17:26 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/785583
09:18:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Simplify device_path check in _detach_encryptor https://review.opendev.org/c/openstack/nova/+/785584
09:19:14 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Simplify device_path check in _detach_encryptor https://review.opendev.org/c/openstack/nova/+/785585
09:32:18 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Remove native LUKS compat code https://review.opendev.org/c/openstack/nova/+/766210
09:32:19 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765771
10:56:01 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Add evacuation migration record assertions to run-evacuate-hook https://review.opendev.org/c/openstack/nova/+/785618
11:32:53 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: Rebase qcow2 images when unshelving an instance https://review.opendev.org/c/openstack/nova/+/785625
11:39:52 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: Rebase qcow2 images when unshelving an instance https://review.opendev.org/c/openstack/nova/+/785627
12:36:31 lyarwood / j #tripleo
12:36:33 lyarwood argh
14:29:10 openstackgerrit Lee Yarwood proposed openstack/nova master: glance: Improve [glance]enable_rbd_download and associated options help text https://review.opendev.org/c/openstack/nova/+/785656
14:33:43 openstackgerrit sean mooney proposed openstack/nova master: Remove RBD swap disk if resizing to a flavor without swap https://review.opendev.org/c/openstack/nova/+/534711
15:08:22 openstackgerrit Merged openstack/nova master: [OVN] Adapt the live-migration job scripts to work with OVN https://review.opendev.org/c/openstack/nova/+/776419
15:13:15 lyarwood gibi: https://bugs.launchpad.net/nova/+bug/1923206 - FYI this has come up downstream and upstream in TripleO
15:13:18 openstack Launchpad bug 1923206 in OpenStack Compute (nova) "libvirt.libvirtError: internal error: unable to execute QEMU command 'device_del': Device $device is already in the process of unplug" [Undecided,New] - Assigned to Lee Yarwood (lyarwood)
15:13:31 lyarwood gibi: I'm looking at fixing it now ahead of your rework series landing in Xena
15:13:50 lyarwood gibi: it's just awkward given the blind retry logic that we had previously
15:14:48 gibi lyarwood: ack, sorry that I had no time to go back to the bugfix
15:16:36 lyarwood gibi: np I'm going to work on this and then I can try to push that along again early in Xena
15:16:44 lyarwood gibi: as once again it would likely avoid this situation
15:18:58 sean-k-mooney lyarwood: did you make the parmater tuneable?
15:19:14 sean-k-mooney lyarwood: could they just increase the retry interval
15:19:27 sean-k-mooney to say 30 seconds
15:20:37 lyarwood sean-k-mooney: with this it's the initial interval so no I don't think we can, it's hardcoded in the decorator iirc
15:21:19 sean-k-mooney ok i tought you added workaround config options for this when we increased them
15:21:29 sean-k-mooney but looks like we did not
15:24:19 lyarwood we tuned the existing ones up but that only covers the time after the initial wait
15:40:43 openstackgerrit Balazs Gibizer proposed openstack/nova-specs master: QoS minimum guaranteed packet rate https://review.opendev.org/c/openstack/nova-specs/+/785014
15:48:56 openstackgerrit Nicolas Parquet proposed openstack/nova-specs master: Adds spec for supporting @ and . in keypair name https://review.opendev.org/c/openstack/nova-specs/+/785674
16:06:32 openstackgerrit Ghanshyam proposed openstack/nova master: Switch to new rolevar for run-tempest role https://review.opendev.org/c/openstack/nova/+/775040
16:47:51 gmann melwitt: stephenfin ^^ fixed the cmt msg.
17:14:06 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/785682
17:18:06 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/785682
17:22:18 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/785682
17:22:21 lyarwood gah tgif++
17:22:58 lyarwood melwitt: ^ if you have any review bandwidth today but it can also wait until next week. More detach device fun with the old retry logic. I'll try to move gibi's rework forward next week ahead of ptg.
17:23:29 melwitt lyarwood: oooh my favorite
17:24:51 lyarwood melwitt: that's exactly what I said :|
17:26:27 melwitt (by that I mean that I will review :) )
17:27:48 lyarwood awesome thanks, I've just respun the TripleO change where I first saw it to include this nova change here https://review.opendev.org/c/openstack/tripleo-heat-templates/+/785027
17:28:48 lyarwood and with that I think I'm done for the week, have a great weekend all \o
17:30:17 lyarwood melwitt: https://review.opendev.org/c/openstack/nova/+/784130 - oh sorry before I go, if you have time this is also ready :)
17:31:06 melwitt lyarwood: will do, thanks for the extra info on the detach patch. have a good weekend o/
17:31:18 melwitt *and thanks
18:23:01 openstackgerrit Merged openstack/python-novaclient master: When creating a client, pass the default logger https://review.opendev.org/c/openstack/python-novaclient/+/783010
20:20:24 openstackgerrit Rodrigo Barbieri proposed openstack/nova master: Error anti-affinity violation on live migrations https://review.opendev.org/c/openstack/nova/+/784166
20:44:00 openstackgerrit Elod Illes proposed openstack/nova stable/train: DNM: nova-grenade-multinode test https://review.opendev.org/c/openstack/nova/+/785731
#openstack-nova - 2021-04-10
03:11:59 openstackgerrit melanie witt proposed openstack/python-novaclient master: Add unit tests for client logger https://review.opendev.org/c/openstack/python-novaclient/+/785741
03:31:55 openstackgerrit Merged openstack/python-novaclient stable/wallaby: Use well named anchor into the microversion history https://review.opendev.org/c/openstack/python-novaclient/+/781947
06:57:49 openstackgerrit Wenping Song proposed openstack/nova-specs master: Repropose vGPU support spec https://review.opendev.org/c/openstack/nova-specs/+/780452
07:01:40 openstackgerrit Merged openstack/nova stable/pike: Add resource_class to fields in ironic node cache https://review.opendev.org/c/openstack/nova/+/755653
07:02:07 openstackgerrit wangzhh proposed openstack/nova master: image_info: make RLIMIT configurable https://review.opendev.org/c/openstack/nova/+/785082
07:02:16 openstackgerrit Merged openstack/nova stable/pike: Include only required fields in ironic node cache https://review.opendev.org/c/openstack/nova/+/754444
07:02:42 openstackgerrit Merged openstack/nova stable/pike: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/c/openstack/nova/+/723055
07:05:12 openstackgerrit wangzhh proposed openstack/nova master: image_info: make RLIMIT configurable https://review.opendev.org/c/openstack/nova/+/785082
#openstack-nova - 2021-04-11
07:49:12 openstackgerrit Takashi Natsume proposed openstack/nova master: Update contributor guide for Xena https://review.opendev.org/c/openstack/nova/+/783534
10:23:22 openstackgerrit Merged openstack/nova master: nova-next: Drop NOVA_USE_SERVICE_TOKEN as it is now True by default https://review.opendev.org/c/openstack/nova/+/775575
15:35:32 openstackgerrit Merged openstack/nova stable/ussuri: Raise InstanceMappingNotFound if StaleDataError is encountered https://review.opendev.org/c/openstack/nova/+/777253

Earlier   Later