| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-08 | |||
| 12:47:23 | lyarwood | nightmare_unreal: we don't list the average transfer rate etc | |
| 12:47:27 | sean-k-mooney | ah we do have disk_remaining_bytes etctra in that responce good | |
| 12:47:33 | lyarwood | nightmare_unreal: iirc it is dumped in the logs | |
| 12:47:39 | nightmare_unreal | ah then i can calculate i think | |
| 12:47:44 | nightmare_unreal | disk remaining / total disk | |
| 12:48:00 | lyarwood | yup | |
| 12:48:07 | nightmare_unreal | thanks :D | |
| 12:48:09 | sean-k-mooney | am not entirly | |
| 12:48:37 | sean-k-mooney | the byts migrated can be larger then total disk | |
| 12:49:02 | lyarwood | right and remaining can also move around | |
| 12:49:12 | sean-k-mooney | disk_processed_bytes can be larger then disk_total_bytes if the vm is wriging to the disk while its being migrated | |
| 12:51:09 | sean-k-mooney | you can get a microsoft style progress bar where it sometiems gose backwared or time remaing goes up as the persentage completeion increases if your try to use disk_remaining_bytes/disk_total_bytes to caulate things | |
| 12:51:29 | sean-k-mooney | the data is not meaning less but unless the guest is idel it wont be linear | |
| 12:51:31 | lyarwood | lol | |
| 12:53:18 | nightmare_unreal | ahah I see | |
| 12:53:28 | nightmare_unreal | i will keep that in mind. | |
| 13:06:00 | sean-k-mooney | gibi: i have a pe8 issue in my port numa policies patch so ill fix the doc issue when i repin it shortly | |
| 13:06:28 | gibi | sean-k-mooney: ack | |
| 13:07:01 | sean-k-mooney | by the way did you want me to add a new doc for this or just fix the comment | |
| 13:07:15 | sean-k-mooney | i could add a doc too but did you have a specific place in mind? | |
| 13:07:33 | gibi | sean-k-mooney: just add a doc to the function about the item in the returned tuple | |
| 13:08:24 | sean-k-mooney | ah cool. i was wondering if you wanted a user facing doc. i could add one of those as a followup but porbly need to think where that should go. ill fix the comment thanks:) | |
| 13:29:41 | gibi | sean-k-mooney: regarding the user facing doc, I hope neturon has at least an API doc for this port parameter | |
| 13:29:43 | gibi | :) | |
| 13:30:21 | sean-k-mooney | i was thinking maybe we should add something to the networking guide | |
| 13:30:43 | gibi | yeah that would be good to have ^^ | |
| 13:30:44 | sean-k-mooney | they did not add the osc command but i submited a patch to do that | |
| 13:31:02 | gibi | the api ref is not missing but pretty short | |
| 13:31:04 | sean-k-mooney | so ill submit a follow up patch to the networkign guide to use that | |
| 13:31:12 | gibi | "The port NUMA affinity policy requested during the virtual machine scheduling. Values: None, requiered, preferred or legacy." | |
| 13:31:33 | sean-k-mooney | ya i can proably extend that too and explain what the policies do | |
| 13:31:49 | gibi | hm and there is a typo in it | |
| 13:31:53 | sean-k-mooney | we have the explaniaton in nova but it better to not need to cross reference | |
| 13:31:54 | gibi | requiered -> required | |
| 13:32:10 | sean-k-mooney | oh yes there is. | |
| 13:32:30 | gibi | also when artom's series lands with the socket value we need to document that for the port attribute too | |
| 13:32:30 | sean-k-mooney | only in the doc however | |
| 13:33:03 | sean-k-mooney | yep although that requires a neutron change to extend the db enum. its just a sting field but the model need to be extended | |
| 13:33:50 | gibi | I see | |
| 13:33:53 | gibi | good to know that | |
| 13:34:04 | gibi | I assumed it is an unrestricted string in neutron | |
| 13:34:12 | gibi | but the it is better | |
| 13:34:47 | sean-k-mooney | i think in terms fo the db coloum defiention yes but they are validating it in python | |
| 13:37:56 | sean-k-mooney | the only thing that is slightly unfortunete is they enabled each policy with a seperate flag in osc | |
| 13:38:21 | sean-k-mooney | rather then a vaule that takes a sting paramter but its not a huge issue | |
| 13:42:52 | admin0 | sean-k-mooney, i am using kvm .. first the flavor was created and the limit was set .. after that the instance was booted up | |
| 13:43:45 | sean-k-mooney | admin0: yes but you use a vaule that is only supproted for vmware | |
| 13:43:54 | sean-k-mooney | admin0: try the same with the one i pointed too | |
| 13:44:12 | admin0 | oh | |
| 13:44:15 | admin0 | i get it | |
| 13:44:16 | admin0 | thank you | |
| 13:44:18 | admin0 | i will retry | |
| 13:44:18 | sean-k-mooney | openstack flavor set g6 --property quota:disk_total_iops_sec=800 | |
| 13:45:29 | sean-k-mooney | that one and the others in the disk tuning section of https://docs.openstack.org/nova/latest/user/flavors.html work with libvirt | |
| 13:46:22 | openstackgerrit | Merged openstack/nova master: Replace md5 for fips https://review.opendev.org/c/openstack/nova/+/777686 | |
| 13:47:18 | openstackgerrit | Merged openstack/nova master: Centralize sqlite FK constraint enforcement https://review.opendev.org/c/openstack/nova/+/773833 | |
| 13:57:32 | openstackgerrit | sean mooney proposed openstack/nova master: support per port numa policies with sriov https://review.opendev.org/c/openstack/nova/+/773792 | |
| 13:58:20 | sean-k-mooney | alex_xu: stephenfin would either of ye have time to review ^ gibi made the doc update and fixed the pep8 issue. | |
| 13:59:39 | gibi | sean-k-mooney: thanks +2 | |
| 14:00:53 | sean-k-mooney | just lookign at your sqlite change that really is just refactoring it so that its cenratlised since we already had the dynamic logic. do we run with 3.7+ in the gate? | |
| 14:01:39 | sean-k-mooney | i assume we have no way to requrie a minium of 3.7 beyond moving to an os that provides it | |
| 14:01:42 | gibi | sean-k-mooney: I think you are referring to melwitt's change :) | |
| 14:01:52 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/773833 | |
| 14:01:57 | sean-k-mooney | the one that just merged | |
| 14:02:03 | gibi | yepp | |
| 14:02:05 | gibi | that one | |
| 14:02:39 | sean-k-mooney | oh did melwitt write that oh she did sorry still find part of the new gerrit confusing | |
| 14:02:53 | gibi | we only have https://github.com/openstack/requirements/blob/master/bindep.txt#L18 in bindep about sqlite | |
| 14:03:16 | gibi | so I guess it depends on the host os | |
| 14:03:28 | sean-k-mooney | ya which is fine i guess | |
| 14:04:00 | sean-k-mooney | aslong as it compatible with our version of sqlalchmy then it should not impact use much since its mainly for testing | |
| 14:04:01 | gibi | yeah, it is just test dep | |
| 14:04:30 | sean-k-mooney | ah yes i rememebr the downstream bug melwitt is fixing | |
| 14:05:10 | sean-k-mooney | the partal archiving of db rows was causing odd behavior for the simple tenant usage api on our internal cloud | |
| 14:06:23 | gibi | yes I just read the fix today for that | |
| 14:06:26 | sean-k-mooney | melwitt: were you going to bring up the idea of using cacade delelets in the ptg | |
| 14:07:50 | sean-k-mooney | gibi: ya melwitt is fixing it in a way that potentally backportable. an alternitive is so use a cascading delete trigger and let the db do it atomically | |
| 14:08:01 | sean-k-mooney | but there were reasons we dont use those in nova in the past | |
| 14:08:18 | sean-k-mooney | neutron does use them but i dont think thye have shadow tables | |
| 14:08:21 | gibi | but we dont delete here we archive here | |
| 14:09:04 | sean-k-mooney | the issue downstream was beacuse we archive and purge at the same time | |
| 14:09:31 | sean-k-mooney | so when the purge is happening with a limit on the rows it can cause half deleted recored until the next time the purge happens | |
| 14:09:33 | gibi | we we replace archiving with direct cascad delete that could work | |
| 14:09:45 | gibi | s/we we/if we/ | |
| 14:10:34 | sean-k-mooney | ya there has been some work done by ovh to do archiving externally with https://github.com/ovh/osarchiver | |
| 14:11:04 | sean-k-mooney | they are going to contbute that to either os-opstools or to a new openstack/ repo | |
| 14:11:32 | sean-k-mooney | if that was adopted instead it could allow us to remove archiving form the project entirly | |
| 14:12:04 | sean-k-mooney | proably somethign we shoudl discuss at the ptg or on the maining list | |
| 14:13:33 | sean-k-mooney | they developed it orginally for manila which dose not have shadow tabels and then started using it for all projects in production to have common way to do it | |
| 14:15:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Remove nova-dsvm-multinode-base https://review.opendev.org/c/openstack/nova/+/778908 | |
| 14:15:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/778885 | |
| 14:15:57 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Create and use nova-libvirt-irrelevant-files https://review.opendev.org/c/openstack/nova/+/778909 | |
| 14:20:30 | lemko | power_state: 4". Any idea? | |
| 14:20:30 | lemko | Hi, since I upgraded my openstack to Victoria (with upgrade of libvirt), my instances are randomly dying for no reason... "shutting down, reason=crashed" from qemu. Otherwise I can see this from nova-compute "Instance shutdown by itself. Calling the stop API. Current vm_state: active, current task_state: None, original DB power_state: 1, current VM | |
| 14:21:07 | sean-k-mooney | lemko: did you check the qemu instance log | |
| 14:21:12 | lemko | Yes. | |
| 14:21:24 | sean-k-mooney | lemko: /var/lib/libvirt/qemu/instace* | |
| 14:21:36 | sean-k-mooney | were there any errors in the log | |
| 14:21:41 | lemko | "shutting down, reason=crashed" is the only line interesting | |
| 14:22:02 | sean-k-mooney | ok that looks like a qemu or kvm issue then not openstack | |