| *** osmanlicilegi is now known as Guest0 | 01:48 | |
| opendevreview | Merged openstack/cinder master: Address G004 in flake8-logging-format 0.7.4 https://review.opendev.org/c/openstack/cinder/+/852461 | 04:46 |
|---|---|---|
| opendevreview | chenwei proposed openstack/cinder master: Remove unicode literal from code https://review.opendev.org/c/openstack/cinder/+/852514 | 06:18 |
| *** pots5 is now known as pots | 08:33 | |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding support for volumegroup https://review.opendev.org/c/openstack/cinder/+/852364 | 09:42 |
| opendevreview | chenwei proposed openstack/cinder master: Remove unicode literal from code https://review.opendev.org/c/openstack/cinder/+/852514 | 10:32 |
| opendevreview | Merged openstack/cinder stable/xena: Pin flake8-logging-format to 0.6.0 https://review.opendev.org/c/openstack/cinder/+/852458 | 10:32 |
| *** tosky is now known as Guest71 | 10:46 | |
| *** tosky_ is now known as tosky | 10:46 | |
| *** dviroel|out is now known as dviroel | 11:32 | |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver consistency groups feature https://review.opendev.org/c/openstack/cinder/+/852369 | 11:37 |
| *** bhagyashris is now known as bhagyashris|afk | 11:44 | |
| *** tosky is now known as Guest77 | 12:01 | |
| *** tosky__ is now known as tosky | 12:01 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/victoria: DNM: checking flake8-logging-format==0.7.4 damage https://review.opendev.org/c/openstack/cinder/+/852565 | 12:06 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/ussuri: DNM: checking flake8-logging-format==0.7.4 damage https://review.opendev.org/c/openstack/cinder/+/852566 | 12:09 |
| jhartkopf | Hey there, I am currently working on a new integration test in cinder-tempest-plugin for this proposed optimization in the Ceph driver (https://review.opendev.org/c/openstack/cinder/+/810457). We talked about this in the Cinder meeting two weeks ago. | 12:22 |
| jhartkopf | Currently my plan is to add these two tests: | 12:22 |
| jhartkopf | 1. Set option to keep all snapshots. Create volume with 3 backups. Verify that all backups are present at the end. | 12:22 |
| jhartkopf | 2. Set option to keep only 1 snapshot. Create volume with 3 backups. Verify that only the latest backup is present at the end. | 12:23 |
| jhartkopf | However, how could we check this via Tempest? As far as I am aware, only Ceph knows how many source snapshots are actually stored, while Cinder returns all backups. | 12:23 |
| *** tosky is now known as Guest78 | 12:30 | |
| *** tosky_ is now known as tosky | 12:30 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/xena: Unpin flake8-logging-format https://review.opendev.org/c/openstack/cinder/+/852570 | 12:33 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding support for volumegroup https://review.opendev.org/c/openstack/cinder/+/852364 | 12:49 |
| *** bhagyashris|afk is now known as bhagyashris | 12:50 | |
| enriquetaso | jhartkopf, i'm not a tempest expert but I'm interesting in this. | 12:58 |
| enriquetaso | From #810457 commit message "This change also creates a new special case which was not present before: If a user deletes at least n backup snapshots, the next incremental snapshot creation would fail (due to how RBD works). This will be handled by forcing a full backup instead." | 12:59 |
| enriquetaso | I think you could test that with tempest [[ | 12:59 |
| *** tosky is now known as Guest85 | 13:13 | |
| *** tosky__ is now known as tosky | 13:13 | |
| jhartkopf | enriquetaso: You're right, that would definitely be another case to test. | 13:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: cinder/volume/flows/api/manage_existing https://review.opendev.org/c/openstack/cinder/+/852581 | 13:20 |
| opendevreview | Walt proposed openstack/cinder master: Fix and unify capacity calculations https://review.opendev.org/c/openstack/cinder/+/831247 | 13:24 |
| opendevreview | Walt proposed openstack/cinder master: Bugfix: Account for consumed space better https://review.opendev.org/c/openstack/cinder/+/826510 | 13:24 |
| opendevreview | Walt proposed openstack/cinder master: Fix tracking allocated_capacity_gb https://review.opendev.org/c/openstack/cinder/+/836083 | 13:26 |
| opendevreview | Merged openstack/cinder stable/xena: SF: Remove compat clone image code https://review.opendev.org/c/openstack/cinder/+/848249 | 14:21 |
| geguileo | eharney: whoami-rajat when you have a minute... https://review.opendev.org/c/openstack/cinder/+/849890 | 14:28 |
| geguileo | you already reviewed that patch before | 14:28 |
| whoami-rajat | geguileo, I've rerun the CI multiple times but not able to get any passing result ... | 14:46 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/victoria: Address G001 in flake8-logging-format 0.7.4 https://review.opendev.org/c/openstack/cinder/+/852595 | 14:54 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/victoria: Add virtualenv requirement to tox.ini https://review.opendev.org/c/openstack/cinder/+/852596 | 14:55 |
| jungleboyj | rosmaita: whoami-rajat Hey guys, I have had a family commitment come up tomorrow that conflicts with the Mid-cycle. Apologies. | 15:01 |
| rosmaita | jungleboyj: ack, and hope it's nothing serious | 15:01 |
| jungleboyj | Nothing too serious. | 15:01 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/ussuri: Address G001 in flake8-logging-format 0.7.4 https://review.opendev.org/c/openstack/cinder/+/852600 | 15:08 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: work around mypy bug #13214 https://review.opendev.org/c/openstack/os-brick/+/852610 | 15:13 |
| *** sfinucan is now known as stephenfin | 15:16 | |
| whoami-rajat | jungleboyj, ack, no worries | 15:22 |
| *** dviroel is now known as dviroel|lunch | 15:39 | |
| rosmaita | whoami-rajat: i finally got the flake8-logging-format thing figured out | 15:39 |
| rosmaita | i think what we should do is | 15:39 |
| rosmaita | merge https://review.opendev.org/c/openstack/cinder/+/852570 into xena | 15:39 |
| rosmaita | wallaby has no problem | 15:39 |
| rosmaita | merge https://review.opendev.org/c/openstack/cinder/+/852595 into victoria | 15:39 |
| rosmaita | and probably also merge https://review.opendev.org/c/openstack/cinder/+/852596 into victoria | 15:39 |
| rosmaita | (the gate doesn't seem to need it, but I can't run pep8 locally in victoria without it) | 15:39 |
| rosmaita | and then for ussuri, merge https://review.opendev.org/c/openstack/cinder/+/852600 | 15:40 |
| rosmaita | train and older don't use flake8-logging-format, so no other changes needed | 15:40 |
| whoami-rajat | rosmaita, looks like we should look more carefully into drivers, all fixes are there only! | 15:46 |
| whoami-rajat | and thanks for working on this, i will take a look at the proposed patches | 15:46 |
| rosmaita | cool | 15:48 |
| whoami-rajat | rosmaita, don't we have u-c or some sort of version limit for flake8-logging-format package for every stable branch? | 15:55 |
| rosmaita | no, no u-c on test requirements | 15:57 |
| whoami-rajat | ok, bummer that for any backward incompatible change, we've to deal with it in all stable branches ... | 15:59 |
| rosmaita | yeah, we can cap it ourselves in test-requirements, but i figure since they were small fixes, it's better to have them all be able to handle the same version | 16:01 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Update volume delete api-ref https://review.opendev.org/c/openstack/cinder/+/826608 | 16:22 |
| whoami-rajat | yeah, i was thinking more in general way but this one looks addressable | 16:33 |
| *** dviroel|lunch is now known as dviroel | 16:51 | |
| hemna | yough | 17:19 |
| opendevreview | Pete Zaitcev proposed openstack/cinder master: WIP: Restore into sparse volumes https://review.opendev.org/c/openstack/cinder/+/852654 | 18:06 |
| *** lifeless_ is now known as lifeless | 18:24 | |
| *** dviroel is now known as dviroel|biab | 19:34 | |
| *** dviroel|biab is now known as dviroel | 20:13 | |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Add flake8-logging-format extension https://review.opendev.org/c/openstack/os-brick/+/852672 | 20:25 |
| opendevreview | Alan Bishop proposed openstack/python-cinderclient master: Bump API max version to 3.70 https://review.opendev.org/c/openstack/python-cinderclient/+/852674 | 20:54 |
| opendevreview | Alan Bishop proposed openstack/cinder-tempest-plugin master: WIP: Test ability to transfer encrypted volumes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/852675 | 21:03 |
| *** dviroel is now known as dviroel|out | 21:17 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!