Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-12-11
17:27:03 sean-k-mooney https://github.com/openstack/nova/commit/8e19ef4173906da0b7c761da4de0728a2fd71e24
17:27:06 sean-k-mooney sorry ^
17:27:10 stephenfin but it doesn't prevent us removing a host from an aggregate (and therefore the AZ)
17:29:09 sean-k-mooney we do it for add https://github.com/openstack/nova/blob/048250a4b76c0b6a85e459f92b436c448d592235/nova/compute/api.py#L6164
17:29:31 sean-k-mooney stephenfin: we dont do it for remove
17:30:06 sean-k-mooney stephenfin: we should here ish https://github.com/openstack/nova/blob/048250a4b76c0b6a85e459f92b436c448d592235/nova/compute/api.py#L6211
17:31:33 sean-k-mooney i guess line 6217 actully to be consitent
17:31:40 sean-k-mooney and do it after the notificaion
17:32:48 sean-k-mooney stephenfin: so that bug is half valid. what they are doing is not supported but we are not blocking the removal
17:32:56 sean-k-mooney it should be 1 line + tests however
17:32:59 stephenfin agreed
17:33:04 stephenfin I'm just creating a test now
17:33:06 stephenfin Easy win
17:33:27 sean-k-mooney i wonder why we did add but not remove in the first place
17:33:44 stephenfin not sure. It's very strange
17:33:56 stephenfin surprised no one has noticed before either
17:35:30 sean-k-mooney people dont do this often and we have a giant warning in the docs
17:36:54 stephenfin we do?
17:37:05 stephenfin about what? messing with AZs on hosts with instances?
17:37:49 sean-k-mooney ya i think spcificaly about moving host
17:39:55 sean-k-mooney https://docs.openstack.org/nova/latest/admin/availability-zones.html#implications-for-moving-servers
17:40:10 sean-k-mooney its got its own section
17:40:39 sean-k-mooney actully that not quite what i was thinking of
17:42:41 sean-k-mooney that partly covers it but i though we had addtion info specific to move compute nodes
17:43:35 sean-k-mooney hum i guess we dont
17:50:12 stephenfin sean-k-mooney: should you be allowed to add a host to an aggregate when it has instances on it?
17:50:58 sean-k-mooney no because if its not in an az it would be in an implcit nova az right
17:51:06 stephenfin good point
17:51:07 sean-k-mooney so that would result itn the instance moving az
17:51:27 stephenfin I was going to say it won't be in an AZ, but it will. The implicit one, as you note
17:51:31 sean-k-mooney it might be ok provide they dont have an explict nova request
17:51:46 stephenfin we tell people not to do that, iirc
17:52:16 sean-k-mooney yep here https://docs.openstack.org/nova/latest/admin/availability-zones.html
17:52:21 sean-k-mooney the warning at the top
17:52:39 sean-k-mooney but thats the edgecase you are hitting if you allow it
18:00:51 JamesBenson hey sean-k-mooney, one follow up question. Is there a setting (or benefit) to passing all of the CPU flags in the nova.conf in that mixed CPU environment we chatted about yesterday?
18:02:57 sean-k-mooney in your case i guess you could
18:03:24 sean-k-mooney since you are enableing flags not on the older host on your newer host you wont be able to migrate form westmer to nehelem anyway
18:03:37 sean-k-mooney you can move the other way
18:03:42 sean-k-mooney nehelem to westmer
18:03:58 sean-k-mooney so you could just use hostmodel or host passthough
18:04:10 sean-k-mooney normally you dont want ot do that if you have a mixed env
18:04:56 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add reproducer for #1907775 https://review.opendev.org/c/openstack/nova/+/766771
18:06:29 stephenfin sean-k-mooney: ^
18:09:11 sean-k-mooney hehe you had to reformat things :P
18:09:56 sean-k-mooney so does that actuly work for both add and remove today
18:10:02 sean-k-mooney i was expect add to be blocked
18:12:27 sean-k-mooney stephenfin: hum they all pass so ya i guess its not blocked on add currently
20:02:20 openstackgerrit Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers https://review.opendev.org/c/openstack/placement/+/760240
20:06:56 openstackgerrit Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers https://review.opendev.org/c/openstack/placement/+/760240
20:06:57 openstackgerrit Lance Bragstad proposed openstack/placement master: Pass context objects to oslo.policy directly https://review.opendev.org/c/openstack/placement/+/766785
20:44:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1907522 in functional test https://review.opendev.org/c/openstack/nova/+/766791
20:44:40 openstack bug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host " [High,Triaged] https://launchpad.net/bugs/1907522 - Assigned to Balazs Gibizer (balazs-gibizer)
21:38:22 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Add 'socket' to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
21:38:23 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
22:42:53 openstackgerrit Merged openstack/nova master: nova-evacuate: Remove leftover playbook from standalone job https://review.opendev.org/c/openstack/nova/+/764618
23:51:48 openstackgerrit Merged openstack/nova master: nova-grenade-multinode: Skip test_live_block_migration_paused https://review.opendev.org/c/openstack/nova/+/766720
#openstack-nova - 2020-12-12
00:33:27 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Omit resource inventories from placement update if zero https://review.opendev.org/c/openstack/nova/+/766825
04:18:57 brinzhang_ gibi:ack
05:27:37 openstackgerrit Wenping Song proposed openstack/nova master: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/759577
06:05:48 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API https://review.opendev.org/c/openstack/nova/+/766380
08:52:10 gibi if there are stable core around today then https://review.opendev.org/c/openstack/nova/+/765749 needs some low
08:52:13 gibi love
10:27:37 zer0def hey there, would there be any reason for a non-monkey-patched eventlet-based nova-conductor to receive RPC messages sent from nova-metadata, but not act on them? the database backend is on psycopg2 and I've disabled monkey patching, because it raised "greenlet.error: cannot switch to a different thread"
10:32:16 zer0def oh yeah, and the reason I'm running eventlet-based nova components is because they're spawned from openstack-helm without certificates
10:33:51 zer0def version is victoria
22:29:25 openstackgerrit Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
22:35:58 openstackgerrit Ghanshyam proposed openstack/nova master: Reuse code from oslo lib for JSON policy migration https://review.opendev.org/c/openstack/nova/+/763487
#openstack-nova - 2020-12-13
03:09:32 openstackgerrit Merged openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4 https://review.opendev.org/c/openstack/nova/+/765749
04:18:35 prometheanfire stephenfin: mind looking at https://review.opendev.org/c/openstack/nova/+/714676 for the mock fixes?
08:02:46 openstackgerrit Mamduh proposed openstack/os-vif stable/ussuri: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765419
08:07:42 openstackgerrit Mamduh proposed openstack/os-vif stable/ussuri: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765419
10:52:43 openstackgerrit Mamduh proposed openstack/os-vif stable/train: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765912
10:57:08 openstackgerrit Mamduh proposed openstack/os-vif stable/stein: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765914
11:25:19 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/c/openstack/nova/+/728094
11:26:07 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/c/openstack/nova/+/728109
11:27:11 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/c/openstack/nova/+/728117
11:28:11 openstackgerrit Takashi Natsume proposed openstack/nova master: Add a description in the PTL guide https://review.opendev.org/c/openstack/nova/+/765682
11:28:36 openstackgerrit Takashi Natsume proposed openstack/nova master: Fix a deprecation warning of 'collections' https://review.opendev.org/c/openstack/nova/+/765646
18:34:59 openstackgerrit Merged openstack/nova master: Fix a deprecation warning of 'collections' https://review.opendev.org/c/openstack/nova/+/765646
19:09:06 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Add 'socket' to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
19:09:06 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
19:09:07 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: db: Add `socket` to pci_devices https://review.opendev.org/c/openstack/nova/+/766862
19:11:01 openstackgerrit Merged openstack/nova master: Add a description in the PTL guide https://review.opendev.org/c/openstack/nova/+/765682
19:38:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: db: Add `socket` to pci_devices https://review.opendev.org/c/openstack/nova/+/766862
19:38:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: objects: Add `socket` to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
19:38:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
#openstack-nova - 2020-12-14
00:19:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: db: Add `socket` to pci_devices https://review.opendev.org/c/openstack/nova/+/766862
00:19:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: objects: Add `socket` to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
00:19:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
02:52:28 openstackgerrit Merged openstack/nova-specs master: Smart-nic Support Specification https://review.opendev.org/c/openstack/nova-specs/+/742785
05:02:36 openstackgerrit Hemanth N proposed openstack/nova stable/ussuri: functional: Clean up PCI tests https://review.opendev.org/c/openstack/nova/+/766879
08:01:55 gibi good morning
08:41:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1907522 in functional test https://review.opendev.org/c/openstack/nova/+/766791
08:41:25 openstack bug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host " [High,Triaged] https://launchpad.net/bugs/1907522 - Assigned to Balazs Gibizer (balazs-gibizer)
08:42:48 openstackgerrit Merged openstack/nova stable/victoria: Reproduce bug 1896463 in func env https://review.opendev.org/c/openstack/nova/+/763152
08:42:48 openstack bug 1896463 in OpenStack Compute (nova) victoria "evacuation failed: Port update failed : Unable to correlate PCI slot " [Low,In progress] https://launchpad.net/bugs/1896463 - Assigned to Balazs Gibizer (balazs-gibizer)
09:56:12 lyarwood gibi: morning, just sent a reply to fungi on the ML without CC'ing you in reply. He raised a number of valid points about our tox envs and requirements files, wanted to ask if you wanted them tracked in LP or somewhere else
10:05:43 gibi lyarwood: hi! ack, replying
10:13:50 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: [stable-only] lower-contraints: Bump various packages https://review.opendev.org/c/openstack/nova/+/766738

Earlier   Later