| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/846842 | 02:18 |
|---|---|---|
| whoami-rajat | hi stephenfin , can you take a look at this patch? https://review.opendev.org/c/openstack/python-openstackclient/+/844268 | 06:41 |
| opendevreview | zack chen proposed openstack/python-cinderclient master: Fix extension loading from python path https://review.opendev.org/c/openstack/python-cinderclient/+/847743 | 07:17 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: WIP: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 07:46 |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: PowerMax Driver - Add a release note for 784603 https://review.opendev.org/c/openstack/cinder/+/847864 | 08:19 |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: PowerMax Driver - Add a release note for 784603 https://review.opendev.org/c/openstack/cinder/+/847864 | 08:19 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver - RDF checks https://review.opendev.org/c/openstack/cinder/+/768105 | 08:42 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver - RDF checks https://review.opendev.org/c/openstack/cinder/+/768105 | 09:11 |
| opendevreview | Oleg proposed openstack/cinder stable/yoga: PowerMax Driver - Add a release note for 784603 https://review.opendev.org/c/openstack/cinder/+/847864 | 09:15 |
| geguileo | eharney whoami-rajat this fixes the NFS job https://review.opendev.org/c/openstack/cinder/+/847953 | 10:00 |
| geguileo | it's a 2 LOC change | 10:01 |
| tosky | work = True; really_work = True | 10:03 |
| whoami-rajat | geguileo, i didn't realize the nfs job was broken | 10:04 |
| geguileo | tosky: rofl rofl rofl That's a great definition of that patch | 10:05 |
| whoami-rajat | will take a look | 10:05 |
| geguileo | tosky: that's why the brach is fix-fix-storage_protocol ;-) | 10:05 |
| geguileo | whoami-rajat: yeah, I broke the job and didn't realize it, eharney told me about it | 10:06 |
| geguileo | I had checked the value locally, but since it was flapping I always got the right value and not the wrong one | 10:06 |
| whoami-rajat | are we referring to devstack-plugin-nfs-tempest-full job? i see it passing in the original change https://review.opendev.org/c/openstack/cinder/+/836069 | 10:09 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Add a check for virtual_size at API layer https://review.opendev.org/c/openstack/cinder/+/847335 | 10:31 |
| opendevreview | Carlos Eduardo proposed openstack/cinder stable/yoga: [NetApp] Fix iSCSI CHAP auth issue during volume attach https://review.opendev.org/c/openstack/cinder/+/848064 | 10:42 |
| geguileo | whoami-rajat: yeah, it worked because of the flapping (value not changed), but then other jobs started failing (because the code changed the value correctly on the call) and a tempest patch was added to fix it for those | 11:00 |
| geguileo | and the union of the cinder patch change and the tempest change broke the NFS job | 11:00 |
| geguileo | whoami-rajat: tempest change: https://review.opendev.org/c/openstack/tempest/+/842432 | 11:02 |
| geguileo | and that tempest change also "broke" backward compatibility (because tempest is branchless), so I proposed a tempest patch to fix that https://review.opendev.org/c/openstack/tempest/+/847952 | 11:03 |
| *** dviroel|out is now known as dviroel | 11:20 | |
| whoami-rajat | geguileo, oh, I'm finding it hard to think of a day when we didn't have a new issue but yeah thanks for working on that | 11:33 |
| geguileo | whoami-rajat: seems like I'm breaking more things than I fix these days :-( | 11:34 |
| whoami-rajat | geguileo, nope, things were broken, you fixed it partially and then you fixed it completely and that too with quick followups so thanks for that :) | 11:37 |
| geguileo | that's an optimistic way of putting it | 11:38 |
| opendevreview | Alexander Malashenko proposed openstack/cinder master: WIP: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 11:38 |
| whoami-rajat | that's how i see it | 11:38 |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 11:51 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver - Fix for force flag https://review.opendev.org/c/openstack/cinder/+/797970 | 13:33 |
| enriquetaso | are we waiting on https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/791921 for any reason? | 13:45 |
| tosky | enriquetaso: eharney voted -1 previously, I'd like to know if all the concerns have been addressed | 13:52 |
| eharney | yeah, i reviewed this to some extent a bit ago but never voted -- it was reworked to fix my initial concerns | 13:54 |
| eharney | will take a look | 13:54 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin | 13:59 |
| whoami-rajat | Meeting link: https://bluejeans.com/556681290 | 13:59 |
| geguileo | whoami-rajat: thanks, I'll be 5 minutes late, finishing another meetng | 14:00 |
| whoami-rajat | geguileo, no issues | 14:00 |
| whoami-rajat | HappyStacker1, can you end the meeting in #openstack-meeting-alt channel? | 14:02 |
| whoami-rajat | #endmeeting -- with this command | 14:02 |
| tosky | can't attend (sadly bj today is overlapping with another video meeting of mine), but I've left a note and I will answer on IRC if needed | 14:02 |
| whoami-rajat | tosky, sure, no worries | 14:02 |
| enriquetaso | #startmeeting cinder_bs | 14:52 |
| opendevmeet | Meeting started Wed Jun 29 14:52:56 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:52 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:52 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 14:52 |
| enriquetaso | This week report | 14:53 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-June/029327.html | 14:53 |
| enriquetaso | #topic Unexpected exception on “image create --volume” | 14:54 |
| enriquetaso | # link https://storyboard.openstack.org/#!/story/2009287 | 14:54 |
| enriquetaso | Summary: If you create an image from an existing volume, "image create --volume " does not work, but just raises error. I guess using glance store. | 14:55 |
| enriquetaso | it seems that this is something we should keep an eye on. | 14:55 |
| enriquetaso | Rajat proposed a patch to solve this on OSC and python cinder client: | 14:55 |
| enriquetaso | #link https://review.opendev.org/c/openstack/python-openstackclient/+/844268 | 14:55 |
| enriquetaso | doesn't affect cinder client | 14:58 |
| enriquetaso | #topic devstack-plugin-nfs-tempest-full CI job failing | 15:00 |
| enriquetaso | #link https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-06-29-14.06.log.html | 15:00 |
| enriquetaso | #topic open discussion | 15:01 |
| enriquetaso | #endmeeting | 15:04 |
| opendevmeet | Meeting ended Wed Jun 29 15:04:51 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-06-29-14.52.html | 15:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-06-29-14.52.txt | 15:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-06-29-14.52.log.html | 15:04 |
| whoami-rajat | just to finish the sentence, there are a bunch of patches that are requested for review in the meetings etherpad so please take a look https://etherpad.opendev.org/p/cinder-zed-meetings | 15:05 |
| enriquetaso | i will | 15:05 |
| enriquetaso | whoami-rajat++ | 15:05 |
| whoami-rajat | thanks enriquetaso ! | 15:06 |
| opendevreview | Eric Harney proposed openstack/cinder master: Move NFS job to voting https://review.opendev.org/c/openstack/cinder/+/848147 | 15:15 |
| *** dviroel is now known as dviroel|lunch | 15:24 | |
| *** dviroel|lunch is now known as dviroel | 16:37 | |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Add multiattach tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/791921 | 16:41 |
| opendevreview | Merged openstack/cinder master: Fix flapping storage_protocol in get-pools https://review.opendev.org/c/openstack/cinder/+/847953 | 18:19 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Driver -- Add support for Unisphere for PowerMax 10.0 https://review.opendev.org/c/openstack/cinder/+/844563 | 19:48 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add a check for virtual_size at API layer https://review.opendev.org/c/openstack/cinder/+/847335 | 20:09 |
| *** dviroel is now known as dviroel|biab | 20:30 | |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 21:41 |
| *** dviroel|biab is now known as dviroel | 22:09 | |
| *** dviroel is now known as dviroel|out | 22:28 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!