| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-25 | |||
| 16:14:31 | bauzas | gibi volunteered for running the Aug 15 meeting | |
| 16:14:38 | bauzas | (and Aug 22) | |
| 16:14:49 | bauzas | but we may need someone to chair the Aug 1 and Aug 8 meetings | |
| 16:14:57 | bauzas | or we skip them | |
| 16:15:40 | gibi | jepp I still OK with the 15 and 22 | |
| 16:15:51 | bauzas | so, anyone fancy volunteering ? if not, nevermind, I'll send an email to cancel the two weeks | |
| 16:16:31 | bauzas | I can try to handle Aug 1 meeting as I won't (yet) be on a plane, but I can't promise | |
| 16:16:48 | bauzas | ok, let's consider the meetings cancelled | |
| 16:17:04 | bauzas | #info Aug 1 and Aug 8 Nova meetings are CANCELLED | |
| 16:17:15 | bauzas | #action bauzas to notify the ML accordingly | |
| 16:17:17 | bauzas | that's it | |
| 16:17:19 | bauzas | for me | |
| 16:17:24 | bauzas | anything anyone ? | |
| 16:18:24 | bauzas | ok, thanks then | |
| 16:18:41 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-07-25-16.00.log.html | |
| 16:18:41 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-07-25-16.00.txt | |
| 16:18:41 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-07-25-16.00.html | |
| 16:18:41 | opendevmeet | Meeting ended Tue Jul 25 16:18:41 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:18:41 | bauzas | #endmeeting | |
| 16:19:00 | gibi | o/ | |
| 16:19:09 | elodilles | thanks, too o/ | |
| 16:21:14 | gmann | thanks o/ | |
| 16:58:39 | opendevreview | sean mooney proposed openstack/nova master: [WIP] use disk caching to hide slow cinder performance https://review.opendev.org/c/openstack/nova/+/889383 | |
| 16:59:11 | sean-k-mooney | if we decied this is the correct approch in general i can do ^ in devstack instead | |
| 16:59:45 | sean-k-mooney | or in the base job but im adding it to all the in tree job to get more input on if it affect the ci jobs positivly or negitivly | |
| 17:02:11 | dansmith | sean-k-mooney: I think doing it in devstack itself is the wrong move, but maybe in some of the devstack job defs.. this is really an optimization for our own CI and not something we'd want anyone running without knowing it, even in devstack, IMHO | |
| 17:02:41 | dansmith | I'll be interested to see if that helps significantly.. I've seen that bug manifest only occasionally in my surveying recently | |
| 17:02:42 | sean-k-mooney | i was thinking of doing it like the mysql memory thing | |
| 17:03:08 | sean-k-mooney | e.g. off by default with a macro/var to turn it on in ci | |
| 17:03:11 | dansmith | yeah, it could be a flag, sure | |
| 17:03:41 | sean-k-mooney | basically i just dont want to have to copy paste that to even more jobs over time :) | |
| 17:04:23 | sean-k-mooney | im not sure if it will help or not but we will see | |
| 17:04:38 | sean-k-mooney | in theory it should not sure | |
| 19:24:20 | opendevreview | Maxim Monin proposed openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | |
| #openstack-nova - 2023-07-26 | |||
| 02:04:51 | opendevreview | Merged openstack/nova master: Add dest_compute_id to Migration object https://review.opendev.org/c/openstack/nova/+/879682 | |
| 10:41:28 | carloss | o/ Uggla I believe we finally have some testable code in the Manila side for the VirtioFS implementation | |
| 10:42:07 | carloss | https://review.opendev.org/c/openstack/manila/+/887725 - Allow restricting access rules fields and deletion | |
| 10:42:07 | carloss | https://review.opendev.org/c/openstack/manila/+/887115 - Resource Locks: Support for share deletion lock | |
| 10:42:07 | carloss | the Manila changes: | |
| 10:43:12 | carloss | https://review.opendev.org/c/openstack/python-manilaclient/+/888355 - Allow restricting access rules | |
| 10:43:12 | carloss | https://review.opendev.org/c/openstack/python-manilaclient/+/888006 - Support for resource locks | |
| 10:43:12 | carloss | OSC changes in case you want to interact with manila through the CLI: | |
| 10:43:50 | carloss | https://review.opendev.org/c/openstack/openstacksdk/+/889519 - Resource locks and access rules restrictions | |
| 10:43:50 | carloss | and finally, the SDK part of the implementation: | |
| 10:44:00 | carloss | please let me know if I can help with something :) | |
| 10:44:40 | carloss | you'll notice some of the changes are still WIP, but that is mostly for some missing coverage in unit/functional tests | |
| 10:57:42 | opendevreview | sean mooney proposed openstack/nova master: use disk caching to hide slow cinder performance https://review.opendev.org/c/openstack/nova/+/889383 | |
| 11:00:31 | sean-k-mooney | carloss: cool the nova code shoudl use the sdk not the manilaclient | |
| 11:00:52 | sean-k-mooney | so we actully only depend on the maniala change and the sdk changes | |
| 11:01:34 | sean-k-mooney | its cutting it kind of tight to get this all landed this cycle | |
| 11:03:17 | carloss | yep... the sdk change should be in a good shape as well... I did some testing on it yesterday and it behave fine when I did stuff with it manually | |
| 11:03:17 | carloss | > carloss: cool the nova code shoudl use the sdk not the manilaclient | |
| 11:03:50 | carloss | sean-k-mooney: ack - we'll do our best to have it landing asap in manila | |
| 11:03:50 | carloss | > its cutting it kind of tight to get this all landed this cycle | |
| 11:04:17 | sean-k-mooney | at least with depend on we can test some of this before its landed | |
| 11:04:26 | carloss | ++ | |
| 11:04:38 | carloss | I think both manila side changes are ready for some feedback as well, so if you want to share your thoughts, please do so | |
| 11:04:56 | sean-k-mooney | have you considered addding tempest coverage? | |
| 11:05:24 | sean-k-mooney | or testing it in the manila ci beyond unit/funcitonal tests | |
| 11:06:34 | carloss | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/889331 - Add access rules restriction tests | |
| 11:06:34 | carloss | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/888618 - Test resource locks | |
| 11:06:34 | carloss | yeah, we have some changes up for functional testing on manila-tempest-plugin as well: | |
| 11:06:46 | carloss | resource locks is failing because of a known issue in our CI | |
| 11:07:25 | carloss | and access rules restrictions is on me... I need to fix some issues with testing, but I also tested it locally and it worked | |
| 11:07:43 | carloss | I should figure out such issues with access rules restriction testing over the course of this week | |
| 11:08:55 | carloss | s/locally/in a devstack env | |
| 11:09:37 | carloss | sean-k-mooney: forgot to share those changes in advance with the other ones I sent, sorry 😅 | |
| 11:09:37 | carloss | sean-k-mooney: forgot to share those changes in advance with the other ones I sent, sorry 😅 | |
| 11:37:37 | sean-k-mooney | no worries im sure they will be useful for Uggla as a refernce for how to use the apis | |
| 12:36:37 | Uggla | Hey carloss, thanks for the information. As sean-k-mooney said I'm currently moving from the manilaclient to sdk. As soon as ready (soon). I will try to implement and test the new "lock and restricted access rules" features. | |
| 12:38:07 | Uggla | carloss, also thanks for the hard work, I can't stand to integrate that. | |
| 12:41:52 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unused mocks https://review.opendev.org/c/openstack/nova/+/867789 | |
| 12:41:53 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | |
| 12:41:53 | 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:41:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | |
| 12:41:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 12:41:55 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | |
| 12:41:55 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | |
| 12:41:56 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining ironic driver calls https://review.opendev.org/c/openstack/nova/+/867795 | |
| 12:41:56 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining network operations https://review.opendev.org/c/openstack/nova/+/867794 | |
| 12:41:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | |
| 12:41:58 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for deprecated baremetal proxy API https://review.opendev.org/c/openstack/nova/+/867796 | |
| 12:42:00 | opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |
| 12:42:00 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | |
| 12:42:32 | sean-k-mooney | stephenfin: just one or two ptaches ya | |
| 12:42:45 | sean-k-mooney | is that all of the convertion to the sdk | |
| 12:43:08 | stephenfin | only two changed, yes 😇 (the rest are just rebases) | |
| 12:43:08 | stephenfin | only two changed, yes 😇 (the rest are just rebases) | |
| 12:43:15 | stephenfin | yup, full conversion of ironic calls to SDK | |
| 12:43:41 | sean-k-mooney | oh just ironic i tough ti say some neutron ones | |
| 12:44:08 | stephenfin | nah, the network operations referred to are still ironic calls | |
| 12:44:17 | stephenfin | neutron is a much bigger fish | |
| 12:44:42 | sean-k-mooney | ok and same for volume it just the ironic volume bits | |
| 12:45:13 | sean-k-mooney | still woul be nice to get that done this cycle | |
| 12:45:29 | carloss | Uggla: yw - happy to help... gouthamr and I did our best to get this to a testable state, so have fun testing! :D | |
| 12:45:35 | stephenfin | yup, ironic volumes | |
| 12:46:11 | Uggla | carloss, \o/ | |
| 14:40:05 | opendevreview | Merged openstack/nova master: db: Store unmasked password https://review.opendev.org/c/openstack/nova/+/880669 | |
| 15:11:55 | opendevreview | Vladislav Belogrudov proposed openstack/nova master: Nova instance snapshot should wait for volumes https://review.opendev.org/c/openstack/nova/+/848638 | |
| 15:49:20 | opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |