*** imacdonn has quit IRC | 01:19 | |
*** imacdonn has joined #openstack-keystone | 01:19 | |
*** gyee has quit IRC | 01:22 | |
openstackgerrit | wangxiyuan proposed openstack/keystone master: Add a test for idp and federated user cascade deleting https://review.openstack.org/591946 | 01:23 |
---|---|---|
*** ducnv has quit IRC | 01:42 | |
lbragstad | kmalloc i think right now it's an empty list | 02:25 |
lbragstad | clarkb i think we're already filling that requirement in keystone? https://git.openstack.org/cgit/openstack/keystone/tree/requirements.txt#n37 | 02:27 |
lbragstad | if we're the only project using pysaml2 (i'd be surprised if we weren't) i can propose a patch to openstack/requirements bumping the lower constraint if needed | 02:28 |
clarkb | hrm I wonder if it is an older branch | 02:36 |
*** shyamb has joined #openstack-keystone | 02:39 | |
*** shyamb has quit IRC | 03:22 | |
*** ykarel has joined #openstack-keystone | 03:24 | |
openstackgerrit | Vishakha Agarwal proposed openstack/keystone master: Incorrect use of translation _() https://review.openstack.org/596683 | 03:29 |
*** ykarel_ has joined #openstack-keystone | 03:31 | |
*** ykarel has quit IRC | 03:33 | |
*** dave-mccowan has quit IRC | 03:38 | |
*** nicolasbock has quit IRC | 03:40 | |
*** ykarel__ has joined #openstack-keystone | 03:58 | |
*** ykarel_ has quit IRC | 03:58 | |
*** shyamb has joined #openstack-keystone | 04:06 | |
*** shyamb has quit IRC | 04:19 | |
*** ykarel__ has quit IRC | 04:22 | |
*** markvoelker has joined #openstack-keystone | 04:34 | |
*** shyamb has joined #openstack-keystone | 04:38 | |
*** shyamb has quit IRC | 04:46 | |
*** ykarel__ has joined #openstack-keystone | 04:58 | |
*** ykarel__ is now known as ykarel | 05:00 | |
*** shyamb has joined #openstack-keystone | 05:26 | |
*** shyamb has quit IRC | 05:37 | |
openstackgerrit | Vishakha Agarwal proposed openstack/keystone master: Mapped Groups don't exist breaks WebSSO https://review.openstack.org/597992 | 05:45 |
*** shyamb has joined #openstack-keystone | 05:46 | |
*** openstackgerrit has quit IRC | 06:07 | |
*** shyamb has quit IRC | 06:15 | |
*** shyamb has joined #openstack-keystone | 06:15 | |
*** hoonetorg has quit IRC | 06:39 | |
*** dklyle has quit IRC | 06:43 | |
*** dklyle has joined #openstack-keystone | 06:44 | |
*** hoonetorg has joined #openstack-keystone | 06:46 | |
*** shyamb has quit IRC | 07:01 | |
*** shyamb has joined #openstack-keystone | 07:12 | |
*** pcaruana has joined #openstack-keystone | 07:14 | |
*** rcernin has quit IRC | 07:20 | |
*** breton has joined #openstack-keystone | 07:35 | |
*** chaconpiza has joined #openstack-keystone | 07:36 | |
*** shyamb has quit IRC | 07:54 | |
*** hoonetorg has quit IRC | 07:55 | |
*** Emine has joined #openstack-keystone | 08:23 | |
*** ykarel is now known as ykarel|lunch | 08:30 | |
*** markvoelker has quit IRC | 08:38 | |
*** shyamb has joined #openstack-keystone | 08:40 | |
*** markvoelker has joined #openstack-keystone | 09:13 | |
*** markvoelker has quit IRC | 09:15 | |
*** ykarel|lunch is now known as ykarel | 09:18 | |
*** shyamb has quit IRC | 09:26 | |
*** shyamb has joined #openstack-keystone | 09:27 | |
*** shyamb has quit IRC | 09:31 | |
*** openstackgerrit has joined #openstack-keystone | 09:44 | |
openstackgerrit | wangxiyuan proposed openstack/oslo.limit master: Add opts file https://review.openstack.org/586760 | 09:44 |
openstackgerrit | wangxiyuan proposed openstack/oslo.limit master: Add limit check func https://review.openstack.org/596520 | 09:44 |
wxy-xiyuan | lbragstad: Please check the new PS. I'm not sure I'm on the right direction. Please ignore the UT failure. I'll fix it once we are on the same page. | 09:45 |
*** shyamb has joined #openstack-keystone | 10:03 | |
vishakha | wxy-xiyuan: Hi. Having a little doubt that code for logging translated string is done https://github.com/openstack/keystone/blob/master/keystone/tests/hacking/checks.py#L287-L300 | 10:10 |
*** deepak_mourya_ has joined #openstack-keystone | 10:11 | |
deepak_mourya_ | cmurphy|vacation: Hi, just a reminder that I have updated the patchset, you can review the same https://review.openstack.org/#/c/484348/ | 10:13 |
*** jaosorior has quit IRC | 10:25 | |
*** jaosorior has joined #openstack-keystone | 10:27 | |
openstackgerrit | Vishakha Agarwal proposed openstack/keystone master: Mapped Groups don't exist breaks WebSSO https://review.openstack.org/597992 | 10:32 |
openstackgerrit | Vishakha Agarwal proposed openstack/keystone master: Mapped Groups don't exist breaks WebSSO https://review.openstack.org/597992 | 10:34 |
*** shyamb has quit IRC | 10:41 | |
*** nicolasbock has joined #openstack-keystone | 10:47 | |
*** shyamb has joined #openstack-keystone | 11:00 | |
*** rnm has joined #openstack-keystone | 11:34 | |
*** rnm is now known as rmart04 | 11:36 | |
*** threestrands has quit IRC | 11:38 | |
*** rmart04 has quit IRC | 11:40 | |
*** rnm has joined #openstack-keystone | 11:40 | |
*** rnm has quit IRC | 11:42 | |
*** rnm has joined #openstack-keystone | 11:42 | |
*** rnm is now known as rmart04 | 11:43 | |
*** shyamb has quit IRC | 11:47 | |
*** dmellado has quit IRC | 11:50 | |
*** shyamb has joined #openstack-keystone | 11:50 | |
wxy-xiyuan | vishakha: good catch. But the pep8 CI didn't catch the problem. So the code is not strong enough. | 11:52 |
wxy-xiyuan | vishakha: https://github.com/openstack/keystone/blob/master/keystone/tests/unit/ksfixtures/hacking.py#L221-L230 So we treat it as OK. It should be fixed IMO. | 12:02 |
*** raildo has joined #openstack-keystone | 12:48 | |
*** mchlumsky has joined #openstack-keystone | 12:50 | |
*** shyamb has quit IRC | 12:55 | |
*** mchlumsky has quit IRC | 13:10 | |
*** mchlumsky has joined #openstack-keystone | 13:12 | |
*** raildo has quit IRC | 13:24 | |
lbragstad | wxy-xiyuan sounds good - i'll take a look | 13:28 |
*** raildo has joined #openstack-keystone | 13:31 | |
*** jistr is now known as jistr|call | 13:51 | |
*** ykarel is now known as ykarel|away | 14:00 | |
*** ykarel|away has quit IRC | 14:06 | |
*** zioproto has joined #openstack-keystone | 14:13 | |
zioproto | hello | 14:13 |
zioproto | I debugging something in the kvs redis backend | 14:13 |
zioproto | I know it is deprecated | 14:14 |
zioproto | but the performance is very good, and we are trying to make it work | 14:14 |
zioproto | it works very well, but we have a problem with some tokens not being found in redis even if they are still valid | 14:14 |
zioproto | I am trying to debug it | 14:14 |
zioproto | but it is not clear to me how the redis key hashes are computed when writing data to the backend | 14:14 |
zioproto | is anyone familiar with that part of the code ? | 14:15 |
*** mpasserini has joined #openstack-keystone | 14:17 | |
*** jistr|call is now known as jistr | 14:21 | |
*** ykarel has joined #openstack-keystone | 14:35 | |
*** r-daneel has joined #openstack-keystone | 14:47 | |
*** ykarel has quit IRC | 14:54 | |
*** ykarel has joined #openstack-keystone | 15:03 | |
*** pcaruana has quit IRC | 15:12 | |
*** r-daneel has quit IRC | 15:13 | |
*** r-daneel_ has joined #openstack-keystone | 15:13 | |
*** r-daneel_ is now known as r-daneel | 15:16 | |
*** eglute has joined #openstack-keystone | 15:25 | |
*** ykarel has quit IRC | 15:25 | |
*** hwoarang has joined #openstack-keystone | 15:26 | |
*** ykarel has joined #openstack-keystone | 15:29 | |
*** Emine has quit IRC | 15:29 | |
*** mpasserini has quit IRC | 15:32 | |
*** ykarel_ has joined #openstack-keystone | 15:38 | |
*** ykarel has quit IRC | 15:39 | |
*** gyee has joined #openstack-keystone | 15:42 | |
*** ykarel_ is now known as ykarel | 15:55 | |
*** ykarel has quit IRC | 16:00 | |
*** fiddletwix has quit IRC | 16:08 | |
*** ykarel has joined #openstack-keystone | 16:14 | |
*** ykarel has quit IRC | 16:17 | |
*** ykarel has joined #openstack-keystone | 16:25 | |
*** ykarel_ has joined #openstack-keystone | 16:29 | |
*** ykarel has quit IRC | 16:31 | |
*** rmart04 has quit IRC | 16:41 | |
*** r-daneel_ has joined #openstack-keystone | 16:47 | |
*** r-daneel has quit IRC | 16:47 | |
*** r-daneel_ is now known as r-daneel | 16:47 | |
kmalloc | zioproto: what version of OpenStack? | 16:48 |
kmalloc | You're using it as a token store? | 16:48 |
kmalloc | I am very familiar with the code, I wrote/fixed/changed a lot of it while it was maintained. | 16:48 |
*** ykarel_ has quit IRC | 16:53 | |
*** ykarel has joined #openstack-keystone | 16:55 | |
*** ykarel has quit IRC | 16:55 | |
*** ykarel has joined #openstack-keystone | 17:01 | |
*** ykarel_ has joined #openstack-keystone | 17:06 | |
*** ykarel has quit IRC | 17:08 | |
*** Emine has joined #openstack-keystone | 17:28 | |
*** ykarel_ is now known as ykarel|away | 17:32 | |
*** ykarel|away has quit IRC | 17:36 | |
*** dmellado has joined #openstack-keystone | 17:44 | |
*** prashkre__ has joined #openstack-keystone | 17:54 | |
*** prashkre__ has quit IRC | 17:55 | |
*** r-daneel has quit IRC | 17:55 | |
*** prashkre has joined #openstack-keystone | 17:55 | |
*** gyee has quit IRC | 17:56 | |
*** r-daneel has joined #openstack-keystone | 17:59 | |
*** r-daneel_ has joined #openstack-keystone | 18:08 | |
*** r-daneel has quit IRC | 18:09 | |
*** r-daneel_ is now known as r-daneel | 18:09 | |
*** Emine has quit IRC | 18:35 | |
*** Emine has joined #openstack-keystone | 18:47 | |
*** fiddletwix has joined #openstack-keystone | 19:00 | |
*** r-daneel has quit IRC | 19:17 | |
gagehugo | lbragstad I think the keystoneauth ocata docs gate is borked | 19:18 |
* lbragstad sigh | 19:26 | |
lbragstad | python3 patches? | 19:27 |
gagehugo | yeah, plus my cherry picked sha256 one | 19:30 |
gagehugo | http://logs.openstack.org/12/598212/1/check/build-openstack-sphinx-docs/dda9886/job-output.txt.gz | 19:31 |
gagehugo | actually maybe not? | 19:34 |
gagehugo | maybe the gate is just having issues | 19:34 |
gagehugo | http://logs.openstack.org/76/597676/1/check/build-openstack-sphinx-docs/5b06397/job-output.txt.gz | 19:35 |
gagehugo | looks like the same issue, so just ocata | 19:35 |
gagehugo | but it passes pep8, so I wonder if ocata bandit is capped at <= 1.4 | 19:35 |
openstackgerrit | Gage Hugo proposed openstack/keystone master: DNM/WIP Convert domains api to flask https://review.openstack.org/597350 | 19:37 |
lbragstad | https://github.com/openstack/requirements/blob/stable/ocata/global-requirements.txt#L333 is the requirement for ocata | 19:39 |
gagehugo | weird | 19:39 |
lbragstad | https://github.com/openstack/requirements/blob/stable/ocata/upper-constraints.txt#L76 | 19:39 |
lbragstad | ^ that's the upper constraint though | 19:39 |
gagehugo | ah ha | 19:40 |
lbragstad | are we proposing something to stable/ocata that requires > 1.4? | 19:40 |
gagehugo | well pike and queens allow 1.5, which is causing the pep8 issue | 19:40 |
lbragstad | 1.5 is causing the pep8 issue? | 19:41 |
gagehugo | yeah | 19:41 |
gagehugo | 1.5 has SHA1 added as unsafe | 19:41 |
gagehugo | but KSA just uses it to hash stuff for logging | 19:41 |
gagehugo | so it's not really a huge concern | 19:41 |
lbragstad | http://logs.openstack.org/76/597676/1/check/build-openstack-sphinx-docs/5b06397/job-output.txt.gz#_2018-08-29_20_54_01_780318 | 19:42 |
lbragstad | that's the failure, right? | 19:42 |
gagehugo | I guess? | 19:43 |
lbragstad | http://logs.openstack.org/12/598212/1/check/build-openstack-sphinx-docs/dda9886/job-output.txt.gz#_2018-08-30_18_32_46_696753 | 19:44 |
gagehugo | https://github.com/openstack/requirements/blob/stable/ocata/upper-constraints.txt#L359 | 19:44 |
lbragstad | have you tried recreating that failure locally? | 19:44 |
gagehugo | requests is capped at 2.12.5, but openstack-requirements is requiring a newer version | 19:45 |
lbragstad | so we need to backport a dependency update? | 19:45 |
* gagehugo is clueless | 19:46 | |
gagehugo | I guess so? | 19:46 |
gagehugo | tox -e docs works for me on stable/ocata | 19:47 |
gagehugo | but I think the gate runs it differently | 19:47 |
openstackgerrit | Merged openstack/keystonemiddleware master: import zuul job settings from project-config https://review.openstack.org/597659 | 19:47 |
openstackgerrit | Merged openstack/keystonemiddleware master: switch documentation job to new PTI https://review.openstack.org/597660 | 19:47 |
lbragstad | gagehugo what if you rebuild your local env? | 19:49 |
lbragstad | (tox -re py27) | 19:49 |
gagehugo | works fine | 19:49 |
lbragstad | huh | 19:50 |
lbragstad | might be worth a gander to the -requirements channel to see if they have any pointers | 19:50 |
gagehugo | ok | 19:50 |
*** r-daneel has joined #openstack-keystone | 19:55 | |
*** mchlumsky has quit IRC | 20:05 | |
openstackgerrit | Merged openstack/keystone master: import zuul job settings from project-config https://review.openstack.org/597652 | 20:12 |
openstackgerrit | Merged openstack/keystone master: switch documentation job to new PTI https://review.openstack.org/597653 | 20:12 |
openstackgerrit | Merged openstack/keystone master: add python 3.6 unit test job https://review.openstack.org/597654 | 20:12 |
lbragstad | gagehugo was this just failing for ksa? | 20:24 |
gagehugo | lbragstad afaik yeah | 20:25 |
* gagehugo looks at ksm | 20:25 | |
gagehugo | oh | 20:25 |
gagehugo | ksm is failing too | 20:25 |
gagehugo | likely the same issue | 20:25 |
*** mugsie has quit IRC | 20:25 | |
*** prashkre has quit IRC | 20:26 | |
gagehugo | https://review.openstack.org/#/c/597677/ | 20:26 |
*** mugsie has joined #openstack-keystone | 20:38 | |
*** harlowja has joined #openstack-keystone | 20:43 | |
*** rmart04 has joined #openstack-keystone | 20:46 | |
*** raildo has quit IRC | 20:50 | |
*** rmart04 has quit IRC | 21:16 | |
*** dmellado has quit IRC | 21:31 | |
*** dmellado has joined #openstack-keystone | 21:31 | |
*** rcernin has joined #openstack-keystone | 21:49 | |
lbragstad | gagehugo i'm not sure what the deal is with https://review.openstack.org/#/c/598346/7 | 22:14 |
kmalloc | lbragstad: ^ that looks like an ansible error | 22:17 |
lbragstad | yeah - ARA specifically | 22:17 |
lbragstad | i resolved the rest of the errors | 22:17 |
lbragstad | apparently a new version of ARA was released today, but i'm not sure how and if it is supposed to affect stable branches | 22:17 |
kmalloc | upper-constraints issue(s) | 22:18 |
clarkb | the ara + devstack-gate issue is being addressed, I don't think that is related to upper constraints or other requirements issues | 22:25 |
lbragstad | oh - so http://logs.openstack.org/46/598346/6/check/legacy-tempest-dsvm-neutron-src/b1624d3/job-output.txt.gz#_2018-08-30_21_29_58_559147 is not related to that patch? | 22:25 |
clarkb | that failure is related to new ara release incompatbile with the version of ansible devstack-gate was using. Fix is to use newer ansible in devstack gate and has been approved | 22:26 |
lbragstad | clarkb awesome | 22:26 |
*** adriant has quit IRC | 23:23 | |
*** adriant has joined #openstack-keystone | 23:23 | |
*** r-daneel has quit IRC | 23:36 | |
*** rcernin has quit IRC | 23:56 | |
*** rcernin has joined #openstack-keystone | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!