| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-25 | |||
| 16:10:29 | bauzas | in theory, today was a Feature Review Day | |
| 16:10:40 | bauzas | but I feel many people having different priorities | |
| 16:10:50 | bauzas | so I haven't really called it | |
| 16:11:14 | bauzas | #topic Review priorities | |
| 16:11:18 | bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | |
| 16:11:22 | bauzas | #info As a reminder, people eager to review changes can +1 to indicate their interest, +2 for asking cores to also review | |
| 16:11:26 | bauzas | #topic Stable Branches | |
| 16:11:31 | bauzas | elodilles: your time | |
| 16:11:35 | elodilles | #info nova-lvm / nova-ceph-multistore jobs are fixed on all branches (2023.1, zed, yoga) \o/ | |
| 16:11:38 | elodilles | #info stable/victoria gate fix has also landed | |
| 16:11:40 | elodilles | #info gates from 2023.1 back till train should be OK | |
| 16:11:47 | elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | |
| 16:11:52 | elodilles | EOM | |
| 16:11:55 | bauzas | thanks | |
| 16:12:02 | bauzas | #info train-eol patch proposed https://review.opendev.org/c/openstack/releases/+/885365 | |
| 16:12:09 | bauzas | we're missing a second release core vote | |
| 16:12:14 | bauzas | but should happen eventually | |
| 16:12:20 | elodilles | yepp | |
| 16:12:32 | bauzas | auniyal had a point, I'll rephrase it | |
| 16:12:44 | bauzas | #info Please review these backport patches of stable 2023.1, zed and yoga for next minor release | |
| 16:12:49 | bauzas | #info most of these already have one +2. | |
| 16:12:52 | bauzas | #link https://etherpad.opendev.org/p/release-liaison-PatchesToReview | |
| 16:13:09 | bauzas | and the stable branches are now back in healthy state | |
| 16:13:18 | bauzas | #topic Open discussion | |
| 16:13:26 | bauzas | I have one item | |
| 16:14:17 | bauzas | as I said previously, I'll be on PTO starting next Tues | |
| 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 | |