Thursday, 2021-08-26

*** abhishekk is now known as akekane|home03:55
*** akekane|home is now known as abhishekk03:55
opendevreviewAbhishek Kekane proposed openstack/glance master: Move metadef property policy checks in the API  https://review.opendev.org/c/openstack/glance/+/79963503:59
opendevreviewAbhishek Kekane proposed openstack/glance master: Move metadef tag policy checks in the API  https://review.opendev.org/c/openstack/glance/+/79963603:59
opendevreviewAbhishek Kekane proposed openstack/glance master: Implement project personas for metadef namespaces  https://review.opendev.org/c/openstack/glance/+/79870004:02
opendevreviewAbhishek Kekane proposed openstack/glance master: Implement project personas for metadef objects  https://review.opendev.org/c/openstack/glance/+/80205404:02
opendevreviewAbhishek Kekane proposed openstack/glance master: Implement project personas for metadef resource-types  https://review.opendev.org/c/openstack/glance/+/79967104:02
opendevreviewAbhishek Kekane proposed openstack/glance master: Implement project personas for metadef properties  https://review.opendev.org/c/openstack/glance/+/80205504:02
opendevreviewAbhishek Kekane proposed openstack/glance master: Implement project personas for metadef tags  https://review.opendev.org/c/openstack/glance/+/80205604:02
opendevreviewMridula Joshi proposed openstack/glance master: Fix overwriting of existing tags while creating new tags  https://review.opendev.org/c/openstack/glance/+/80496604:40
opendevreviewMerged openstack/glance master: Check policies for Image Tags in API  https://review.opendev.org/c/openstack/glance/+/80458804:45
opendevreviewMridula Joshi proposed openstack/glance master: Fix overwriting of existing tags while creating new tags  https://review.opendev.org/c/openstack/glance/+/80496605:25
opendevreviewAbhishek Kekane proposed openstack/glance-tempest-plugin master: Add protection testing for metadef namespaces  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80090205:57
*** akekane_ is now known as abhishekk06:01
opendevreviewAbhishek Kekane proposed openstack/glance-tempest-plugin master: Add protection testing for metadef namespaces  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80090208:51
opendevreviewAbhishek Kekane proposed openstack/glance-tempest-plugin master: Add protection testing for metadef namespaces  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80090209:38
opendevreviewHanGuangyu proposed openstack/glance master: Make "killed" images list available using the CLI  https://review.opendev.org/c/openstack/glance/+/80613410:28
*** jokke_ is now known as jokke11:07
*** jokke is now known as jokke_11:07
opendevreviewAbhishek Kekane proposed openstack/glance-tempest-plugin master: Implement API protection testing for metadef objects  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80279311:45
opendevreviewMridula Joshi proposed openstack/glance master: Fix overwriting of existing tags while creating new tags  https://review.opendev.org/c/openstack/glance/+/80496613:23
opendevreviewMridula Joshi proposed openstack/glance master: Add doc support for delete-from-store API  https://review.opendev.org/c/openstack/glance/+/80618013:23
rajiv_mucheliihi, how can we calculate glance-api memory requirement?13:29
rajiv_mucheliiI would like to understand, glance-api process patterns when several images of different sizes are uploaded together 13:30
rajiv_mucheliiI see approx 300Mb is required but when I try upload several images, this values spikes to 3.8Gb and kills the process13:31
opendevreviewRajat Dhasmana proposed openstack/glance master: Add doc support for delete-from-store API  https://review.opendev.org/c/openstack/glance/+/80618013:35
opendevreviewRajat Dhasmana proposed openstack/glance master: Fix overwriting of existing tags while creating new tags  https://review.opendev.org/c/openstack/glance/+/80496613:35
opendevreviewErno Kuvaja proposed openstack/glance master: Cache management API endpoints  https://review.opendev.org/c/openstack/glance/+/79202213:50
abhishekkjokke_, rosmaita, dansmith, croelandt, smcginnis glance weekly meeting in 10 minutes13:50
abhishekkat #openstack-meeting13:50
jokke_rajiv_: so yeah the worker count is one factor on the memory consumption. I think that defaults to number of logical cores in the system capped to 4, but each of those workers has pool of 1000 greenlet threads on their disposal15:01
rajiv_in my prod, 5 workers are running in parallel15:02
jokke_these are spun up as needed. but in general you might have 4000 requests being processed at any given time as default config and those will consume decent amount of memory just on their own15:02
jokke_kk15:02
jokke_so that means you might potentially handling 5000 connections per server15:02
jokke_we capped that default few years back as the machines started to get big and suddenly 64 logical cores meant that gapi had 64 workers running and the memory consumption went through the roof just on idle15:03
rajiv_okay, as you aware, my prod runs on k8s. if any image_upload job is assigned to a pod, does the upload get shared among all the workers ? i guess its not ? there is a dedicated PID to follow the process, correct ?15:03
opendevreviewPranali Deore proposed openstack/glance-tempest-plugin master: Add protection testing for metadef namespaces  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80090215:04
opendevreviewPranali Deore proposed openstack/glance-tempest-plugin master: Implement API protection testing for metadef objects  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80279315:04
opendevreviewPranali Deore proposed openstack/glance-tempest-plugin master: Implement API protection testing for metadef resource types  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80279215:04
jokke_The upload is single socket that is attached to a single greenlet thread on a single worker15:05
jokke_and that thread will get eventually garbage collected/recycled after the socket is closed15:05
jokke_I dont think the greenlet threads gets their own pids 'though. Workers definitely do as they are proper child processes15:06
rajiv_yes, i see a PID for the worker jobs in the DEBUG logs 15:07
rajiv_but i am still figuring out on how to set a value or limit upload processes to prevent another crash.15:08
rajiv_are there any tempest tests which can assist here ?15:09
rajiv_can we use the glance-tempest-plugin repo now ? i still dont see tests added here.15:09
jokke_I don't think there is any performance/scale testing happening upstream at all. we used to have rally jobs but I think those haven't been ran for years anymore15:10
jokke_you might need to just set the memory limits very liberal and hope none of your apis gets hit by basically all your uploads15:11
jokke_somewhere like you know 32-64 gigs might give you pretty good margin 15:12
jokke_I don't know how k8s deals with that 'though does it allocate that memory or is it just limit out of shared pool?15:12
jokke_Might be worth of probing other operators on the mailing list too for their experiences and tips 15:13
rajiv_FYR : https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/#requests-and-limits15:13
rajiv_sure, any names ?15:14
rajiv_to be precise : https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/#meaning-of-memory15:15
jokke_rajiv_: yeah just got to that point. It's tricky ... like said I have no experience imposing artificial limits to the process, so won't be much of help unfortunately :(15:17
rajiv_okay, i will set the maximum value possible and monitor for few days, something like hit and try method.15:18
jokke_openstack-discuss mailing list and I think the subject tag is [ops] that lots of the operators are following there. Shoot an e-mail and hopefully there is someone round who would have better experience running the services under k8s15:18
rajiv_thanks a lot jokke_ like always :) 15:19
jokke_NP ... I might have some interesting numbers in the future as I'll be looking closer into the perfscale, but it won't help now 15:20
jokke_feel free to share your findings too. I'm sure your experiences will help others to move on with their deployments too15:21
rajiv_roger that!15:21
abhishekklbragstad, could you please have a look at metadef rbac protection test patch ?15:46
abhishekk#link https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80090215:47
lbragstadabhishekk yeah - i can take a look today 15:50
abhishekklbragstad, ack, thank you15:50
*** akekane_ is now known as abhishekk16:18
* abhishekk going for dinner16:18
* abhishekk signing out for the day17:13
opendevreviewPranali Deore proposed openstack/glance-tempest-plugin master: Add protection testing for metadef namespaces  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80090218:58
opendevreviewPranali Deore proposed openstack/glance-tempest-plugin master: Implement API protection testing for metadef objects  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80279318:58
opendevreviewPranali Deore proposed openstack/glance-tempest-plugin master: Implement API protection testing for metadef resource types  https://review.opendev.org/c/openstack/glance-tempest-plugin/+/80279218:58
lbragstadi generated a sample policy file for glance and noticed that this policy has nested, unescaped double quotes - is that a problem?20:28
lbragstadhttps://github.com/openstack/glance/blob/master/glance/policies/base.py#L6820:28
lbragstad"download_image": "role:admin or (role:member and (project_id:%(project_id)s or project_id:%(member_id)s or "community":%(visibility)s or "public":%(visibility)s or "shared":%(visibility)s))"20:28
dansmithpolicy json? I'm confused21:14

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