*** lamt has quit IRC | 00:02 | |
*** catintheroof has joined #openstack-keystone | 00:04 | |
openstackgerrit | Morgan Fainberg proposed openstack/keystone master: Remove KVS code https://review.openstack.org/424862 | 00:10 |
---|---|---|
*** Ephur has quit IRC | 00:10 | |
*** adrian_otto has joined #openstack-keystone | 00:19 | |
*** jperry has quit IRC | 00:20 | |
*** jose-phillips has quit IRC | 00:22 | |
*** agrebennikov__ has quit IRC | 00:24 | |
*** catintheroof has quit IRC | 00:30 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: WIP add db_sync check https://review.openstack.org/416383 | 00:40 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: WIP add db_sync check https://review.openstack.org/416383 | 00:41 |
*** jaosorior has quit IRC | 00:46 | |
*** lamt has joined #openstack-keystone | 00:50 | |
*** dave-mccowan has joined #openstack-keystone | 00:56 | |
*** tqtran has quit IRC | 00:59 | |
*** thorst_ has quit IRC | 01:20 | |
*** lamt has quit IRC | 01:21 | |
*** thorst_ has joined #openstack-keystone | 01:21 | |
*** thorst_ has quit IRC | 01:25 | |
*** lamt has joined #openstack-keystone | 01:26 | |
*** rdo has quit IRC | 01:26 | |
*** rdo has joined #openstack-keystone | 01:28 | |
*** lamt has quit IRC | 01:28 | |
*** portdirect is now known as portdirect_away | 01:30 | |
*** lamt has joined #openstack-keystone | 01:31 | |
*** chris_hultin|AWA is now known as chris_hultin | 01:41 | |
*** chris_hultin is now known as chris_hultin|AWA | 01:43 | |
*** edmondsw has joined #openstack-keystone | 01:45 | |
*** edmondsw has quit IRC | 01:51 | |
*** browne has quit IRC | 01:55 | |
*** anteaya has quit IRC | 01:57 | |
*** thorst_ has joined #openstack-keystone | 02:00 | |
*** dave-mccowan has quit IRC | 02:03 | |
*** anteaya has joined #openstack-keystone | 02:04 | |
openstackgerrit | Steve Martinelli proposed openstack/keystone-specs master: move not implemented ocata specs to backlog https://review.openstack.org/428540 | 02:04 |
*** adrian_otto has quit IRC | 02:06 | |
openstackgerrit | Merged openstack/keystone-specs master: move not implemented ocata specs to backlog https://review.openstack.org/428540 | 02:12 |
*** lamt has quit IRC | 02:13 | |
*** lamt has joined #openstack-keystone | 02:13 | |
*** lamt has quit IRC | 02:18 | |
openstackgerrit | Steve Martinelli proposed openstack/keystone master: Remove KVS code https://review.openstack.org/424862 | 02:25 |
openstackgerrit | Gage Hugo proposed openstack/pycadf master: Allow multiple joined uuids to be considered valid https://review.openstack.org/428543 | 02:25 |
openstackgerrit | Gage Hugo proposed openstack/pycadf master: Allow multiple joined uuids to be considered valid https://review.openstack.org/428543 | 02:28 |
*** thorst_ has joined #openstack-keystone | 02:32 | |
*** thorst_ has quit IRC | 02:32 | |
stevemar | gagehugo: i love that ^ | 02:32 |
gagehugo | stevemar: yay | 02:33 |
gagehugo | that fixes the sha256 issue | 02:33 |
stevemar | and the 'default' issue :) | 02:33 |
gagehugo | yes | 02:33 |
*** spzala has quit IRC | 02:34 | |
gagehugo | this pycadf thing is tricky, it only complains once per test run | 02:34 |
*** spzala has joined #openstack-keystone | 02:34 | |
gagehugo | so once you patch the hole, another one pops up | 02:35 |
stevemar | hehe | 02:35 |
*** adu has joined #openstack-keystone | 02:38 | |
*** spzala has quit IRC | 02:39 | |
gagehugo | might need to break that one up a bit, there is a lot there | 02:40 |
stevemar | gagehugo: the pycadf patch? | 02:40 |
stevemar | nah | 02:40 |
stevemar | gagehugo: once that merges you'll have a much easier time getting the keystone bits passing and probably much fewer changes | 02:41 |
stevemar | gagehugo: changing pycadf should be easy | 02:41 |
gagehugo | yeah the pycadf one | 02:42 |
gagehugo | ah ok | 02:42 |
*** adu has quit IRC | 02:42 | |
stevemar | gagehugo: keystone folks have +2 on that project | 02:42 |
stevemar | or at least most of us do | 02:43 |
gagehugo | that works lol | 02:44 |
gagehugo | stevemar: do you want keystone test logs as an example in the message? | 02:46 |
*** MasterOfBugs has quit IRC | 02:47 | |
stevemar | gagehugo: meh, see what others say | 02:49 |
stevemar | gagehugo: morgan has an idea of whats up, he can review it | 02:49 |
gagehugo | ok | 02:50 |
morgan | ? | 02:51 |
gagehugo | morgan: https://review.openstack.org/428543 | 02:52 |
morgan | ah | 02:53 |
morgan | ok will review | 02:53 |
*** dave-mccowan has joined #openstack-keystone | 02:53 | |
gagehugo | thanks! | 02:53 |
*** dave-mccowan has quit IRC | 03:15 | |
*** adu has joined #openstack-keystone | 03:18 | |
*** martinlopes has quit IRC | 03:28 | |
*** martinlopes has joined #openstack-keystone | 03:31 | |
*** edmondsw has joined #openstack-keystone | 03:33 | |
*** edmondsw has quit IRC | 03:38 | |
*** thorst_ has joined #openstack-keystone | 03:58 | |
*** adu has quit IRC | 04:01 | |
*** thorst_ has joined #openstack-keystone | 04:21 | |
*** thorst_ has quit IRC | 04:21 | |
*** stingaci has quit IRC | 04:23 | |
*** adriant has quit IRC | 04:31 | |
*** dikonoor has joined #openstack-keystone | 04:32 | |
*** adu has joined #openstack-keystone | 04:47 | |
*** lucas_ has joined #openstack-keystone | 04:49 | |
*** nicolasbock has quit IRC | 04:52 | |
*** lucas_ has quit IRC | 04:52 | |
*** lucas_ has joined #openstack-keystone | 04:53 | |
*** lucas_ has quit IRC | 05:01 | |
*** stingaci has joined #openstack-keystone | 05:05 | |
*** stingaci has quit IRC | 05:06 | |
*** adrian_otto has joined #openstack-keystone | 05:14 | |
*** adu has quit IRC | 05:14 | |
openstackgerrit | Merged openstack/keystone master: Stop reading local config dirs for domain-specific SQL config driver https://review.openstack.org/427940 | 05:16 |
*** lucas_ has joined #openstack-keystone | 05:17 | |
*** edmondsw has joined #openstack-keystone | 05:21 | |
*** lucas_ has quit IRC | 05:22 | |
*** edmondsw has quit IRC | 05:26 | |
*** phalmos has quit IRC | 05:58 | |
*** MasterOfBugs has joined #openstack-keystone | 06:01 | |
*** pramodrj07 has joined #openstack-keystone | 06:04 | |
*** MasterOfBugs has quit IRC | 06:05 | |
*** thorst_ has joined #openstack-keystone | 06:22 | |
*** martinlopes has quit IRC | 06:23 | |
*** thorst_ has quit IRC | 06:27 | |
*** mvk has quit IRC | 06:37 | |
*** mvk has joined #openstack-keystone | 06:39 | |
*** edmondsw has joined #openstack-keystone | 07:10 | |
*** edmondsw has quit IRC | 07:14 | |
*** tesseract has joined #openstack-keystone | 07:20 | |
*** adrian_otto has quit IRC | 07:20 | |
*** adrian_otto has joined #openstack-keystone | 07:26 | |
*** rcernin has joined #openstack-keystone | 07:26 | |
*** blake has joined #openstack-keystone | 07:28 | |
*** blake has quit IRC | 07:28 | |
*** blake has joined #openstack-keystone | 07:29 | |
*** adrian_otto has quit IRC | 07:33 | |
*** nkinder has joined #openstack-keystone | 07:56 | |
*** aloga has quit IRC | 07:59 | |
*** aloga has joined #openstack-keystone | 07:59 | |
*** pramodrj07 has quit IRC | 08:18 | |
*** blake has quit IRC | 08:21 | |
*** thorst_ has joined #openstack-keystone | 08:23 | |
*** thorst_ has quit IRC | 08:28 | |
*** ravelar has quit IRC | 08:47 | |
*** edmondsw has joined #openstack-keystone | 08:58 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-keystone | 09:00 | |
*** edmondsw has quit IRC | 09:02 | |
*** spzala has joined #openstack-keystone | 09:35 | |
*** chlong has quit IRC | 09:38 | |
*** spzala has quit IRC | 09:40 | |
*** erlon has joined #openstack-keystone | 09:57 | |
*** thorst_ has joined #openstack-keystone | 10:24 | |
*** thorst_ has quit IRC | 10:29 | |
*** bauruine has joined #openstack-keystone | 10:41 | |
*** edmondsw has joined #openstack-keystone | 10:46 | |
*** edmondsw has quit IRC | 10:50 | |
*** thiagolib has joined #openstack-keystone | 11:09 | |
*** nicolasbock has joined #openstack-keystone | 11:24 | |
*** timss has quit IRC | 11:30 | |
*** timss has joined #openstack-keystone | 11:30 | |
*** bigjools has quit IRC | 11:36 | |
*** bigjools has joined #openstack-keystone | 11:37 | |
*** bigjools has joined #openstack-keystone | 11:37 | |
*** pnavarro has quit IRC | 11:41 | |
*** dave-mccowan has joined #openstack-keystone | 12:10 | |
*** catintheroof has joined #openstack-keystone | 12:20 | |
*** catintheroof has quit IRC | 12:20 | |
*** catintheroof has joined #openstack-keystone | 12:21 | |
*** jvarlamova_ has quit IRC | 12:28 | |
*** rcernin has quit IRC | 12:31 | |
*** edmondsw has joined #openstack-keystone | 12:34 | |
*** thorst_ has joined #openstack-keystone | 12:37 | |
*** edmondsw has quit IRC | 12:38 | |
*** edmondsw has joined #openstack-keystone | 12:43 | |
*** raildo has joined #openstack-keystone | 13:00 | |
*** markvoelker_ has joined #openstack-keystone | 13:29 | |
*** markvoelker has quit IRC | 13:32 | |
*** hoonetorg has quit IRC | 13:35 | |
*** spzala has joined #openstack-keystone | 13:35 | |
*** spzala has quit IRC | 13:36 | |
*** spzala has joined #openstack-keystone | 13:36 | |
*** hoonetorg has joined #openstack-keystone | 13:48 | |
*** lamt has joined #openstack-keystone | 13:56 | |
*** spilla has joined #openstack-keystone | 14:09 | |
*** lamt has quit IRC | 14:13 | |
*** jperry has joined #openstack-keystone | 14:14 | |
*** mvk has quit IRC | 14:15 | |
*** Dinesh_Bhor has quit IRC | 14:20 | |
*** spzala has quit IRC | 14:23 | |
*** jperry has quit IRC | 14:36 | |
*** lamt has joined #openstack-keystone | 14:41 | |
*** mvk has joined #openstack-keystone | 14:44 | |
*** lucas_ has joined #openstack-keystone | 14:45 | |
lbragstad | office hours!! | 14:52 |
*** dikonoor has quit IRC | 14:54 | |
*** spzala has joined #openstack-keystone | 14:55 | |
*** spzala_ has joined #openstack-keystone | 14:57 | |
*** spzala_ has quit IRC | 14:57 | |
*** spzala_ has joined #openstack-keystone | 14:57 | |
*** mvk has quit IRC | 14:58 | |
*** spzala has quit IRC | 15:00 | |
*** chris_hultin|AWA is now known as chris_hultin | 15:01 | |
*** lamt has quit IRC | 15:02 | |
*** haplo37_ has quit IRC | 15:09 | |
*** chris_hultin is now known as chris_hultin|AWA | 15:10 | |
*** pcaruana has quit IRC | 15:11 | |
*** haplo37_ has joined #openstack-keystone | 15:12 | |
*** ravelar has joined #openstack-keystone | 15:16 | |
*** pcaruana has joined #openstack-keystone | 15:17 | |
*** adrian_otto has joined #openstack-keystone | 15:19 | |
knikolla | o/ | 15:20 |
*** stingaci has joined #openstack-keystone | 15:20 | |
knikolla | lbragstad: reporting in for office hours. | 15:21 |
lbragstad | knikolla sweet! | 15:21 |
lbragstad | knikolla anything you're planning on tackling today? | 15:21 |
knikolla | lbragstad: open for suggestions | 15:22 |
*** chlong has joined #openstack-keystone | 15:24 | |
lbragstad | knikolla couple reviews that close bugs - https://review.openstack.org/#/c/420085/5 | 15:24 |
*** stingaci has quit IRC | 15:24 | |
lbragstad | knikolla and https://review.openstack.org/#/c/424862/ | 15:24 |
*** adrian_otto has quit IRC | 15:25 | |
*** stingaci has joined #openstack-keystone | 15:25 | |
knikolla | lbragstad: the second on is for pike | 15:25 |
lbragstad | knikolla yeah - kvs removal | 15:26 |
knikolla | lbragstad: did we cut stable/ocata yet? | 15:26 |
gagehugo | o/ | 15:26 |
lbragstad | knikolla that will probably be one of the first patches to land in pike (after hearing morgan and stevemar talk about it) | 15:26 |
lbragstad | knikolla we have an RC | 15:27 |
lbragstad | gagehugo o/ | 15:27 |
lbragstad | knikolla http://lists.openstack.org/pipermail/openstack-dev/2017-February/111534.html | 15:27 |
lbragstad | knikolla and https://github.com/openstack/keystone/tree/stable/ocata | 15:28 |
knikolla | lbragstad: so master is open for merging pike stuff now | 15:28 |
lbragstad | knikolla correct | 15:28 |
knikolla | lbragstad: awesome! | 15:28 |
* knikolla goes back to reviewing | 15:29 | |
*** chris_hultin|AWA is now known as chris_hultin | 15:29 | |
*** nkinder has quit IRC | 15:32 | |
*** dave-mccowan has quit IRC | 15:40 | |
*** jaosorior has joined #openstack-keystone | 15:40 | |
*** edtubill has joined #openstack-keystone | 15:42 | |
lbragstad | heads up https://review.openstack.org/#/c/420085/5 closes a bug for us in case anyone wants to review | 15:45 |
*** lamt has joined #openstack-keystone | 15:48 | |
samueldmq | hi all | 15:52 |
samueldmq | lbragstad: done | 15:52 |
lbragstad | samueldmq o/ thanks! | 15:52 |
samueldmq | :) | 15:52 |
*** dave-mccowan has joined #openstack-keystone | 15:54 | |
*** mvk has joined #openstack-keystone | 16:07 | |
lbragstad | dstanek stevemar samueldmq do any of you remember when we removed the stable driver interfaces work? | 16:08 |
dstanek | lbragstad: hmmm | 16:08 |
lbragstad | I'm looking through Launchpad and can't seem to find it (go figure) | 16:08 |
samueldmq | newton ? | 16:08 |
dstanek | lbragstad: jas...looking | 16:08 |
lbragstad | dstanek something tells me Newton | 16:08 |
lbragstad | samueldmq ++ that's what I was thinking - but I don't think we tracked the work with a spec | 16:09 |
samueldmq | lbragstad: that's what I think it is too | 16:09 |
samueldmq | :) | 16:09 |
*** BlackDex has quit IRC | 16:09 | |
stevemar | lbragstad: i think this round | 16:09 |
stevemar | lbragstad: https://blueprints.launchpad.net/keystone/+spec/removed-as-of-ocata | 16:09 |
stevemar | "all stable driver interfaces" | 16:10 |
spilla | lbragstad: for https://review.openstack.org/#/c/420085/ would you think `region_id_required` and `region_id_optional` would be a good naming change? | 16:10 |
lbragstad | stevemar ah - yes! | 16:10 |
lbragstad | stevemar thanks! | 16:10 |
dstanek | stevemar: nice... just found it too | 16:10 |
lbragstad | spilla I'm open to that - yeah... there are other conventions already established in the parameters module, too | 16:11 |
spilla | oh, there seem to be a few of the `param_1` s. I may go through a bunch of these. Also ty, I'll check those conventions | 16:11 |
lbragstad | spilla i had a note on my todo list to go through through, i remember them bugging me the last time I was mucking around in the docs | 16:12 |
lbragstad | through those* | 16:12 |
spilla | lbragstad if you'd like I can take it off yours and add it to mine :) | 16:13 |
* lbragstad hands spilla the Post-It note from his monitor | 16:14 | |
*** BlackDex has joined #openstack-keystone | 16:16 | |
*** BlackDex has quit IRC | 16:16 | |
*** BlackDex has joined #openstack-keystone | 16:17 | |
lbragstad | antwash o/ I updated https://review.openstack.org/#/c/277707/10 with a comment regarding our discussion yesterday | 16:18 |
*** BlackDex has quit IRC | 16:20 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: WIP add db_sync check https://review.openstack.org/416383 | 16:22 |
*** BlackDex has joined #openstack-keystone | 16:25 | |
*** agrebennikov__ has joined #openstack-keystone | 16:27 | |
*** adrian_otto has joined #openstack-keystone | 16:28 | |
*** stingaci has quit IRC | 16:32 | |
*** chlong has quit IRC | 16:36 | |
openstackgerrit | Gage Hugo proposed openstack/keystone master: WIP Fix multiple uuid warnings with pycadf https://review.openstack.org/426411 | 16:37 |
*** ravelar has quit IRC | 16:44 | |
*** ravelar has joined #openstack-keystone | 16:45 | |
*** adrian_otto has quit IRC | 16:51 | |
*** chlong has joined #openstack-keystone | 16:52 | |
openstackgerrit | Gage Hugo proposed openstack/pycadf master: Make `is_valid` more flexible with uuid validation https://review.openstack.org/428543 | 16:53 |
*** d0ugal is now known as d0ugal|skiing | 16:54 | |
*** stingaci has joined #openstack-keystone | 16:58 | |
antwash | lbragstad : gotcha, I just looked at it -- i'll go ahead an implement still, just in case Kalaswan doesn't respond or push by end of day | 17:00 |
*** adrian_otto has joined #openstack-keystone | 17:02 | |
*** lucas_ has quit IRC | 17:03 | |
*** chlong has quit IRC | 17:16 | |
*** mvk has quit IRC | 17:17 | |
*** lamt has quit IRC | 17:19 | |
*** lamt has joined #openstack-keystone | 17:24 | |
*** tqtran has joined #openstack-keystone | 17:25 | |
*** browne has joined #openstack-keystone | 17:30 | |
lbragstad | antwash you might be able to do a rebase, too | 17:31 |
lbragstad | antwash i'd personally try and shoot for that case, that way it keeps Kalaswan as the original author | 17:32 |
antwash | Yeah that's what I'm doing lol | 17:32 |
lbragstad | antwash awesome | 17:32 |
lbragstad | antwash you're always two steps ahead :) | 17:32 |
antwash | lbragstad : no point in writing all that over again lol #LAzy | 17:32 |
lbragstad | antwash lazy people make great developers | 17:33 |
*** tqtran_ has joined #openstack-keystone | 17:34 | |
*** tqtran has quit IRC | 17:34 | |
lbragstad | ravelar it'd be good to get odyssey4me's eyes on https://review.openstack.org/#/c/416383/ :) | 17:35 |
*** stingaci has quit IRC | 17:35 | |
*** lamt has quit IRC | 17:35 | |
*** lamt has joined #openstack-keystone | 17:36 | |
*** dave-mccowan has quit IRC | 17:38 | |
openstackgerrit | Samuel Pilla proposed openstack/keystone master: Renaming of api parameters https://review.openstack.org/428830 | 17:40 |
ravelar | lbragstad that it would lol | 17:44 |
openstackgerrit | Tin Lam proposed openstack/keystone master: WIP Fix multiple uuid warnings with pycadf https://review.openstack.org/426411 | 17:45 |
*** adrian_otto1 has joined #openstack-keystone | 17:48 | |
*** adrian_otto has quit IRC | 17:50 | |
*** EmilienM has quit IRC | 17:56 | |
*** pkoraca has quit IRC | 17:56 | |
*** Kimmo_ has quit IRC | 17:56 | |
*** Adobeman has quit IRC | 17:56 | |
*** baffle has quit IRC | 17:56 | |
*** tonyb has quit IRC | 17:56 | |
*** rodrigods has quit IRC | 17:56 | |
*** Tahvok has quit IRC | 17:56 | |
*** baffle has joined #openstack-keystone | 17:56 | |
*** Nakato has quit IRC | 17:56 | |
*** bknudson has quit IRC | 17:56 | |
*** mkoderer___ has quit IRC | 17:56 | |
*** d34dh0r53 has quit IRC | 17:56 | |
*** Anticimex has quit IRC | 17:56 | |
*** peterstac has quit IRC | 17:56 | |
*** alex_xu has quit IRC | 17:56 | |
*** Adobeman has joined #openstack-keystone | 17:56 | |
*** tonyb has joined #openstack-keystone | 17:56 | |
*** rodrigods has joined #openstack-keystone | 17:56 | |
*** mkoderer__ has joined #openstack-keystone | 17:56 | |
*** Kimmo_ has joined #openstack-keystone | 17:56 | |
*** Tahvok has joined #openstack-keystone | 17:56 | |
*** andreaf has quit IRC | 17:56 | |
*** agarner has quit IRC | 17:56 | |
*** SamYaple has quit IRC | 17:56 | |
*** topol has quit IRC | 17:56 | |
*** kevinbenton has quit IRC | 17:56 | |
*** d34dh0r53 has joined #openstack-keystone | 17:56 | |
*** EmilienM has joined #openstack-keystone | 17:56 | |
*** Anticimex has joined #openstack-keystone | 17:56 | |
*** sirushti has quit IRC | 17:56 | |
*** ktychkova has quit IRC | 17:56 | |
*** SamYaple has joined #openstack-keystone | 17:56 | |
*** peterstac has joined #openstack-keystone | 17:56 | |
*** topol_ has joined #openstack-keystone | 17:56 | |
*** alex_xu has joined #openstack-keystone | 17:56 | |
*** bknudson has joined #openstack-keystone | 17:56 | |
*** ChanServ sets mode: +v bknudson | 17:56 | |
*** ktychkova has joined #openstack-keystone | 17:56 | |
*** pkoraca has joined #openstack-keystone | 17:57 | |
*** dave-mccowan has joined #openstack-keystone | 17:57 | |
*** Nakato has joined #openstack-keystone | 17:57 | |
*** kevinbenton has joined #openstack-keystone | 17:57 | |
*** sirushti has joined #openstack-keystone | 17:57 | |
*** andreaf has joined #openstack-keystone | 17:58 | |
*** jose-phillips has joined #openstack-keystone | 17:59 | |
*** aleph1 has joined #openstack-keystone | 17:59 | |
*** DuncanT has quit IRC | 17:59 | |
*** DuncanT has joined #openstack-keystone | 18:01 | |
*** lamt has quit IRC | 18:03 | |
*** alex_xu has quit IRC | 18:06 | |
*** alex_xu has joined #openstack-keystone | 18:08 | |
openstackgerrit | Merged openstack/keystone master: Update endpoint api for optional region_id https://review.openstack.org/420085 | 18:12 |
*** lamt has joined #openstack-keystone | 18:13 | |
*** Ephur has joined #openstack-keystone | 18:17 | |
lbragstad | morgan bang up job on https://review.openstack.org/#/c/424862/7 | 18:17 |
openstackgerrit | David Stanek proposed openstack/keystone master: Federated mapping doc improvements https://review.openstack.org/422853 | 18:21 |
dstanek | ^ i hope that help clear up some of the confusion | 18:23 |
*** lucas_ has joined #openstack-keystone | 18:23 | |
lbragstad | dstanek nice! | 18:23 |
samueldmq | dstanek: is "Federated mapping doc improvements" ready for reviews ? | 18:28 |
samueldmq | or is there more to come in that patch yet ? | 18:28 |
*** stingaci has joined #openstack-keystone | 18:33 | |
dstanek | samueldmq: that's it for that patch | 18:34 |
stevemar | does someone want to do the sql migration placeholder patch for ocata backports? https://github.com/openstack/keystone/commit/3a84987ce0388a7131d9f13b1ae2ced3ba083ada ? | 18:34 |
dstanek | i'm working on more docs stuff, but i'll be submitting other patches | 18:34 |
stevemar | i was just going to do it, but meeting time | 18:34 |
stevemar | and i don't want us to merge a migration before thats in :O | 18:34 |
*** stingaci has quit IRC | 18:36 | |
*** tesseract has quit IRC | 18:38 | |
*** chlong has joined #openstack-keystone | 18:38 | |
openstackgerrit | Tin Lam proposed openstack/pycadf master: Make `is_valid` more flexible with uuid validation https://review.openstack.org/428543 | 18:39 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: WIP add db_sync check https://review.openstack.org/416383 | 18:40 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: WIP add db_sync check https://review.openstack.org/416383 | 18:41 |
*** chris_hultin is now known as chris_hultin|AWA | 18:44 | |
samueldmq | stevemar: I can do that if you're not on it yet | 19:00 |
samueldmq | stevemar: how many slots do we need ? | 19:00 |
samueldmq | 5 as well ? | 19:02 |
*** Mech422 has joined #openstack-keystone | 19:10 | |
Mech422 | Hi! Anyone had problems with Newton on a percona cluster when trying to run 'keystone manage bootstrap' it fails with a FK_CONSTRAINT error | 19:10 |
Mech422 | for some reason it thinks RegionOne already exists, when in fact, the region table is empty | 19:10 |
Mech422 | Odd thing is - it only fails like HALF the time ? | 19:10 |
*** chlong has quit IRC | 19:11 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/keystone master: Add placeholder migrations for Ocata https://review.openstack.org/428984 | 19:12 |
samueldmq | stevemar: ^ | 19:12 |
*** lucas_ has quit IRC | 19:12 | |
*** lucas_ has joined #openstack-keystone | 19:12 | |
*** lucas_ has quit IRC | 19:14 | |
*** jdennis has quit IRC | 19:15 | |
*** lucas_ has joined #openstack-keystone | 19:15 | |
*** chlong has joined #openstack-keystone | 19:21 | |
openstackgerrit | Samuel Pilla proposed openstack/keystone master: Renaming of api parameters https://review.openstack.org/428830 | 19:23 |
openstackgerrit | Samuel Pilla proposed openstack/keystone master: Renaming of api parameters https://review.openstack.org/428830 | 19:24 |
*** lamt has quit IRC | 19:25 | |
*** lamt has joined #openstack-keystone | 19:26 | |
*** ravelar has quit IRC | 19:26 | |
*** chlong has quit IRC | 19:27 | |
*** lucas_ has quit IRC | 19:27 | |
samueldmq | spilla: hey, just a nit on ^ | 19:27 |
*** lucas_ has joined #openstack-keystone | 19:27 | |
samueldmq | spilla: looking good, thanks for doing that | 19:27 |
spilla | samuelmq: ah, good call! and no problem | 19:29 |
*** ravelar has joined #openstack-keystone | 19:30 | |
Mech422 | Hmm - it appears I'm getting a Conflict error trying to create RegionOne during bootstrap ? | 19:30 |
Mech422 | thats....odd | 19:30 |
openstackgerrit | Samuel Pilla proposed openstack/keystone master: Renaming of api parameters https://review.openstack.org/428830 | 19:33 |
*** stingaci has joined #openstack-keystone | 19:35 | |
*** nkinder has joined #openstack-keystone | 19:37 | |
*** chlong has joined #openstack-keystone | 19:41 | |
*** stingaci has quit IRC | 19:48 | |
morgan | stevemar: we should convert to alembic | 19:52 |
morgan | stevemar: trhen we don't need placeholders | 19:52 |
*** Marcellin__ has joined #openstack-keystone | 19:52 | |
*** adrian_otto1 has quit IRC | 19:55 | |
*** lucas_ has quit IRC | 19:55 | |
*** adrian_otto has joined #openstack-keystone | 19:57 | |
*** lucas_ has joined #openstack-keystone | 20:04 | |
*** lucas_ has quit IRC | 20:08 | |
mtreinish | morgan: hooray for linked lists! :) | 20:10 |
morgan | mtreinish: right?! | 20:12 |
openstackgerrit | Morgan Fainberg proposed openstack/keystone master: Deprecate (and slate for removal) UUID tokens https://review.openstack.org/428388 | 20:18 |
openstackgerrit | Morgan Fainberg proposed openstack/keystone master: Deprecate (and slate for removal) UUID tokens https://review.openstack.org/428388 | 20:19 |
*** pramodrj07 has joined #openstack-keystone | 20:24 | |
*** pramodrj07 has quit IRC | 20:24 | |
*** MasterOfBugs has joined #openstack-keystone | 20:25 | |
*** lucas_ has joined #openstack-keystone | 20:26 | |
openstackgerrit | David Stanek proposed openstack/keystone master: Federated mapping doc improvements https://review.openstack.org/422853 | 20:33 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: WIP add db_sync check https://review.openstack.org/416383 | 20:33 |
*** lucas_ has quit IRC | 20:33 | |
*** stingaci has joined #openstack-keystone | 20:34 | |
*** chris_hultin|AWA is now known as chris_hultin | 20:34 | |
*** lucas_ has joined #openstack-keystone | 20:34 | |
*** lucas_ has quit IRC | 20:39 | |
*** adrian_otto has quit IRC | 20:40 | |
*** lucas_ has joined #openstack-keystone | 20:44 | |
lbragstad | antwash these two are failing? | 20:48 |
lbragstad | antwash https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1508-L1539 ? | 20:48 |
antwash | lbragstad : yeah the assert methods are complaining about the key -- i'm trying to figure out how to print the user <k,v> so I can see what all keys are there | 20:49 |
antwash | print isn't working | 20:50 |
antwash | I'm going to try and use log as a work around and see if it works | 20:51 |
Mech422 | so it appears my problem with keystone bootstrap goes away if I disable memcached caching | 20:52 |
Mech422 | commenting out [cache] backend/enabled/memcache_servers allows me to (re-)install keystone several times in a row | 20:53 |
lbragstad | antwash this line? https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1517 | 20:54 |
openstackgerrit | David Stanek proposed openstack/keystone master: Federated mapping doc improvements https://review.openstack.org/422853 | 20:55 |
dstanek | Mech422: you're trying to run bootstrap a second time? | 20:55 |
*** lucas_ has quit IRC | 20:56 | |
Mech422 | dstanek: I'm testing some ansible roles, and installing / purging keystone | 20:56 |
Mech422 | dstanek: I thought my role was broken because it failed to re-install about every other time | 20:57 |
dstanek | Mech422: yeah, that doesn't surprise me at all. when you purge the database you'll have to purge the cache too | 20:57 |
Mech422 | dstanek: instead, it appears that keystone is picking up stale cache entries from memcache | 20:57 |
Mech422 | dstanek: heh - yeah, only cost me 6 hours :-P *sigh* TGIF!! | 20:58 |
antwash | lbragstad : yeap -- 1517, 1521, 1539 | 20:59 |
lbragstad | antwash huh - what's in the user reference there? | 21:00 |
stevemar | samueldmq: pep8 o_O | 21:00 |
stevemar | https://review.openstack.org/#/c/428984/1 | 21:00 |
antwash | lbragstad : I didn't change anything it's the same, that's why I'm not understand why it's not passing | 21:01 |
lbragstad | antwash well - it is a stale patch, so something could have merged into master that changes things in the user reference | 21:01 |
lbragstad | and it might be failing on what ever merged into master now that you've rebased it, right? | 21:01 |
antwash | lbragstad if only I could see the <k.v> :( | 21:02 |
antwash | I'm going to search thru some more test and see | 21:02 |
lbragstad | antwash how are you running the tests? | 21:03 |
antwash | tox -e py27 <test.method> | 21:03 |
dstanek | should we be calling is 'keystone-to-keystone' instead of 'Keystone to Keystone' federation? | 21:04 |
stevemar | antwash: you can add pdb breakpoints to debug | 21:04 |
lbragstad | dstanek I think I'd opt for keystone-to-keystone? | 21:04 |
stevemar | antwash: "import pdb; pdb.set_trace()" in the code, and run it with "tox -e debug <test>" | 21:05 |
dstanek | antwash: do you have a failing test? | 21:05 |
openstackgerrit | David Stanek proposed openstack/keystone master: Federated mapping doc improvements https://review.openstack.org/422853 | 21:05 |
dstanek | lbragstad: yeah, i already jumped ahead and fix it ^ | 21:06 |
dstanek | does anyone know if there are guide for using codenames at the begininng of a sentence? i was treating it like iPhone, but maybe i shouldn't? | 21:06 |
lbragstad | antwash I was able to add http://cdn.pasteraw.com/20v41xhby9t2dgc2qf7b9wjcub9fnhf to the tests and get user output from `tox -e py27 -- test_delete_project_clears_default_project_id` | 21:06 |
lbragstad | dstanek I usually try and find a wording that doesn't put it at the beginning of the sentence so that I can avoid making a decision about the convention | 21:07 |
dstanek | lol | 21:07 |
* lbragstad is probably a bad person | 21:08 | |
lbragstad | gonna step away for an ultra late lunch and coffee quick | 21:09 |
lbragstad | back in 15 | 21:09 |
*** lucas_ has joined #openstack-keystone | 21:11 | |
*** catintheroof has quit IRC | 21:11 | |
*** raildo has quit IRC | 21:11 | |
*** thiagolib has quit IRC | 21:18 | |
lbragstad | antwash you haven't pushed a patch for review yet have you? | 21:23 |
openstackgerrit | Samuel Pilla proposed openstack/keystone master: Renaming of api parameters https://review.openstack.org/428830 | 21:24 |
antwash | lbragstad : no I haven't but I figured out why it's failing | 21:24 |
lbragstad | antwash nice! | 21:25 |
lbragstad | antwash what was it? | 21:25 |
antwash | lbragstad : when the identity_api deletes the project, it removes the <k> from the dic as well -- so that's why it's saying KeyError -- the key doesn't exist anymore lol | 21:25 |
lbragstad | antwash aha | 21:26 |
lbragstad | antwash so 'default_project_id' isn't actually in the reference as opposed to it being None or something like that | 21:26 |
antwash | lbragstad : exactly, it should be there but as None, but it's not soo yeah something needs to be modified with core.py | 21:28 |
dstanek | antwash: is it failing on jenkins? | 21:28 |
antwash | or maybe another place -- least I know why it's failing at least | 21:28 |
antwash | dstanek : nope in my local env unit test | 21:28 |
lbragstad | antwash can you push what you have? | 21:28 |
antwash | lbragstad : sure I can | 21:28 |
dstanek | antwash: publish the code and i'll take a look | 21:28 |
lbragstad | dstanek antwash ++ | 21:28 |
*** r-daneel has joined #openstack-keystone | 21:33 | |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Clear the project ID from user information https://review.openstack.org/429047 | 21:35 |
dstanek | antwash: which tests are failing? | 21:36 |
antwash | not sure why the commit message says conflicts but *shrugs* | 21:36 |
antwash | dstanek : test_delete_project_clears_default_project_id / test_delete_project_with_roles_clears_default_project_id | 21:37 |
lbragstad | antwash if you cherry pick a patch from something and there is a conflict, it will automatically be amended to the commit message | 21:37 |
lbragstad | dstanek https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1508-L1539 | 21:37 |
dstanek | antwash: git does that automatically when you have local conflicts; you have to remove that by hand | 21:37 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Clear the project ID from user information https://review.openstack.org/429047 | 21:38 |
openstackgerrit | Gage Hugo proposed openstack/keystone master: Fix multiple uuid warnings with pycadf https://review.openstack.org/426411 | 21:39 |
gagehugo | stevemar: no more warnings with that ^ when using the updated pycadf patch | 21:39 |
dstanek | antwash: this passes for me "tox -e py27 -- unit.resource" - which specific test is failing? | 21:43 |
*** chlong has quit IRC | 21:44 | |
antwash | dstanek : this the traceback I was getting http://paste.openstack.org/show/597634/ -- maybe my env is screwed up some where. | 21:45 |
lbragstad | antwash pulling it down locally, too | 21:46 |
dstanek | antwash: you don't have any changes you didn't push? | 21:46 |
antwash | dstanek : I know when I was able to see the user <k,v> it was removing the default_project_id completely instead of setting to None -- which isn't the behavior I was expecting to get | 21:46 |
antwash | them are all the files that were modified on my end | 21:47 |
dstanek | antwash: ah, ok. i know the test that is failing now :-) | 21:47 |
lbragstad | antwash dstanek passes locally form e | 21:47 |
lbragstad | for me* | 21:47 |
*** chris_hultin is now known as chris_hultin|AWA | 21:47 | |
dstanek | ok, i have it failing for me now | 21:53 |
morgan | lbragstad: see responses to comments: https://review.openstack.org/#/c/428388/ | 21:54 |
*** ChanServ sets mode: +v topol_ | 21:54 | |
*** topol_ is now known as topol | 21:54 | |
*** lucas_ has quit IRC | 21:54 | |
* antwash about to get food <late lunch> brb | 21:55 | |
*** nkinder has quit IRC | 21:55 | |
*** stingaci has quit IRC | 21:55 | |
*** spzala_ has quit IRC | 21:55 | |
*** adrian_otto has joined #openstack-keystone | 21:57 | |
*** edtubill has quit IRC | 21:59 | |
dstanek | lbragstad: antwash: i think that test is just incorrect | 22:03 |
dstanek | lbragstad: does it pass for you? | 22:03 |
lbragstad | dstanek https://github.com/openstack/keystone/blob/9bb57c1eb7f295d1c7d37f880f5ed99bbee766a4/keystone/tests/unit/resource/test_backends.py#L1523-L1539 ? | 22:03 |
dstanek | specifically running 'tox -e py27 -- test_delete_project' - i don't think it can work as written | 22:03 |
lbragstad | dstanek huh - yeah it's broken | 22:04 |
*** edmondsw has quit IRC | 22:04 | |
*** stingaci has joined #openstack-keystone | 22:04 | |
dstanek | lbragstad: antwash: we don't return that key if it's None - http://git.openstack.org/cgit/openstack/keystone/tree/keystone/identity/backends/sql_model.py#n219 | 22:04 |
*** edmondsw has joined #openstack-keystone | 22:05 | |
dstanek | lbragstad: antwash: and that test is looking for a None value | 22:05 |
lbragstad | dstanek yeah - so the test is wrong | 22:05 |
dstanek | lbragstad: antwash: it should just make sure that key doesn't exist | 22:05 |
lbragstad | dstanek but it probably merged that way because it was wrapped with a wip decorator | 22:05 |
*** edmondsw has quit IRC | 22:05 | |
*** edmondsw has joined #openstack-keystone | 22:05 | |
stevemar | gagehugo: booya | 22:11 |
*** spilla has quit IRC | 22:11 | |
dstanek | lbragstad: yeah, most likely | 22:16 |
*** thorst_ has quit IRC | 22:25 | |
*** Marcellin__ has quit IRC | 22:29 | |
*** mvk has joined #openstack-keystone | 22:38 | |
*** edmondsw has quit IRC | 22:41 | |
*** chlong has joined #openstack-keystone | 22:42 | |
openstackgerrit | Lance Bragstad proposed openstack/keystone master: Ensure token expiration is always consistent https://review.openstack.org/429067 | 22:44 |
*** stingaci has quit IRC | 22:51 | |
*** jaosorior has quit IRC | 22:52 | |
*** stingaci has joined #openstack-keystone | 22:53 | |
openstackgerrit | Eric Brown proposed openstack/keystone master: Use httplib constants for http status codes https://review.openstack.org/379855 | 22:54 |
*** stingaci has quit IRC | 22:57 | |
*** spzala has joined #openstack-keystone | 23:02 | |
*** chris_hultin|AWA is now known as chris_hultin | 23:02 | |
*** chris_hultin is now known as chris_hultin|AWA | 23:03 | |
*** adrian_otto has quit IRC | 23:05 | |
openstackgerrit | Steve Martinelli proposed openstack/keystone master: clean up newton release notes https://review.openstack.org/429071 | 23:06 |
*** spzala has quit IRC | 23:06 | |
antwash | dstanek : cool I see thats what I was thinking as well, when I realized the key isn't even there | 23:26 |
antwash | lbragstad: dstanek : thanks for looking at it | 23:26 |
*** jdennis has joined #openstack-keystone | 23:27 | |
*** adrian_otto has joined #openstack-keystone | 23:30 | |
*** Ephur has quit IRC | 23:36 | |
*** Ephur has joined #openstack-keystone | 23:43 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Add --check to keystone-manage db_sync command https://review.openstack.org/416383 | 23:44 |
*** adu has joined #openstack-keystone | 23:46 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Add unit test for db_sync run out of order https://review.openstack.org/429113 | 23:54 |
*** adu has quit IRC | 23:55 | |
openstackgerrit | Merged openstack/keystone master: Renaming of api parameters https://review.openstack.org/428830 | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!