| opendevreview | Merged openstack/os-brick master: encryptors: Unbind LuksEncryptor and CryptsetupEncryptor https://review.opendev.org/c/openstack/os-brick/+/791271 | 00:04 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from ProphetStor driver https://review.opendev.org/c/openstack/cinder/+/907390 | 00:54 |
| opendevreview | Merged openstack/cinder master: Remove six from Zadara driver https://review.opendev.org/c/openstack/cinder/+/907387 | 01:08 |
| opendevreview | Merged openstack/cinder master: Remove six from Hedvig driver https://review.opendev.org/c/openstack/cinder/+/907418 | 01:32 |
| opendevreview | Merged openstack/cinder master: Remove six from Fujitsu drivers https://review.opendev.org/c/openstack/cinder/+/907373 | 01:34 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/2023.1: Skip sparse copy during volume reimage https://review.opendev.org/c/openstack/cinder/+/908808 | 05:31 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907770 | 06:17 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM - Do not merge - For test only https://review.opendev.org/c/openstack/cinder/+/907771 | 06:24 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907772 | 07:16 |
| opendevreview | Merged openstack/cinder master: Remove six from test codes https://review.opendev.org/c/openstack/cinder/+/907384 | 07:57 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907773 | 08:04 |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907768 | 08:48 |
| *** tosky_ is now known as tosky | 09:48 | |
| opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax: Added timeout into rest API call. https://review.opendev.org/c/openstack/cinder/+/907768 | 10:56 |
| opendevreview | Andrei proposed openstack/cinder master: JovianDSS: Rework Open-E JovianDSS driver https://review.opendev.org/c/openstack/cinder/+/889284 | 11:03 |
| mnasiadka | rosmaita: 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/upgrade | 11:42 |
| opendevreview | Jan Hartkopf proposed openstack/cinder master: Ceph: Add option to keep only last n snapshots per backup https://review.opendev.org/c/openstack/cinder/+/810457 | 12:54 |
| rosmaita | mnasiadka: i think whoami-rajat has a fix posted | 13:31 |
| rosmaita | mnasiadka: https://review.opendev.org/c/openstack/cinder/+/908578 | 13:32 |
| whoami-rajat | mnasiadka, rosmaita yes, here is the fix: https://review.opendev.org/c/openstack/cinder/+/908578 | 13:32 |
| whoami-rajat | and here is the DNM patch to verify it: https://review.opendev.org/c/openstack/kolla-ansible/+/908581 | 13:32 |
| whoami-rajat | currently working on the suggestions | 13:33 |
| mnasiadka | nice, thanks! | 13:33 |
| *** abishop_ is now known as abishop | 15:11 | |
| simondodsley | Cores- 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 |
| jbernard | simondodsley: ^ it looks sound to me, had a small question about .2 | 16:01 |
| simondodsley | what question? | 16:02 |
| jbernard | just wether it should be first proposed to 2023.2 before .1 | 16:05 |
| jbernard | rosmaita: ^ | 16:06 |
| rosmaita | looking | 16:06 |
| rosmaita | looks like it merged to master when master was 2023.2 development | 16:07 |
| simondodsley | yup | 16:08 |
| jbernard | ahh, indeed you're right | 16:09 |
| simondodsley | so jbernard and rosmaita can you approve this backport please? | 16:21 |
| rosmaita | i'm looking at it, but would be more comfortable with one of the original approvers doing it | 16:23 |
| simondodsley | so that would be hemna: and whoami-rajat: ^ | 16:24 |
| opendevreview | Rajat Dhasmana proposed openstack/os-brick master: Fix: FC partial target scan https://review.opendev.org/c/openstack/os-brick/+/906743 | 16:44 |
| whoami-rajat | simondodsley, rosmaita done | 16:50 |
| whoami-rajat | geguileo, rosmaita ^ updated the fc scanning patch with given comments | 16:51 |
| rosmaita | thanks for both, will take a look now | 16:51 |
| simondodsley | Awesome - thank you rosmaita whoami-rajat. When merged I'll raise the zed backport | 16:51 |
| whoami-rajat | rosmaita, thanks, I've tested with pep8 and py310 locally but would be good to see gate results | 16:51 |
| rosmaita | ok, you just gave me an excuse to wait to look until after the tc meeting! | 16:52 |
| whoami-rajat | I'm always skeptical if gate is doing something differently but it should pass | 16:53 |
| whoami-rajat | I will log off, thanks for the review | 16:53 |
| opendevreview | Merged openstack/os-brick stable/2023.1: SCSI: Support non SAM LUN addressing https://review.opendev.org/c/openstack/os-brick/+/905127 | 18:53 |
| opendevreview | Simon Dodsley proposed openstack/os-brick stable/zed: SCSI: Support non SAM LUN addressing https://review.opendev.org/c/openstack/os-brick/+/908923 | 18:54 |
| opendevreview | Merged openstack/cinder master: Recognize Dell PowerMax Unisphere 10.x (x>0) https://review.opendev.org/c/openstack/cinder/+/900118 | 19:47 |
| simondodsley | hemna: 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 |
| rosmaita | ack | 20:48 |
| simondodsley | rosmaita: what are the timescales for cutting new os-brick builds based on stable branches? | 20:52 |
| rosmaita | it can be done pretty quickly ... just need to propose a release patch, and then bug the release team | 20:55 |
| rosmaita | but i think jbernard is getting some release patches together right now | 20:56 |
| rosmaita | doesn't look like any are proposed yet, maybe he's waiting until after the midcycle | 20:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!