| *** whoami-rajat has quit IRC | 00:20 | |
| *** abhishekk has joined #openstack-glance | 04:39 | |
| *** abhishekk has quit IRC | 04:41 | |
| *** abhishekk has joined #openstack-glance | 04:41 | |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Spec for Glance cache API https://review.opendev.org/c/openstack/glance-specs/+/665258 | 05:48 |
|---|---|---|
| *** bhagyashris has joined #openstack-glance | 06:16 | |
| *** bhagyashris has quit IRC | 07:23 | |
| *** bhagyashris has joined #openstack-glance | 07:23 | |
| *** abhishekk has quit IRC | 07:24 | |
| *** bhagyashris_ has joined #openstack-glance | 07:38 | |
| *** bhagyashris has quit IRC | 07:44 | |
| *** icey has joined #openstack-glance | 08:20 | |
| *** Luzi has joined #openstack-glance | 09:10 | |
| *** abhishekk has joined #openstack-glance | 09:19 | |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/glance_store master: Allow any Keystone domain for cinder store https://review.opendev.org/c/openstack/glance_store/+/793826 | 10:34 |
| *** whoami-rajat has joined #openstack-glance | 10:49 | |
| *** icey has quit IRC | 10:59 | |
| *** belmoreira has joined #openstack-glance | 11:02 | |
| *** icey has joined #openstack-glance | 11:33 | |
| *** bhagyashris_ is now known as bhagyashris | 12:09 | |
| *** abhishekk has quit IRC | 12:15 | |
| *** Luzi has quit IRC | 13:21 | |
| *** abhishekk has joined #openstack-glance | 13:31 | |
| *** abhishekk is now known as akekane|home | 13:32 | |
| *** akekane|home is now known as abhishekk | 13:32 | |
| abhishekk | dansmith, o/ | 14:01 |
| dansmith | hey | 14:01 |
| abhishekk | replied to your concerns on the spec | 14:01 |
| abhishekk | the client related concern is valid one, I need some time to put a spec together for the same | 14:02 |
| dansmith | cool, I think requiring the override will be good | 14:03 |
| abhishekk | yes, that is definitely required | 14:03 |
| dansmith | can you update the perms part to say "new" and update queue_image -> cache_image? if so, I'll be +2 assuming the lite spec matches your promises :) | 14:04 |
| abhishekk | yes will do it asap | 14:05 |
| dansmith | FYI, I have a new rev of the quota series to push up with the changes we discussed on the spec. it works, but struggling with getting the tests (specifically with copy async stuff) just right | 14:06 |
| abhishekk | great | 14:07 |
| abhishekk | you should change your nick | 14:07 |
| abhishekk | superdan :D | 14:07 |
| dansmith | that used to be my friday nick | 14:08 |
| dansmith | not feeling very super with these dang tests tho :/ | 14:08 |
| abhishekk | ohh | 14:09 |
| abhishekk | you will fix it, I know | 14:10 |
| dansmith | hopefully :) | 14:11 |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Spec for Glance cache API https://review.opendev.org/c/openstack/glance-specs/+/665258 | 14:22 |
| *** opendevreview has quit IRC | 14:38 | |
| *** opendevreview has joined #openstack-glance | 15:08 | |
| opendevreview | Dan Smith proposed openstack/glance master: Drop lower-constraints jobs https://review.opendev.org/c/openstack/glance/+/782768 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Refactor SynchronousAPIBase for more cases https://review.opendev.org/c/openstack/glance/+/788065 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas infrastructure https://review.opendev.org/c/openstack/glance/+/788054 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Enforce keystone limits for image upload https://review.opendev.org/c/openstack/glance/+/788055 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Update image.size after conversion https://review.opendev.org/c/openstack/glance/+/788091 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Make image stage set image.size https://review.opendev.org/c/openstack/glance/+/788075 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API https://review.opendev.org/c/openstack/glance/+/788076 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_stage_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788077 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_image_count() to DB API https://review.opendev.org/c/openstack/glance/+/788326 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788327 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Make taskflow_executor log get_flow() exceptions https://review.opendev.org/c/openstack/glance/+/794244 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_uploading_count() to DB API https://review.opendev.org/c/openstack/glance/+/794245 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Fix missing context args to get_flow() https://review.opendev.org/c/openstack/glance/+/794246 | 15:08 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement https://review.opendev.org/c/openstack/glance/+/794247 | 15:08 |
| abhishekk | :o | 15:10 |
| dansmith | abhishekk: still need to update the tempest patch | 15:14 |
| dansmith | also, a bunch of those patches are cleanups or fixes that could go earlier and be merged before the rest, but I left them in "logical order" for the moment | 15:15 |
| abhishekk | ack | 15:15 |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Spec for Glance cache API https://review.opendev.org/c/openstack/glance-specs/+/665258 | 15:25 |
| *** bhagyashris_ has joined #openstack-glance | 16:26 | |
| *** bhagyashris has quit IRC | 16:34 | |
| opendevreview | Merged openstack/glance-specs master: Add support of cinder's new attachment API https://review.opendev.org/c/openstack/glance-specs/+/787515 | 16:45 |
| *** bhagyashris_ is now known as bhagyashris | 16:50 | |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Revert "Add support of cinder's new attachment API" https://review.opendev.org/c/openstack/glance-specs/+/794187 | 16:59 |
| whoami-rajat | rosmaita: ^ can you ninja approve this patch so i can propose a new patch with needed changes? | 17:04 |
| abhishekk | dansmith, around? | 17:08 |
| dansmith | yeah | 17:08 |
| abhishekk | I have one question related to quota | 17:08 |
| abhishekk | so our global quota does have image cap | 17:09 |
| abhishekk | right? | 17:09 |
| dansmith | there are a couple limits, which one are you talking about? | 17:09 |
| abhishekk | on number of images? | 17:09 |
| abhishekk | do we have it for number of images or image properties? | 17:10 |
| dansmith | both I think, just a sec | 17:10 |
| dansmith | https://github.com/openstack/glance/blob/922e544ca2556994450e6972403ba4313318c5e0/glance/quota/__init__.py#L102-L116 | 17:11 |
| dansmith | oh, actually I think there's no limit on number of images, yeah | 17:11 |
| dansmith | as rosmaita brought up in the spec, that's new, but mirrors per-item count limits in other projects (like nova has a quota on instances) | 17:12 |
| abhishekk | ack | 17:12 |
| abhishekk | also, there is limit on storage per tenant | 17:12 |
| abhishekk | so if I set user_storage_quota 10GB and new unified quota is 20GB then what will happen? | 17:13 |
| abhishekk | means if we are using new quotas, will it also enforce or check for global or earlier quotas ? | 17:14 |
| dansmith | you'll still be limited by the global one | 17:14 |
| abhishekk | hmm | 17:14 |
| abhishekk | this will be confusing right, unless we document the same | 17:15 |
| dansmith | well, we can surely doc it, but we could: | 17:15 |
| dansmith | 1. Disable the equivalent global quota if limit quotas are enabled | 17:15 |
| abhishekk | I am in favor of 1 | 17:15 |
| dansmith | 2. Leave it, doc it, and remove the global quotas later | 17:16 |
| dansmith | 3. Just doc it | 17:16 |
| dansmith | abhishekk: ack, good thing we kept the config knob for turning them on then :) | 17:16 |
| abhishekk | I think option 1 is better | 17:16 |
| abhishekk | ++ | 17:16 |
| dansmith | sounds good to me | 17:17 |
| rosmaita | whoami-rajat: i +2'd, only abhishekk can +A a glance-spec | 17:24 |
| abhishekk | rosmaita, thanks | 17:25 |
| * abhishekk signing out for the day | 17:30 | |
| *** abhishekk has quit IRC | 17:30 | |
| opendevreview | Dan Smith proposed openstack/glance master: Make taskflow_executor log get_flow() exceptions https://review.opendev.org/c/openstack/glance/+/794244 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Refactor SynchronousAPIBase for more cases https://review.opendev.org/c/openstack/glance/+/788065 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Update image.size after conversion https://review.opendev.org/c/openstack/glance/+/788091 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Make image stage set image.size https://review.opendev.org/c/openstack/glance/+/788075 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Fix missing context args to get_flow() https://review.opendev.org/c/openstack/glance/+/794246 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Drop lower-constraints jobs https://review.opendev.org/c/openstack/glance/+/782768 | 17:48 |
| opendevreview | Merged openstack/glance-specs master: Revert "Add support of cinder's new attachment API" https://review.opendev.org/c/openstack/glance-specs/+/794187 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas infrastructure https://review.opendev.org/c/openstack/glance/+/788054 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Enforce keystone limits for image upload https://review.opendev.org/c/openstack/glance/+/788055 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API https://review.opendev.org/c/openstack/glance/+/788076 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_stage_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788077 | 17:48 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_image_count() to DB API https://review.opendev.org/c/openstack/glance/+/788326 | 17:49 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788327 | 17:49 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_uploading_count() to DB API https://review.opendev.org/c/openstack/glance/+/794245 | 17:49 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement https://review.opendev.org/c/openstack/glance/+/794247 | 17:49 |
| *** belmoreira has quit IRC | 19:11 | |
| opendevreview | Dan Smith proposed openstack/glance master: Refactor gateway get_repo auth layer https://review.opendev.org/c/openstack/glance/+/789913 | 19:24 |
| opendevreview | Dan Smith proposed openstack/glance master: Make property protection tests use member role https://review.opendev.org/c/openstack/glance/+/789914 | 19:24 |
| opendevreview | Dan Smith proposed openstack/glance master: WIP: Make image update check policy at API layer https://review.opendev.org/c/openstack/glance/+/789915 | 19:24 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!