| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-13 | |||
| 16:26:53 | bauzas | (bauzas) Document the volume detach issue | |
| 16:27:17 | bauzas | as a reminder, this volume detach issue we have is related to a specific libvirt release | |
| 16:27:35 | bauzas | so, I don't know how we could document it | |
| 16:27:57 | bauzas | but I'd like to have a way to explain our users that they need to make sure to ask to detach once they can ssh | |
| 16:28:10 | bauzas | and ideally attach as well | |
| 16:28:38 | bauzas | thoughts on that ? | |
| 16:29:31 | bauzas | hah | |
| 16:30:00 | bauzas | you know what ? | |
| 16:30:26 | bauzas | let's discuss this once more people are around like next year :) | |
| 16:30:42 | gibi | I guess you are not happy with a simple reno | |
| 16:31:02 | bauzas | gibi: yes, because this is not related to our repository, right? | |
| 16:31:09 | gibi | right | |
| 16:31:09 | bauzas | I mean, this isn't a nova bug | |
| 16:31:18 | bauzas | by a specific release | |
| 16:31:36 | gibi | it is only a nova bug if we assume nova should prevent the attach / detach | |
| 16:31:39 | bauzas | we can track this in Launchpad and I'd be okay | |
| 16:31:56 | gibi | but we never promised to prevent such attach / detach | |
| 16:31:56 | bauzas | that's one way of documenting an open issue | |
| 16:32:03 | bauzas | gibi: correct | |
| 16:33:05 | bauzas | you know what ? | |
| 16:33:13 | bauzas | lemme repropose this topic next year | |
| 16:33:17 | gibi | OK | |
| 16:33:27 | bauzas | (gmann) updates on switching the RBAC(scope and new defaults) defaults in nova | |
| 16:33:29 | clarkb | as an end user I look at your api docs when tryingto understand expectations | |
| 16:33:42 | clarkb | if anything is done I would suggest adding a note to the api docs for attach and detach | |
| 16:33:46 | bauzas | clarkb: that's a solutioon | |
| 16:34:17 | bauzas | which lgtm btw. | |
| 16:34:20 | bauzas | I can propose this | |
| 16:34:28 | bauzas | and people can vote on it | |
| 16:34:53 | bauzas | gmann: you had a thought | |
| 16:34:58 | gmann | for RBAC, | |
| 16:35:05 | bauzas | on the RBAC defaults, yeah | |
| 16:35:08 | gmann | I have added a new tempest job enabling the new defaults for Nova, neutron, cinder, glance and it is working fine #link https://zuul.openstack.org/builds?job_name=tempest-full-enforce-scope-new-defaults&skip=0 | |
| 16:35:33 | gmann | and this patch which need more work on functional tests flipping the defaults in nova #link https://review.opendev.org/c/openstack/nova/+/866218 | |
| 16:35:55 | gmann | just a info, nova patch is not ready to review as I need to fix functional tests there | |
| 16:36:18 | gmann | especially on admin to be changed from context.is_admin to roles['admin'] | |
| 16:36:23 | gmann | I should be ready by this week | |
| 16:36:28 | gmann | that is all | |
| 16:36:48 | gmann | there were few neutron bugs we found during tempest testing and all are fixed | |
| 16:38:32 | gmann | that's it from me unless any question on rbac defaults | |
| 16:38:50 | bauzas | gmann: ++ | |
| 16:39:02 | bauzas | thanks for working on it, that's good to know it works :) | |
| 16:39:31 | bauzas | gmann: you can ping me once you're done with the functests | |
| 16:39:38 | gmann | sure, will do | |
| 16:41:37 | bauzas | ok, then we're done | |
| 16:41:54 | bauzas | any other item anyone wants to discuss ? | |
| 16:42:24 | gibi | - | |
| 16:42:44 | bauzas | thanks all | |
| 16:42:49 | bauzas | #endmeeting | |
| 16:42:49 | opendevmeet | Meeting ended Tue Dec 13 16:42:49 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:42:49 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-12-13-16.00.html | |
| 16:42:49 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-12-13-16.00.txt | |
| 16:42:49 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-12-13-16.00.log.html | |
| 16:42:50 | gmann | thanks | |
| 16:42:54 | Uggla | thx | |
| 16:43:26 | gibi | thanks bauzas | |
| 16:47:14 | elodilles | thanks bauzas o/ | |
| 16:48:13 | clarkb | bauzas: to clarify a bit more on the tox stuff I'm trying to keep on top of it from a high level and in shared library tooling like the ensure-tox role. But I'm not planning to edit all the various tox.ini files to make them work :) | |
| 16:48:25 | opendevreview | Merged openstack/nova stable/xena: Retry attachment delete API call for 504 Gateway Timeout https://review.opendev.org/c/openstack/nova/+/866083 | |
| 16:48:26 | bauzas | ok | |
| 16:48:27 | clarkb | but nova's actually looks really clean and if that test change passes then you are probably fine | |
| 17:04:14 | gmann | elodilles: gibi: clarkb: bauzas: on tox. we have pinned it in devstack side for tox based job also https://github.com/openstack/devstack/blob/master/playbooks/tox/pre.yaml#L14 | |
| 17:04:56 | gmann | so ensure-tox role unpinning on dev 21 should not impact devstack based tox job. but yes we need to move master to tox 4.0.0 soon | |
| 17:05:48 | clarkb | right I think the main concern at this point is developers aren't going to want to maintain different versions of tox for different projects. Generally keeping up with latest avoids this problem. We pinned to stop the bleeding and now we can take a controlled approach to adding compatibility | |
| 17:06:28 | gmann | yeah | |
| 17:33:36 | opendevreview | Merged openstack/nova master: Create RequestGroups from InstancePCIRequests https://review.opendev.org/c/openstack/nova/+/852771 | |
| 17:33:43 | opendevreview | Merged openstack/nova master: Support resource_class and traits in PCI alias https://review.opendev.org/c/openstack/nova/+/853316 | |
| 17:33:50 | opendevreview | Merged openstack/nova master: Split PCI pools per PF https://review.opendev.org/c/openstack/nova/+/854440 | |
| #openstack-nova - 2022-12-14 | |||
| 08:35:44 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 08:35:44 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 08:36:16 | bauzas | happy spec review day everyone | |
| 08:50:18 | sahid | o/ I will be around if needed | |
| 13:23:29 | opendevreview | Merged openstack/nova stable/wallaby: Fix the wrong exception used to retry detach API calls https://review.opendev.org/c/openstack/nova/+/866084 | |
| 13:23:35 | opendevreview | Merged openstack/nova stable/wallaby: Retry attachment delete API call for 504 Gateway Timeout https://review.opendev.org/c/openstack/nova/+/866085 | |
| 14:11:41 | opendevreview | Merged openstack/nova master: Ironic nodes with instance reserved in placement https://review.opendev.org/c/openstack/nova/+/864773 | |
| 14:33:10 | andry | Hi to all, on my openstack installation I have one kvm that has been emptied and rebooted, but when i launch openstack hypervisor show podto2-kvm06 I see running_vms 7 but if I see on kvm I don't see vm and if I list openstack server list --all --host kvm, nothing appear. How is it possible?Thank you. | |
| 14:39:02 | andry | now i'm seeing on nova of kvm Error updating resources for node podto2-kvm06.: ValueError: No JSON object could be decoded | |
| 14:41:18 | amorin | hey sean-k-mooney and other, I'd love having a review on this: https://review.opendev.org/c/openstack/nova/+/853682 | |
| 14:44:07 | amorin | and also: https://review.opendev.org/c/openstack/nova/+/863891 | |
| 14:46:03 | opendevreview | Manuel Bentele proposed openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 15:23:37 | sean-k-mooney | amorin:perhaps tomorrow. today is intented to be a spec review day | |
| 15:24:04 | sean-k-mooney | for reasons i have not been able to spend much time on reviews today but tomrrow is ment to be an unoffical implemantion reveiw day | |
| 15:24:15 | sean-k-mooney | ill see if i can take a look at it then | |
| 16:11:02 | amorin | ack, thanks | |
| 17:44:38 | opendevreview | Merged openstack/nova-specs master: add spec for fqdn in hostname https://review.opendev.org/c/openstack/nova-specs/+/862626 | |
| 17:57:16 | opendevreview | Merged openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 21:05:40 | opendevreview | Merged openstack/python-novaclient master: trivial: Remove use of kwargs https://review.opendev.org/c/openstack/python-novaclient/+/806918 | |
| #openstack-nova - 2022-12-15 | |||
| 05:26:10 | opendevreview | Ghanshyam proposed openstack/nova master: Enable new defaults and scope checks by default https://review.opendev.org/c/openstack/nova/+/866218 | |
| 07:56:15 | opendevreview | Nobuhiro MIKI proposed openstack/nova-specs master: Add maxphysaddr support for Libvirt https://review.opendev.org/c/openstack/nova-specs/+/861033 | |
| 08:00:21 | opendevreview | Hiroki Narukawa proposed openstack/nova master: libvirt: retry libvirt connection on live_migration_monitor https://review.opendev.org/c/openstack/nova/+/867077 | |
| 10:54:11 | sean-k-mooney | bauzas: hi i was wrogn yestereday johns ironic spec is not merged yet but im more or less happy with it https://review.opendev.org/c/openstack/nova-specs/+/862833 do you want to review or will i add +w | |
| 10:54:40 | sean-k-mooney | there are a few nits that i think can be adressed in a followup but i dont see anything that would prevent us merging it as is | |
| 11:31:01 | sean-k-mooney | gibi: i need to do one thing then grab coffee after that i will be staring to review the pci series | |
| 11:31:25 | gibi | sean-k-mooney: OK. I replied to your comments about the rp_uuid vs rp_uuids | |
| 11:32:31 | sean-k-mooney | ya i was just checkign that and going to ping you | |
| 11:32:38 | sean-k-mooney | so based on your answer | |
| 11:33:15 | sean-k-mooney | we are using that array for two different types | |
| 11:33:30 | gibi | nope | |
| 11:33:41 | sean-k-mooney | pool vs spec | |
| 11:34:06 | gibi | pool has a single rp uuid, spec has a list of rp uuids serialized as comma separates string | |