Friday, 2021-06-04

*** bhagyashris_ has joined #openstack-glance00:31
*** bhagyashris has quit IRC00:38
*** benj_ has quit IRC01:37
*** benj_ has joined #openstack-glance02:01
*** abhishekk has joined #openstack-glance05:11
*** abhishekk is now known as akekane|home05:12
opendevreviewAbhishek Kekane proposed openstack/glance-specs master: Spec for Glance cache API  https://review.opendev.org/c/openstack/glance-specs/+/66525805:44
opendevreviewAbhishek Kekane proposed openstack/glance-specs master: Spec Lite: CLI support for new cache APIs  https://review.opendev.org/c/openstack/glance-specs/+/79470906:19
*** bhagyashris_ is now known as bhagyashris06:33
*** akekane|home has quit IRC07:41
*** CeeMac has joined #openstack-glance08:17
opendevreviewRajat Dhasmana proposed openstack/glance_store master: Add cinder's new attachment support  https://review.opendev.org/c/openstack/glance_store/+/78220011:14
*** bhagyashris_ has joined #openstack-glance12:50
*** bhagyashris has quit IRC12:56
*** bhagyashris_ is now known as bhagyashris12:58
*** abhishekk has joined #openstack-glance13:44
opendevreviewDan Smith proposed openstack/glance master: Fix missing context args to get_flow()  https://review.opendev.org/c/openstack/glance/+/79424614:10
opendevreviewDan Smith proposed openstack/glance master: Refactor SynchronousAPIBase for more cases  https://review.opendev.org/c/openstack/glance/+/78806514:10
opendevreviewDan Smith proposed openstack/glance master: Update image.size after conversion  https://review.opendev.org/c/openstack/glance/+/78809114:10
opendevreviewDan Smith proposed openstack/glance master: Make image stage set image.size  https://review.opendev.org/c/openstack/glance/+/78807514:10
opendevreviewDan Smith proposed openstack/glance master: Drop lower-constraints jobs  https://review.opendev.org/c/openstack/glance/+/78276814:10
opendevreviewDan Smith proposed openstack/glance master: Add unified quotas infrastructure  https://review.opendev.org/c/openstack/glance/+/78805414:10
opendevreviewDan Smith proposed openstack/glance master: Enforce keystone limits for image upload  https://review.opendev.org/c/openstack/glance/+/78805514:10
opendevreviewDan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API  https://review.opendev.org/c/openstack/glance/+/78807614:10
opendevreviewDan Smith proposed openstack/glance master: Add image_stage_total quota enforcement  https://review.opendev.org/c/openstack/glance/+/78807714:10
opendevreviewDan Smith proposed openstack/glance master: Add user_get_image_count() to DB API  https://review.opendev.org/c/openstack/glance/+/78832614:10
opendevreviewDan Smith proposed openstack/glance master: Add image_count_total quota enforcement  https://review.opendev.org/c/openstack/glance/+/78832714:10
opendevreviewDan Smith proposed openstack/glance master: Add user_get_uploading_count() to DB API  https://review.opendev.org/c/openstack/glance/+/79424514:11
opendevreviewDan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement  https://review.opendev.org/c/openstack/glance/+/79424714:11
opendevreviewDan Smith proposed openstack/glance master: Add unified quotas documentation  https://review.opendev.org/c/openstack/glance/+/79467214:11
*** gmann is now known as gmann_afk14:40
*** gmann_afk is now known as gmann15:13
abhishekkdansmith, around?15:45
dansmithyup15:45
abhishekkI am going through Add image_stage_total quota enforcement15:45
abhishekkhttps://review.opendev.org/c/openstack/glance/+/78807715:46
abhishekkYou enforced the quota for stage call15:46
abhishekkbut we also need to enforce it for web-download and copy-image plugins as staging is used there as well right?15:47
dansmithoh, did I not do that? I might have ben focused on the new upload count limit and forgotten to do the size for those15:49
abhishekkyep, adding comment now15:50
dansmiththanks15:50
abhishekkno problem15:50
dansmithabhishekk: I see all your comments, thanks15:59
abhishekkdansmith, ack16:00
dansmithI'm working on getting oslo.limit to expose actual limits so we can write an API,16:00
dansmithso when I get to a stopping point I will circle back on those16:00
abhishekkcool, thank you16:01
* abhishekk will be back from dinner soon16:02
opendevreviewDan Smith proposed openstack/glance master: ULTRAWIP: Example usage API  https://review.opendev.org/c/openstack/glance/+/79486016:19
*** akekane_ has joined #openstack-glance16:44
*** abhishekk has quit IRC16:44
*** abhishekk has joined #openstack-glance16:51
*** akekane_ has quit IRC16:52
dansmithabhishekk: I'm pushing up all the actionable things from your comments (I think) other than the import staging change17:08
abhishekkack17:09
dansmithabhishekk: specifically, if you could look at the doc thing I added about "what to do about over quota" that would be good17:09
dansmith(as soon as gerrit catches up)17:09
abhishekkack17:09
abhishekkI will probably have a look at them next week17:11
dansmithokay, it's still waiting17:11
dansmithbut yeah, no problem, I'm not sure I'll get to the import staging thing today either17:11
abhishekkregarding the debug/error log comment I guess leave it as it is if that is consistent as you replied to the comment17:11
abhishekkack17:12
dansmithokay, it's consistently wrong, and could use some cleanup for sure, but.. the stack is already large17:12
abhishekkBut full marks for functional tests17:12
dansmiththanks :)17:13
opendevreviewMerged openstack/glance master: Fix missing context args to get_flow()  https://review.opendev.org/c/openstack/glance/+/79424617:16
abhishekksomething is wrong with bot I guess17:18
dansmithyup17:18
abhishekkokies, thank you for quick fixes17:20
abhishekktime for me to wrap up17:20
abhishekkhave a nice week ahead o/~17:20
dansmithcool, enjoy your weekend!17:21
abhishekkthank you17:22
*** abhishekk has quit IRC17:22
opendevreviewDan Smith proposed openstack/glance master: Refactor SynchronousAPIBase for more cases  https://review.opendev.org/c/openstack/glance/+/78806518:34
opendevreviewDan Smith proposed openstack/glance master: Update image.size after conversion  https://review.opendev.org/c/openstack/glance/+/78809118:34
opendevreviewDan Smith proposed openstack/glance master: Make image stage set image.size  https://review.opendev.org/c/openstack/glance/+/78807518:34
opendevreviewDan Smith proposed openstack/glance master: Drop lower-constraints jobs  https://review.opendev.org/c/openstack/glance/+/78276818:34
opendevreviewDan Smith proposed openstack/glance master: Add unified quotas infrastructure  https://review.opendev.org/c/openstack/glance/+/78805418:34
opendevreviewDan Smith proposed openstack/glance master: Enforce keystone limits for image upload  https://review.opendev.org/c/openstack/glance/+/78805518:34
opendevreviewDan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API  https://review.opendev.org/c/openstack/glance/+/78807618:34
opendevreviewDan Smith proposed openstack/glance master: Add image_stage_total quota enforcement  https://review.opendev.org/c/openstack/glance/+/78807718:34
opendevreviewDan Smith proposed openstack/glance master: Add user_get_image_count() to DB API  https://review.opendev.org/c/openstack/glance/+/78832618:34
opendevreviewDan Smith proposed openstack/glance master: Add image_count_total quota enforcement  https://review.opendev.org/c/openstack/glance/+/78832718:34
opendevreviewDan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement  https://review.opendev.org/c/openstack/glance/+/79424718:34
opendevreviewDan Smith proposed openstack/glance master: Add unified quotas documentation  https://review.opendev.org/c/openstack/glance/+/79467218:34
opendevreviewDan Smith proposed openstack/glance master: ULTRAWIP: Example usage API  https://review.opendev.org/c/openstack/glance/+/79486018:34
opendevreviewDan Smith proposed openstack/glance master: Add image_stage_total quota enforcement  https://review.opendev.org/c/openstack/glance/+/78807718:57
opendevreviewDan Smith proposed openstack/glance master: Add user_get_image_count() to DB API  https://review.opendev.org/c/openstack/glance/+/78832618:57
opendevreviewDan Smith proposed openstack/glance master: Add image_count_total quota enforcement  https://review.opendev.org/c/openstack/glance/+/78832718:57
opendevreviewDan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement  https://review.opendev.org/c/openstack/glance/+/79424718:57
opendevreviewDan Smith proposed openstack/glance master: Add unified quotas documentation  https://review.opendev.org/c/openstack/glance/+/79467218:57
opendevreviewDan Smith proposed openstack/glance master: ULTRAWIP: Example usage API  https://review.opendev.org/c/openstack/glance/+/79486018:57
*** gmann is now known as gmann_afk21:51
*** whoami-rajat has quit IRC23:40

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!