Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-23
19:38:36 openstackgerrit Merged openstack/nova master: Handle instance = None in _local_delete_cleanup https://review.opendev.org/c/openstack/nova/+/775308
19:39:41 openstackgerrit Merged openstack/nova master: Add net & utils methods for routed nets & segments https://review.opendev.org/c/openstack/nova/+/773976
19:40:58 openstackgerrit Merged openstack/nova master: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068
20:10:02 openstackgerrit melanie witt proposed openstack/nova stable/victoria: Add regression test for bug 1914777 https://review.opendev.org/c/openstack/nova/+/777121
20:10:03 openstack bug 1914777 in OpenStack Compute (nova) "Possible race condition between n-cpu and n-api when deleting a building instance" [High,In progress] https://launchpad.net/bugs/1914777 - Assigned to melanie witt (melwitt)
20:13:03 openstackgerrit melanie witt proposed openstack/nova stable/victoria: Handle instance = None in _local_delete_cleanup https://review.opendev.org/c/openstack/nova/+/777209
20:26:43 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Add regression test for bug 1893284 https://review.opendev.org/c/openstack/nova/+/777216
20:26:44 openstack bug 1893284 in OpenStack Compute (nova) ussuri "Per-user quotas not scoped correctly when creating a server" [Low,New] https://launchpad.net/bugs/1893284
20:26:45 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Default user_id when not specified in check_num_instances_quota https://review.opendev.org/c/openstack/nova/+/777217
20:26:45 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Add regression test for bug 1914777 https://review.opendev.org/c/openstack/nova/+/777218
20:26:48 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Handle instance = None in _local_delete_cleanup https://review.opendev.org/c/openstack/nova/+/777219
20:26:48 openstack bug 1914777 in OpenStack Compute (nova) victoria "Possible race condition between n-cpu and n-api when deleting a building instance" [High,In progress] https://launchpad.net/bugs/1914777 - Assigned to melanie witt (melwitt)
22:04:23 openstackgerrit Merged openstack/nova master: FUP: Catch and reraise routed nets exception https://review.opendev.org/c/openstack/nova/+/776694
22:41:38 openstackgerrit Merged openstack/nova master: Refactor and rename test_tcp_rst_no_compute_rpcapi https://review.opendev.org/c/openstack/nova/+/741351
22:44:36 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Raise InstanceMappingNotFound if StaleDataError is encountered https://review.opendev.org/c/openstack/nova/+/777253
22:45:57 openstackgerrit melanie witt proposed openstack/nova stable/train: Raise InstanceMappingNotFound if StaleDataError is encountered https://review.opendev.org/c/openstack/nova/+/777254
#openstack-nova - 2021-02-24
00:40:13 openstackgerrit Brin Zhang proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336
00:44:08 brinzhang gibi, stephenfin: as current implementation of noVNC that we can support open console with password, but not used x509+vnc, we would like to optimize it later, can we let this feature go?
00:46:49 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs https://review.opendev.org/c/openstack/nova/+/764292
00:47:19 openstackgerrit Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311
00:48:05 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
00:48:36 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726
00:49:26 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704
00:50:19 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs https://review.opendev.org/c/openstack/nova/+/768509
00:50:56 openstackgerrit Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315
00:51:41 openstackgerrit Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852
00:52:06 openstackgerrit Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in os-quota-sets path https://review.opendev.org/c/openstack/nova/+/768851
00:52:27 openstackgerrit Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in Limits API https://review.opendev.org/c/openstack/nova/+/768862
00:52:56 openstackgerrit Brin Zhang proposed openstack/nova master: Replace tenant* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329
04:18:44 prometheanfire sean-k-mooney: let me know if https://review.opendev.org/777295 is needed when / if you get a chance (for reference https://review.opendev.org/777170 is the test for 0.16)
08:39:59 jkulik found the answer to my question regarding sync_aggregates: ironic host are not supported with aggregates.
08:40:26 jkulik how does AZ assignment work there, if they don't support aggregates?
08:49:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enable mypy on scheduler/report.py https://review.opendev.org/c/openstack/nova/+/765152
09:00:11 openstackgerrit Balazs Gibizer proposed openstack/nova master: Follow up type hints for a634103 https://review.opendev.org/c/openstack/nova/+/777159
09:21:23 bauzas jkulik: the problem with aggregates is that only service hostnames are used
09:22:01 bauzas so, if you use the same nova-compute service for all the ironic nodes, then you'll have all the nova compute nodes having the same "host" name
09:22:13 bauzas but...
09:22:22 bauzas you can have placement aggregates
09:22:31 bauzas using compute node UUIDs
09:23:06 bauzas so, you could be able to provide flavors for each of them
09:23:14 bauzas but not AZs
09:25:29 jkulik bauzas: we have different racks of baremetal nodes assigned to different hostnames in nova. they are assigned to aggregates for the AZs in our case. that seemed to work
09:25:45 bauzas cool then
09:26:01 jkulik well yes and no ;)
09:26:25 jkulik we cannot sync those aggregates automatically to placement, because Nova doesn't support the case.
09:26:59 jkulik but it looks like that's just a matter of patching it (badly) to allow multiple resource providers/compute nodes for ironic
09:35:51 bauzas we discussed this a bit in the past but providing nova aggregates support for ironic wasn't pursued
09:36:10 bauzas as there are a ton of implications and some breaking changes
09:36:19 bauzas and then we moved to placement
09:36:47 bauzas the idea here is maybe to say we could use placement queries with some kind of prefilter that would be smarter
09:38:03 jkulik because the admin can target specific ironic nodes, I guess? makes sense.
09:38:14 jkulik thank you for helping me get a clearer picture on that
09:46:22 gibi lyarwood: I'm OK going with the python based solution in the machine_type series. We can optimize it later if we get reports about slowness
09:47:48 lyarwood gibi: ack, I'm going to respin the series later today, apologies for not getting to it just yet
09:48:01 lyarwood FWIW I'm about to admit defeat with the ephemeral encryption series
09:48:16 lyarwood there's just too much debt in the libvirt driver to land this in 2 weeks
09:48:39 gibi no problem, I also was slow responding
09:48:40 lyarwood and I've under estimated the required test coverage given the removal of some of that debt in this series
09:48:56 gibi lyarwood: ack, thanks for being upfront about that
10:14:56 stephenfin gibi: artom is on PTO so I'm going to address my own nits on https://review.opendev.org/c/openstack/nova/+/772748 to keep it moving
10:15:01 stephenfin is that okay?
10:15:23 stephenfin I can also fix up the next one since you agree with my comments
10:25:38 Fl1nt Hi there!
10:29:56 Fl1nt Quick question, let say I want to implement garbage collection/housekeeping and a mechanism to auto-balance my cluster when I had new nodes to a zone/cell/aggregate/whatever, is there already WIP or SIG working on this for nova? If not, would it be better to create a new project that work on nova/placement ressources or is it prefered to create it as a nova/placement feature?
10:34:38 gibi stephenfin: sure, it works for me
10:35:56 gibi Fl1nt: I'm not sure what is the status of https://wiki.openstack.org/wiki/Watcher but I think it has a scope you are looking for
10:37:14 Fl1nt gibi, perfect, if it already exist then I'll watch (no pun intended) it and contribut to it if it's not a living dead project ^^
10:37:28 gibi :)
10:49:12 gibi brinzhang: I need stephenfin and melwitt to agree on the vnc password patch
10:51:06 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'socket' PCI NUMA affinity https://review.opendev.org/c/openstack/nova/+/772748
10:51:07 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Add 'socket' PCI NUMA affinity policy trait https://review.opendev.org/c/openstack/nova/+/772749
10:51:08 openstackgerrit Stephen Finucane proposed openstack/nova master: Track host NUMA topology in PCI manager https://review.opendev.org/c/openstack/nova/+/774149
10:51:10 openstackgerrit Stephen Finucane proposed openstack/nova master: pci: implement the 'socket' NUMA affinity policy https://review.opendev.org/c/openstack/nova/+/772779
10:51:35 Fl1nt thanks gibi that's absolutely what I was looking for!
10:51:50 Fl1nt thanks and cheers!
10:52:28 gibi Fl1nt: I'm happy that I could help
10:53:04 belmoreira one line patch review: https://review.opendev.org/c/openstack/nova/+/776428 Let me know if this makes sense. Created a lot of confusion some time ago :)
10:54:35 gibi belmoreira: make sense to me
11:39:28 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Add tests for insecure UEFI configuration https://review.opendev.org/c/openstack/nova/+/777367
11:39:32 stephenfin bauzas: ^
11:39:43 bauzas ta
11:39:51 bauzas will look
11:54:52 gibi sean-k-mooney: could you take a look on https://review.opendev.org/c/openstack/nova/+/750552 we have +2s from cores, but you had issues with the ipVersion vs ip_version in the spec. The impl goes with ipVersion. Is it a dealbreakre for you?
11:55:16 sean-k-mooney no its not
11:55:23 sean-k-mooney i prefer ip_version
11:55:28 sean-k-mooney but both work
11:56:05 gibi thanks, then I will approve the impl with ipVersion as there was no real preference in the existing attribute names for either ways
11:56:09 sean-k-mooney we coudl fix it with a follow up to the spec or code to bring them in line
11:56:33 gibi as far as I understand there is no definit line
11:56:34 gibi https://review.opendev.org/c/openstack/nova/+/750552/10/nova/virt/libvirt/config.py#3459
11:56:56 sean-k-mooney i dont think we currently have any 2 word elemnts
11:57:02 sean-k-mooney at least not nova ones
11:57:30 gibi no, not nova ones
11:57:36 gibi but in general
11:57:50 sean-k-mooney well libvirt uses _
11:58:07 sean-k-mooney <on_reboot>restart</on_reboot>
11:58:37 sean-k-mooney but its a minor nit
11:59:08 sean-k-mooney not a deal breaker i would just prefer to use _ when i extend it with the flavor info next cycle since i never got to it this cycle
11:59:41 sean-k-mooney acutlly it uses both
12:00:13 sean-k-mooney <currentMemory unit='KiB'>1048576</currentMemory>

Earlier   Later