| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-05 | |||
| 12:40:52 | sean-k-mooney | bauzas: https://codesearch.opendev.org/?q=six&i=nope&literal=nope&files=&excludeFiles=&repos=openstack%2Fnova | |
| 12:41:00 | bauzas | that's what I was verifying | |
| 12:42:30 | sean-k-mooney | bauzas: tox -e pep8 check all files and i verified that with pre-commit run --all-files also | |
| 12:42:45 | sean-k-mooney | so i know that the only files that it flagged on where the two places i added it for testing | |
| 12:43:25 | sean-k-mooney | so on master at least we are all good. replied in the revew with the codesearch link too | |
| 12:49:49 | bauzas | sean-k-mooney: sent to the gate | |
| 13:07:47 | opendevreview | Merged openstack/nova master: Remove the code that never reach https://review.opendev.org/c/openstack/nova/+/809572 | |
| 13:07:56 | opendevreview | Merged openstack/nova master: Fix eventlet.tpool import https://review.opendev.org/c/openstack/nova/+/830383 | |
| 13:08:04 | opendevreview | Merged openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/836113 | |
| 13:18:33 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: stable/xena: fix openstack-tox-lower-constraints https://review.opendev.org/c/openstack/nova/+/836650 | |
| 13:19:34 | tobias-urdin | now when running_vms key is removed from /os-hypervisors/detail after microversion 2.87 what is the best way (without looping every single servers) to get total amount of instance and running vms using api? | |
| 13:24:03 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: stable/xena: fix openstack-tox-lower-constraints https://review.opendev.org/c/openstack/nova/+/836650 | |
| 13:33:53 | opendevreview | Maxim Korezkij proposed openstack/nova master: Handle volume attachments https://review.opendev.org/c/openstack/nova/+/833234 | |
| 13:40:02 | bauzas | reminder : we start our PTG in 20 mins \o/ | |
| 13:45:07 | artom | Precarious Tricky Game? | |
| 13:46:44 | bauzas | muh | |
| 13:47:14 | tobias-urdin | hm I guess I could hit os-instance_usage_audit_log but that isn't reliable | |
| 14:00:01 | sean-k-mooney | normally you would get the totals form placment | |
| 14:00:38 | sean-k-mooney | and the runnign instance i guess via a openstack server list --all-tenant and filter that for running | |
| 14:00:51 | sean-k-mooney | it depends on what you actually are trying to do | |
| 14:00:54 | sean-k-mooney | tobias-urdin: ^ | |
| 14:01:19 | sean-k-mooney | https://www.openstack.org/ptg/rooms/newton | |
| 14:01:23 | sean-k-mooney | nova room ^ | |
| 14:03:00 | bauzas | what sean-k-mooney said | |
| 14:03:03 | bauzas | opening the PTG | |
| 14:03:58 | stephenfin | I'll be along at :30. Got a conflict | |
| 14:08:04 | opendevreview | Maxim Korezkij proposed openstack/nova master: Handle volume attachments https://review.opendev.org/c/openstack/nova/+/833234 | |
| 14:10:19 | tobias-urdin | sean-k-mooney: we get allocations from placement (vcpu, memory, disk, gpu etc) instead of the old hypervisor statistics API but cannot find a way to get number of instances only resource usage | |
| 14:11:49 | sean-k-mooney | you would count allocations of type instance | |
| 14:12:33 | tobias-urdin | oh there is a consumer_type on allocations, i see, thanks :) | |
| 14:12:36 | sean-k-mooney | although honestly openstack server list --all-projects is proably the best way to do this | |
| 14:12:52 | sean-k-mooney | if you are an admin at least | |
| 14:13:11 | sean-k-mooney | placement wont actully count shelved instnaces | |
| 14:13:24 | sean-k-mooney | since they are not on a host | |
| 14:13:57 | tobias-urdin | it's just to get an overview the easiest way (an less intensive way), isn't actually used for anything else since we have resource usage for planning | |
| 14:14:40 | sean-k-mooney | so one of the reasons we remvoed this is it was cached info that can get out of sync if the resouce tracker had a problem | |
| 14:15:08 | sean-k-mooney | e.g. any that broke the update_available_resouces preiodic task ment the data in the hypervior api was out of date | |
| 14:15:39 | sean-k-mooney | server list on the other hand will always be correct | |
| 14:15:57 | sean-k-mooney | if you do no hit the server detail api then i think it also only hits the api db | |
| 14:16:00 | sean-k-mooney | so its fast | |
| 14:25:51 | tobias-urdin | for now i'll just lazy guess with len(allocations) when hitting /resource_provider/<uuid>/allocations since the returned allocations doesn't include consumer_type like /allocations does | |
| 14:26:20 | tobias-urdin | it doesn't have to be perfect, seems inefficient to just loop all allocations to check for VCPU resource or similar, so just using that for now | |
| 14:29:37 | sean-k-mooney | consumer types is new | |
| 14:29:45 | sean-k-mooney | so unless you are on yoga it wont be there | |
| 14:30:04 | sean-k-mooney | before yoga they are always vms | |
| 14:30:10 | sean-k-mooney | well no vms or migrations | |
| 14:30:32 | tobias-urdin | ah, ack good to know then I can check that when we move to yoga | |
| 14:30:41 | sean-k-mooney | yep | |
| 14:32:09 | tobias-urdin | thanks! | |
| 14:45:25 | dansmith | bauzas: are ya'll on track for cadence discussion at 1500? | |
| 14:45:35 | bauzas | yup | |
| 14:45:39 | dansmith | cool | |
| 14:47:43 | sean-k-mooney | dansmith: technially we are slightly behind but going to jump to it at the top of the hour | |
| 14:47:57 | dansmith | okay, I'm planning to come over for that, FYI | |
| 14:50:47 | bauzas | + | |
| 14:58:25 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: stable/xena: fix openstack-tox-lower-constraints https://review.opendev.org/c/openstack/nova/+/836650 | |
| 14:59:35 | opendevreview | Andre Aranha proposed openstack/nova master: Remove Centos 8 jobs https://review.opendev.org/c/openstack/nova/+/836665 | |
| 15:01:24 | sean-k-mooney | gmann: we are staring the release cadance discussion | |
| 15:02:58 | gmann | sean-k-mooney: thanks, I am in keystone too but will jpin | |
| 15:03:37 | stephenfin | sean-k-mooney: Are you using a monospace font as your system font? What is _wrong_ with you? :-D | |
| 15:03:54 | sean-k-mooney | yes | |
| 15:03:56 | sean-k-mooney | yes i am | |
| 15:04:49 | sean-k-mooney | i think im using source code pro | |
| 15:05:36 | sean-k-mooney | source code pro regualar 10pt :) | |
| 15:05:57 | sean-k-mooney | althoug my desktop font is notserif Nerd Font medium | |
| 15:06:11 | sean-k-mooney | so it depends on the app | |
| 15:10:22 | clarkb | source code pro is a great font | |
| 15:27:09 | opendevreview | Merged openstack/nova master: hacking: Prevent use of six https://review.opendev.org/c/openstack/nova/+/836588 | |
| 15:49:49 | gibi | bauzas: i will be 10 minutes late for the next session | |
| 15:51:35 | bauzas | we're restarting the sessions | |
| 17:05:29 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Deprecate nova CLI https://review.opendev.org/c/openstack/python-novaclient/+/836682 | |
| 17:05:30 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: docs: Update docs to reflect deprecation status https://review.opendev.org/c/openstack/python-novaclient/+/836683 | |
| 17:08:40 | stephenfin | dansmith: export NOVACLIENT_IAMDAN=1 ^ yw | |
| 17:10:23 | dansmith | lol | |
| 17:11:04 | dansmith | fine with me, but probably should make that something else.. but definitely keep the comment :) | |
| 17:12:37 | stephenfin | I initially used NOVACLIENT_IGNORE_DEPRECATION but I wanted to avoid people using this in e.g. ansible playbooks (vs. just changing their used commands). Giving it a weird name should make it sound less official, right? | |
| 17:25:54 | dansmith | stephenfin: yeah that's a good goal... what you have might be too unofficial, but whatever :) | |
| 19:16:53 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: extend_volume of libvirt/volume/iscsi should not use device_path https://review.opendev.org/c/openstack/nova/+/836604 | |
| 19:17:23 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: extend_volume of libvirt/volume/iscsi should not use device_path https://review.opendev.org/c/openstack/nova/+/836605 | |
| 19:17:34 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: extend_volume of libvirt/volume/iscsi should not use device_path https://review.opendev.org/c/openstack/nova/+/836606 | |
| 19:17:45 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: extend_volume of libvirt/volume/iscsi should not use device_path https://review.opendev.org/c/openstack/nova/+/836607 | |
| 20:27:09 | opendevreview | Merged openstack/nova master: Fix wrong attribute to find remote address https://review.opendev.org/c/openstack/nova/+/836248 | |
| #openstack-nova - 2022-04-06 | |||
| 06:19:38 | opendevreview | Maxim Korezkij proposed openstack/nova master: Handle volume attachments https://review.opendev.org/c/openstack/nova/+/833234 | |
| 07:16:01 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1944759 https://review.opendev.org/c/openstack/nova/+/836733 | |
| 07:16:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/836734 | |
| 08:55:30 | kashyap | gibi[m]: Hey, when you get 25 minutes, I've addrressed various things from your past feedback here: https://review.opendev.org/c/openstack/nova/+/762330/ | |
| 08:56:39 | kashyap | gibi[m]: It requires to reload some context, so do let me know if you see any further glaring holes there. Also see my last comment on PS24 for a short changelog | |
| 08:57:13 | kashyap | I'm badgering you because you actually gave it a close look in the past. :) | |
| 09:03:57 | gibi | kashyap: ack, I will look at it | |
| 09:09:11 | opendevreview | kiran pawar proposed openstack/nova master: VMware: Early fail spawn if memory is not multiple of 4. https://review.opendev.org/c/openstack/nova/+/835739 | |
| 09:53:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: trigger nova-emulation on the parent patch https://review.opendev.org/c/openstack/nova/+/836764 | |
| 10:04:45 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Deprecate nova CLI https://review.opendev.org/c/openstack/python-novaclient/+/836682 | |
| 10:04:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Update pre-commit hook, hacking versions https://review.opendev.org/c/openstack/python-novaclient/+/836768 | |
| 10:04:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: docs: Update docs to reflect deprecation status https://review.opendev.org/c/openstack/python-novaclient/+/836683 | |
| 10:06:58 | gibi | kashyap: replied | |
| 10:07:50 | kashyap | gibi: Thank you! Will look shortly | |
| 10:07:53 | gibi | kashyap: and I have to update your estimator, it took ~1 hour to load the context and review the patch again :) Maybe I need more coffee | |
| 10:08:32 | kashyap | gibi: Haha, really sorry; I was being selfish there. I almost wrote 40 minutes, then I thought: "No, that might dissuade Gibi" :D | |
| 10:08:45 | gibi | no worries :) | |
| 10:11:41 | kashyap | gibi: Just read; good questions. I need a couple more hours to study it carefully to answer your questions well | |