| *** topol has quit IRC | 00:20 | |
| *** matsuhas_ has quit IRC | 01:24 | |
| *** morganfainberg_Z is now known as morganfainberg | 01:33 | |
| *** diegows has quit IRC | 02:06 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/keystone: Convert explicit session get/being to transaction context https://review.openstack.org/97058 | 02:13 |
|---|---|---|
| openstackgerrit | Morgan Fainberg proposed a change to openstack/keystone: Remove `with_lockmode` use from Trust SQL backend. https://review.openstack.org/97059 | 02:17 |
| *** dims_ has quit IRC | 02:43 | |
| *** nsquare has quit IRC | 02:46 | |
| *** openstackgerrit has quit IRC | 02:51 | |
| *** openstackgerrit has joined #openstack-keystone | 02:52 | |
| *** mberlin has joined #openstack-keystone | 02:57 | |
| *** mberlin1 has quit IRC | 02:58 | |
| *** openstackgerrit has quit IRC | 03:00 | |
| *** mfisch has quit IRC | 03:46 | |
| *** mfisch has joined #openstack-keystone | 03:50 | |
| *** mfisch has joined #openstack-keystone | 03:50 | |
| *** nsquare has joined #openstack-keystone | 04:33 | |
| *** openstackgerrit has joined #openstack-keystone | 04:34 | |
| *** arunkant has quit IRC | 04:36 | |
| *** huats_ has quit IRC | 04:39 | |
| *** huats has joined #openstack-keystone | 04:40 | |
| *** huats has quit IRC | 04:40 | |
| *** huats has joined #openstack-keystone | 04:40 | |
| *** arunkant has joined #openstack-keystone | 04:43 | |
| *** stevemar has joined #openstack-keystone | 04:55 | |
| *** ukalifon1 has joined #openstack-keystone | 04:59 | |
| morganfainberg | stevemar, isn't it like the weekend or something... (/pot, meet kettle) | 05:27 |
| *** stevemar has quit IRC | 05:30 | |
| *** pheadron has joined #openstack-keystone | 05:37 | |
| *** stevemar has joined #openstack-keystone | 05:49 | |
| *** nsquare has quit IRC | 05:52 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/keystone: Imported Translations from Transifex https://review.openstack.org/97005 | 06:00 |
| *** ukalifon1 has quit IRC | 06:02 | |
| *** nsquare has joined #openstack-keystone | 06:32 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/keystone: Properly invalidate cache for get_*_by_name methods https://review.openstack.org/97082 | 06:34 |
| *** stevemar has quit IRC | 06:40 | |
| *** stevemar has joined #openstack-keystone | 06:44 | |
| *** bearhands is now known as comstud | 07:01 | |
| *** stevemar has quit IRC | 07:51 | |
| *** nsquare has quit IRC | 07:54 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/keystone: pep8: do not test locale files https://review.openstack.org/97089 | 08:19 |
| *** leseb has joined #openstack-keystone | 09:17 | |
| *** bvandenh has quit IRC | 10:24 | |
| *** leseb has quit IRC | 10:57 | |
| *** henrynash has joined #openstack-keystone | 11:03 | |
| *** leseb has joined #openstack-keystone | 11:58 | |
| *** leseb has quit IRC | 12:03 | |
| *** leseb has joined #openstack-keystone | 12:12 | |
| *** leseb has quit IRC | 12:16 | |
| *** leseb has joined #openstack-keystone | 13:02 | |
| *** dims has joined #openstack-keystone | 13:03 | |
| *** leseb has quit IRC | 13:15 | |
| *** leseb has joined #openstack-keystone | 13:18 | |
| *** dims has quit IRC | 13:22 | |
| *** leseb has quit IRC | 13:48 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/keystone: Add v3 curl examples https://review.openstack.org/96973 | 15:07 |
| openstackgerrit | Brant Knudson proposed a change to openstack/keystone: Fix curl example refs in docs https://review.openstack.org/96966 | 15:07 |
| *** rodrigods has joined #openstack-keystone | 16:00 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token token cache members to class https://review.openstack.org/97109 | 16:16 |
| *** nsquare has joined #openstack-keystone | 16:32 | |
| *** hrybacki has joined #openstack-keystone | 16:45 | |
| *** dims has joined #openstack-keystone | 16:47 | |
| *** diegows has joined #openstack-keystone | 17:12 | |
| *** leseb has joined #openstack-keystone | 17:25 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/keystone-specs: Add spec for non-persistent-tokens https://review.openstack.org/95976 | 17:47 |
| *** dims has quit IRC | 18:02 | |
| *** rodrigods has quit IRC | 18:11 | |
| *** leseb has quit IRC | 18:12 | |
| *** rodrigods has joined #openstack-keystone | 18:15 | |
| *** leseb has joined #openstack-keystone | 18:20 | |
| *** Ajaeger has joined #openstack-keystone | 18:26 | |
| Ajaeger | Hi, I would love to have some review for https://review.openstack.org/#/c/97089/ so that translation imports work again. | 18:27 |
| *** leseb has quit IRC | 18:50 | |
| *** ukalifon1 has joined #openstack-keystone | 18:55 | |
| *** ukalifon1 has quit IRC | 19:07 | |
| *** rodrigods has quit IRC | 19:16 | |
| *** hrybacki has quit IRC | 19:17 | |
| *** dims has joined #openstack-keystone | 19:19 | |
| *** topol has joined #openstack-keystone | 19:44 | |
| *** radez_g0n3 has quit IRC | 19:54 | |
| *** radez_g0n3 has joined #openstack-keystone | 19:59 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token, move identity server members to class https://review.openstack.org/97124 | 20:20 |
| *** dims has quit IRC | 20:22 | |
| *** topol has quit IRC | 20:22 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token, move identity server members to class https://review.openstack.org/97124 | 20:24 |
| Ajaeger | bknudson: could you have a look at 97089, please? | 20:25 |
| bknudson | Ajaeger: can you move 97005 to be dependent on 97089 ? | 20:25 |
| Ajaeger | bknudson: I fear I would break the scripts if I do so ;( | 20:26 |
| bknudson | ok, I'll try it myself | 20:27 |
| Ajaeger | The daily job is not looking at the previous content, just reusing the ChangeID. So tomorrow it would be broken again... | 20:27 |
| bknudson | that's weird. I wonder why pep8 is checking the pot files. | 20:27 |
| Ajaeger | bknudson: exactly ;( it shouldn't it... | 20:28 |
| Ajaeger | bknudson: this happened for trove and keystone but not for any of the other projects... | 20:29 |
| Ajaeger | bknudson: might be because of "filename= *.py,keystone-*" in tox.ini | 20:29 |
| Ajaeger | Other projects don't do it this way | 20:30 |
| bknudson | keystone has a bin/keystone-all and bin/keystone-manage to check | 20:30 |
| bknudson | Ajaeger: https://review.openstack.org/#/c/97089/ doesn't fix the problem for me. | 20:34 |
| Ajaeger | Strange, it fixed it for me. Let me recheck... | 20:34 |
| bknudson | tox -e pep8 still fails for me | 20:34 |
| Ajaeger | bknudson: Running "flake8" locally I see http://paste.opensuse.org/40089427 | 20:36 |
| * Ajaeger tests tox now as well... | 20:37 | |
| bknudson | Ajaeger: I get a bunch of errors when I run flake8, too. | 20:38 |
| Ajaeger | bknudson: the same 3? | 20:38 |
| Ajaeger | I mean 4 | 20:38 |
| bknudson | Ajaeger: I get those ones and a bunch more | 20:38 |
| bknudson | 2.1.0 (pep8: 1.5.7, pyflakes: 0.8.1, mccabe: 0.2.1) CPython 2.7.3 on Linux | 20:39 |
| Ajaeger | pep 1.5.6 on my system | 20:39 |
| Ajaeger | flake8 2.0 | 20:40 |
| Ajaeger | Seems our gate runs something different ;) | 20:40 |
| bknudson | I think it does tox -e pep8 | 20:40 |
| Ajaeger | You're right, let's try rebasing these so that we see that it works in the gate... | 20:41 |
| Ajaeger | Yes, but different packages versions I guess | 20:41 |
| *** leseb has joined #openstack-keystone | 20:41 | |
| *** leseb has quit IRC | 20:43 | |
| Ajaeger | bknudson: does "tox -e pep8" on head give a clean result for you? | 20:44 |
| Ajaeger | Or the same you get with 97089 applied on top of 97005? | 20:45 |
| * Ajaeger has rebased the two patches properly and tests | 20:45 | |
| bknudson | tox -e pep8 on master is clean | 20:45 |
| Ajaeger | I'm puzzled, neither of these changes should affect it... | 20:46 |
| bknudson | $ git log --oneline -n2 -- 435873f Imported Translations from Transifex -- 3b2b50e pep8: do not test locale files | 20:46 |
| bknudson | Ajaeger: that fails with the same errors as are in the 97005 log | 20:47 |
| bknudson | git-review -d 97089 -r cg ; git-review -x 97005 -r cg ; tox -e pep8 | 20:48 |
| bknudson | cg is the remote that I use for review.openstack.org | 20:49 |
| Ajaeger | The new thing in these files is the "keystone-log-$LEVEL.po" files | 20:51 |
| bknudson | Ajaeger: maybe tox.ini could be changed to bin/keysone-* rather than keystone-*? | 20:54 |
| bknudson | bin/keystone-* | 20:54 |
| bknudson | there must be some reason pep8 ignores other keystone-* files. | 20:55 |
| *** leseb has joined #openstack-keystone | 21:01 | |
| bknudson | Ajaeger: setting exclude= *.po,*.pot seems to work | 21:01 |
| Ajaeger | then let's do that one instead. Will you send a patch or shall I update mine? | 21:02 |
| *** leseb has quit IRC | 21:03 | |
| *** pheadron has quit IRC | 21:06 | |
| Ajaeger | yeah, does the job for me as well. I'll update my patch... | 21:08 |
| *** pheadron has joined #openstack-keystone | 21:10 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/keystone: pep8: do not test locale files https://review.openstack.org/97089 | 21:10 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/keystone: Imported Translations from Transifex https://review.openstack.org/97005 | 21:11 |
| Ajaeger | Now rebased on top of each other... | 21:11 |
| Ajaeger | bknudson: passed pep8: https://jenkins04.openstack.org/job/gate-keystone-pep8/1803/console | 21:15 |
| *** pheadron has quit IRC | 21:17 | |
| bknudson | Ajaeger: are there really any differences in en_AU vs en_US? | 21:18 |
| Ajaeger | fifield would tell you that there are some like there are differences between en_US and en_GB. | 21:19 |
| Ajaeger | Not much, but "c" instead of "z" in some words | 21:19 |
| Ajaeger | flavour vs. flavor | 21:20 |
| Ajaeger | Sorry, I'm not the expert here | 21:20 |
| bknudson | I assume the command is still -flavor | 21:20 |
| Ajaeger | I hope so | 21:20 |
| bknudson | Ajaeger: those changes worked for me locally, too | 21:20 |
| bknudson | +2 | 21:20 |
| Ajaeger | seen it,thanks! | 21:21 |
| *** hrybacki has joined #openstack-keystone | 21:25 | |
| *** Ajaeger has quit IRC | 21:26 | |
| *** Ajaeger has joined #openstack-keystone | 21:28 | |
| Ajaeger | bknudson: I should have set the topic of the translation import to transifex/translations - tomorrow's import will screw this up | 21:29 |
| Ajaeger | the import scripts search for this topic | 21:29 |
| bknudson | Ajaeger: I think you can set the topic right in gerrit | 21:30 |
| Ajaeger | bknudson: I'm not the owner of that patch ;( | 21:30 |
| bknudson | Ajaeger: https://review.openstack.org/#/c/97005/ ? | 21:31 |
| Ajaeger | I push the patches again and reset the topic | 21:31 |
| Ajaeger | yep, I can't edit 97005's topic via the gui | 21:31 |
| Ajaeger | can you? | 21:31 |
| bknudson | Ajaeger: No, I can't. | 21:32 |
| bknudson | Ajaeger: it's weird because you just changed the topic when you rebased it anyways | 21:33 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/keystone: Imported Translations from Transifex https://review.openstack.org/97005 | 21:33 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/keystone: pep8: do not test locale files https://review.openstack.org/97089 | 21:33 |
| Ajaeger | Yeah it is -and the above worked... | 21:33 |
| Ajaeger | it even kept you +2s ;) | 21:34 |
| Ajaeger | good night! | 21:34 |
| *** Ajaeger has quit IRC | 21:35 | |
| bknudson | sleep tight, Ajaeger | 21:36 |
| *** dstanek_zzz is now known as dstanek | 21:39 | |
| *** rodrigods has joined #openstack-keystone | 21:43 | |
| *** dims has joined #openstack-keystone | 21:50 | |
| *** henrynash has quit IRC | 21:53 | |
| *** hrybacki has quit IRC | 21:56 | |
| *** pheadron has joined #openstack-keystone | 22:17 | |
| *** rodrigods has quit IRC | 22:26 | |
| *** dstanek is now known as dstanek_zzz | 22:30 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token, move identity server members to class https://review.openstack.org/97124 | 22:36 |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token revocation list members to new class https://review.openstack.org/97127 | 22:36 |
| *** harlowja_away is now known as harlowja_ | 22:48 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token, move identity server members to class https://review.openstack.org/97124 | 22:49 |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token revocation list members to new class https://review.openstack.org/97127 | 22:56 |
| *** pheadron has quit IRC | 23:02 | |
| *** pheadron has joined #openstack-keystone | 23:03 | |
| openstackgerrit | Brant Knudson proposed a change to openstack/python-keystoneclient: Refactor auth_token revocation list members to new class https://review.openstack.org/97127 | 23:07 |
| *** jamielennox|away is now known as jamielennox | 23:15 | |
| *** pheadron has quit IRC | 23:21 | |
| *** pheadron has joined #openstack-keystone | 23:25 | |
| *** hrybacki has joined #openstack-keystone | 23:35 | |
| *** pheadron has quit IRC | 23:42 | |
| *** pheadron has joined #openstack-keystone | 23:43 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!