| whoami-rajat | thanks rosmaita , LGTM | 04:48 |
|---|---|---|
| gksk | Hi Team | 04:56 |
| gksk | Can someone pls review the below patch | 04:57 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/869704 | 04:57 |
| opendevreview | Merged openstack/cinder master: Require tooz>=2.8.0 https://review.opendev.org/c/openstack/cinder/+/873435 | 05:14 |
| gksk | Can some one review the below patch https://review.opendev.org/c/openstack/cinder/+/869704 | 05:17 |
| Sathya | Hi Reviewers, Can I get some review on the patch https://review.opendev.org/c/openstack/cinder/+/869302 | 06:45 |
| Mounika | Hi Reviewers, Requesting to review the documentation patch https://review.opendev.org/c/openstack/cinder/+/873083 | 07:09 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: Doc: Branding change for Spectrum Virtulize family https://review.opendev.org/c/openstack/cinder/+/873083 | 08:18 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: Doc: Branding change for Spectrum Virtualize family https://review.opendev.org/c/openstack/cinder/+/873083 | 08:19 |
| opendevreview | Sathyanarayana R proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding config parameter to support volumegroup https://review.opendev.org/c/openstack/cinder/+/869302 | 08:22 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Update API returns https://review.opendev.org/c/openstack/cinder/+/856041 | 08:40 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding config parameter to support volumegroup https://review.opendev.org/c/openstack/cinder/+/869302 | 08:41 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add the clone_across_pools driver capability https://review.opendev.org/c/openstack/cinder/+/847131 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: declare the clone_across_pools capability https://review.opendev.org/c/openstack/cinder/+/847132 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop _attach_volume() and _detach_volume() https://review.opendev.org/c/openstack/cinder/+/787296 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool driver: remove the obsolete backup_volume() https://review.opendev.org/c/openstack/cinder/+/803823 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop copy_image_to_volume() and copy_volume_to_image() https://review.opendev.org/c/openstack/cinder/+/870550 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the "rename volume" unit test emulation https://review.opendev.org/c/openstack/cinder/+/870681 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the retype volume flow https://review.opendev.org/c/openstack/cinder/+/870551 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 09:20 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: cosmetic: comment headings instead of empty lines https://review.opendev.org/c/openstack/cinder/+/873507 | 09:20 |
| roquej | Hi team, I still have a few patches waiting for your final review, can you take a look at them ? https://etherpad.opendev.org/p/cinder-antelope-meetings#L121 | 10:51 |
| roquej | especially this one https://review.opendev.org/c/openstack/cinder/+/844563 | 11:19 |
| *** priteau_ is now known as priteau | 12:25 | |
| opendevreview | Konrad Gube proposed openstack/cinder master: Add the os-extend_volume_completion volume action https://review.opendev.org/c/openstack/cinder/+/873557 | 12:38 |
| opendevreview | Konrad Gube proposed openstack/python-cinderclient master: Add os-extend_volume_completion volume action. https://review.opendev.org/c/openstack/python-cinderclient/+/873558 | 12:45 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Fix NFS cloned volume https://review.opendev.org/c/openstack/cinder/+/872052 | 13:20 |
| opendevreview | Rajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Update minimum requirements https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/873595 | 15:41 |
| whoami-rajat | rosmaita, hey, can you take a look at brick-cinderclient-ext requirements bump for the release this week (there are only 3 requirement change) ^ | 15:43 |
| rosmaita | sure | 15:43 |
| whoami-rajat | thanks | 15:43 |
| whoami-rajat | looking into cinderclient requirements as well | 15:43 |
| rosmaita | ok, let me know when you have a patch up | 15:44 |
| whoami-rajat | sure | 15:44 |
| roquej | Hey team, don't want to hassle, did you see my messages with the patches for which I request a review? Don't want to miss our antelope release | 15:46 |
| whoami-rajat | roquej, hi, surely we will try to get it in along with other features | 16:02 |
| ganso | whoami-rajat, jungleboyj, rosmaita: hi! Could you please review this backport that is required to fix the stable/yoga branch? I had a +2 before but needed to be rebased: https://review.opendev.org/c/openstack/cinder/+/868444 thanks in advance! | 16:04 |
| jungleboyj | ganso: Looking. | 16:05 |
| roquej | thanks whoami-rajat | 16:05 |
| whoami-rajat | ganso, the zed patch isn't merged yet so we need to review that first | 16:06 |
| ganso | whoami-rajat: oh I see, I thought the zed one had merged. Looking at it now, it is passing CI, so that looks good to go to unblock the rest of the backports | 16:07 |
| whoami-rajat | rosmaita, is this change still the way to go? I remember you mentioning that this wasn't the actual fix and there was going to be further discussion ^ | 16:07 |
| whoami-rajat | roquej, np | 16:07 |
| whoami-rajat | rosmaita, it's regarding pinning tox < 4 | 16:07 |
| jungleboyj | I am +2 on the Zed one as well. | 16:08 |
| simondodsley | Looking for some guidance. Getting the following error `Exception during message handling: nova.exception.Forbidden: Policy doesn't allow volume_extension:volume_admin_actions:migrate_volume_completion to be performed. (HTTP 403)` | 16:16 |
| opendevreview | Rajat Dhasmana proposed openstack/python-cinderclient master: Update minimum requirements https://review.opendev.org/c/openstack/python-cinderclient/+/873604 | 16:16 |
| simondodsley | this is during a volume migration using `on-demand` | 16:17 |
| simondodsley | any idea why this might be failing? | 16:17 |
| rosmaita | whoami-rajat: i think the further discussion was that the pinning tox patches are ok as-is | 16:21 |
| rosmaita | simondodsley: which release? | 16:21 |
| simondodsley | I think this is Xena - it's happening in PowerVC latest version | 16:21 |
| whoami-rajat | rosmaita, ack, thanks for confirming | 16:21 |
| whoami-rajat | rosmaita, also this is the cinderclient requirement update patch (only 1 req change) https://review.opendev.org/c/openstack/python-cinderclient/+/873604 | 16:22 |
| rosmaita | whoami-rajat: ack | 16:22 |
| rosmaita | simondodsley: that action has to be done by a user with the 'admin' role | 16:31 |
| rosmaita | api-ref says about migration_policy=on_demand , "If the volume is attached to a server instance and will be migrated, then by default policy only users with the administrative role should attempt the retype operation. " | 16:31 |
| simondodsley | rosmaita: thanks - I'll check that out. | 16:31 |
| whoami-rajat | Hi cinder team, please check my mail regarding review priorities https://lists.openstack.org/pipermail/openstack-discuss/2023-February/032179.html | 16:40 |
| simondodsley | whoami-rajat: I'd like https://review.opendev.org/c/openstack/cinder/+/871261 to be added to the features for antelope please | 16:54 |
| ganso | whoami-rajat, rosmaita: then I guess we are green to +2+W on the zed tox patch ? | 16:57 |
| rosmaita | ganso: i think so, but I can't do it because it's my patch | 16:59 |
| whoami-rajat | simondodsley, sure, please add it to the list | 17:01 |
| simondodsley | whoami-rajat: oops - my bad - already in the list | 17:02 |
| whoami-rajat | np | 17:02 |
| whoami-rajat | ganso, done | 17:02 |
| ganso | whoami-rajat: thanks! | 17:03 |
| *** jgwentworth is now known as melwitt | 18:33 | |
| opendevreview | Merged openstack/python-cinderclient master: Add support for collect-timing option https://review.opendev.org/c/openstack/python-cinderclient/+/828380 | 18:53 |
| opendevreview | Rajat Dhasmana proposed openstack/python-brick-cinderclient-ext master: Fix: tox4 issues https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/873635 | 19:04 |
| whoami-rajat | rosmaita, jungleboyj turns out we didn't do the tox4 changes in brick-cinderclient ext and gate is failing ^ | 19:04 |
| jungleboyj | whoami-rajat: Ok. Will give it my vote when it passes the check. | 19:06 |
| whoami-rajat | jungleboyj, sure, thanks | 19:06 |
| whoami-rajat | though I've checked locally with pep8 and py38 runs, gate still sometimes report failures | 19:07 |
| opendevreview | Merged openstack/python-cinderclient master: Add test coverage for shell snapshot-create w/ metadata https://review.opendev.org/c/openstack/python-cinderclient/+/863299 | 19:26 |
| *** dviroel is now known as dviroel|out | 19:26 | |
| *** JohnnyW0 is now known as JohnnyW | 20:19 | |
| opendevreview | Merged openstack/cinder stable/zed: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/cinder/+/868429 | 21:34 |
| opendevreview | Merged openstack/cinder stable/yoga: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/cinder/+/868444 | 21:34 |
| opendevreview | Merged openstack/python-cinderclient master: Update minimum requirements https://review.opendev.org/c/openstack/python-cinderclient/+/873604 | 21:55 |
| opendevreview | Pete Zaitcev proposed openstack/cinder master: Restore into sparse volumes https://review.opendev.org/c/openstack/cinder/+/852654 | 22:36 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!