Tuesday, 2024-02-13

opendevreviewMerged openstack/os-brick master: encryptors: Unbind LuksEncryptor and CryptsetupEncryptor  https://review.opendev.org/c/openstack/os-brick/+/79127100:04
opendevreviewTakashi Kajinami proposed openstack/cinder master: Remove six from ProphetStor driver  https://review.opendev.org/c/openstack/cinder/+/90739000:54
opendevreviewMerged openstack/cinder master: Remove six from Zadara driver  https://review.opendev.org/c/openstack/cinder/+/90738701:08
opendevreviewMerged openstack/cinder master: Remove six from Hedvig driver  https://review.opendev.org/c/openstack/cinder/+/90741801:32
opendevreviewMerged openstack/cinder master: Remove six from Fujitsu drivers  https://review.opendev.org/c/openstack/cinder/+/90737301:34
opendevreviewRajat Dhasmana proposed openstack/cinder stable/2023.1: Skip sparse copy during volume reimage  https://review.opendev.org/c/openstack/cinder/+/90880805:31
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call.  https://review.opendev.org/c/openstack/cinder/+/90777006:17
opendevreviewRaghavendra Tilay proposed openstack/cinder master: DNM - Do not merge - For test only  https://review.opendev.org/c/openstack/cinder/+/90777106:24
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call.  https://review.opendev.org/c/openstack/cinder/+/90777207:16
opendevreviewMerged openstack/cinder master: Remove six from test codes  https://review.opendev.org/c/openstack/cinder/+/90738407:57
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call.  https://review.opendev.org/c/openstack/cinder/+/90777308:04
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call.  https://review.opendev.org/c/openstack/cinder/+/90776808:48
*** tosky_ is now known as tosky09:48
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call.  https://review.opendev.org/c/openstack/cinder/+/90776810:56
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: Rework Open-E JovianDSS driver  https://review.opendev.org/c/openstack/cinder/+/88928411:03
mnasiadkarosmaita: the problem that I mentioned only happens on Kolla-Ansible multinode runs - any ideas? https://abc5ddca5f0f58a1c47b-510d79facabc0af50e4fd5842aa5e4cb.ssl.cf1.rackcdn.com/908405/5/check/kolla-ansible-rocky9-slurp-upgrade-cephadm/53fdf01/primary/logs/ansible/upgrade11:42
opendevreviewJan Hartkopf proposed openstack/cinder master: Ceph: Add option to keep only last n snapshots per backup  https://review.opendev.org/c/openstack/cinder/+/81045712:54
rosmaitamnasiadka: i think whoami-rajat has a fix posted13:31
rosmaitamnasiadka: https://review.opendev.org/c/openstack/cinder/+/90857813:32
whoami-rajatmnasiadka, rosmaita yes, here is the fix: https://review.opendev.org/c/openstack/cinder/+/90857813:32
whoami-rajatand here is the DNM patch to verify it: https://review.opendev.org/c/openstack/kolla-ansible/+/90858113:32
whoami-rajatcurrently working on the suggestions13:33
mnasiadkanice, thanks!13:33
*** abishop_ is now known as abishop15:11
simondodsleyCores- can we get some reviews for this backport from geguileo? https://review.opendev.org/c/openstack/os-brick/+/905127. Need to evntually get this back to Zed for a customer.15:18
jbernardsimondodsley: ^ it looks sound to me, had a small question about .216:01
simondodsleywhat question?16:02
jbernardjust wether it should be first proposed to 2023.2 before .116:05
jbernardrosmaita: ^16:06
rosmaitalooks like it merged to master when master was 2023.2 development16:07
jbernardahh, indeed you're right16:09
simondodsleyso jbernard and rosmaita can you approve this backport please?16:21
rosmaitai'm looking at it, but would be more comfortable with one of the original approvers doing it16:23
simondodsleyso that would be hemna: and whoami-rajat: ^16:24
opendevreviewRajat Dhasmana proposed openstack/os-brick master: Fix: FC partial target scan  https://review.opendev.org/c/openstack/os-brick/+/90674316:44
whoami-rajatsimondodsley, rosmaita done16:50
whoami-rajatgeguileo, rosmaita ^ updated the fc scanning patch with given comments16:51
rosmaitathanks for both, will take a look now16:51
simondodsleyAwesome - thank you rosmaita whoami-rajat. When merged I'll raise the zed backport16:51
whoami-rajatrosmaita, thanks, I've tested with pep8 and py310 locally but would be good to see gate results16:51
rosmaitaok, you just gave me an excuse to wait to look until after the tc meeting!16:52
whoami-rajatI'm always skeptical if gate is doing something differently but it should pass16:53
whoami-rajatI will log off, thanks for the review16:53
opendevreviewMerged openstack/os-brick stable/2023.1: SCSI: Support non SAM LUN addressing  https://review.opendev.org/c/openstack/os-brick/+/90512718:53
opendevreviewSimon Dodsley proposed openstack/os-brick stable/zed: SCSI: Support non SAM LUN addressing  https://review.opendev.org/c/openstack/os-brick/+/90892318:54
opendevreviewMerged openstack/cinder master: Recognize Dell PowerMax Unisphere 10.x (x>0)  https://review.opendev.org/c/openstack/cinder/+/90011819:47
simondodsleyhemna: whoami-rajat: rosmaita: could you look at https://review.opendev.org/c/openstack/os-brick/+/908923 to see if you are OK to backport the os-brick fix to Zed?20:47
simondodsleyrosmaita: what are the timescales for cutting new os-brick builds based on stable branches?20:52
rosmaitait can be done pretty quickly ... just need to propose a release patch, and then bug the release team20:55
rosmaitabut i think jbernard is getting some release patches together right now20:56
rosmaitadoesn't look like any are proposed yet, maybe he's waiting until after the midcycle20:57

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!