| opendevreview | Takashi Kajinami proposed openstack/cinder master: Move boto3 to driver requirement https://review.opendev.org/c/openstack/cinder/+/902104 | 03:37 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/os-brick master: Add releasenote for unchecked LVM versions https://review.opendev.org/c/openstack/os-brick/+/909612 | 07:54 |
| opendevreview | Rajat Dhasmana proposed openstack/os-brick master: Add releasenote for unchecked LVM versions https://review.opendev.org/c/openstack/os-brick/+/909612 | 07:55 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907768 | 07:55 |
| Andrei-1 | simondodsley, hay! youve once reviewed https://review.opendev.org/c/openstack/cinder/+/889284 and gave positive review, since then Ive addressed several typos and minor warning messages. Can you please take a look on my patch again | 10:35 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: OSP 18.0 RFE https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/908164 | 11:02 |
| frickler | is sometimes getting timeouts in cinder unit tests a known issue? https://zuul.opendev.org/t/openstack/build/620cd280b9a34a2eadeac0b0f52ecdc6 or could that be a regression caused by one of the version bumps? | 11:11 |
| frickler | eharney: thanks, missed your comment yesterday | 11:12 |
| whoami-rajat_ | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:00 |
| whoami-rajat_ | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov | 14:00 |
| opendevreview | Jon Bernard proposed openstack/cinder master: Add unit test for _run_ssh() in Cisco driver https://review.opendev.org/c/openstack/cinder/+/909672 | 14:25 |
| opendevreview | Rajat Dhasmana proposed openstack/os-brick master: Add releasenote for unchecked LVM versions https://review.opendev.org/c/openstack/os-brick/+/909612 | 14:32 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Deprecate Windows OS support https://review.opendev.org/c/openstack/os-brick/+/900913 | 15:00 |
| eharney | whoami-rajat: https://review.opendev.org/c/openstack/cinder/+/861883 turned into a requirements change, so we should land it soon too | 15:01 |
| eharney | (rosmaita +2'd it in September) | 15:01 |
| rosmaita | i'm still in favor of it, too | 15:02 |
| rosmaita | eharney and zaitcev: can you re-look at https://review.opendev.org/c/openstack/os-brick/+/900913 ? Takashi pushed a revision based on my comment, but i was asking because I wasn't sure what the correct thing was ... need a sanity check | 15:04 |
| whoami-rajat_ | eharney, i see those changes were introduced in oslo.utils in 2022! approved | 15:05 |
| crohmann | rosmaita: Isn't that hacking change also related to e.g https://review.opendev.org/c/openstack/keystone/+/906312 ? | 15:05 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Move boto3 to driver requirement https://review.opendev.org/c/openstack/cinder/+/902104 | 15:06 |
| rosmaita | crohmann: gosh, i hope not! | 15:06 |
| zaitcev | rosmaita: done | 15:08 |
| rosmaita | thanks! | 15:08 |
| crohmann | Sorry hit send to quick ... I mean to say, that that Python 3.12 requires, apart from other tings, datetime changes. And this affects more than one project https://review.opendev.org/q/topic:%22python-3.12%22. Seeing those flake8 checks were about checkons on datetime I kinda made the connection that there is now another reason to check for "wrong usages" ;-) | 15:09 |
| crohmann | Since I kinda bombarded the Request for review column with improvements for cinder backup. Would anybody be willing to do a little review session on them in a closer loop / live? | 15:11 |
| tkajinam | rosmaita, I think 2 is correct, according to https://paste.opendev.org/show/bqz6MVfJiAfpHMZRHYzO/ | 15:13 |
| opendevreview | Merged openstack/cinder stable/2023.2: Tests: Make NEC tests faster https://review.opendev.org/c/openstack/cinder/+/905534 | 15:15 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Deprecate Windows OS support https://review.opendev.org/c/openstack/os-brick/+/900913 | 15:15 |
| rosmaita | tkajinam: thanks for the example, i think you are correct | 15:18 |
| tkajinam | no thank YOU for pointing that out | 15:18 |
| tkajinam | I should have noticed earlier. probably did dumb blind pick from somewhere else... | 15:19 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907768 | 15:30 |
| ganso | jungleboyj, whoami-rajat hi! could you please take a look at this backport? it already has a +2, thanks in advance! https://review.opendev.org/c/openstack/os-brick/+/907930 | 15:36 |
| jungleboyj | ganso: Done. :-) | 15:39 |
| ganso | jungleboyj: thanks! :D | 15:40 |
| jungleboyj | Welcome. | 15:40 |
| opendevreview | Eric Harney proposed openstack/cinder master: Provide message info in continue_backup https://review.opendev.org/c/openstack/cinder/+/909688 | 16:00 |
| opendevreview | Merged openstack/cinder master: Hacking: Remove C306, C308 checks https://review.opendev.org/c/openstack/cinder/+/861883 | 16:00 |
| opendevreview | Merged openstack/cinder stable/2023.2: [stable-only] fix relnote markup https://review.opendev.org/c/openstack/cinder/+/905039 | 16:00 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: Expose bug 2009483 with mypy https://review.opendev.org/c/openstack/cinder/+/909689 | 16:09 |
| opendevreview | Merged openstack/os-brick master: LVM: Assume LVM >= 2.02.115 https://review.opendev.org/c/openstack/os-brick/+/891319 | 16:35 |
| opendevreview | Merged openstack/os-brick master: Dell PowerFlex: Unnecessary login happen https://review.opendev.org/c/openstack/os-brick/+/876284 | 16:47 |
| opendevreview | Merged openstack/os-brick master: Add releasenote for unchecked LVM versions https://review.opendev.org/c/openstack/os-brick/+/909612 | 16:47 |
| opendevreview | Merged openstack/cinder stable/2023.1: Skip sparse copy during volume reimage https://review.opendev.org/c/openstack/cinder/+/908808 | 16:57 |
| opendevreview | Merged openstack/os-brick stable/zed: nvmeof: Call findmnt with -v https://review.opendev.org/c/openstack/os-brick/+/907930 | 17:18 |
| opendevreview | Merged openstack/os-brick master: Deprecate Windows OS support https://review.opendev.org/c/openstack/os-brick/+/900913 | 19:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!