| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-13 | |||
| 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 | |
| 11:34:27 | sean-k-mooney | yes but ignored_tags is beign applied to both | |
| 11:35:14 | gibi | ahh, yes that is applied to both yes | |
| 11:35:26 | sean-k-mooney | ya so that is what was confusitng me | |
| 11:35:48 | sean-k-mooney | since each type only have one fo the two field but we have on array of ignored tags | |
| 11:36:12 | sean-k-mooney | anyway based on your comments and this converstaion i now understand the intent | |
| 11:36:53 | sean-k-mooney | we might want to consider doing this diffently but i think im ok with this for now. but we my want to split this in a followup | |
| 11:37:31 | gibi | I made a note and I will look into this | |
| 11:37:32 | sean-k-mooney | ill leave a comment to that effect and see what stephen thinks when he gets to it and upgrade to +2 | |
| 12:51:08 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/c/openstack/nova/+/659691 | |
| 12:51:09 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | |
| 12:51:09 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove use of removeprefix https://review.opendev.org/c/openstack/nova/+/867788 | |
| 12:51:10 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unused mocks https://review.opendev.org/c/openstack/nova/+/867789 | |
| 12:51:10 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 12:51:11 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | |
| 12:51:11 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | |