Earlier  
Posted Nick Remark
#openstack-nova - 2020-11-30
10:57:13 stephenfin lyarwood: Could you hit https://review.opendev.org/c/openstack/nova/+/749315/ https://review.opendev.org/c/openstack/nova/+/749312/ and https://review.opendev.org/c/openstack/nova/+/759114/ (remove XenAPI final bits)?
11:03:30 viks____ kashyap: ok.. thanks a lot.. will check with fio once
11:04:55 lyarwood stephenfin: ack sure
11:29:46 lyarwood stephenfin: https://review.opendev.org/c/openstack/nova/+/759114/ - As it isn't passing CI anyway can you remove `Remove` from the commit subject ;)
11:30:10 stephenfin um, yeah, fair /o\
11:31:55 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Neuter 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/c/openstack/nova/+/759114
11:31:59 stephenfin lyarwood: ^
11:32:43 lyarwood ta
11:34:45 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Drop models for removed services, features https://review.opendev.org/c/openstack/nova/+/698706
11:41:55 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Remove KeyPair model from main DB https://review.opendev.org/c/openstack/nova/+/763989
11:55:44 stephenfin gibi: I corrected the docstring as requested at https://review.opendev.org/c/openstack/nova/+/764376/ fwiw
12:00:18 priteau Hello. I have an affinity server group, with a single instance in it. The instance is shut off. I can't cold migrate this instance, ServerGroupAffinityFilter is removing all hosts. Is this a known issue?
12:01:11 priteau I found https://bugs.launchpad.net/nova/+bug/1890065 but I am not sure it's the same problem, since it is about 2 instances being live migrated
12:01:11 openstack Launchpad bug 1890065 in OpenStack Compute (nova) "Impossible to migrate affinity instances" [Wishlist,Opinion]
12:58:07 gibi stephenfin: ack, looks good
12:59:20 gibi priteau: I think it is the general behavior is known, if you have instances in an affinity group then you cannot migrate them. (the power state does not matter either). The edge case when you have only one instance in the affinity group is not handled spcially. Do you have a use case where you need a single instance in an affinity group?
13:06:21 priteau gibi: I've encountered this while migrating every instance on an OpenStack cloud during a c7 to c8 migration, I don't know why the user set it up that way. I suppose one use case would be to a server group for all instances of a cluster, which could potentially scale down to 1.
13:09:09 gibi priteau: solving the migration problem generally is hard, as that needs a way to say ignore-the-affinity-rule-for-this-migration which is most probably means an API impact. The single instance case could be solved by always ignoring the affinity rule, so that is easier but I don't know that solving only the single instance case would really help much
13:09:52 priteau Agreed.
13:10:26 priteau On the advice of johnthetubaguy I temporarily changed the policy in the instance_group_policy table and reverted it after the migration.
13:42:33 openstackgerrit Merged openstack/nova master: doc: Fix rendering in the PTL guide https://review.opendev.org/c/openstack/nova/+/764175
13:56:38 gibi did anybody see pip error like "ERROR: Links are not allowed as constraints" I got hit by it recently in grenade https://39757d668f2e13d2ecb5-94a4d63114c603d5be2910eb980b9a2e.ssl.cf1.rackcdn.com/756894/3/check/grenade/fe26d2e/job-output.txt
14:02:31 gibi both grenade and grenade-multinode hits it
14:02:42 gibi this is from the same patch running the grenade-multinode https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_57e/756894/3/check/nova-grenade-multinode/57ea016/logs/grenade.sh.txt
14:12:27 gibi it seems it is started happening a lot in the last hour or so http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Links%20are%20not%20allowed%20as%20constraints%5C%22
14:14:08 jangutter gibi: I was just about to start sending up signal flares!
14:14:21 gibi jangutter: thanks
14:14:45 jangutter gibi: happened to me too, I was working on stable/victoria and at first I thought I did something wrong.
14:17:31 jangutter best I can figure out: https://github.com/openstack/devstack/blob/master/inc/python#L386 <--- devstack has been using this method for ages now. I wonder if a pip update converted the warning to an error.
14:20:45 gibi yeah, that sound like a possibility
14:24:06 jangutter Removing that line (and the preceding backslash) seems to unblock it for me, but figuring out the consequences of that needs someone with a lot more experience on requirements, pip and the projects needs.
14:37:13 gibi hm, I cannot push to opendev gerrit now
14:37:24 gibi fatal: Unpack error, check server log
14:37:24 gibi error: remote unpack failed: error Missing tree 466cd024745bbeffebf3f8e598759fcc9b75df4d
14:47:19 kashyap stephenfin: gibi: I'm still to comb through some review comments on Secure Boot spec before the spec-review day tomm. Excuse my delay.
14:48:18 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM https://review.opendev.org/c/openstack/nova/+/764766
14:52:16 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: Add regression test for bug #1899649 https://review.opendev.org/c/openstack/nova/+/764625
14:52:16 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: compute: Don't detach volumes when RescheduledException raised without retry https://review.opendev.org/c/openstack/nova/+/764626
14:52:17 openstack bug 1899649 in OpenStack Compute (nova) "Volume marked as available after a failure to build" [Undecided,In progress] https://launchpad.net/bugs/1899649 - Assigned to Lee Yarwood (lyarwood)
14:52:40 gibi kashyap: no worries, Milestone 1 is this week but we only freeze specs at M2
14:53:44 kashyap gibi: Ah, thank you. I didn't realize that spec-freeze is at M2.
14:54:40 gibi kashyap: M2 is spec freze M3 is feature freeze
14:55:06 gibi I guess in the past there was a time when we freezed specs at M1 but we don't do that in the last couple of cycles
14:55:20 kashyap I see
15:00:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow extending an existing instance allocation https://review.opendev.org/c/openstack/nova/+/757109
15:00:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make remove allocation symmetric with add allocation https://review.opendev.org/c/openstack/nova/+/757110
15:12:42 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM https://review.opendev.org/c/openstack/nova/+/764771
15:14:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM https://review.opendev.org/c/openstack/nova/+/764772
15:16:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM https://review.opendev.org/c/openstack/nova/+/764773
15:17:30 gibi (sorry for the noise)
15:18:18 jangutter gibi: still checking the pip thing?
15:18:32 gibi jangutter: nope, I got hit by a gerrit issue
15:36:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out port resource deallocation https://review.opendev.org/c/openstack/nova/+/757111
15:42:39 stephenfin lyarwood: Don't know if I linked these already, but those changes to improve logging for PCI requests are here https://review.opendev.org/q/topic:%2522bug/1852727%2522+status:open
15:42:50 stephenfin If you care to cast an eye over them :)
15:43:31 lyarwood ack can do
15:43:35 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-resume job https://review.opendev.org/c/openstack/nova/+/764777
15:46:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor update_pci_request_spec_with_allocated_interface_name https://review.opendev.org/c/openstack/nova/+/757614
15:58:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/c/openstack/nova/+/756530
15:58:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove unused Instance method https://review.opendev.org/c/openstack/nova/+/757615
16:19:12 gibi fyi: grenade seems to be broken due to fresh pip 20.3 release http://lists.openstack.org/pipermail/openstack-discuss/2020-November/019128.html
16:19:52 lyarwood oh joy I was wondering what that was going to break today :)
16:20:59 spatel sean-k-mooney: Hey! i have replied to your email regarding my DPDK performance issue.
16:39:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: stabilize unit test asserting log https://review.opendev.org/c/openstack/nova/+/764796
16:39:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: stabilize set_host libvirt driver unit test https://review.opendev.org/c/openstack/nova/+/764797
16:39:47 gibi stephenfin: small unit test stability fixes ^^ after these two I run 6 hours of repeated random unit tests without any instability
16:53:04 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-resume job https://review.opendev.org/c/openstack/nova/+/764777
16:57:45 stephenfin gibi: ack, looking
17:22:10 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-resume job https://review.opendev.org/c/openstack/nova/+/764777
17:40:41 openstackgerrit Clark Boylan proposed openstack/nova master: DO NOT MERGE https://review.opendev.org/c/openstack/nova/+/764805
17:51:59 openstackgerrit Stephen Finucane proposed openstack/nova master: rpc: Rework 'get_notifier', 'wrap_exception' https://review.opendev.org/c/openstack/nova/+/741663
17:51:59 openstackgerrit Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.opendev.org/c/openstack/nova/+/676209
17:52:00 openstackgerrit Stephen Finucane proposed openstack/nova master: mypy: Add nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657
17:52:01 openstackgerrit Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to top-level modules https://review.opendev.org/c/openstack/nova/+/705658
17:52:01 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: Expand type hints for nova.block_device https://review.opendev.org/c/openstack/nova/+/742170
17:52:01 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Clean manager.Manager, service.Service signatures https://review.opendev.org/c/openstack/nova/+/764806
#openstack-nova - 2020-12-01
01:12:41 Jorhson lyarwood: Hi, I have commit the nova-spec for "758920: Add instance_state to ensure volume attachment successful | https://review.opendev.org/c/openstack/nova-specs/+/758920", please help to review, thanks
05:12:19 openstackgerrit Hemanth N proposed openstack/nova stable/ussuri: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761701
06:24:49 openstackgerrit Adit Sarfaty proposed openstack/nova master: Retry on vmware create_vm when it fails https://review.opendev.org/c/openstack/nova/+/764586
06:25:56 openstackgerrit Adit Sarfaty proposed openstack/nova stable/train: Retry on vmware create_vm when it fails https://review.opendev.org/c/openstack/nova/+/764515
08:42:31 bauzas good morning Nova
09:41:36 openstackgerrit Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336
09:56:51 lyarwood morning all and happy(?!) spec review day
10:04:55 lyarwood bauzas / gibi / melwitt ; https://review.opendev.org/c/openstack/nova-specs/+/752284 could use another nova-specs core review today if anyone has time
10:05:12 bauzas fwiw, I already reviewed a few specs
10:06:51 lyarwood kashyap: https://review.opendev.org/c/openstack/nova-specs/+/631154 - can you close this out btw?
10:07:39 kashyap lyarwood: Yes, I'm finishing up writing something; will get to it in the afternoon
10:08:47 lyarwood kashyap: wait we don't need your q35 spec anymore however right?
10:09:07 kashyap lyarwood: Nope; sorry, I was talking about secure boot spec
10:09:12 kashyap Poor wording here
10:09:14 lyarwood kashyap: ah kk np
10:23:51 openstackgerrit Lee Yarwood proposed openstack/nova-specs master: libvirt: Store and allow the default machine type to be changed https://review.opendev.org/c/openstack/nova-specs/+/762199
10:33:11 kashyap lyarwood: Closed out my no-longer-required spec
10:49:26 openstackgerrit Aleksey Ulyanov proposed openstack/nova master: Fix graceful shutdown timeout https://review.opendev.org/c/openstack/nova/+/764927
10:52:38 kashyap [OT, but people here might appreciate it] Meanwhile, this goes live: https://www.qemu-advent-calendar.org/2020/
11:04:57 stephenfin bauzas: I'm suggesting doing this change as part of the new microversion, btw https://review.opendev.org/c/openstack/nova-specs/+/763043
11:05:16 bauzas stephenfin: ack, then all good
11:06:02 bauzas sent to the gate
11:20:59 openstackgerrit Merged openstack/nova-specs master: Update modernize-os-hypervisors-api spec https://review.opendev.org/c/openstack/nova-specs/+/763043
11:37:24 gibi lyarwood: left questions in https://review.opendev.org/c/openstack/nova-specs/+/752284 for you

Earlier   Later