| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-24 | |||
| 13:36:38 | bauzas | | resource_class | allocation_ratio | min_unit | max_unit | reserved | step_size | total | used | | |
| 13:36:38 | bauzas | [root@hab-19 devstack]# openstack resource provider inventory list 7853e09b-5f5c-4b79-99c6-a550c56eb7e0 | |
| 13:36:39 | bauzas | +----------------+------------------+----------+----------+----------+-----------+-------+------+ | |
| 13:36:41 | bauzas | dammit | |
| 13:36:50 | gibi | sean-k-mooney: thanks for the heads up, enjoy your time off | |
| 13:37:03 | bauzas | so instances could pick those resources | |
| 13:37:14 | sean-k-mooney | bauzas: ya the inventories would because when you remove the confi the virt driver will just not update the rp or its inventories again | |
| 13:37:29 | bauzas | yup | |
| 13:37:30 | sean-k-mooney | this is also a problem for provider.yaml by the way | |
| 13:37:49 | sean-k-mooney | if we remove something form provider.yaml it will never get removed form placment | |
| 13:38:12 | bauzas | definitely worth a PTG discussion then | |
| 13:38:23 | sean-k-mooney | bauzas: we could perhaps write a file to disk in the nova state dir | |
| 13:38:25 | bauzas | sean-k-mooney: fancy adding a 10th bullet point or want me to write it ? | |
| 13:38:45 | bauzas | I mean, I couldf | |
| 13:38:54 | sean-k-mooney | that way the agent could read it on start up and use that to figure out if something was removed | |
| 13:39:02 | sean-k-mooney | if we did not want to store it in the db | |
| 13:39:26 | sean-k-mooney | bauzas: am i can i guess i have another to add anyway | |
| 13:39:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810909 | |
| 13:39:48 | sean-k-mooney | just assume the stuff i add is an aggreate of stuff we have mentioned in downstream or upstream meetings/discussions | |
| 13:42:56 | gibi | bauzas, sean-k-mooney: I will check what today's neutron does if I remove the bw config. I guess we have the same missing cleanup problem in neutron too. If we have, then I would argue for a solution that is not nova specific or at least easy to apply to neutron too | |
| 13:44:38 | bauzas | gibi: that's why a audit tool seems important | |
| 13:44:59 | bauzas | but the working logic is owned by neutron or the virt driver | |
| 13:45:14 | bauzas | so I don't know how a tool could be prescriptive | |
| 13:45:28 | gibi | bauzas: RP cleanup is OK to be left for the nova-manage but inventory cleanup is needed to be automatic to avoid wrong scheduling | |
| 13:45:42 | bauzas | agreed | |
| 13:46:03 | bauzas | and RP cleanup would be easily manageable in the audit command | |
| 13:46:15 | bauzas | if inventories are left empty | |
| 13:47:35 | gibi | yepp | |
| 13:47:48 | gibi | no inventory is a good indication to a RP to be deleted | |
| 13:48:00 | gibi | there are exceptions though :D | |
| 13:48:19 | gibi | a neturn sriov agent without any PF configured | |
| 13:48:27 | gibi | creates an agent RP and no PF RPs | |
| 13:48:33 | gibi | but the agent RP has no inventory | |
| 13:48:41 | gibi | so the audit tool would delete it | |
| 13:48:46 | gibi | then neutron would re-create it :D | |
| 13:48:54 | gibi | there is no harm, just noise | |
| 13:49:21 | gibi | and having an sriov agent without any PF is a pretty useless config :D | |
| 13:50:24 | gibi | btw if we have a proper logic to null out inventory on an RP then the agent nulling out the last inventory could delete the RP automatically too | |
| 13:50:57 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Reproduce bug 1944759 https://review.opendev.org/c/openstack/nova/+/810910 | |
| 13:50:58 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810911 | |
| 13:53:31 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Reproduce bug 1944759 https://review.opendev.org/c/openstack/nova/+/810912 | |
| 13:53:53 | gibi | one less escalation ^^ | |
| 13:54:27 | gibi | I'm pushing this back to victoria without delay as I need this upstream on victoria | |
| 13:54:48 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810913 | |
| 13:56:06 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Reproduce bug 1944759 https://review.opendev.org/c/openstack/nova/+/810914 | |
| 13:56:07 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810915 | |
| 14:00:41 | sean-k-mooney | i added two more itmes to the ptg adgenda | |
| 14:01:15 | sean-k-mooney | lines 182 for the inventory cleanup https://etherpad.opendev.org/p/nova-yoga-ptg | |
| 14:01:46 | gibi | sean-k-mooney: thanks! | |
| 14:02:05 | sean-k-mooney | can you update that with what you find for neturon | |
| 14:02:51 | gibi | sure | |
| 14:02:55 | gibi | I will do that today | |
| 14:03:44 | sean-k-mooney | no rush we have time before the ptg | |
| 14:04:07 | sean-k-mooney | bauzas: speaking of which i assuem you are going to take the etherpad and turn it into an adgenda at some point | |
| 14:04:10 | gibi | sean-k-mooney: I just closed an escalation I have a lot of motivation :D | |
| 14:05:09 | sean-k-mooney | thats alwasy a nice feeling espically on a friday | |
| 14:05:55 | sean-k-mooney | gibi: by the way i have only been skimming what happing with setuptool. are we any closer to a decision on what to do for the stable branches | |
| 14:07:05 | bauzas | sean-k-mooney: yup will work on it | |
| 14:07:11 | bauzas | (the agenda) | |
| 14:07:37 | gibi | sean-k-mooney: I have a solution based on you idea of [tox]requires config that proved to be working. It seems others are more for dropping lower constraints testing althogheter which is fine by me. | |
| 14:08:20 | gibi | sean-k-mooney: however a recent mail might means that we have the same issue outside of lower constraint testing http://lists.openstack.org/pipermail/openstack-discuss/2021-September/025040.html | |
| 14:08:27 | sean-k-mooney | ok and are we going to look into the project.yaml or what ever the new thing is for yoga on? | |
| 14:08:49 | gibi | pyproject.yaml is the thing but honestly I has not time to look into that as a direction | |
| 14:09:02 | gibi | I assume it also needs some common agreement in OpenStack | |
| 14:09:10 | sean-k-mooney | ya we proably should not be using any deps that rely on use_2to3 realistically in ussuri | |
| 14:09:22 | sean-k-mooney | gibi: yep we would | |
| 14:09:31 | sean-k-mooney | it would fundemtally change how we do dep managment | |
| 14:09:52 | sean-k-mooney | its a replacment for requirements.txt and tox.ini all wrapped into one file | |
| 14:10:07 | sean-k-mooney | i think it can still use the other but its basically one file to rule them all | |
| 14:12:54 | gibi | OK then it is definitely bigger that I can chew right now | |
| 14:13:09 | sean-k-mooney | yep it is | |
| 14:15:26 | gibi | and it is Friday :) | |
| 14:17:11 | priteau | Hello. Who has rights to update the Nova bug template on LP? I noticed an issue with the rpm command. | |
| 14:17:24 | priteau | See https://wiki.openstack.org/wiki/Nova/BugsTeam/BugReportTemplate | |
| 14:17:38 | priteau | `rpm -ql | grep <projectname>` should be `rpm -qa | grep <projectname>` | |
| 14:18:45 | gibi | priteau: I think anybody can edit the wiki I'm not sure about LP | |
| 14:19:15 | priteau | I can make the wiki edit of course, but what matters is syncing with LP ;-) | |
| 14:19:53 | gibi | priteau: I found I can update the LP | |
| 14:20:10 | gibi | let me know what needs to be fixed and I will do it in LP | |
| 14:20:21 | gibi | ahh I see what is needed | |
| 14:20:35 | priteau | I've updated the wiki page | |
| 14:21:59 | gibi | hm, do we need both -a and -l ? | |
| 14:23:07 | gibi | is -l a shortcut for --last ? | |
| 14:23:11 | gibi | (I'm on debian :D)_ | |
| 14:23:16 | sean-k-mooney | we proably should jsut remove it from the template | |
| 14:23:44 | sean-k-mooney | i think for deb packages -l is for list | |
| 14:23:47 | sean-k-mooney | i can check | |
| 14:24:00 | priteau | Just -qa | |
| 14:24:07 | priteau | -ql is for listing files inside a package | |
| 14:24:15 | priteau | List files in package. | |
| 14:24:15 | priteau | -l, --list | |
| 14:24:29 | gibi | ohh OK then -a it is | |
| 14:24:49 | gibi | fixed the LP | |
| 14:24:56 | gibi | priteau: thanks for reporting it | |
| 14:24:58 | sean-k-mooney | yes -qa | |
| 14:25:16 | sean-k-mooney | but really we dont care about the package version most of the time we just care about the openstack version | |
| 14:25:59 | priteau | This is an easy way to find the version when using binary packages | |
| 14:26:01 | sean-k-mooney | we care for libvirt and qemu sometimes but we porably could make it more generic | |
| 14:26:16 | sean-k-mooney | priteau: right but upstream that normally not useful | |
| 14:26:29 | sean-k-mooney | since we cant easially map it to the source code | |