| *** bhagyashris__ <bhagyashris__!~bhagyashr@103.98.79.86> has joined #openstack-glance | 01:51 | |
| *** bhagyashris_ <bhagyashris_!~bhagyashr@103.98.79.86> has quit IRC (Ping timeout: 480 seconds) | 01:58 | |
| *** priteau <priteau!~priteau@93.186.40.84> has quit IRC (Ping timeout: 480 seconds) | 02:04 | |
| *** priteau <priteau!~priteau@93.186.40.84> has joined #openstack-glance | 02:06 | |
| *** abhishekk <abhishekk!~akekane@116.74.162.143> has joined #openstack-glance | 05:18 | |
| *** ratailor_ <ratailor_!~ratailor@103.115.207.241> has joined #openstack-glance | 05:43 | |
| *** ratailor_ is now known as ratailor | 05:48 | |
| *** icey <icey!~icey@0001bbad.user.oftc.net> has quit IRC () | 06:18 | |
| *** icey <icey!~icey@0001bbad.user.oftc.net> has joined #openstack-glance | 06:18 | |
| *** bhagyashris__ is now known as bhagyashris | 06:36 | |
| *** pdeore <pdeore!~pdeore@2401:4900:1b82:1ebf:7a02:6805:1a5c:6475> has joined #openstack-glance | 06:47 | |
| *** tkajinam <tkajinam!~tkajinam@240d:1a:6af:1b00:3462:14d1:c5f6:8946> has quit IRC (Quit: Leaving) | 07:16 | |
| *** tkajinam <tkajinam!~tkajinam@240d:1a:6af:1b00:3462:14d1:c5f6:8946> has joined #openstack-glance | 07:17 | |
| *** tosky <tosky!~luigi@dynamic-adsl-78-13-253-141.clienti.tiscali.it> has joined #openstack-glance | 07:45 | |
| *** CeeMac <CeeMac!uid366483@id-366483.brockwell.irccloud.com> has joined #openstack-glance | 08:05 | |
| *** whoami-rajat <whoami-rajat!uid314320@id-314320.brockwell.irccloud.com> has joined #openstack-glance | 08:14 | |
| *** abhishekk <abhishekk!~akekane@116.74.162.143> has quit IRC (Quit: Bye...) | 09:19 | |
| *** Luzi <Luzi!~Luzi@ipservice-092-208-056-099.092.208.pools.vodafone-ip.de> has joined #openstack-glance | 09:47 | |
| *** ratailor_ <ratailor_!~ratailor@2409:4052:2204:5df:d46:7ddd:22bd:3dc0> has joined #openstack-glance | 10:40 | |
| *** ratailor_ <ratailor_!~ratailor@2409:4052:2204:5df:d46:7ddd:22bd:3dc0> has quit IRC (Remote host closed the connection) | 10:40 | |
| *** ratailor_ <ratailor_!~ratailor@103.115.207.228> has joined #openstack-glance | 10:46 | |
| *** ratailor <ratailor!~ratailor@103.115.207.241> has quit IRC (Ping timeout: 480 seconds) | 10:46 | |
| *** smcginnis <smcginnis!~smcginnis@72-50-207-44.fttp.usinternet.com> has quit IRC (Remote host closed the connection) | 11:27 | |
| *** smcginnis <smcginnis!~smcginnis@72-50-207-44.fttp.usinternet.com> has joined #openstack-glance | 11:37 | |
| *** smcginnis <smcginnis!~smcginnis@72-50-207-44.fttp.usinternet.com> has quit IRC (Quit: leaving) | 11:43 | |
| *** smcginnis <smcginnis!~smcginnis@72-50-207-44.fttp.usinternet.com> has joined #openstack-glance | 11:43 | |
| *** smcginnis <smcginnis!~smcginnis@72-50-207-44.fttp.usinternet.com> has quit IRC () | 11:45 | |
| *** smcginnis <smcginnis!~smcginnis@72-50-207-44.fttp.usinternet.com> has joined #openstack-glance | 11:46 | |
| *** rosmaita <rosmaita!~rosmaita@2601:5c0:c380:b8e0::3397> has joined #openstack-glance | 12:09 | |
| *** ratailor_ <ratailor_!~ratailor@103.115.207.228> has quit IRC (Quit: Leaving) | 12:46 | |
| *** abhishekk <abhishekk!~akekane@116.74.162.143> has joined #openstack-glance | 13:37 | |
| *** abhishekk is now known as akekane|home | 13:37 | |
| *** akekane|home is now known as abhishekk | 13:37 | |
| abhishekk | dansmith, o/, around | 13:58 |
|---|---|---|
| dansmith | abhishekk: yep | 13:58 |
| abhishekk | good morning!! | 13:59 |
| abhishekk | what will be the good time to sync on policy refactoring | 13:59 |
| abhishekk | (will be better if it is tomorrow) | 14:00 |
| dansmith | tomorrow is probably okay, tuesdays are busy for me.. this hour tomorrow should be okay though | 14:01 |
| *** Luzi <Luzi!~Luzi@ipservice-092-208-056-099.092.208.pools.vodafone-ip.de> has quit IRC (Quit: Leaving) | 14:01 | |
| dansmith | I pushed up some more very WIP code on friday just to get it out there | 14:01 |
| dansmith | and I have some notes I can go over with you | 14:02 |
| abhishekk | ack | 14:02 |
| dansmith | I think the default rules we merged last cycle for "experimental" RBAC support are not right | 14:02 |
| abhishekk | ohh | 14:03 |
| dansmith | they might be right enough considering our otherwise loose model, I'm not sure, but they're definitely not right from the perspective of making sense, nor for what we need them to be for this refactor effort | 14:03 |
| dansmith | anyway, I'll get those cleaned up and we can discuss tomorrow | 14:03 |
| dansmith | I wish lance was here, but...alas | 14:04 |
| abhishekk | yes | 14:04 |
| abhishekk | also, I want to ask one thing about quotas documentation | 14:04 |
| abhishekk | should we also explain parameters needs to set under oslo_limit? | 14:05 |
| *** pdeore <pdeore!~pdeore@2401:4900:1b82:1ebf:7a02:6805:1a5c:6475> has quit IRC (Ping timeout: 480 seconds) | 14:06 | |
| dansmith | the config parameters? | 14:06 |
| abhishekk | yeah | 14:07 |
| abhishekk | as per documentation if you just enable by turning on use_keystone_limits to True | 14:07 |
| dansmith | https://docs.openstack.org/oslo.limit/latest/user/usage.html#configuration | 14:07 |
| dansmith | but I can link to this ^ in the docs, sure | 14:07 |
| abhishekk | then while creating image it ruturns 500 error with something containing oslo_limit auth_plugin requires | 14:08 |
| dansmith | I think it's similar config to anything that uses ks_client, you just have to know to put that in [oslo_limit] as well as somethign else | 14:08 |
| dansmith | ah, okay, what does it raise? we can catch and log that with a helpful message | 14:09 |
| abhishekk | yep, I think better to add a one line with above link | 14:09 |
| abhishekk | give me a minute | 14:09 |
| abhishekk | Jun 14 14:10:38 akekane-wallaby-dev devstack@g-api.service[1874771]: ERROR glance.common.wsgi oslo_limit.exception.SessionInitError: Can't initialise OpenStackSDK session: An auth plugin is required to determine endpoint URL. | 14:11 |
| abhishekk | Jun 14 14:10:38 akekane-wallaby-dev devstack@g-api.service[1874771]: ERROR glance.common.wsgi File "/opt/stack/glance/glance/quota/keystone.py", line 55, in _enforce_some | 14:12 |
| dansmith | okay, will add something for that | 14:13 |
| abhishekk | ack | 14:14 |
| dansmith | you know, reading this oslo.limit doc, I think lance and I identified a bug in what is prescribed here, which I totally forgot about | 14:18 |
| dansmith | so I will make our docs look like what devstack uses (which works) | 14:19 |
| dansmith | and I need to circle back with him afterwards | 14:19 |
| abhishekk | :D | 14:19 |
| abhishekk | Better to file a bug then, otherwise ... | 14:20 |
| dansmith | yep, done | 14:25 |
| dansmith | https://bugs.launchpad.net/oslo.limit/+bug/1931875 | 14:25 |
| abhishekk | great | 14:26 |
| abhishekk | tomorrow 1400 UTC confirm then | 14:28 |
| dansmith | sounds good | 14:32 |
| abhishekk | cool, thank you | 14:33 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas infrastructure https://review.opendev.org/c/openstack/glance/+/788054 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Enforce keystone limits for image upload https://review.opendev.org/c/openstack/glance/+/788055 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API https://review.opendev.org/c/openstack/glance/+/788076 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_stage_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788077 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_image_count() to DB API https://review.opendev.org/c/openstack/glance/+/788326 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788327 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement https://review.opendev.org/c/openstack/glance/+/794247 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas documentation https://review.opendev.org/c/openstack/glance/+/794672 | 14:41 |
| opendevreview | Dan Smith proposed openstack/glance master: ULTRAWIP: Example usage API https://review.opendev.org/c/openstack/glance/+/794860 | 14:41 |
| dansmith | abhishekk: doc add and SessionInitError handling ^ | 14:41 |
| abhishekk | dansmith, ack, looking | 14:41 |
| abhishekk | one mistake | 14:43 |
| dansmith | WAT | 14:43 |
| abhishekk | commented | 14:44 |
| abhishekk | Doc looks perfect now, thank you | 14:46 |
| dansmith | ah, dang, that was a rebase conflict | 14:46 |
| dansmith | fixed, thanks | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas infrastructure https://review.opendev.org/c/openstack/glance/+/788054 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Enforce keystone limits for image upload https://review.opendev.org/c/openstack/glance/+/788055 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API https://review.opendev.org/c/openstack/glance/+/788076 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_stage_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788077 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_image_count() to DB API https://review.opendev.org/c/openstack/glance/+/788326 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788327 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement https://review.opendev.org/c/openstack/glance/+/794247 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas documentation https://review.opendev.org/c/openstack/glance/+/794672 | 14:47 |
| opendevreview | Dan Smith proposed openstack/glance master: ULTRAWIP: Example usage API https://review.opendev.org/c/openstack/glance/+/794860 | 14:47 |
| abhishekk | you work really fast, may be faster than light | 14:48 |
| abhishekk | :D | 14:48 |
| dansmith | always under the speed limit.. it's the law ;) | 14:48 |
| abhishekk | haha | 14:48 |
| abhishekk | I think some thing is wrong with rebasing | 14:51 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/794245/4 | 14:51 |
| abhishekk | this patch is on top of ULTRAWIP patch | 14:52 |
| dansmith | hmm, it looks correct locally | 14:52 |
| dansmith | oh wait, maybe not | 14:53 |
| abhishekk | also it doesn't shown in above bot updates | 14:53 |
| dansmith | yeah, hmm | 14:53 |
| dansmith | I think it's right locally, I'm forcing an update, we'll see if that helps | 14:57 |
| abhishekk | ack | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Refactor SynchronousAPIBase for more cases https://review.opendev.org/c/openstack/glance/+/788065 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Update image.size after conversion https://review.opendev.org/c/openstack/glance/+/788091 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Make image stage set image.size https://review.opendev.org/c/openstack/glance/+/788075 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Drop lower-constraints jobs https://review.opendev.org/c/openstack/glance/+/782768 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas infrastructure https://review.opendev.org/c/openstack/glance/+/788054 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Enforce keystone limits for image upload https://review.opendev.org/c/openstack/glance/+/788055 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_staging_usage() to DB API https://review.opendev.org/c/openstack/glance/+/788076 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_stage_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788077 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_image_count() to DB API https://review.opendev.org/c/openstack/glance/+/788326 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788327 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement https://review.opendev.org/c/openstack/glance/+/794247 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas documentation https://review.opendev.org/c/openstack/glance/+/794672 | 14:58 |
| opendevreview | Dan Smith proposed openstack/glance master: ULTRAWIP: Example usage API https://review.opendev.org/c/openstack/glance/+/794860 | 14:58 |
| dansmith | okay, looks right all the way down from here right? https://review.opendev.org/c/openstack/glance/+/794860/9 | 14:59 |
| abhishekk | looking | 15:00 |
| abhishekk | yeah | 15:00 |
| dansmith | I blame gerrit.. all I did was tweak the bottom patch and force a rebase in git-review | 15:00 |
| abhishekk | but you go to first patch in the chain then its blown up | 15:02 |
| abhishekk | that is refactor synchronous patch | 15:02 |
| dansmith | this looks right to me: https://review.opendev.org/c/openstack/glance/+/788065/8 | 15:02 |
| dansmith | does it not to you? | 15:02 |
| dansmith | oh, maybe the get_uploading_count patch got kicked out? | 15:03 |
| abhishekk | yeah | 15:03 |
| abhishekk | I see user_get_uploading_count on top of ultrawip patch | 15:04 |
| dansmith | it's not really, it's just gerrit showing you that because it's an ancestor | 15:04 |
| dansmith | it's not in my stack locally, so that's the problem It hink | 15:04 |
| abhishekk | hmm, and if we see from top, https://review.opendev.org/c/openstack/glance/+/794860/9 then I don't see user_get_uploading_count patch in tree | 15:05 |
| dansmith | right | 15:06 |
| dansmith | I'll stitch it back together, just a sec | 15:06 |
| abhishekk | ack | 15:06 |
| dansmith | man, this was seriously messed up | 15:19 |
| dansmith | that patch got squashed into the one above it, but with no squashed commit message | 15:19 |
| abhishekk | ohh | 15:24 |
| dansmith | let's try this again | 15:24 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_total quota enforcement https://review.opendev.org/c/openstack/glance/+/788327 | 15:24 |
| opendevreview | Dan Smith proposed openstack/glance master: Add user_get_uploading_count() to DB API https://review.opendev.org/c/openstack/glance/+/794245 | 15:24 |
| opendevreview | Dan Smith proposed openstack/glance master: Add image_count_uploading quota enforcement https://review.opendev.org/c/openstack/glance/+/794247 | 15:24 |
| opendevreview | Dan Smith proposed openstack/glance master: Add unified quotas documentation https://review.opendev.org/c/openstack/glance/+/794672 | 15:24 |
| opendevreview | Dan Smith proposed openstack/glance master: ULTRAWIP: Example usage API https://review.opendev.org/c/openstack/glance/+/794860 | 15:24 |
| abhishekk | Now its in right order | 15:26 |
| dansmith | yeah, I dunno what happened there, but thanks for noticing | 15:30 |
| abhishekk | no worries, its good to sorted out in time | 15:34 |
| * abhishekk going out for dinner break | 15:34 | |
| abhishekk | rosmaita, if you around have some time, then quotas patches are in good shape now | 15:35 |
| rosmaita | abhishekk: ack | 15:39 |
| opendevreview | Merged openstack/glance master: Allow member creation when using db.simple api https://review.opendev.org/c/openstack/glance/+/366354 | 15:41 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!