Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-23
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: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)
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
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>
12:00:28 sean-k-mooney so both are fine lets leave it as is
12:00:35 sean-k-mooney ill comment the same in the bug
12:00:39 sean-k-mooney *patch
12:02:11 gibi thanks
12:04:50 sean-k-mooney done
12:06:01 sean-k-mooney un related to that change but when we use https://www.openstack.org/xmlns/libvirt/nova/1.1 as a namespace url that is actully ment to be a vaild path to the xml schema
12:06:32 sean-k-mooney i dont know if we care to fix that at some point since we dont actully have a schema for the metadta xml elemnet just said i would note that in passing
12:11:44 gibi good point maybe we can fix that next cycle when we extend the schema further
12:13:07 sean-k-mooney ya not sure how we would fix it unless we put it in the docs and linked to the doc site or to the git repo on opendev.org

Earlier   Later