*** ramishra has joined #openstack-qa | 00:05 | |
*** ramishra has quit IRC | 00:08 | |
*** ramishra has joined #openstack-qa | 00:25 | |
*** ramishra has quit IRC | 00:25 | |
*** ramishra has joined #openstack-qa | 00:35 | |
openstackgerrit | Wenping Song proposed openstack/tempest master: Remove usage of six https://review.opendev.org/c/openstack/tempest/+/776863 | 00:36 |
---|---|---|
openstackgerrit | Ghanshyam proposed openstack/devstack stable/wallaby: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785544 | 01:16 |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/victoria: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785545 | 01:17 |
*** ramishra has quit IRC | 01:17 | |
*** ramishra has joined #openstack-qa | 01:18 | |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/ussuri: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785546 | 01:18 |
*** ramishra has quit IRC | 01:18 | |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/train: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785547 | 01:20 |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/stein: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785548 | 01:21 |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/rocky: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785388 | 01:22 |
*** ramishra has joined #openstack-qa | 01:28 | |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/queens: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785549 | 01:30 |
openstackgerrit | Ghanshyam proposed openstack/devstack stable/pike: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785550 | 01:31 |
*** hamalq has quit IRC | 01:32 | |
*** spotz has quit IRC | 01:36 | |
*** enriquetaso has quit IRC | 01:37 | |
*** rh-jelabarre has quit IRC | 02:24 | |
*** ramishra has quit IRC | 02:30 | |
*** ramishra has joined #openstack-qa | 02:35 | |
*** rcernin has quit IRC | 02:45 | |
*** ramishra has quit IRC | 02:57 | |
*** ramishra has joined #openstack-qa | 03:03 | |
*** rcernin has joined #openstack-qa | 03:03 | |
*** rcernin has quit IRC | 03:08 | |
*** rcernin has joined #openstack-qa | 03:08 | |
*** enriquetaso has joined #openstack-qa | 03:19 | |
*** ramishra has quit IRC | 03:20 | |
*** ramishra has joined #openstack-qa | 03:31 | |
gmann | yoctozepto: stackviz non-failing patches are ready - https://review.opendev.org/q/Ifee04f28ecee52e74803f1623aba5cfe5ee5ec90 | 03:37 |
*** enriquetaso has quit IRC | 03:44 | |
*** psahoo has joined #openstack-qa | 03:58 | |
*** enriquetaso has joined #openstack-qa | 04:00 | |
*** vishalmanchanda has joined #openstack-qa | 04:16 | |
*** ramishra has quit IRC | 04:30 | |
*** ramishra has joined #openstack-qa | 04:30 | |
*** gcheresh has joined #openstack-qa | 04:58 | |
*** enriquetaso has quit IRC | 04:59 | |
*** gfidente has joined #openstack-qa | 05:14 | |
*** gfidente has quit IRC | 05:20 | |
*** gfidente has joined #openstack-qa | 05:43 | |
*** gcheresh has quit IRC | 05:48 | |
*** slaweq has joined #openstack-qa | 06:11 | |
*** eandersson has quit IRC | 06:15 | |
*** eandersson has joined #openstack-qa | 06:16 | |
*** sboyron has joined #openstack-qa | 06:21 | |
*** eolivare has joined #openstack-qa | 06:27 | |
*** rcernin has quit IRC | 06:50 | |
*** rcernin has joined #openstack-qa | 07:09 | |
*** rcernin has quit IRC | 07:11 | |
*** Yarboa has quit IRC | 07:31 | |
*** Yarboa has joined #openstack-qa | 07:32 | |
*** rpittau|afk is now known as rpittau | 07:33 | |
*** gcheresh has joined #openstack-qa | 07:34 | |
*** ChanServ has quit IRC | 07:58 | |
*** ramishra has quit IRC | 07:58 | |
*** chaconpiza has quit IRC | 07:58 | |
*** dviroel has quit IRC | 07:59 | |
*** vishalmanchanda has quit IRC | 07:59 | |
*** iurygregory has quit IRC | 07:59 | |
*** mugsie has quit IRC | 07:59 | |
*** dasp has quit IRC | 07:59 | |
*** sboyron has quit IRC | 07:59 | |
*** trident has quit IRC | 07:59 | |
*** ccamposr__ has quit IRC | 07:59 | |
*** early has quit IRC | 07:59 | |
*** dansmith has quit IRC | 07:59 | |
*** TViernion has quit IRC | 07:59 | |
*** brtknr_ has quit IRC | 07:59 | |
*** ianw has quit IRC | 07:59 | |
*** hberaud has quit IRC | 07:59 | |
*** priteau has quit IRC | 07:59 | |
*** dtantsur|afk has quit IRC | 07:59 | |
*** tkajinam has quit IRC | 07:59 | |
*** soniya29 has quit IRC | 07:59 | |
*** mjturek has quit IRC | 07:59 | |
*** donnyd has quit IRC | 07:59 | |
*** gmann has quit IRC | 07:59 | |
*** vdrok has quit IRC | 07:59 | |
*** jrosser has quit IRC | 07:59 | |
*** erbarr has quit IRC | 07:59 | |
*** jroll has quit IRC | 07:59 | |
*** csatari has quit IRC | 07:59 | |
*** knikolla has quit IRC | 07:59 | |
*** vhari has quit IRC | 07:59 | |
*** eandersson has quit IRC | 07:59 | |
*** gfidente has quit IRC | 07:59 | |
*** brinzhang_ has quit IRC | 07:59 | |
*** redrobot has quit IRC | 07:59 | |
*** zbr has quit IRC | 07:59 | |
*** pleia2 has quit IRC | 07:59 | |
*** kevinz has quit IRC | 07:59 | |
*** cgoncalves has quit IRC | 07:59 | |
*** zigo has quit IRC | 07:59 | |
*** jpena|off has quit IRC | 07:59 | |
*** ajitha has quit IRC | 07:59 | |
*** amodi has quit IRC | 07:59 | |
*** rpittau has quit IRC | 07:59 | |
*** vkmc has quit IRC | 07:59 | |
*** masayukig has quit IRC | 07:59 | |
*** lxkong has quit IRC | 07:59 | |
*** mgagne has quit IRC | 07:59 | |
*** TheJulia has quit IRC | 07:59 | |
*** clarkb has quit IRC | 07:59 | |
*** mnasiadka has quit IRC | 07:59 | |
*** tbarron has quit IRC | 07:59 | |
*** fungi has quit IRC | 07:59 | |
*** jamespage has quit IRC | 07:59 | |
*** flaviof has quit IRC | 07:59 | |
*** rpioso has quit IRC | 07:59 | |
*** johnsom has quit IRC | 07:59 | |
*** mnaser has quit IRC | 07:59 | |
*** dpaterson has quit IRC | 07:59 | |
*** melwitt has quit IRC | 07:59 | |
*** rnoriega_ has quit IRC | 07:59 | |
*** openstackgerrit has quit IRC | 07:59 | |
*** freerunner has quit IRC | 07:59 | |
*** dosaboy has quit IRC | 07:59 | |
*** frickler has quit IRC | 07:59 | |
*** icey has quit IRC | 07:59 | |
*** dtruong has quit IRC | 07:59 | |
*** freefood_ has quit IRC | 07:59 | |
*** stephenfin has quit IRC | 07:59 | |
*** mtreinish has quit IRC | 07:59 | |
*** amotoki has quit IRC | 07:59 | |
*** abhishekk has quit IRC | 07:59 | |
*** psahoo has quit IRC | 07:59 | |
*** arxcruz has quit IRC | 07:59 | |
*** andreaf has quit IRC | 07:59 | |
*** chandankumar has quit IRC | 07:59 | |
*** jhesketh has quit IRC | 07:59 | |
*** mcarden has quit IRC | 07:59 | |
*** eolivare has quit IRC | 07:59 | |
*** artom has quit IRC | 07:59 | |
*** pots has quit IRC | 07:59 | |
*** yoctozepto has quit IRC | 07:59 | |
*** felixhuettner[m] has quit IRC | 07:59 | |
*** ttx has quit IRC | 07:59 | |
*** DinaBelova has quit IRC | 07:59 | |
*** gouthamr has quit IRC | 07:59 | |
*** rm_work has quit IRC | 07:59 | |
*** akahat has quit IRC | 07:59 | |
*** ralonsoh has quit IRC | 07:59 | |
*** lbragstad has quit IRC | 07:59 | |
*** dmellado has quit IRC | 07:59 | |
*** elod has quit IRC | 07:59 | |
*** bhagyashris has quit IRC | 07:59 | |
*** gcheresh has quit IRC | 07:59 | |
*** Yarboa has quit IRC | 07:59 | |
*** yamamoto has quit IRC | 07:59 | |
*** kopecmartin has quit IRC | 07:59 | |
*** dave-mccowan has quit IRC | 07:59 | |
*** soniya29 has joined #openstack-qa | 08:04 | |
*** tkajinam has joined #openstack-qa | 08:04 | |
*** ianw has joined #openstack-qa | 08:04 | |
*** brtknr_ has joined #openstack-qa | 08:04 | |
*** TViernion has joined #openstack-qa | 08:04 | |
*** dansmith has joined #openstack-qa | 08:04 | |
*** early has joined #openstack-qa | 08:04 | |
*** ccamposr__ has joined #openstack-qa | 08:04 | |
*** trident has joined #openstack-qa | 08:04 | |
*** sboyron has joined #openstack-qa | 08:04 | |
*** ChanServ has joined #openstack-qa | 08:04 | |
*** verne.freenode.net sets mode: +o ChanServ | 08:04 | |
*** vishalmanchanda has joined #openstack-qa | 08:04 | |
*** iurygregory has joined #openstack-qa | 08:04 | |
*** mugsie has joined #openstack-qa | 08:04 | |
*** dasp has joined #openstack-qa | 08:04 | |
*** gcheresh has joined #openstack-qa | 08:04 | |
*** Yarboa has joined #openstack-qa | 08:04 | |
*** yamamoto has joined #openstack-qa | 08:04 | |
*** kopecmartin has joined #openstack-qa | 08:04 | |
*** dave-mccowan has joined #openstack-qa | 08:04 | |
*** mjturek has joined #openstack-qa | 08:05 | |
*** akahat has joined #openstack-qa | 08:05 | |
*** ralonsoh has joined #openstack-qa | 08:05 | |
*** lbragstad has joined #openstack-qa | 08:05 | |
*** dmellado has joined #openstack-qa | 08:05 | |
*** elod has joined #openstack-qa | 08:05 | |
*** bhagyashris has joined #openstack-qa | 08:05 | |
*** psahoo has joined #openstack-qa | 08:05 | |
*** arxcruz has joined #openstack-qa | 08:05 | |
*** andreaf has joined #openstack-qa | 08:05 | |
*** chandankumar has joined #openstack-qa | 08:05 | |
*** mcarden has joined #openstack-qa | 08:05 | |
*** jhesketh has joined #openstack-qa | 08:05 | |
*** ramishra has joined #openstack-qa | 08:05 | |
*** chaconpiza has joined #openstack-qa | 08:05 | |
*** dviroel has joined #openstack-qa | 08:05 | |
*** ttx has joined #openstack-qa | 08:05 | |
*** DinaBelova has joined #openstack-qa | 08:05 | |
*** gouthamr has joined #openstack-qa | 08:05 | |
*** rm_work has joined #openstack-qa | 08:05 | |
*** jroll has joined #openstack-qa | 08:06 | |
*** erbarr has joined #openstack-qa | 08:06 | |
*** jrosser has joined #openstack-qa | 08:06 | |
*** vdrok has joined #openstack-qa | 08:06 | |
*** donnyd has joined #openstack-qa | 08:06 | |
*** gmann has joined #openstack-qa | 08:06 | |
*** knikolla has joined #openstack-qa | 08:06 | |
*** csatari has joined #openstack-qa | 08:06 | |
*** vhari has joined #openstack-qa | 08:06 | |
*** abhishekk has joined #openstack-qa | 08:06 | |
*** amotoki has joined #openstack-qa | 08:06 | |
*** mtreinish has joined #openstack-qa | 08:06 | |
*** stephenfin has joined #openstack-qa | 08:06 | |
*** freefood_ has joined #openstack-qa | 08:06 | |
*** dosaboy has joined #openstack-qa | 08:06 | |
*** frickler has joined #openstack-qa | 08:06 | |
*** icey has joined #openstack-qa | 08:06 | |
*** dtruong has joined #openstack-qa | 08:06 | |
*** eandersson has joined #openstack-qa | 08:06 | |
*** gfidente has joined #openstack-qa | 08:06 | |
*** brinzhang_ has joined #openstack-qa | 08:06 | |
*** redrobot has joined #openstack-qa | 08:06 | |
*** zbr has joined #openstack-qa | 08:06 | |
*** kevinz has joined #openstack-qa | 08:06 | |
*** pleia2 has joined #openstack-qa | 08:06 | |
*** zigo has joined #openstack-qa | 08:06 | |
*** cgoncalves has joined #openstack-qa | 08:06 | |
*** jpena has joined #openstack-qa | 08:06 | |
*** ajitha has joined #openstack-qa | 08:06 | |
*** amodi has joined #openstack-qa | 08:06 | |
*** mgagne has joined #openstack-qa | 08:06 | |
*** rpittau has joined #openstack-qa | 08:06 | |
*** vkmc has joined #openstack-qa | 08:06 | |
*** tbarron has joined #openstack-qa | 08:06 | |
*** masayukig has joined #openstack-qa | 08:06 | |
*** lxkong has joined #openstack-qa | 08:06 | |
*** TheJulia has joined #openstack-qa | 08:06 | |
*** mnasiadka has joined #openstack-qa | 08:06 | |
*** clarkb has joined #openstack-qa | 08:06 | |
*** fungi has joined #openstack-qa | 08:06 | |
*** jamespage has joined #openstack-qa | 08:06 | |
*** rpioso has joined #openstack-qa | 08:06 | |
*** flaviof has joined #openstack-qa | 08:06 | |
*** mnaser has joined #openstack-qa | 08:06 | |
*** johnsom has joined #openstack-qa | 08:06 | |
*** melwitt has joined #openstack-qa | 08:06 | |
*** dpaterson has joined #openstack-qa | 08:06 | |
*** lucasagomes has joined #openstack-qa | 08:06 | |
*** rnoriega_ has joined #openstack-qa | 08:06 | |
*** openstackgerrit has joined #openstack-qa | 08:06 | |
*** freerunner has joined #openstack-qa | 08:06 | |
*** hberaud has joined #openstack-qa | 08:06 | |
*** priteau has joined #openstack-qa | 08:06 | |
*** dtantsur|afk has joined #openstack-qa | 08:06 | |
*** jrosser has quit IRC | 08:07 | |
*** gouthamr has quit IRC | 08:08 | |
*** mnasiadka has quit IRC | 08:08 | |
*** mnasiadka has joined #openstack-qa | 08:09 | |
*** gouthamr has joined #openstack-qa | 08:09 | |
*** tosky has joined #openstack-qa | 08:09 | |
*** jrosser has joined #openstack-qa | 08:10 | |
*** mnasiadka has quit IRC | 08:20 | |
*** mnasiadka has joined #openstack-qa | 08:20 | |
*** gouthamr has quit IRC | 08:20 | |
*** gouthamr has joined #openstack-qa | 08:20 | |
*** jrosser has quit IRC | 08:20 | |
*** jrosser has joined #openstack-qa | 08:20 | |
*** felixhuettner[m] has joined #openstack-qa | 08:40 | |
*** eolivare has joined #openstack-qa | 08:49 | |
*** artom has joined #openstack-qa | 08:49 | |
*** pots has joined #openstack-qa | 08:49 | |
*** yoctozepto has joined #openstack-qa | 08:49 | |
*** yoctozepto has quit IRC | 08:50 | |
*** yoctozepto has joined #openstack-qa | 08:50 | |
*** yoctozepto has quit IRC | 08:53 | |
*** yoctozepto has joined #openstack-qa | 08:53 | |
*** dtantsur|afk is now known as dtantsur | 09:05 | |
*** yamamoto has quit IRC | 09:14 | |
*** dtantsur is now known as dtantsur|afk | 09:29 | |
*** yamamoto has joined #openstack-qa | 10:41 | |
*** brinzhang_ has quit IRC | 10:51 | |
*** enriquetaso has joined #openstack-qa | 11:09 | |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Fix auth ID caps for manila with ceph-mgr https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/782855 | 11:26 |
*** yamamoto has quit IRC | 11:27 | |
*** paras333_ has joined #openstack-qa | 11:27 | |
*** gcheresh has quit IRC | 11:29 | |
*** paras333_ has quit IRC | 11:29 | |
*** paras333 has joined #openstack-qa | 11:29 | |
*** jpena is now known as jpena|lunch | 11:30 | |
*** paras333 has quit IRC | 11:40 | |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Enable testing Ceph master https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/772157 | 11:51 |
*** yamamoto has joined #openstack-qa | 12:04 | |
*** yamamoto has quit IRC | 12:09 | |
*** rh-jelabarre has joined #openstack-qa | 12:15 | |
*** whoami-rajat_ has joined #openstack-qa | 12:17 | |
whoami-rajat_ | hi gmann is there a way to do a raw_request with keystone auth? i want to hit a custom URL that doesn't match the cinder endpoint url structure | 12:18 |
*** jpena|lunch is now known as jpena | 12:29 | |
*** whoami-rajat_ is now known as whoami-rajat | 12:39 | |
yoctozepto | gmann: https://review.opendev.org/c/openstack/devstack/+/761246 haha :D | 12:43 |
gmann | yoctozepto: nice, i was searching in tempest :) | 13:21 |
yoctozepto | :-) | 13:21 |
gmann | kopecmartin: can you check this https://review.opendev.org/q/Ifee04f28ecee52e74803f1623aba5cfe5ee5ec90 | 13:22 |
kopecmartin | sure | 13:23 |
gmann | whoami-rajat: yes, raw_request is meant for that. if you have a valid url then you can do. like we do in version url but that is with modifying the keystone auth url only - https://github.com/openstack/tempest/blob/7e96c8e854386f43604ad098a6ec7606ee676145/tempest/lib/services/volume/v3/versions_client.py#L35 | 13:24 |
*** rpittau is now known as rpittau|bbl | 13:29 | |
whoami-rajat | gmann: but i think versions API doesn't require keystone authentication and in my case i get the error that my raw_request requires authentication | 13:34 |
whoami-rajat | gmann: L#69 which I've commented https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/751990/3/cinder_tempest_plugin/services/default_types_client.py#69 | 13:35 |
gmann | whoami-rajat: oh i thought different endpoint which is there in keystone. | 13:36 |
gmann | whoami-rajat: is new endpoint you are trying is present in keystone? | 13:36 |
gmann | whoami-rajat: 'headers' is arg in raw_request where you need to pass the keystone token otherwise it will fail if service API need to be authenticated via keystone | 13:39 |
whoami-rajat | gmann: oh ok. is there a method in tempest by which i can generate the keystone token to pass into the header? | 13:43 |
whoami-rajat | gmann: also the def request() method uses the same headers as raw_request but doesn't require keystone auth? | 13:44 |
gmann | whoami-rajat: sending keystone auth from tempest is optional but if service need that for quested API then you will get error | 13:44 |
gmann | whoami-rajat: i am not understanding your use case completely. why you are not using current way of Tempest via request() which generate the keystone auth etc | 13:45 |
whoami-rajat | gmann: so the url i pass is changed with the request method here https://github.com/openstack/tempest/blob/master/tempest/lib/common/rest_client.py#L564 | 13:47 |
whoami-rajat | gmann: the url which tempest creates is v3/<project_id>/resource | 13:47 |
whoami-rajat | gmann: the url i want to hit is v3/resource/<project_id> | 13:47 |
gmann | whoami-rajat: ah got it. | 13:47 |
whoami-rajat | it is a new API i added recently and is different from other API formats | 13:47 |
gmann | that is for rbac if i remember right | 13:48 |
whoami-rajat | yes this was added with consideration of RBAC | 13:48 |
gmann | basically you are removing project-id form v3/<project_id> | 13:48 |
gmann | yeah | 13:48 |
gmann | so that is for all the cinder APIs right | 13:49 |
gmann | in that case we need to change the devstack to register the new endpoint for cinder and then tempest to use them. | 13:49 |
gmann | whoami-rajat: but question is which one old or new endpoint to test as default in tempest(cinder-temepst-plugin) | 13:50 |
whoami-rajat | gmann: hmm, isn't there a way to authenticate the raw request? like you said passing the token in headers | 13:50 |
gmann | whoami-rajat: we can do but how you will do for all the APIs | 13:51 |
gmann | for that it is kind of shifting/provide-a-way-to-configure tempest to newendpoint | 13:51 |
whoami-rajat | gmann: so for other APIs, the change isn't decided yet, i just want to test 4 API operations i added recently | 13:51 |
gmann | whoami-rajat: we already have a configurable way for that, we do for nova | 13:52 |
gmann | whoami-rajat: yeah that is something we should do for all or discuss what we should do as default | 13:52 |
gmann | whoami-rajat: I think we can discuss it in PTG with best way forward, what you think? | 13:53 |
whoami-rajat | gmann: sure, i will let you know what cinder plans to do with other APIs and work on a general way to handle this. | 13:54 |
gmann | whoami-rajat: as i mentioned, we already have way to request new endpoint from Tempest if devstack register it so you doing it via raw_request will be duplicate work | 13:54 |
whoami-rajat | ack | 13:54 |
gmann | whoami-rajat: i did not get. you are saying new endpoint is only for few APIs not for all ? | 13:54 |
whoami-rajat | gmann: so the APIs i added recently have this format which i mentioned. we still haven't decided what to do with other (i mean ALL APIs) so i will let you know about that | 13:56 |
whoami-rajat | other = ALL old APIs with /v3/<project_id> format | 13:56 |
gmann | whoami-rajat: humm, that seems confusing for users. and how user will use those new APIs vs existing old APIs | 13:56 |
gmann | whoami-rajat: they would not like to maintain two separate endpoint format for requesting APIs form same service. | 13:57 |
gmann | whoami-rajat: I think this is good topic to discuss in PTG, can you add it in https://etherpad.opendev.org/p/qa-xena-ptg | 13:57 |
gmann | whoami-rajat: based on when cinder team is available. and we can see how we can test the way cinder decide about endpoint | 13:58 |
whoami-rajat | yeah, so I've added 4 new APIs for a feature maybe we will reformat those also and decide on a general format | 13:58 |
whoami-rajat | sure | 13:58 |
gmann | ok | 13:58 |
whoami-rajat | i will discuss with Brian and see what he thinks | 13:58 |
gmann | sure | 13:59 |
whoami-rajat | thanks for your help | 13:59 |
gmann | whoami-rajat: or if cinder team have sessions then can have cross project sessions with qa | 13:59 |
gmann | whoami-rajat: also main point is we thought of new endpoint for testing the RBAC for all the APIs, so if old APIs are not going with new endpoint then how we will solve the original issue of testing new RBAC | 14:00 |
whoami-rajat | agreed | 14:13 |
whoami-rajat | gmann: so i discussed with Brian and the new endpoint proposed will be /v3/ without the <project-id> so the new APIs i added are compatible | 14:20 |
whoami-rajat | gmann: and the solution you suggested is to register a new endpoint with devstack and then tempest can use it right? | 14:20 |
gmann | whoami-rajat: and user can request existing APIs also with new endpoint? | 14:22 |
gmann | whoami-rajat: and hope you already modified the cinder wsgi layer to accept the url without project-id ? | 14:24 |
whoami-rajat | gmann: yeah, that's the change we intent to do with the existing APIs | 14:24 |
whoami-rajat | gmann: no, that's the plan for Xena | 14:24 |
*** gcheresh has joined #openstack-qa | 14:25 | |
gmann | whoami-rajat: ok. | 14:25 |
gmann | whoami-rajat: here you can register the new endpoint in devstack - https://github.com/openstack/devstack/blob/e53142ed0d314f07d974a104005be2120056d629/lib/cinder#L363 | 14:26 |
gmann | whoami-rajat: you can give new name also which we can use in tempest to query for | 14:27 |
whoami-rajat | gmann: i see lbragstad has already pushed one patch https://review.opendev.org/c/openstack/devstack/+/776520 | 14:35 |
gmann | whoami-rajat: yeah like that. | 14:36 |
whoami-rajat | gmann: i think that will work for my case (and for other future API changes), it's the same url i intend to use | 14:37 |
lbragstad | context on that change if you need it http://lists.openstack.org/pipermail/openstack-discuss/2021-February/020605.html | 14:38 |
whoami-rajat | thanks lbragstad , one ques: why is it WIP? | 14:40 |
lbragstad | because i hit an issue where if the first endpoint in a project-scoped token is the cinder endpoint without the project-id, the request failed | 14:41 |
gmann | lbragstad: whoami-rajat we need to change in wsgi layer in cinder also. | 14:41 |
*** gcheresh has quit IRC | 14:42 | |
lbragstad | project-scoped tokens will contain all cinder endpoints, including the one without project IDs in the URl | 14:42 |
lbragstad | if a project client attempts to use the endpoint without that project ID, cinder is going to respond with a 400 | 14:42 |
lbragstad | https://review.opendev.org/c/openstack/cinder/+/776468 is the WIP patch i have for relaxing cinder's project ID validation | 14:43 |
gmann | yeah this one https://review.opendev.org/c/openstack/cinder/+/776468/2/cinder/api/openstack/wsgi.py | 14:43 |
*** psahoo has quit IRC | 14:57 | |
*** gcheresh has joined #openstack-qa | 15:20 | |
gmann | lbragstad: replied on ML thread on cinder endpoints testing http://lists.openstack.org/pipermail/openstack-discuss/2021-April/021740.html | 16:00 |
gmann | sorry for late response, I missed that email | 16:00 |
*** gcheresh has quit IRC | 16:01 | |
gmann | that is how we did for nova also when nova provided the new endpoint without-project-id | 16:01 |
*** lucasagomes has quit IRC | 16:01 | |
gmann | kopecmartin: would you like to +A on these or waiting for master patch to be green in zuul ? - https://review.opendev.org/q/Ifee04f28ecee52e74803f1623aba5cfe5ee5ec90 | 16:05 |
kopecmartin | gmann: oh, yeah, i'll +a them , i got distracted and forgot about the tab where i had it opened :D | 16:09 |
gmann | kopecmartin: thanks | 16:09 |
lbragstad | gmann ack - cc whoami-rajat ^ | 16:12 |
lbragstad | gmann i really only hacked on it to see what i absolutely needed to do to get cinder to consume a system-scoped token | 16:12 |
lbragstad | it was only a PoC to show a path forward, i'm not sure what direction the cinder folks want to take with that work | 16:13 |
*** yamamoto has joined #openstack-qa | 16:13 | |
gmann | lbragstad: whoami-rajat point to discuss or decide is which endpoint we want to test as default. | 16:13 |
gmann | lbragstad: yeah, depends on cinder team which endpoint they want to keep as API contract and which one as optional | 16:14 |
gmann | nova did without project-id as recommended and keep supporting project-id one too for backword compatibility | 16:14 |
*** yamamoto has quit IRC | 16:21 | |
*** yamamoto has joined #openstack-qa | 16:23 | |
lbragstad | gmann yeah - that makes sense | 16:23 |
lbragstad | i just think we need to work through ensuring project clients don't trip using the non-project id endpoint for cinder | 16:24 |
lbragstad | so - maybe make the cinderclient smarter? | 16:24 |
gmann | lbragstad: +1, same for osc also. we need to move them to new endpoint at some time (I will say soon) | 16:25 |
lbragstad | well - i think we'll need to make that change regardless if cinder decides to leave the project_id in the API | 16:26 |
gmann | agree | 16:27 |
lbragstad | i think the lowest possible bar here would be for cinder to just leave the project_id in there, and have the client tack on a fake project_id or something, when dealing with system scope | 16:27 |
lbragstad | then just relax the project_id matching assumptions in cinder to allow it to consume system-scope | 16:27 |
lbragstad | then we're in a position to start making the policy changes to consume system-scope | 16:27 |
gmann | but supporting both would not change much or existing client/users | 16:28 |
lbragstad | supporting both? | 16:29 |
gmann | project-id and no project-id in url | 16:29 |
lbragstad | oh - i'm saying cinder would still require a project-id in the URL | 16:29 |
gmann | humm why | 16:30 |
lbragstad | but - we'd need to 1.) add a non-project id endpoint in the catalog for system users 2.) patch cinderclient to stub in a fake project id for system-scoped requests | 16:30 |
lbragstad | i think it would be the lowest possible bar for cinder to actually consume system-scope | 16:30 |
lbragstad | i'm not advocating that's the right or only approach, but i think it's the least amount of work required to actually get system-scope integrated into cinder | 16:31 |
gmann | so for 1st cinder need to support non-project-id right? this one https://review.opendev.org/c/openstack/cinder/+/776468/2/cinder/api/openstack/wsgi.py | 16:31 |
lbragstad | yeah - or it would need to ignore the URL project_id if context.systme_scope | 16:32 |
gmann | I hope that project-id form url is not populated in DB or so | 16:32 |
lbragstad | in keystone or in cinder? | 16:32 |
gmann | cinder | 16:32 |
* lbragstad shrugs | 16:32 | |
lbragstad | that's a good question - i'm not sure | 16:33 |
lbragstad | it could be persisted as the project_id of the object (e.g., volume) | 16:33 |
lbragstad | maybe? | 16:33 |
gmann | yeah, that is my worry. | 16:33 |
lbragstad | i'd need to dig through the code or find a cinder expert | 16:33 |
gmann | +1 | 16:34 |
gmann | this is sessions for this topic right? tuesday 13 UTC L83 ?https://etherpad.opendev.org/p/policy-popup-xena-ptg | 16:35 |
lbragstad | gmann yep | 16:40 |
gmann | ok, | 16:40 |
*** yamamoto has quit IRC | 16:49 | |
*** dviroel has quit IRC | 16:57 | |
*** jpena is now known as jpena|off | 16:58 | |
openstackgerrit | James Parker proposed openstack/whitebox-tempest-plugin master: WIP: Test allow disabling CPU flags https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/776112 | 17:22 |
*** yamamoto has joined #openstack-qa | 17:24 | |
*** gfidente is now known as gfidente|afk | 17:28 | |
*** yamamoto has quit IRC | 17:29 | |
kopecmartin | gmann: I see and email on ML - "[ptl] Wallaby Release Community Meeting" .. have we done this in the past? I'm not sure what kind of updates we would like to announce | 17:30 |
kopecmartin | * I see *this* email .. | 17:30 |
whoami-rajat | lbragstad: gmann the volume, snapshot, backup etc resources take project_id from context and store it in DB. i think context object will still have a project_id even if it isn't in the url right? | 17:36 |
openstackgerrit | James Parker proposed openstack/whitebox-tempest-plugin master: WIP: Test allow disabling CPU flags https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/776112 | 17:36 |
*** gcheresh has joined #openstack-qa | 17:41 | |
*** gcheresh has quit IRC | 17:53 | |
*** gcheresh has joined #openstack-qa | 17:55 | |
gmann | kopecmartin: we have not done that, QA does not have release highlights as such so we did not do | 18:00 |
kopecmartin | gmann: good, just wanted to double check | 18:01 |
gmann | whoami-rajat: yeah, context has the project_id for project scope. project-id in url is different than that | 18:01 |
lbragstad | whoami-rajat contents objects are not guaranteed to have project_ids | 18:08 |
lbragstad | s/contents/contexts/ | 18:08 |
lbragstad | context.project_id will be None for system-scoped tokens | 18:08 |
lbragstad | whoami-rajat we have a similar hurdle with nova http://lists.openstack.org/pipermail/openstack-discuss/2021-March/021131.html | 18:09 |
*** dviroel has joined #openstack-qa | 18:17 | |
*** eolivare has quit IRC | 18:19 | |
*** sboyron has quit IRC | 18:22 | |
*** spotz has joined #openstack-qa | 18:41 | |
*** vishalmanchanda has quit IRC | 18:55 | |
*** gcheresh has quit IRC | 18:57 | |
*** ajitha has quit IRC | 19:04 | |
*** slaweq has quit IRC | 19:26 | |
*** gfidente|afk has quit IRC | 19:37 | |
*** ralonsoh has quit IRC | 19:39 | |
openstackgerrit | James Parker proposed openstack/whitebox-tempest-plugin master: WIP: Test allow disabling CPU flags https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/776112 | 19:44 |
*** whoami-rajat has quit IRC | 20:00 | |
openstackgerrit | Elod Illes proposed openstack/grenade stable/train: WIP: Fix upper-constrain path for smoke test https://review.opendev.org/c/openstack/grenade/+/785730 | 20:38 |
*** rpittau|bbl is now known as rpittau | 20:39 | |
*** yamamoto has joined #openstack-qa | 20:42 | |
*** yamamoto has quit IRC | 20:47 | |
openstackgerrit | Merged openstack/devstack stable/ussuri: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785546 | 21:10 |
openstackgerrit | Merged openstack/devstack stable/pike: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785550 | 21:10 |
*** amodi has quit IRC | 21:12 | |
openstackgerrit | Elod Illes proposed openstack/grenade stable/train: WIP: Fix upper-constrain path for smoke test https://review.opendev.org/c/openstack/grenade/+/785730 | 21:38 |
*** rpittau is now known as rpittau|afk | 22:04 | |
*** Yarboa has quit IRC | 22:12 | |
*** Yarboa has joined #openstack-qa | 22:14 | |
*** yamamoto has joined #openstack-qa | 22:16 | |
*** ajitha has joined #openstack-qa | 22:51 | |
openstackgerrit | Merged openstack/devstack stable/rocky: Make stackviz tasks not to fail jobs https://review.opendev.org/c/openstack/devstack/+/785388 | 23:17 |
*** yamamoto has quit IRC | 23:18 | |
*** tosky has quit IRC | 23:29 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!