| opendevreview | Aneesh Pachilangottil proposed openstack/cinder master: NVMe-TCP volume driver for Fungible Storage https://review.opendev.org/c/openstack/cinder/+/849143 | 00:24 |
|---|---|---|
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Clean up formatting in dplcommon.py https://review.opendev.org/c/openstack/cinder/+/855359 | 03:18 |
| aneeeshp1 | Hi, I am seeing two failures (openstack-tox-py39 & openstack-tox-py310) from Zuul on the Fungible driver patch https://review.opendev.org/c/openstack/cinder/+/849143. But log shows failures are from other drivers (not from the Fungible driver). I am not able figure out the reason. Could you please help to understand the reason for the failure? | 04:09 |
| whoami-rajat | aneeeshp1, hi, we've recently started running randomized tests hence we're discovering new issues in driver tests, I will fix them | 05:26 |
| whoami-rajat | aneeeshp1, looks like there are some issues with your tests, you're mocking some of the cinder methods in a wrong way | 06:36 |
| whoami-rajat | rosmaita, hey, can you take a look at this small backport? https://review.opendev.org/c/openstack/cinder/+/858624 | 11:35 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Test the ability to transfer encrypted volumes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/852675 | 11:47 |
| rosmaita | whoami-rajat: done | 11:53 |
| whoami-rajat | thanks | 11:53 |
| whoami-rajat | trying to finish up the list https://etherpad.opendev.org/p/cinder-zed-rc2-bugfix | 11:54 |
| whoami-rajat | rosmaita, I'm updating the tempest plugin release patch (since the encrypted vol transfer test is merged), did we have anything else? | 11:55 |
| whoami-rajat | the rbac tests are in merge conflict so won't be able to make it | 11:55 |
| rosmaita | i think those were the only things that came up yesterday | 11:55 |
| rosmaita | it's not a big deal for the plugin really since don't branch | 11:56 |
| whoami-rajat | cool, will update it | 11:56 |
| rosmaita | (about the rbac tests, i mean) | 11:56 |
| whoami-rajat | ack, will try to get them in after the release | 11:56 |
| HappyStacker | Good morning team. I have two questions for you. 1. How do you compare two branches (for example Zed vs Yoga) to list all patches not part of Yoga? 2. Does anyone of you use Pycharm with Gerrit? | 12:14 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Add UTs to improve REST client coverage https://review.opendev.org/c/openstack/cinder/+/857434 | 13:02 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Add UTs to improve REST client coverage https://review.opendev.org/c/openstack/cinder/+/857434 | 13:04 |
| abishop | whoami-rajat, rosmaita: I think you'll want this one in zed - https://review.opendev.org/c/openstack/cinder/+/855307 | 13:09 |
| whoami-rajat | abishop, ack, added to https://etherpad.opendev.org/p/cinder-zed-rc2-bugfix | 13:11 |
| rosmaita | abishop: i think you need to separate those 2 changes. The release note change should go directly to stable/zed; the api-ref can go directly to master and be backported | 13:12 |
| abishop | rosmaita: ack, can do (I assume you'll respond the same in the patch) | 13:13 |
| rosmaita | i will now! | 13:13 |
| rosmaita | abishop: done | 13:18 |
| abishop | HappyStacker: I can help with 1. assuming your current branch is stable/yoga and you want to focus on changes to a particular driver, try "git log --no-merges HEAD..master -- cinder/volume/drivers/<etc>" | 13:19 |
| abishop | there are many similar alternatives | 13:19 |
| HappyStacker | OK, thank you abishop | 13:21 |
| sfv880_ | whoami-rajat, please tell me - what are the requirements for fixes to be added to the https://etherpad.opendev.org/p/cinder-zed-rc2-bugfix ? Thank you! | 13:49 |
| opendevreview | Alan Bishop proposed openstack/cinder stable/zed: Clarify encrypted volume transfer release note https://review.opendev.org/c/openstack/cinder/+/859094 | 14:11 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Clarify description of encrypted volume transfer mv https://review.opendev.org/c/openstack/cinder/+/855307 | 14:12 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder/+/855013 | 14:14 |
| TusharTgite | rosmaita: update your comment take a look https://review.opendev.org/c/openstack/cinder/+/855359 | 14:14 |
| rosmaita | TusharTgite: ack | 14:15 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Add support for Unisphere for PowerMax 10.0 https://review.opendev.org/c/openstack/cinder/+/844563 | 14:17 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/854989 | 14:29 |
| opendevreview | Tushar Trambak Gite proposed openstack/os-brick master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/os-brick/+/855012 | 14:29 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder/+/855013 | 14:30 |
| opendevreview | Tushar Trambak Gite proposed openstack/python-brick-cinderclient-ext master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/854988 | 14:31 |
| opendevreview | Merged openstack/cinder stable/zed: rbd: Fix snapshot delete when the source volume doesn't exist https://review.opendev.org/c/openstack/cinder/+/858624 | 15:32 |
| opendevreview | Merged openstack/cinder stable/zed: Clarify encrypted volume transfer release note https://review.opendev.org/c/openstack/cinder/+/859094 | 18:49 |
| opendevreview | Merged openstack/cinder master: Clarify description of encrypted volume transfer mv https://review.opendev.org/c/openstack/cinder/+/855307 | 18:49 |
| opendevreview | Alan Bishop proposed openstack/cinder stable/zed: Clarify description of encrypted volume transfer mv https://review.opendev.org/c/openstack/cinder/+/858909 | 18:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!