| *** rchurch has quit IRC | 00:23 | |
| *** rchurch has joined #openstack-glance | 00:26 | |
| *** k_mouza has joined #openstack-glance | 01:30 | |
| *** k_mouza has quit IRC | 01:35 | |
| *** baojg has joined #openstack-glance | 01:51 | |
| *** rcernin has joined #openstack-glance | 02:39 | |
| *** udesale has joined #openstack-glance | 03:55 | |
| openstackgerrit | Lance Bragstad proposed openstack/glance master: WIP: add logic to properly enforce secure RBAC when handling images https://review.opendev.org/c/openstack/glance/+/765210 | 05:00 |
|---|---|---|
| *** ratailor has joined #openstack-glance | 05:16 | |
| *** whoami-rajat__ has joined #openstack-glance | 05:23 | |
| *** k_mouza has joined #openstack-glance | 05:54 | |
| *** k_mouza has quit IRC | 05:58 | |
| *** m75abrams has joined #openstack-glance | 06:47 | |
| *** zzzeek has quit IRC | 07:07 | |
| *** zzzeek has joined #openstack-glance | 07:08 | |
| *** lpetrut has joined #openstack-glance | 07:11 | |
| *** nikparasyr has joined #openstack-glance | 07:15 | |
| *** ralonsoh has joined #openstack-glance | 07:36 | |
| *** rcernin has quit IRC | 09:07 | |
| *** rcernin has joined #openstack-glance | 09:17 | |
| *** rcernin has quit IRC | 09:25 | |
| *** rcernin has joined #openstack-glance | 09:43 | |
| *** udesale_ has joined #openstack-glance | 10:32 | |
| openstackgerrit | Abhishek Kekane proposed openstack/glance master: [PoC] New API to get task information from image https://review.opendev.org/c/openstack/glance/+/763739 | 10:34 |
| *** udesale has quit IRC | 10:35 | |
| *** baojg has quit IRC | 10:35 | |
| *** baojg has joined #openstack-glance | 10:35 | |
| *** baojg has quit IRC | 10:45 | |
| *** rcernin has quit IRC | 10:48 | |
| *** rcernin has joined #openstack-glance | 10:53 | |
| *** rcernin has quit IRC | 10:57 | |
| *** k_mouza has joined #openstack-glance | 11:18 | |
| *** rcernin has joined #openstack-glance | 11:49 | |
| openstackgerrit | Rajat Dhasmana proposed openstack/glance master: Add functional tests for cinder multiple store https://review.opendev.org/c/openstack/glance/+/750144 | 11:51 |
| whoami-rajat__ | hi dansmith abhishekk jokke , can I have reviews on ^^ as it's been sitting for very long and gets in merge conflict from time to time | 11:53 |
| *** k_mouza has quit IRC | 12:31 | |
| *** k_mouza has joined #openstack-glance | 12:38 | |
| *** ratailor has quit IRC | 12:42 | |
| openstackgerrit | Abhishek Kekane proposed openstack/glance-specs master: New Image API /v2/images/{id}/tasks for task information https://review.opendev.org/c/openstack/glance-specs/+/763740 | 13:41 |
| *** rcernin has quit IRC | 13:43 | |
| dansmith | whoami-rajat__: yeah I will try to take a look at that today, sorry about that | 13:49 |
| abhishekk | jokke, rosmaita, dansmith, smcginnis, glance weekly meeting in 5 minutes at #openstack-meeting | 13:55 |
| *** m75abrams has quit IRC | 14:01 | |
| *** Underknowledge has quit IRC | 14:17 | |
| *** Underknowledge1 has joined #openstack-glance | 14:17 | |
| *** Underknowledge1 is now known as Underknowledge | 14:18 | |
| abhishekk | dansmith, are you recommending not to have index (regarding tasks patch) | 14:41 |
| dansmith | abhishekk: no | 14:42 |
| dansmith | abhishekk: I'm just calling out that it's going to do that in the middle of an upgrade, and wanted to make sure we assume there is not a huge amount of data to be indexed there | 14:42 |
| dansmith | rosmaita mentioned something about tasks might not be getting purged for everyone unless they hit the tasks api, | 14:42 |
| dansmith | so if someone has 5 years of tasks in that table, it could be really problematic to block their upgrade at 100% IO for several hours just to add that index | 14:43 |
| abhishekk | hmm | 14:43 |
| rosmaita | yeah, it's a lazy delete based on the owner doing a task-list call | 14:43 |
| dansmith | these are the kinds of things that seem trivial to us in a devstack, but make operators hate us during upgrades, so it's important to make sure we're being cognizant of the potential harm | 14:43 |
| rosmaita | but i thihk it's a hard-delete? | 14:43 |
| abhishekk | I think so | 14:44 |
| rosmaita | so we could probably do the delete first and then add the index as par tof the db migration | 14:44 |
| dansmith | we create tasks for imports now, but if nobody is hitting the task api to check on those, they're all just sitting there probably? | 14:44 |
| dansmith | rosmaita: ack, that's one option that would be a lot lighter | 14:45 |
| rosmaita | yeah, and there may be a lot of old 'pending' tasks | 14:45 |
| rosmaita | should probably clean those out too, though they won't have expires_at set | 14:45 |
| dansmith | yeah, I mean, you don't want to make a lot of work to scan and delete and then index, but.. we just don't want to add that and not be mindful of the impact I think | 14:45 |
| rosmaita | but we could look at creation date and kill any over a week old or something | 14:45 |
| dansmith | if you do that, do it in sql for sure and don't load everything into python to look at one field :) | 14:46 |
| rosmaita | i agree | 14:46 |
| * abhishekk going offline, please add your suggestions on the specs/patch | 14:46 | |
| *** lpetrut has quit IRC | 15:23 | |
| *** udesale_ has quit IRC | 15:41 | |
| whoami-rajat__ | dansmith: np, thanks | 15:45 |
| *** rosmaita has quit IRC | 15:53 | |
| *** rosmaita has joined #openstack-glance | 15:58 | |
| openstackgerrit | Dan Smith proposed openstack/glance-specs master: Distributed Image Import spec https://review.opendev.org/c/openstack/glance-specs/+/774097 | 16:02 |
| openstackgerrit | Dan Smith proposed openstack/glance master: Add get_ksa_client() helper https://review.opendev.org/c/openstack/glance/+/770682 | 16:11 |
| openstackgerrit | Dan Smith proposed openstack/glance master: Distributed image import https://review.opendev.org/c/openstack/glance/+/769976 | 16:11 |
| openstackgerrit | Dan Smith proposed openstack/glance master: Enable second glance worker for import testing https://review.opendev.org/c/openstack/glance/+/770629 | 16:11 |
| *** hoonetorg has joined #openstack-glance | 16:20 | |
| *** nikparasyr has left #openstack-glance | 17:15 | |
| *** openstack has joined #openstack-glance | 17:22 | |
| *** ChanServ sets mode: +o openstack | 17:22 | |
| *** zzzeek has quit IRC | 17:42 | |
| *** zzzeek has joined #openstack-glance | 17:44 | |
| *** k_mouza has quit IRC | 18:17 | |
| *** ralonsoh has quit IRC | 18:18 | |
| jokke | lbragstad: you around? | 18:19 |
| lbragstad | i am | 18:22 |
| jokke | about the client patch | 18:23 |
| jokke | I'm just pushing review to that. Quick question, what do we expect as value for that --os-system-scope? | 18:24 |
| *** hoonetorg has quit IRC | 18:24 | |
| lbragstad | it's a string and right now the only sensible value is 'all' | 18:25 |
| jokke | kk | 18:26 |
| jokke | that was more of a curiosity thing. Flagged the actual issue in th review. Not a big change needed, couple of lines | 18:26 |
| openstackgerrit | Dan Smith proposed openstack/glance-specs master: Distributed Image Import spec https://review.opendev.org/c/openstack/glance-specs/+/774097 | 18:35 |
| dansmith | gmann: can you help me remember what we had to do in nova to get rid of this crazy verbose logging of the policy deprecation stuff? https://zuul.opendev.org/t/openstack/build/71f0813ac0dc4a559974298a457d41e1/log/job-output.txt#1017 | 19:36 |
| gmann | dansmith: there are two flag we can set False for those warnings to skip | 19:38 |
| gmann | https://github.com/openstack/nova/blob/master/nova/policy.py#L89-L91 | 19:39 |
| dansmith | gmann: ack | 19:39 |
| dansmith | thanks, I'll see if I can do that in glance | 19:40 |
| *** rcernin has joined #openstack-glance | 19:40 | |
| *** rcernin has quit IRC | 19:45 | |
| *** hoonetorg has joined #openstack-glance | 20:01 | |
| *** stand has quit IRC | 20:12 | |
| *** hoonetorg has quit IRC | 20:13 | |
| *** rcernin has joined #openstack-glance | 20:17 | |
| *** k_mouza has joined #openstack-glance | 20:17 | |
| *** k_mouza has quit IRC | 20:21 | |
| *** k_mouza has joined #openstack-glance | 20:21 | |
| *** k_mouza has quit IRC | 20:26 | |
| *** hoonetorg has joined #openstack-glance | 20:30 | |
| *** rcernin has quit IRC | 20:31 | |
| *** rcernin has joined #openstack-glance | 21:00 | |
| *** whoami-rajat__ has quit IRC | 21:10 | |
| *** rcernin has quit IRC | 21:39 | |
| *** rcernin has joined #openstack-glance | 21:56 | |
| *** lbragstad_ has joined #openstack-glance | 22:22 | |
| *** lbragstad has quit IRC | 22:24 | |
| *** rcernin has quit IRC | 22:26 | |
| *** rcernin has joined #openstack-glance | 22:26 | |
| *** tkajinam has quit IRC | 22:59 | |
| *** tkajinam has joined #openstack-glance | 22:59 | |
| *** lbragstad_ is now known as lbragstad | 23:03 | |
| *** lbragstad has quit IRC | 23:21 | |
| *** lbragstad has joined #openstack-glance | 23:23 | |
| openstackgerrit | Dan Smith proposed openstack/glance master: DNM: Test volume quota increase https://review.opendev.org/c/openstack/glance/+/774174 | 23:27 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!