| *** ianw is now known as ianw_pto | 00:00 | |
| *** hoonetorg has quit IRC | 00:04 | |
| *** rcernin_ has quit IRC | 00:07 | |
| *** hoonetorg has joined #openstack-cinder | 00:22 | |
| *** rcernin_ has joined #openstack-cinder | 00:29 | |
| *** rcernin_ has quit IRC | 00:35 | |
| *** ociuhandu has joined #openstack-cinder | 01:02 | |
| *** rcernin_ has joined #openstack-cinder | 01:06 | |
| *** ociuhandu has quit IRC | 01:07 | |
| *** rcernin_ has quit IRC | 01:11 | |
| *** ociuhandu has joined #openstack-cinder | 01:21 | |
| *** sapd1 has joined #openstack-cinder | 01:23 | |
| *** ociuhandu has quit IRC | 01:25 | |
| *** sapd1 has quit IRC | 01:28 | |
| *** ociuhandu has joined #openstack-cinder | 01:32 | |
| *** ociuhandu has quit IRC | 01:36 | |
| *** rcernin_ has joined #openstack-cinder | 01:42 | |
| *** rcernin_ has quit IRC | 01:47 | |
| *** arne_wiebalck has quit IRC | 01:51 | |
| *** jopdorp has quit IRC | 01:51 | |
| *** simondodsley has quit IRC | 01:51 | |
| *** almir-okato has quit IRC | 01:52 | |
| *** almir-okato has joined #openstack-cinder | 01:56 | |
| *** jopdorp has joined #openstack-cinder | 01:56 | |
| *** arne_wiebalck has joined #openstack-cinder | 01:56 | |
| *** simondodsley has joined #openstack-cinder | 01:56 | |
| *** zenkuro has quit IRC | 02:47 | |
| *** ociuhandu has joined #openstack-cinder | 02:53 | |
| *** ociuhandu has quit IRC | 02:58 | |
| *** rcernin_ has joined #openstack-cinder | 03:00 | |
| *** sapd1 has joined #openstack-cinder | 03:03 | |
| *** rcernin_ has quit IRC | 03:04 | |
| *** baojg has joined #openstack-cinder | 03:09 | |
| openstackgerrit | zhangboye proposed openstack/cinderlib master: Update master for stable/wallaby https://review.opendev.org/c/openstack/cinderlib/+/784721 | 03:15 |
|---|---|---|
| *** rcernin_ has joined #openstack-cinder | 03:20 | |
| *** rcernin_ has quit IRC | 03:26 | |
| *** psachin has joined #openstack-cinder | 03:32 | |
| *** ykarel has joined #openstack-cinder | 04:00 | |
| *** rcernin_ has joined #openstack-cinder | 04:10 | |
| *** rcernin_ has quit IRC | 04:15 | |
| *** rcernin_ has joined #openstack-cinder | 04:24 | |
| *** m75abrams has joined #openstack-cinder | 04:25 | |
| *** GirishChilukuri has joined #openstack-cinder | 04:29 | |
| *** rcernin_ has quit IRC | 04:29 | |
| *** benj_ has quit IRC | 04:37 | |
| *** benj_ has joined #openstack-cinder | 04:38 | |
| *** logan- has quit IRC | 04:39 | |
| *** ociuhandu has joined #openstack-cinder | 04:42 | |
| *** logan- has joined #openstack-cinder | 04:42 | |
| *** ociuhandu has quit IRC | 04:47 | |
| *** rcernin_ has joined #openstack-cinder | 04:55 | |
| *** rcernin_ has quit IRC | 05:12 | |
| *** udesale has joined #openstack-cinder | 05:15 | |
| *** whoami-rajat_ has joined #openstack-cinder | 05:47 | |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Quota: Fix until_refresh config changes https://review.opendev.org/c/openstack/cinder/+/784587 | 06:02 |
| *** baojg has quit IRC | 06:06 | |
| *** baojg has joined #openstack-cinder | 06:07 | |
| *** baojg has quit IRC | 06:07 | |
| *** baojg has joined #openstack-cinder | 06:07 | |
| *** baojg has quit IRC | 06:08 | |
| *** baojg has joined #openstack-cinder | 06:08 | |
| *** baojg has quit IRC | 06:08 | |
| *** baojg has joined #openstack-cinder | 06:09 | |
| *** baojg has quit IRC | 06:09 | |
| *** baojg has joined #openstack-cinder | 06:10 | |
| *** baojg has quit IRC | 06:10 | |
| *** baojg has joined #openstack-cinder | 06:10 | |
| *** baojg has quit IRC | 06:11 | |
| *** baojg has joined #openstack-cinder | 06:11 | |
| *** baojg has quit IRC | 06:12 | |
| *** baojg has joined #openstack-cinder | 06:12 | |
| *** baojg has quit IRC | 06:12 | |
| *** baojg has joined #openstack-cinder | 06:13 | |
| *** baojg has quit IRC | 06:13 | |
| *** baojg has joined #openstack-cinder | 06:13 | |
| *** baojg has quit IRC | 06:14 | |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Quota: Fix until_refresh config changes https://review.opendev.org/c/openstack/cinder/+/784587 | 06:17 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Remove nested quota leftovers https://review.opendev.org/c/openstack/cinder/+/784588 | 06:20 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Fix quota usage duplicate entries https://review.opendev.org/c/openstack/cinder/+/784589 | 06:21 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Update cinder manage quota commands https://review.opendev.org/c/openstack/cinder/+/784590 | 06:22 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Prevent quota and reservations to go into negative https://review.opendev.org/c/openstack/cinder/+/784591 | 06:24 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Quota: Fix multiple race conditions https://review.opendev.org/c/openstack/cinder/+/784592 | 06:26 |
| *** GirishChilukuri has quit IRC | 06:36 | |
| *** GirishChilukuri has joined #openstack-cinder | 06:40 | |
| *** sfernand has quit IRC | 06:42 | |
| *** almir-okato has quit IRC | 07:40 | |
| openstackgerrit | Andrey Bolgov proposed openstack/cinder stable/train: Add support for RBD fast-diff feature for backups stored in Ceph https://review.opendev.org/c/openstack/cinder/+/784747 | 07:51 |
| openstackgerrit | Andrey Bolgov proposed openstack/cinder stable/train: Add support for RBD fast-diff feature for backups stored in Ceph https://review.opendev.org/c/openstack/cinder/+/784041 | 07:59 |
| *** ociuhandu has joined #openstack-cinder | 08:54 | |
| *** ociuhandu has quit IRC | 09:01 | |
| *** ociuhandu_ has joined #openstack-cinder | 09:02 | |
| *** ociuhandu_ has quit IRC | 09:11 | |
| *** ociuhandu has joined #openstack-cinder | 09:11 | |
| *** ociuhandu has quit IRC | 09:16 | |
| *** sapd1 has quit IRC | 09:24 | |
| *** sapd1 has joined #openstack-cinder | 09:37 | |
| *** ociuhandu has joined #openstack-cinder | 09:52 | |
| *** ociuhandu has quit IRC | 09:53 | |
| *** ociuhandu has joined #openstack-cinder | 09:53 | |
| *** ociuhandu has quit IRC | 09:55 | |
| *** ociuhandu has joined #openstack-cinder | 09:56 | |
| *** martinkennelly has quit IRC | 10:02 | |
| *** rcernin_ has joined #openstack-cinder | 10:09 | |
| *** rcernin_ has quit IRC | 10:24 | |
| *** ykarel_ has joined #openstack-cinder | 10:32 | |
| *** ykarel has quit IRC | 10:34 | |
| *** ykarel_ is now known as ykarel | 10:34 | |
| *** e0ne has quit IRC | 10:37 | |
| *** lpetrut has joined #openstack-cinder | 10:38 | |
| *** ociuhandu has quit IRC | 10:40 | |
| *** lpetrut has quit IRC | 10:51 | |
| *** zenkuro has joined #openstack-cinder | 11:01 | |
| *** ociuhandu has joined #openstack-cinder | 11:11 | |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Add project validation in quotas https://review.opendev.org/c/openstack/cinder/+/784763 | 11:19 |
| *** ociuhandu has quit IRC | 11:20 | |
| whoami-rajat_ | geguileo: ^^ this adds the project validation for quotas | 11:20 |
| *** e0ne has joined #openstack-cinder | 11:28 | |
| *** tosky has joined #openstack-cinder | 11:29 | |
| *** carloss has joined #openstack-cinder | 11:30 | |
| geguileo | whoami-rajat_: we have an issue... | 11:33 |
| whoami-rajat_ | geguileo: oh, what is it? | 11:33 |
| geguileo | whoami-rajat_: what about the "garbage" we can have in the DB | 11:33 |
| geguileo | ? | 11:33 |
| geguileo | if we just make that change it means that we won't allow things to be cleaned up | 11:33 |
| whoami-rajat_ | hmm, true, didn't think about it | 11:33 |
| *** ociuhandu has joined #openstack-cinder | 11:34 | |
| whoami-rajat_ | geguileo: one thing we can do is to avoid the validation check on delete | 11:34 |
| geguileo | whoami-rajat_: but if you cannot show, how do you know if they are in the DB in the first place? | 11:35 |
| geguileo | mmmm, what if we added to the cinder-manage quota command one to do the cleaning up? | 11:35 |
| geguileo | because I really like the patch code | 11:36 |
| geguileo | and I'm not sure I would want us to add weird code around it to allow showing and deleting existing garbage | 11:36 |
| *** sapd1 has quit IRC | 11:37 | |
| whoami-rajat_ | ok, i will try to write a working manage command to clean the non existing project records | 11:38 |
| whoami-rajat_ | geguileo: should that be in the same patch or a different one? | 11:38 |
| geguileo | whoami-rajat_: I would prefer it to be in the same patch, but I'm ok if it's in a different one | 11:39 |
| whoami-rajat_ | geguileo: ok, thanks for looking into this | 11:39 |
| geguileo | whoami-rajat_: thank you for fixing this annoyance!! | 11:40 |
| whoami-rajat_ | :D | 11:40 |
| geguileo | whoami-rajat_: I have also added to the review that you haven't added any tests for the new 404 behavior, you just updated existing tests for the new error that's returned | 11:45 |
| geguileo | by new error, I mean the replacement of the not authorized one | 11:46 |
| *** ociuhandu has quit IRC | 11:46 | |
| whoami-rajat_ | geguileo: hmm, so we already have tests for update,show, delete with non-admin and i am not sure what additional tests i can add... | 11:51 |
| geguileo | whoami-rajat_: mmmm, none of those raise 404 | 11:52 |
| whoami-rajat_ | oh sorry | 11:52 |
| geguileo | but I see the point, since we already have tests in the method we call for 404 | 11:52 |
| geguileo | there's no need to add them to the caller as well | 11:53 |
| *** eharney has joined #openstack-cinder | 11:55 | |
| geguileo | whoami-rajat_: mmmmm, I think it will be enough if you add a test that mocks the get_project_hierarchy method and confirms that it being called from _validate_project_and_authorize | 11:55 |
| geguileo | whoami-rajat_: and then make sure that _validate_project_and_authorize is being called from the delete, show, update methods | 11:56 |
| whoami-rajat_ | geguileo: that should be easy, will add a test | 11:58 |
| whoami-rajat_ | thanks | 11:58 |
| geguileo | np | 11:58 |
| *** ociuhandu has joined #openstack-cinder | 12:02 | |
| *** e0ne has quit IRC | 12:02 | |
| *** e0ne has joined #openstack-cinder | 12:02 | |
| *** e0ne has quit IRC | 12:02 | |
| *** enriquetaso has joined #openstack-cinder | 12:03 | |
| hemna | mornin | 12:04 |
| *** ociuhandu has quit IRC | 12:06 | |
| *** tosky has quit IRC | 12:13 | |
| *** tosky has joined #openstack-cinder | 12:19 | |
| *** ociuhandu has joined #openstack-cinder | 12:36 | |
| *** rosmaita has joined #openstack-cinder | 12:38 | |
| *** ociuhandu has quit IRC | 12:46 | |
| *** zenkuro has quit IRC | 12:51 | |
| *** zenkuro has joined #openstack-cinder | 12:52 | |
| *** sfernand has joined #openstack-cinder | 12:58 | |
| *** sapd1 has joined #openstack-cinder | 12:58 | |
| *** psachin has quit IRC | 13:06 | |
| *** ociuhandu has joined #openstack-cinder | 13:08 | |
| *** e0ne has joined #openstack-cinder | 13:09 | |
| *** lbragstad has joined #openstack-cinder | 13:10 | |
| *** hemna has quit IRC | 13:32 | |
| *** hemna has joined #openstack-cinder | 13:33 | |
| *** ociuhandu has quit IRC | 13:41 | |
| *** GirishChilukuri has quit IRC | 13:42 | |
| *** mailingsam has joined #openstack-cinder | 13:42 | |
| *** ociuhandu has joined #openstack-cinder | 13:42 | |
| *** lbragstad has quit IRC | 13:43 | |
| *** whoami-rajat_ is now known as whoami-rajat | 13:43 | |
| *** tosky has quit IRC | 13:43 | |
| *** tosky has joined #openstack-cinder | 13:45 | |
| *** lbragstad has joined #openstack-cinder | 13:47 | |
| *** ociuhandu has quit IRC | 13:48 | |
| *** ociuhandu has joined #openstack-cinder | 13:50 | |
| *** ociuhandu has quit IRC | 13:51 | |
| *** ociuhandu has joined #openstack-cinder | 13:51 | |
| *** tosky has quit IRC | 13:52 | |
| *** tosky has joined #openstack-cinder | 13:54 | |
| *** ociuhandu has quit IRC | 13:59 | |
| *** m75abrams has quit IRC | 14:07 | |
| *** rcernin_ has joined #openstack-cinder | 14:07 | |
| *** ociuhandu has joined #openstack-cinder | 14:13 | |
| *** m75abrams has joined #openstack-cinder | 14:16 | |
| *** ociuhandu has quit IRC | 14:18 | |
| *** rcernin_ has quit IRC | 14:21 | |
| *** e0ne has quit IRC | 14:42 | |
| *** _erlon_ has joined #openstack-cinder | 14:43 | |
| openstackgerrit | saikrishnagk proposed openstack/cinder master: [SVF] Fixed add volumes to clone group issue https://review.opendev.org/c/openstack/cinder/+/784779 | 14:50 |
| *** ociuhandu has joined #openstack-cinder | 14:52 | |
| *** dsariel has joined #openstack-cinder | 15:15 | |
| *** ircuser-1 has joined #openstack-cinder | 15:16 | |
| *** ykarel has quit IRC | 15:16 | |
| *** whoami-rajat has quit IRC | 15:46 | |
| openstackgerrit | katari manoj kumar proposed openstack/cinder master: [SVF]:Fix add volumes to GMCV group https://review.opendev.org/c/openstack/cinder/+/784092 | 15:46 |
| openstackgerrit | Venkata krishna Thumu proposed openstack/cinder master: [SVF] Support volume-resize to hyperswap volume https://review.opendev.org/c/openstack/cinder/+/774594 | 15:51 |
| *** GirishChilukuri has joined #openstack-cinder | 15:56 | |
| *** ociuhandu_ has joined #openstack-cinder | 16:02 | |
| *** ociuhandu has quit IRC | 16:06 | |
| *** e0ne has joined #openstack-cinder | 16:06 | |
| *** ociuhandu_ has quit IRC | 16:07 | |
| *** m75abrams has quit IRC | 16:20 | |
| *** jv has quit IRC | 16:41 | |
| *** hamalq has joined #openstack-cinder | 16:50 | |
| *** jv has joined #openstack-cinder | 16:53 | |
| *** sapd1 has quit IRC | 16:56 | |
| *** udesale has quit IRC | 16:58 | |
| *** dsariel has quit IRC | 16:59 | |
| *** e0ne has quit IRC | 17:05 | |
| *** e0ne has joined #openstack-cinder | 17:11 | |
| *** e0ne has quit IRC | 17:28 | |
| *** GirishChilukuri has quit IRC | 17:57 | |
| openstackgerrit | Amar proposed openstack/cinder master: [SVF]: Fix extend issue for a clone of rep-volume https://review.opendev.org/c/openstack/cinder/+/782405 | 17:58 |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder master: New Job: Ceph multibackend and c-bak https://review.opendev.org/c/openstack/cinder/+/784815 | 19:07 |
| *** dsariel has joined #openstack-cinder | 19:25 | |
| *** elod has quit IRC | 19:54 | |
| *** elod has joined #openstack-cinder | 19:56 | |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder master: Fix creating encrypted volume from unencrypt image https://review.opendev.org/c/openstack/cinder/+/783135 | 20:22 |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder master: Fix creating encrypted volume from unencrypt image https://review.opendev.org/c/openstack/cinder/+/783135 | 20:24 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Change snapshots type with volume retype https://review.opendev.org/c/openstack/cinder/+/784361 | 20:25 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Add quota utils to cinder-manage https://review.opendev.org/c/openstack/cinder/+/784362 | 20:25 |
| *** rcernin_ has joined #openstack-cinder | 20:25 | |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Remove unused quota python classes https://review.opendev.org/c/openstack/cinder/+/784399 | 20:26 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Quota: Fix until_refresh config changes https://review.opendev.org/c/openstack/cinder/+/784587 | 20:26 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Remove nested quota leftovers https://review.opendev.org/c/openstack/cinder/+/784588 | 20:30 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Fix quota usage duplicate entries https://review.opendev.org/c/openstack/cinder/+/784589 | 20:30 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Update cinder manage quota commands https://review.opendev.org/c/openstack/cinder/+/784590 | 20:33 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: mypy: generate stubs for imported libs https://review.opendev.org/c/openstack/cinder/+/775007 | 20:36 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Prevent quota and reservations to go into negative https://review.opendev.org/c/openstack/cinder/+/784591 | 20:39 |
| openstackgerrit | Merged openstack/cinder stable/wallaby: Quota: Fix multiple race conditions https://review.opendev.org/c/openstack/cinder/+/784592 | 20:39 |
| *** rcernin_ has quit IRC | 21:02 | |
| *** eharney_ has joined #openstack-cinder | 21:05 | |
| *** eharney has quit IRC | 21:07 | |
| *** eharney_ is now known as eharney | 21:08 | |
| *** rcernin_ has joined #openstack-cinder | 21:24 | |
| *** rcernin_ has quit IRC | 21:26 | |
| *** rcernin has joined #openstack-cinder | 21:26 | |
| *** enriquetaso has quit IRC | 21:36 | |
| *** muesono has joined #openstack-cinder | 21:44 | |
| openstackgerrit | Marcelo Uesono proposed openstack/cinder master: Removes inconsistency from docs https://review.opendev.org/c/openstack/cinder/+/784833 | 21:54 |
| *** muesono has quit IRC | 21:59 | |
| *** penick has joined #openstack-cinder | 22:07 | |
| *** zenkuro has quit IRC | 22:16 | |
| *** zenkuro has joined #openstack-cinder | 22:17 | |
| *** rcernin has quit IRC | 22:19 | |
| *** rcernin has joined #openstack-cinder | 22:19 | |
| *** tosky has quit IRC | 23:19 | |
| *** _erlon_ has quit IRC | 23:32 | |
| *** penick has quit IRC | 23:38 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!