*** markvoelker has joined #openstack-keystone | 00:01 | |
*** jlopezgu_ has quit IRC | 00:01 | |
*** markvoelker has quit IRC | 00:05 | |
*** elle has joined #openstack-keystone | 00:06 | |
*** elle is now known as Guest41254 | 00:06 | |
openstackgerrit | David Stanek proposed openstack/keystone master: Replace wip with skip https://review.openstack.org/450992 | 00:06 |
---|---|---|
openstackgerrit | David Stanek proposed openstack/keystone master: Remove decorator for asserting validation errors https://review.openstack.org/450985 | 00:06 |
openstackgerrit | David Stanek proposed openstack/keystone master: Remove conflict guards in load_fixtures https://review.openstack.org/450987 | 00:06 |
openstackgerrit | David Stanek proposed openstack/keystone master: Remove orphaned _create_context test helper https://review.openstack.org/450986 | 00:06 |
openstackgerrit | David Stanek proposed openstack/keystone master: Reduce fixture setup in test_backend_ldap https://review.openstack.org/450989 | 00:06 |
openstackgerrit | David Stanek proposed openstack/keystone master: Consolidate and cleanup test_backend_ldap setup https://review.openstack.org/450988 | 00:06 |
openstackgerrit | David Stanek proposed openstack/keystone master: Removed domain conflict guard in load_fixtures https://review.openstack.org/450991 | 00:06 |
openstackgerrit | David Stanek proposed openstack/keystone master: Remove create_container_group from tests https://review.openstack.org/450990 | 00:06 |
Guest41254 | Hello Identity/Keystone: I'm a potential Outreachy intern | 00:07 |
ayoung | knikolla, answered your email, sorry took so long | 00:08 |
ayoung | where is the final policy-in-code patch in the process? | 00:08 |
*** pnavarro has quit IRC | 00:11 | |
*** thorst has quit IRC | 00:12 | |
*** Guest41254 has quit IRC | 00:14 | |
openstackgerrit | David Stanek proposed openstack/keystone master: Removed the deprecated pki_setup command https://review.openstack.org/430938 | 00:15 |
*** dave-mccowan has joined #openstack-keystone | 00:21 | |
dstanek | ayoung: i think merged | 00:22 |
ayoung | dstanek, ah, good | 00:22 |
ayoung | pki setup is still in the codebase? | 00:22 |
ayoung | wow | 00:22 |
dstanek | lol, yep | 00:22 |
*** nkinder has quit IRC | 00:24 | |
ayoung | dstanek, that whole stack looks good. Thanks for cleaning up my messes. | 00:28 |
dstanek | ayoung: lol, the ldap tests? | 00:29 |
ayoung | dstanek, some cruft I wrote in there before you were on the project | 00:30 |
ayoung | that groups container thing? | 00:30 |
ayoung | that was from before the identity/assignment split | 00:30 |
dstanek | we have lots of cruft :-) | 00:30 |
dstanek | i'm not a software engineer, i'm a code janitor | 00:30 |
ayoung | Now we know who custodiates the custodians! | 00:32 |
ayoung | dstanek, consider my +2s there sticky. They are all good cleanup patches | 00:32 |
*** hoonetorg has quit IRC | 00:32 | |
dstanek | ayoung: muchas gracias | 00:32 |
ayoung | De Nada | 00:32 |
dstanek | ayoung: how's everything going? haven't seen you for a while | 00:33 |
ayoung | dstanek, we got live migration working. | 00:33 |
dstanek | awesome | 00:33 |
ayoung | Now I'm doing code clean up, and trying to learn go and the kubernetes coding style. | 00:33 |
dstanek | has that been your focus? | 00:33 |
ayoung | yeah | 00:33 |
ayoung | we're doing a Kubernetes based VM management API. | 00:34 |
*** catintheroof has quit IRC | 00:34 | |
dstanek | i've been playing with go a bit. pretty cool language | 00:35 |
ayoung | Its github based development, with some changes of mindsets when dealing with tooling | 00:35 |
ayoung | Go is OK. I've done enough Java over my life to appreciate where they streamlined something basic things | 00:35 |
ayoung | Its kindof like typesafe Python. If you squint | 00:36 |
ayoung | I do like the compiler support. Automated refactroing could be better. | 00:37 |
ayoung | dstanek, you coming to Boston? | 00:37 |
dstanek | ayoung: that's the plan so far | 00:39 |
ayoung | Excellent. | 00:39 |
*** elleobot has joined #openstack-keystone | 00:41 | |
openstackgerrit | Merged openstack/keystone master: Remove orphaned AuthTestMixin from test_v3 https://review.openstack.org/450984 | 00:45 |
*** hoonetorg has joined #openstack-keystone | 00:45 | |
*** masterjcool has quit IRC | 00:59 | |
*** nkinder has joined #openstack-keystone | 00:59 | |
*** namnh has joined #openstack-keystone | 00:59 | |
*** jamielennox is now known as jamielennox|away | 01:02 | |
*** thorst has joined #openstack-keystone | 01:08 | |
*** masterjcool has joined #openstack-keystone | 01:11 | |
*** liujiong has joined #openstack-keystone | 01:11 | |
*** thorst has quit IRC | 01:12 | |
*** jamielennox|away is now known as jamielennox | 01:16 | |
*** adrian_otto has quit IRC | 01:18 | |
*** thorst has joined #openstack-keystone | 01:19 | |
*** david-lyle has joined #openstack-keystone | 01:24 | |
*** nkinder has quit IRC | 01:25 | |
*** thorst has quit IRC | 01:29 | |
*** hoonetorg has quit IRC | 01:42 | |
*** agrebennikov has quit IRC | 01:42 | |
dstanek | woot, almost all of those patches are passing...only one not just needed a recheck | 01:44 |
*** hoonetorg has joined #openstack-keystone | 01:54 | |
*** pramodrj07 has quit IRC | 01:58 | |
*** MasterOfBugs has quit IRC | 01:58 | |
*** zhurong has joined #openstack-keystone | 02:04 | |
*** adrian_otto has joined #openstack-keystone | 02:21 | |
*** dikonoo has joined #openstack-keystone | 02:26 | |
*** adrian_otto has quit IRC | 02:27 | |
*** thorst has joined #openstack-keystone | 02:30 | |
*** david-lyle has quit IRC | 02:36 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/keystone master: Updated from global requirements https://review.openstack.org/451014 | 02:39 |
*** agrebennikov has joined #openstack-keystone | 02:42 | |
*** thorst has quit IRC | 02:49 | |
*** aojea has joined #openstack-keystone | 02:49 | |
*** rderose has quit IRC | 02:53 | |
*** aojea has quit IRC | 02:54 | |
*** dave-mccowan has quit IRC | 03:03 | |
*** dikonoo has quit IRC | 03:04 | |
*** prashkre has joined #openstack-keystone | 03:07 | |
*** links has joined #openstack-keystone | 03:12 | |
*** raginbajin has quit IRC | 03:14 | |
*** nicolasbock has quit IRC | 03:15 | |
*** raginbajin has joined #openstack-keystone | 03:21 | |
*** elleobot has quit IRC | 03:21 | |
*** agrebennikov has quit IRC | 03:23 | |
*** erlon has quit IRC | 03:35 | |
*** david-lyle has joined #openstack-keystone | 03:37 | |
*** prashkre has quit IRC | 03:39 | |
*** thorst has joined #openstack-keystone | 03:46 | |
*** thorst has quit IRC | 03:51 | |
*** Dinesh_Bhor has joined #openstack-keystone | 04:00 | |
*** thorst has joined #openstack-keystone | 04:47 | |
*** thorst has quit IRC | 04:52 | |
*** obre_ has joined #openstack-keystone | 05:06 | |
*** vryzhenkin has joined #openstack-keystone | 05:07 | |
*** obre has quit IRC | 05:07 | |
*** freerunner has quit IRC | 05:07 | |
*** vryzhenkin is now known as freerunner | 05:07 | |
*** hlo323 has quit IRC | 05:09 | |
*** adrian_otto has joined #openstack-keystone | 05:27 | |
*** oomichi has quit IRC | 05:29 | |
*** ayoung has quit IRC | 05:29 | |
*** edmondsw has joined #openstack-keystone | 05:29 | |
*** oomichi has joined #openstack-keystone | 05:33 | |
*** edmondsw has quit IRC | 05:34 | |
*** rcernin has joined #openstack-keystone | 05:39 | |
*** richm has quit IRC | 05:44 | |
*** thorst has joined #openstack-keystone | 05:48 | |
*** thorst has quit IRC | 05:52 | |
*** zhurong has quit IRC | 05:54 | |
*** luzC has quit IRC | 05:55 | |
*** luzC has joined #openstack-keystone | 05:56 | |
*** ayoung has joined #openstack-keystone | 05:58 | |
*** jamielennox is now known as jamielennox|away | 06:10 | |
*** jaosorior has joined #openstack-keystone | 06:14 | |
*** pcaruana has joined #openstack-keystone | 06:24 | |
*** tovin07 has joined #openstack-keystone | 06:25 | |
*** jamielennox|away is now known as jamielennox | 06:25 | |
*** ayoung has quit IRC | 06:38 | |
*** thorst has joined #openstack-keystone | 06:49 | |
*** jamielennox is now known as jamielennox|away | 06:50 | |
*** ayoung has joined #openstack-keystone | 06:51 | |
*** thorst has quit IRC | 06:53 | |
*** jamielennox|away is now known as jamielennox | 06:57 | |
*** alex_xu has quit IRC | 07:13 | |
*** tesseract has joined #openstack-keystone | 07:14 | |
*** alex_xu has joined #openstack-keystone | 07:16 | |
*** aojea has joined #openstack-keystone | 07:20 | |
*** Aqsa has joined #openstack-keystone | 07:24 | |
*** oomichi has quit IRC | 07:29 | |
*** oomichi has joined #openstack-keystone | 07:33 | |
*** zhurong has joined #openstack-keystone | 07:34 | |
*** aojea_ has joined #openstack-keystone | 07:41 | |
*** aojea has quit IRC | 07:44 | |
*** sileht has quit IRC | 07:45 | |
*** tovin07 has quit IRC | 07:45 | |
*** thorst has joined #openstack-keystone | 07:49 | |
*** zzzeek has quit IRC | 08:00 | |
*** pnavarro has joined #openstack-keystone | 08:02 | |
*** zzzeek has joined #openstack-keystone | 08:03 | |
*** thorst has quit IRC | 08:08 | |
*** adrian_otto has quit IRC | 08:16 | |
*** jamielennox is now known as jamielennox|away | 08:17 | |
Dinesh_Bhor | lbragstad, dstanek: Hi, I have added some tests to request-id patch: https://review.openstack.org/#/c/329913/. Could you please take a look at it and give your valuable suggestions? | 08:48 |
*** prashkre has joined #openstack-keystone | 08:59 | |
*** thorst has joined #openstack-keystone | 09:05 | |
*** thorst has quit IRC | 09:09 | |
*** henrynash has quit IRC | 09:24 | |
*** prashkre_ has joined #openstack-keystone | 09:37 | |
*** prashkre has quit IRC | 09:37 | |
*** edmondsw has joined #openstack-keystone | 09:43 | |
*** edmondsw has quit IRC | 09:48 | |
*** mvk has quit IRC | 10:00 | |
*** thorst has joined #openstack-keystone | 10:05 | |
*** thorst has quit IRC | 10:10 | |
*** richm has joined #openstack-keystone | 10:14 | |
*** liujiong has quit IRC | 10:19 | |
*** henrynash has joined #openstack-keystone | 10:27 | |
*** oomichi has quit IRC | 10:29 | |
*** mvk has joined #openstack-keystone | 10:30 | |
*** nicolasbock has joined #openstack-keystone | 10:31 | |
*** oomichi has joined #openstack-keystone | 10:33 | |
*** knangia has quit IRC | 10:41 | |
*** raildo has joined #openstack-keystone | 11:01 | |
*** thorst has joined #openstack-keystone | 11:06 | |
*** thorst has quit IRC | 11:11 | |
breton | why can't we use regex with whitelist and blacklist conditions in mapping? | 11:32 |
breton | was it a choice or just nobody implemented it? | 11:32 |
*** zhurong has quit IRC | 11:34 | |
*** henrynash has quit IRC | 11:37 | |
*** spotz is now known as spotz_zzz | 11:40 | |
*** henrynash has joined #openstack-keystone | 11:43 | |
*** henrynash has quit IRC | 11:48 | |
breton | ok, nevermind, i just used field in "type" twice | 11:49 |
breton | http://paste.openstack.org/show/604647/ like this | 11:49 |
*** thorst has joined #openstack-keystone | 11:54 | |
*** spotz_zzz is now known as spotz | 11:58 | |
*** chlong has quit IRC | 12:05 | |
gagehugo | o/ | 12:11 |
*** henrynash has joined #openstack-keystone | 12:13 | |
*** bjornar_ has joined #openstack-keystone | 12:20 | |
*** edmondsw has joined #openstack-keystone | 12:20 | |
*** henrynash has quit IRC | 12:24 | |
*** mvk has quit IRC | 12:25 | |
*** henrynash has joined #openstack-keystone | 12:26 | |
*** henrynash has quit IRC | 12:26 | |
*** bauruine_ has quit IRC | 12:27 | |
*** prashkre has joined #openstack-keystone | 12:28 | |
*** namnh has quit IRC | 12:29 | |
*** prashkre_ has quit IRC | 12:30 | |
*** bauruine has joined #openstack-keystone | 12:32 | |
*** mvk has joined #openstack-keystone | 12:38 | |
*** dave-mccowan has joined #openstack-keystone | 12:46 | |
*** prashkre has quit IRC | 12:47 | |
*** lamt has joined #openstack-keystone | 12:48 | |
*** bauruine has quit IRC | 12:49 | |
*** agrebennikov has joined #openstack-keystone | 12:50 | |
*** bauruine has joined #openstack-keystone | 12:51 | |
*** lamt has quit IRC | 12:52 | |
*** prashkre has joined #openstack-keystone | 12:59 | |
*** lamt has joined #openstack-keystone | 13:00 | |
*** spilla has joined #openstack-keystone | 13:05 | |
dstanek | breton: we chose to leave it out. we thought that it was too complicated for some reason | 13:09 |
breton | .цшт 23 | 13:10 |
breton | :( | 13:10 |
dstanek | i don't remember why. i just remember the discussion | 13:12 |
breton | ok. Anyway, paste above worked for me. | 13:14 |
*** lamt has quit IRC | 13:17 | |
*** ravelar has joined #openstack-keystone | 13:18 | |
*** sileht has joined #openstack-keystone | 13:19 | |
*** links has quit IRC | 13:20 | |
*** erlon has joined #openstack-keystone | 13:20 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Validate rolling upgrade is run in order https://review.openstack.org/437441 | 13:21 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Validate rolling upgrade is run in order https://review.openstack.org/437441 | 13:21 |
*** zhurong has joined #openstack-keystone | 13:23 | |
*** knangia has joined #openstack-keystone | 13:25 | |
*** catintheroof has joined #openstack-keystone | 13:30 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove policy file from source and refactor tests https://review.openstack.org/449675 | 13:30 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/keystone master: Updated from global requirements https://review.openstack.org/451014 | 13:33 |
dolphm | dstanek: do you remember the fix for testr reporting double the number of actual test failures? | 13:36 |
dolphm | if not just double the number of actual tests | 13:37 |
*** jaosorior has quit IRC | 13:38 | |
*** chlong has joined #openstack-keystone | 13:40 | |
*** catintheroof has quit IRC | 13:41 | |
*** jaosorior has joined #openstack-keystone | 13:43 | |
dstanek | dolphm: it was an upstream but that i thought they closed. are you still seeing it? | 13:43 |
dolphm | dstanek: yes, in oslo.config's py27 tests | 13:44 |
dstanek | i actually think the counts are off. i was trying to make sure that my changes didn't stop some of the tests from running since i changed the inheritence tree and they were off | 13:44 |
dstanek | i added 1 tests, but when i compared the list of tests printed they were exactly the same | 13:45 |
dolphm | dstanek: upstream in testr i assume, right? | 13:46 |
dstanek | dolphm: yes | 13:46 |
dstanek | actually it might have been a bug that i reported | 13:46 |
*** jlopezgu_ has joined #openstack-keystone | 13:46 | |
*** Aqsam has joined #openstack-keystone | 13:49 | |
*** Aqsa has quit IRC | 13:52 | |
*** jrist has quit IRC | 13:55 | |
*** Aqsam has quit IRC | 13:55 | |
*** jrist has joined #openstack-keystone | 13:57 | |
*** ayoung has quit IRC | 13:58 | |
*** nicolasbock has quit IRC | 14:05 | |
*** sjain has joined #openstack-keystone | 14:06 | |
*** zhurong_ has joined #openstack-keystone | 14:07 | |
*** lucasxu has joined #openstack-keystone | 14:07 | |
*** zhurong has quit IRC | 14:07 | |
*** pnavarro has quit IRC | 14:11 | |
*** zhurong_ has quit IRC | 14:18 | |
*** nkinder has joined #openstack-keystone | 14:19 | |
*** Aqsa has joined #openstack-keystone | 14:27 | |
*** adrian_otto has joined #openstack-keystone | 14:35 | |
*** rderose has joined #openstack-keystone | 14:39 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove policy file from source and refactor tests https://review.openstack.org/449675 | 14:43 |
*** ma9_ has joined #openstack-keystone | 14:45 | |
*** lamt has joined #openstack-keystone | 14:46 | |
*** pnavarro has joined #openstack-keystone | 14:59 | |
*** jistr is now known as jistr|mtg | 15:01 | |
*** ayoung has joined #openstack-keystone | 15:02 | |
*** rcernin has quit IRC | 15:05 | |
*** henrynash has joined #openstack-keystone | 15:12 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove unused revocation check in revoke_models https://review.openstack.org/451452 | 15:12 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove unused revocation check in revoke_models https://review.openstack.org/451452 | 15:19 |
stevemar | someone needs to punt this through: https://review.openstack.org/#/c/447562/ !! | 15:22 |
stevemar | and this https://review.openstack.org/#/c/448186/ !! | 15:22 |
lbragstad | stevemar i can review the second one for sure | 15:23 |
stevemar | lbragstad: <3 | 15:23 |
stevemar | thanks rodrigods ! :) | 15:23 |
rodrigods | :) | 15:24 |
rodrigods | left the second for lbragstad | 15:24 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove unused revocation check in revoke_models https://review.openstack.org/451452 | 15:27 |
*** sjain has quit IRC | 15:27 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Validate rolling upgrade is run in order https://review.openstack.org/437441 | 15:29 |
*** ma9_ has quit IRC | 15:29 | |
*** adrian_otto has quit IRC | 15:30 | |
ravelar | stevemar lbragstad rodrigods: thanks all! | 15:31 |
*** alex_xu has quit IRC | 15:33 | |
*** davechen has quit IRC | 15:34 | |
*** alex_xu has joined #openstack-keystone | 15:35 | |
*** ma9_ has joined #openstack-keystone | 15:40 | |
*** ma9_ has left #openstack-keystone | 15:44 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove redundant code in test_revoke https://review.openstack.org/451463 | 15:47 |
*** davechen has joined #openstack-keystone | 15:47 | |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Add policy sample generation https://review.openstack.org/443344 | 15:48 |
lbragstad | ravelar antwash if i'm looking through all the reviews correctly - we only need https://review.openstack.org/#/c/449675/ and https://review.openstack.org/#/c/448826/ to merge for policy-in-code to be fully implemented? | 15:52 |
antwash | lbragstad: yeah and plus the patch I just rebased then it'll done :) | 15:53 |
lbragstad | antwash aha yep | 15:54 |
lbragstad | we also have this - https://review.openstack.org/#/c/450818/ | 15:54 |
*** aojea_ has quit IRC | 15:56 | |
*** pnavarro has quit IRC | 15:57 | |
*** tesseract has quit IRC | 15:58 | |
knikolla | o/ | 15:59 |
*** bjornar_ has quit IRC | 16:01 | |
knikolla | lbragstad: is there a policy meeting now? | 16:02 |
notmorgan | knikolla: i thought i saw an email saying cancelled | 16:02 |
lbragstad | knikolla we didn't have anything proposed so i cancelled | 16:02 |
*** adrian_otto has joined #openstack-keystone | 16:02 | |
lbragstad | knikolla figured we could use the time to review policy patch instead | 16:02 |
knikolla | lbragstad notmorgan: alright, didn't check my mail :) | 16:03 |
lbragstad | knikolla http://lists.openstack.org/pipermail/openstack-dev/2017-March/114705.html | 16:03 |
knikolla | lbragstad: thanks, just opened my mail client for the first time today. I'll help with reviewing. | 16:04 |
lbragstad | knikolla that'd be much appreciated, thanks! | 16:04 |
*** lamt has quit IRC | 16:10 | |
*** pnavarro has joined #openstack-keystone | 16:12 | |
*** d0ugal has quit IRC | 16:17 | |
lbragstad | ravelar one pep8 thing here https://review.openstack.org/#/c/449675/9 otherwise it should pass | 16:19 |
lbragstad | ravelar aha - i see you're already on it, never mind | 16:19 |
lbragstad | :) | 16:19 |
knikolla | reviewing the policy docs patches feels like gaming stackalytics | 16:30 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move service provider to DocumentedRuleDefault https://review.openstack.org/449347 | 16:30 |
ravelar | lbragstad lol i was in a hurry to throw a commit up and went to the gym. little pep8 tried to pull a fast one on me while I was afk | 16:32 |
antwash | knikolla: originally it was just 5 patches, but it was recommended to separate them in single patches for better reviewing | 16:33 |
ravelar | knikolla lol! | 16:33 |
lbragstad | yeah - we opted to have a bunch of smaller reviews in an effort to get them gating in parallel as well as easier reviewability | 16:33 |
knikolla | it definitely resulted in easier reviewability. | 16:34 |
*** Aqsa has quit IRC | 16:35 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove policy file from source and refactor tests https://review.openstack.org/449675 | 16:35 |
*** d0ugal has joined #openstack-keystone | 16:38 | |
*** d0ugal has quit IRC | 16:43 | |
*** pnavarro has quit IRC | 16:45 | |
*** jistr|mtg is now known as jistr | 16:47 | |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move protocol to DocumentedRuleDefault https://review.openstack.org/449345 | 16:55 |
*** ravelar has quit IRC | 16:55 | |
openstackgerrit | Merged openstack/keystone master: Remove unnecessary revocation events https://review.openstack.org/447562 | 17:01 |
openstackgerrit | Merged openstack/keystone master: Remove unnecessary revocation events https://review.openstack.org/448186 | 17:02 |
openstackgerrit | Merged openstack/keystone master: Remove unnecessary revocation events revoke grant https://review.openstack.org/448192 | 17:02 |
openstackgerrit | Merged openstack/keystone master: Remove decorator for asserting validation errors https://review.openstack.org/450985 | 17:02 |
openstackgerrit | Merged openstack/keystone master: Remove orphaned _create_context test helper https://review.openstack.org/450986 | 17:03 |
openstackgerrit | Merged openstack/keystone master: Remove conflict guards in load_fixtures https://review.openstack.org/450987 | 17:03 |
openstackgerrit | Merged openstack/keystone master: Consolidate and cleanup test_backend_ldap setup https://review.openstack.org/450988 | 17:03 |
openstackgerrit | Merged openstack/keystone master: Reduce fixture setup in test_backend_ldap https://review.openstack.org/450989 | 17:03 |
*** chlong has quit IRC | 17:03 | |
dstanek | the merge fairy strikes again | 17:04 |
stevemar | noice | 17:05 |
*** links has joined #openstack-keystone | 17:07 | |
dstanek | grrrrr...one of my changed failed to merge | 17:07 |
*** ravelar has joined #openstack-keystone | 17:08 | |
samueldmq | hi keystone | 17:10 |
samueldmq | rs | 17:10 |
samueldmq | stevemar o/ | 17:10 |
stevemar | samueldmq: howdy sir | 17:11 |
*** links has quit IRC | 17:19 | |
*** MasterOfBugs has joined #openstack-keystone | 17:29 | |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove redundant code in test_revoke https://review.openstack.org/451463 | 17:29 |
*** bjornar_ has joined #openstack-keystone | 17:32 | |
*** mvk has quit IRC | 17:41 | |
*** pcaruana has quit IRC | 17:52 | |
*** chlong has joined #openstack-keystone | 17:58 | |
*** chlong has quit IRC | 18:07 | |
*** alex_xu has quit IRC | 18:07 | |
*** bjornar_ has quit IRC | 18:08 | |
*** alex_xu has joined #openstack-keystone | 18:08 | |
*** lamt has joined #openstack-keystone | 18:12 | |
dstanek | samueldmq: ! | 18:12 |
*** mvk has joined #openstack-keystone | 18:13 | |
*** chlong has joined #openstack-keystone | 18:21 | |
*** aojea has joined #openstack-keystone | 18:24 | |
lbragstad | antwash just a heads up - but we're going to hit some py35 failures in the gate here - https://review.openstack.org/#/c/443344/ | 18:25 |
*** ayoung has quit IRC | 18:28 | |
*** chlong has quit IRC | 18:34 | |
antwash | lbragstad: :( | 18:34 |
samueldmq | dstanek: o/ | 18:34 |
antwash | lbragstad: the traceback error message is interesting | 18:36 |
lbragstad | antwash yeah - the error was obvious to me | 18:36 |
lbragstad | but I also didn't dig spend a bunch of time digging into it either | 18:37 |
lbragstad | s/dig// | 18:37 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/keystone master: Updated from global requirements https://review.openstack.org/451014 | 18:42 |
*** openstackstatus has joined #openstack-keystone | 18:44 | |
*** ChanServ sets mode: +v openstackstatus | 18:44 | |
*** lucasxu has quit IRC | 18:45 | |
*** lucasxu has joined #openstack-keystone | 18:45 | |
samueldmq | lbragstad: antwash: can any of you walk me through the current state of changes for policy | 18:49 |
samueldmq | I do see a LOT of patches up for review | 18:49 |
*** chlong has joined #openstack-keystone | 18:49 | |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move mapping to DocumentedRuleDefault https://review.openstack.org/449341 | 18:50 |
*** Aqsa has joined #openstack-keystone | 18:51 | |
lbragstad | samueldmq we have two implementations up for review right now | 18:52 |
lbragstad | samueldmq policy-in-code and policy-docs | 18:52 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move service provider to DocumentedRuleDefault https://review.openstack.org/449347 | 18:52 |
lbragstad | samueldmq i'd probably start with reviewing the remaining policy-in-code patches - https://review.openstack.org/#/q/status:open+project:openstack/keystone+branch:master+topic:bp/policy-in-code | 18:52 |
lbragstad | the policy-docs patches are replacing the RuleDefault object we're using from oslo.policy with another, more strict object, that will require us to provide descriptions of all policies in keysotne | 18:53 |
lbragstad | the descriptions will be rendered when generating sample policy files - just like we do when generating sample configds | 18:53 |
lbragstad | configs* | 18:53 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move identity provider to DocumentedRuleDefault https://review.openstack.org/449275 | 18:53 |
samueldmq | lbragstad: gotcha, so one instance of RuleDefault per policy rule or something like that ? | 18:54 |
lbragstad | samueldmq this is an example https://review.openstack.org/#/c/449234/ | 18:54 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move domain policies to DocumentedRuleDefault https://review.openstack.org/449234 | 18:54 |
lbragstad | samueldmq it switches RuleDefault for DocumentedRuleDefault and supplies the required attributes of that class | 18:55 |
antwash | lbragstad: let me know how that commit message sounds, making the adjustments now | 18:55 |
lbragstad | so we need to make sure the documentation we're supplying is accurate when compared to the actual api | 18:55 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move endpoint policies to DocumentedRuleDefault https://review.openstack.org/449212 | 18:56 |
lbragstad | antwash sounds perfect, thanks for respinning | 18:56 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move policy association to DocumentedRuleDefault https://review.openstack.org/449344 | 18:57 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move ec2 credential policies to DocumentedRuleDefault https://review.openstack.org/449235 | 18:58 |
antwash | lbragstad: np, I wont overload Zuul and the chat with the changes I'll break them up lol | 18:58 |
lbragstad | antwash ++ | 18:59 |
lbragstad | antwash i was thinking about that last night | 18:59 |
lbragstad | antwash since there is such a large number of patches, maybe we pick 5 everyday and focus on them until they merge | 18:59 |
lbragstad | then pick a new 5 | 18:59 |
lbragstad | and so on and so forth, that way you don't have to respin 20 patches every time someone has a comment | 19:00 |
lbragstad | and reviewers can prioritize which ones to review | 19:00 |
samueldmq | lbragstad: done with policy-in-code. all those approved. | 19:01 |
samueldmq | lbragstad: https://review.openstack.org/#/q/topic:bp/policy-docs+status:open now right ? | 19:01 |
samueldmq | well, adding project:openstack/keystone to this ^ | 19:01 |
lbragstad | samueldmq yeah - essentially | 19:02 |
antwash | samueldmq: hey there, just read you comment on https://review.openstack.org/#/c/443344/ | 19:02 |
antwash | samueldmq: the 'yaml' format was adopted from the nova team | 19:02 |
lbragstad | samueldmq let's prioritize https://review.openstack.org/#/c/449344/ https://review.openstack.org/#/c/449235/ https://review.openstack.org/#/c/449212/ and https://review.openstack.org/#/c/449234/ | 19:02 |
*** spilla has quit IRC | 19:02 | |
samueldmq | antwash: nice. I am fine with that, was just curious about the change, but it's easily possible to generate the .json too, correct? | 19:03 |
samueldmq | antwash: as people are more used to it after all these years | 19:03 |
lbragstad | samueldmq nope | 19:03 |
samueldmq | lbragstad: not easy to generate policy.json anymore ? | 19:04 |
lbragstad | samueldmq https://bugs.launchpad.net/oslo.policy/+bug/1607968 | 19:04 |
openstack | Launchpad bug 1607968 in oslo.policy "oslopolicy-sample-generator does not output json, but docs always reference json" [Undecided,Confirmed] | 19:04 |
antwash | thanks for the answer lance :) | 19:04 |
lbragstad | antwash no problem | 19:04 |
lbragstad | samueldmq but we should totally try and fix oslo.policy so that is supports that case | 19:04 |
antwash | yeah I agree that way you're not limited to a cetain format | 19:05 |
lbragstad | but if the new hotness is to move towards yaml, then we also need to go through and start correcting our documentation | 19:05 |
lbragstad | because the reporter of that bug has a point | 19:05 |
lbragstad | and so does robcresswell | 19:05 |
samueldmq | lbragstad: I just commented on the bug. oslopolicy-sample-generator should generate .json too | 19:06 |
lbragstad | to be backwards compatible, we can create a new flag for policy generation that denotes the format to output, if it's not specified we can assume .yaml | 19:07 |
samueldmq | lbragstad: I am fine if we change the default format. that's totally okay. then we fix the docs | 19:07 |
lbragstad | (which would maintain backwards compat) | 19:07 |
samueldmq | lbragstad: but we should still support the .json format, and people should be able to generate it imo | 19:07 |
lbragstad | yeah | 19:07 |
samueldmq | lbragstad: nice | 19:07 |
samueldmq | lbragstad: this is why we need to deprecate/remove things that nobody uses | 19:09 |
samueldmq | https://review.openstack.org/#/c/449344/4/keystone/common/policies/policy_association.py | 19:09 |
samueldmq | it's been a good amount of effort put on that policy backend :( | 19:10 |
lbragstad | i'm not sure we'd ever be able to remove it | 19:10 |
samueldmq | lbragstad: didn't we agree to deprecate it for removal during the PTG? | 19:12 |
lbragstad | samueldmq that was the policy API | 19:12 |
lbragstad | if i remember correctly | 19:12 |
lbragstad | and i think that was something notmorgan already proposed | 19:12 |
samueldmq | lbragstad: hmm yes, but if policy API goes away, there is no point in keeping policy association | 19:13 |
samueldmq | as associations map policy entities to targets :-) | 19:14 |
lbragstad | samueldmq i think the outcome of that discussion was that we would deprecate the policy API, since it's effectively useless at the moment, but we'd never be able to remove it because that would break backwards compatibiltiy | 19:14 |
lbragstad | if we ever did a v4, we just wouldn't include a policy api | 19:14 |
*** dulek has quit IRC | 19:15 | |
*** Aqsa has quit IRC | 19:15 | |
*** gyee has joined #openstack-keystone | 19:15 | |
lbragstad | samueldmq https://etherpad.openstack.org/p/pike-ptg-keystone-deprecations line 35 | 19:15 |
samueldmq | lbragstad: k. so my point is that we should deprecate endpojnt_policy too | 19:17 |
samueldmq | lbragstad: if policy endpoint is useless, something that needs it to exist is useless as well | 19:18 |
lbragstad | samueldmq yeah | 19:18 |
*** dulek has joined #openstack-keystone | 19:18 | |
*** david-lyle has quit IRC | 19:19 | |
*** david-lyle has joined #openstack-keystone | 19:19 | |
samueldmq | antwash: btw well done on all those patches, you're doing a great job. | 19:20 |
lbragstad | ++ it's a lot of work | 19:20 |
antwash | Thanks you too, just ready to get them merged now lol | 19:22 |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/keystone master: Move ec2 credential policies to DocumentedRuleDefault https://review.openstack.org/449235 | 19:23 |
samueldmq | lbragstad: ^ | 19:23 |
*** bjornar_ has joined #openstack-keystone | 19:26 | |
*** david-lyle has quit IRC | 19:27 | |
samueldmq | bp/policy-docs sprint ? | 19:29 |
samueldmq | :-) | 19:29 |
*** david-lyle has joined #openstack-keystone | 19:30 | |
lbragstad | we could! | 19:31 |
*** ravelar1 has joined #openstack-keystone | 19:32 | |
*** openstackgerrit has quit IRC | 19:33 | |
*** jamielennox|away is now known as jamielennox | 19:33 | |
*** ravelar1 has quit IRC | 19:34 | |
*** chlong has quit IRC | 19:34 | |
*** lamt has quit IRC | 19:41 | |
rodrigods | lbragstad, maybe https://review.openstack.org/#/c/445505/6 uncovered a bug | 19:44 |
rodrigods | or the fix is wrong | 19:44 |
rodrigods | see my last comment | 19:44 |
*** rarora has quit IRC | 19:45 | |
lbragstad | rodrigods sounds good - thanks for the update | 19:46 |
samueldmq | lbragstad: I wonder if we did something like "Create AN identity provider." rather than "Create identity provider." | 19:47 |
samueldmq | for all policy docs... if that would be something more correct | 19:48 |
samueldmq | or both are okay. I am talking from an English perspective, and you can say it better than I can :-) | 19:48 |
*** d0ugal has joined #openstack-keystone | 19:49 | |
*** raildo has quit IRC | 19:53 | |
*** swatson has joined #openstack-keystone | 19:58 | |
*** lamt has joined #openstack-keystone | 19:58 | |
swatson | Hey channel, I'm trying to start a devstack and I'm having issues that seem related to keystone | 19:58 |
swatson | Here's my relevant output from devstack: http://paste.openstack.org/show/604729/ | 19:58 |
*** openstackgerrit has joined #openstack-keystone | 19:59 | |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move policy association to DocumentedRuleDefault https://review.openstack.org/449344 | 19:59 |
swatson | Any ideas about what that error is about? | 19:59 |
*** d0ugal has quit IRC | 20:00 | |
lbragstad | swatson looks like the admin project isn't there "++lib/keystone:create_keystone_accounts:384 openstack project show admin -f value -c id" | 20:03 |
lbragstad | swatson is there any other information prior to that point that suggests an issue? | 20:04 |
swatson | lbragstad: I'll take a look | 20:05 |
lbragstad | swatson whenever i see issues like that, they are typically caused from a prior step failing in some way | 20:07 |
swatson | I had to start the stack script again because I forgot I had just unstacked as well >_< | 20:07 |
lbragstad | swatson is it a relatively quick failure? | 20:07 |
swatson | lbragstad: Yeah, within a few minutes that I've noticed | 20:07 |
lbragstad | cool | 20:08 |
antwash | lbragstad: looking at the https://github.com/openstack/keystone/blob/stable/mitaka/etc/policy.json -- the authenticate for ec2 isn't there -- would you happen to know the rule applied to it? | 20:08 |
*** prashkre has quit IRC | 20:11 | |
swatson | lbragstad: I hit the same error again. If I scroll up a bit further I see something about "key_repository does not appear to exist" a few times | 20:12 |
swatson | It looks like maybe key_repository ends up created, and then I see: | 20:13 |
swatson | start_service apache2 | 20:13 |
swatson | +functions-common:start_service:2305 '[' -x /bin/systemctl ']' | 20:13 |
swatson | +functions-common:start_service:2308 sudo service apache2 start | 20:13 |
swatson | * Starting web server apache2 AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using 127.0.0.1. Set the 'ServerName' directive globally to suppress this message | 20:13 |
breton | this is probably just warning | 20:14 |
breton | i think you can ignore it for now | 20:14 |
swatson | Is there a devstack log somewhere? | 20:15 |
lbragstad | hmm | 20:15 |
swatson | My shell didn't store much past the repository stuff | 20:16 |
lbragstad | swatson you could check the keystone logs, too | 20:16 |
lbragstad | swatson you could tee the devstack logs out to a file as well | 20:16 |
lbragstad | s/logs/output/ | 20:16 |
swatson | are the keystone logs in /var/log? I don't have a keystone dir in there | 20:16 |
swatson | lbragstad I'm re-stacking now, gathering everything in another file | 20:18 |
dstanek | swatson: if you are running under apache it'll be something like /var/log/{httpd,apache2}/keystone.log | 20:20 |
swatson | dstanek: Ah, that makes sense. I'll let this stack finish and then take a look | 20:21 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move mapping to DocumentedRuleDefault https://review.openstack.org/449341 | 20:22 |
openstackgerrit | Anthony Washington proposed openstack/keystone master: Move identity provider to DocumentedRuleDefault https://review.openstack.org/449275 | 20:23 |
*** Guest6666 has quit IRC | 20:28 | |
*** Guest6666 has joined #openstack-keystone | 20:29 | |
swatson | Keystone logs don't seem out of the ordinary | 20:30 |
*** pramodrj07 has joined #openstack-keystone | 20:31 | |
*** david-lyle has quit IRC | 20:33 | |
*** MasterOfBugs has quit IRC | 20:34 | |
knikolla | is the python35 gate broken? | 20:34 |
*** rdo has quit IRC | 20:34 | |
knikolla | seems like it | 20:34 |
antwash | ++ | 20:35 |
*** david-lyle has joined #openstack-keystone | 20:35 | |
*** rdo has joined #openstack-keystone | 20:36 | |
*** ayoung has joined #openstack-keystone | 20:38 | |
*** david-lyle has quit IRC | 20:40 | |
*** david-lyle has joined #openstack-keystone | 20:42 | |
knikolla | dstanek: ^^ | 20:43 |
knikolla | seems to me a webob thing | 20:43 |
knikolla | TypeError: You cannot set the body to a text value without a charset | 20:44 |
*** henrynash has quit IRC | 20:44 | |
*** david-lyle has quit IRC | 20:46 | |
openstackgerrit | Kristi Nikolla proposed openstack/keystone master: Add charset to webob.Response https://review.openstack.org/451559 | 20:49 |
*** dulek has quit IRC | 20:50 | |
*** dulek has joined #openstack-keystone | 20:52 | |
*** Aqsa has joined #openstack-keystone | 20:52 | |
*** david-lyle has joined #openstack-keystone | 20:52 | |
*** spilla has joined #openstack-keystone | 20:56 | |
knikolla | that fixed it for me. lbragstad, dstanek ^^ | 20:56 |
knikolla | antwash | 20:57 |
swatson | lbragstad: I tried another stack after pulling the latest from devstack | 21:02 |
swatson | I went to check my screen sessions and it took me to keystone's first. | 21:02 |
swatson | "key-access failed to start. Exit code: $?" | 21:02 |
swatson | It's also referencing "/opt/stack/status/stack/key-access.failure" but I don't seem to have that file | 21:03 |
swatson | I do have key-access.pid and key.pid instead however | 21:04 |
*** Aqsa has quit IRC | 21:07 | |
* lbragstad dstanek it'd be good to get your eyes on this, too https://review.openstack.org/#/c/451559/1 | 21:13 | |
lbragstad | swatson hmm | 21:13 |
lbragstad | swatson did stack.sh fail with the same error as before? | 21:15 |
swatson | yeah | 21:16 |
lbragstad | swatson can you confirm keystone is running? | 21:17 |
swatson | "screen -x" shows 1) dstat 2) key 3) key-access | 21:18 |
swatson | key looks like it's running--no errors showing | 21:18 |
lbragstad | can you curl the version endpoint of keystone and verify a response? | 21:19 |
swatson | key-access looks like it died. It's what gave me that error I mentioned before | 21:19 |
swatson | what's the version endpoint? http://192.168.1.96/identity/version? | 21:20 |
lbragstad | http://192.168.1.96:35357/v3/ | 21:21 |
lbragstad | for example | 21:21 |
swatson | If I try that I get a 500 error | 21:21 |
swatson | "curl http://192.168.1.96:35357/v3/" | 21:21 |
lbragstad | swatson do you have a paste? | 21:21 |
lbragstad | swatson specifically of the logs | 21:22 |
*** andreykurilin has quit IRC | 21:22 | |
swatson | which logs would you like to see? | 21:22 |
lbragstad | keystone's | 21:22 |
lbragstad | would probably be a good start | 21:22 |
*** andreykurilin has joined #openstack-keystone | 21:22 | |
swatson | lbragstad: http://paste.openstack.org/show/604745/ | 21:23 |
swatson | That's my /opt/stack/logs/key.log | 21:23 |
swatson | http://paste.openstack.org/show/604746/ | 21:23 |
swatson | And that's my key-access.log from the same dir | 21:23 |
lbragstad | swatson could you try and cat your entire keystone.log to pasteraw? | 21:25 |
openstackgerrit | Richard Avelar proposed openstack/keystone master: Remove policy file from source and refactor tests https://review.openstack.org/449675 | 21:25 |
*** spilla has quit IRC | 21:25 | |
swatson | lbragstad: the apache2 one? | 21:25 |
lbragstad | `pip install pasteraw; cat /var/log/apache2/keystone_access.log | pasteraw` | 21:25 |
lbragstad | swatson yeah - that one | 21:25 |
swatson | I'm looking at it now actually | 21:25 |
swatson | the last entry has an exceptoin | 21:26 |
swatson | mod_wsgi (pid=32056): Target WSGI script '/usr/local/bin/keystone-wsgi-admin' cannot be loaded as Python module. | 21:26 |
lbragstad | bah | 21:26 |
swatson | Shortly after that: ContextualVersionConflict: (oslo.messaging 5.17.0 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('oslo.messaging>=5.19.0'), set(['keystone'])) | 21:26 |
lbragstad | bah! | 21:26 |
lbragstad | requirements | 21:26 |
swatson | I actually blew away my dist-packages when I first saw a requirements exception in stacking | 21:27 |
swatson | and then I started seeing keystone-related issues | 21:27 |
swatson | after stacking again, that is | 21:27 |
lbragstad | looks like keystone is requiring oslo.messaging 5.19 or greater https://github.com/openstack/keystone/blob/master/requirements.txt#L27 | 21:27 |
lbragstad | swatson try upgrading oslo.messaging manually | 21:28 |
swatson | So some other project is requiring 5.17 only maybe? | 21:28 |
lbragstad | swatson that's a good question | 21:28 |
knikolla | >=5.19 is specified in global-reqs too | 21:29 |
lbragstad | yep = https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L135 | 21:29 |
lbragstad | upper is pinned to 5.19 | 21:29 |
swatson | Hm. I wonder why mine wouldn't upgrade | 21:29 |
swatson | Upgrading manually seems to have worked okay | 21:29 |
swatson | I'll re-stack and see how it goes | 21:29 |
lbragstad | swatson chances are that might be the only one | 21:29 |
lbragstad | swatson sorry, chances are that might *not* be the only one | 21:30 |
swatson | lbragstad ah | 21:31 |
lbragstad | swatson IME whenever i hit requirements issues, i typically have to update a few of them at a time | 21:31 |
swatson | Weird...is this a known issue then? | 21:32 |
lbragstad | swatson well - chances are something got stale in your environment and didn't get updated | 21:32 |
lbragstad | swatson is this a fresh installation? | 21:33 |
lbragstad | of devstack? | 21:33 |
knikolla | i usually respin the vm. i figure if gates aren't broken upstream i may have messed up my env. | 21:33 |
*** lucasxu has quit IRC | 21:34 | |
*** aojea has quit IRC | 21:34 | |
swatson | lbragstad: What's your definition of "fresh installation"? :P | 21:34 |
swatson | I unstacked earlier today and then tried to run stack again | 21:34 |
lbragstad | swatson by fresh i mean an brand new VM with a fresh clone of devstack | 21:35 |
lbragstad | knikolla ++ same here | 21:35 |
swatson | lbragstad: I did a git pull of devstack, but otherwise no. I'm running on my dev machine, no VM | 21:36 |
lbragstad | swatson aha | 21:36 |
*** bjornar_ has quit IRC | 21:36 | |
lbragstad | swatson fwiw - i typically run devstack on things I can dispose of | 21:36 |
knikolla | ++ | 21:37 |
swatson | I'd like that option but with magnum some distros don't quite like VMs yet in my experience :P | 21:37 |
lbragstad | based on the warning here - https://docs.openstack.org/developer/devstack/ | 21:37 |
lbragstad | swatson what distro are you using? | 21:37 |
lbragstad | just out of curiosity | 21:37 |
swatson | I'm running Ubuntu 14.04 | 21:38 |
swatson | I've had issues creating containers through magnum on VMs before, so switched to a regular machine | 21:38 |
lbragstad | aha | 21:38 |
lbragstad | i'm not entirely familiar with magnum development, but that makes sense | 21:38 |
swatson | I can't recall having devstack issues like this before though. | 21:39 |
swatson | I've been running this way for several months now | 21:39 |
*** pramodrj07 has quit IRC | 21:41 | |
*** pramodrj07 has joined #openstack-keystone | 21:41 | |
*** DinaBelova has quit IRC | 21:44 | |
*** timss has quit IRC | 21:44 | |
*** NikitaKonovalov has quit IRC | 21:44 | |
*** NikitaKonovalov has joined #openstack-keystone | 21:45 | |
*** timss has joined #openstack-keystone | 21:45 | |
*** DinaBelova has joined #openstack-keystone | 21:46 | |
openstackgerrit | Samriddhi proposed openstack/keystone master: Updated description of scope parameter in the corresponding strings for scoped and explicit unscoped authentication. https://review.openstack.org/450038 | 21:51 |
*** erlon has quit IRC | 21:55 | |
openstackgerrit | Samriddhi proposed openstack/keystone master: Updated description of scope parameter. https://review.openstack.org/450038 | 21:55 |
lbragstad | knikolla looks like your webob patch failed - http://logs.openstack.org/59/451559/1/check/gate-tempest-dsvm-py35-ubuntu-xenial/0304010/console.html#_2017-03-29_21_49_26_452791 | 22:01 |
*** bjornar_ has joined #openstack-keystone | 22:06 | |
swatson | lbragstad: One interesting thing...even after manually upgrading oslo.messaging it seems like it reverts somehow | 22:08 |
swatson | If I run "sudo pip install --upgrade oslo.messaging" I see all the usual upgrade stuff and it looks successful | 22:08 |
swatson | After running stack again I get the same requirements error of oslo.messaging being 5.17 instead of 5.19 | 22:09 |
lbragstad | swatson do you have a pip config somewhere that's preventing it from upgrading? | 22:09 |
*** jamielennox is now known as jamielennox|away | 22:09 | |
*** adrian_otto has quit IRC | 22:09 | |
swatson | lbragstad: I'm not sure. I haven't done any pip configuration of my own that I can recall | 22:09 |
dtroyer | jamielennox: Around yet? I've got my head full of microversions and am wondering about what is in ksa to do discovery | 22:09 |
*** adrian_otto has joined #openstack-keystone | 22:10 | |
lbragstad | swatson hmm - i wonder if it's using a venv somewhere | 22:11 |
*** jamielennox|away is now known as jamielennox | 22:12 | |
*** rarora has joined #openstack-keystone | 22:13 | |
swatson | hm.. | 22:13 |
jamielennox | dtroyer: i am | 22:13 |
jamielennox | dtroyer: and there is not that much in ksa - i've always wanted there to be more, but no one has given me a straight answer on how we should actually work with them | 22:13 |
*** henrynash has joined #openstack-keystone | 22:14 | |
lbragstad | swatson https://github.com/openstack-dev/devstack/blob/1e7f738f284b85ed95d514fb13fbc1afb6b31087/stack.sh#L737-L749 might be relevant | 22:14 |
swatson | lbragstad: It looks like that'll default to False, and mine is currently unset | 22:15 |
lbragstad | swatson what does your requirements repo look like? | 22:16 |
lbragstad | https://github.com/openstack-dev/devstack/blob/1e7f738f284b85ed95d514fb13fbc1afb6b31087/stack.sh#L720 | 22:16 |
lbragstad | because if that's older it might be out of sync with what devstack is expecting | 22:16 |
swatson | Should I have anything in particular in there? | 22:17 |
dtroyer | jamielennox: ok, I'm going to take a crack at it since a big chunk of what is in novaclient isn't working the way I want | 22:17 |
swatson | lbragstad: Would blowing away this dir be an okay thing to try? | 22:17 |
swatson | since devstack should re-DL anyway | 22:17 |
jamielennox | dtroyer: maybe -> #sdks | 22:18 |
dtroyer | sure, I'm never there anymore it seems :) | 22:18 |
lbragstad | swatson see what oslo.messaging is set to in global-requirements.txt and upper-constraints.txt | 22:18 |
lbragstad | swatson devstack has a specific option for recloning stuff that you can toggle | 22:19 |
swatson | lbragstad: Aha :) "oslo.messaging===5.17.0" in my requirements/upper-constraints.txt | 22:19 |
lbragstad | swatson https://github.com/openstack-dev/devstack/blob/d7bc9486aa548d97d14a8af05f98954bb2b774d5/functions-common#L490 | 22:19 |
lbragstad | swatson yeah - i bet that's what's causing oslo.messaging to be pinned for you | 22:20 |
lbragstad | swatson I'd either set RECLONE = true or forcibly update requirements using `git pull` | 22:20 |
swatson | the latter option sounds faster | 22:21 |
lbragstad | i'm curious to see if that is the issue | 22:22 |
swatson | well my git pull resulted in about 1k line changes in requirements-related files alone | 22:23 |
lbragstad | hm | 22:24 |
swatson | And my stack failed pretty quickly this time | 22:24 |
lbragstad | yeah - i bet that will have impact on your next run | 22:24 |
swatson | IOError: [Errno 2] No such file or directory: '/opt/stack/requirements/.venv/lib/python2.7/site-packages/pyparsing-2.1.10.dist-info/METADATA' | 22:24 |
lbragstad | hm | 22:24 |
swatson | I'm trying again with reclone | 22:28 |
swatson | could take awhile | 22:28 |
swatson | Nope, same error, though it was a little longer to get to | 22:31 |
knikolla | lbragstad: i hope it's only a transient error. the keystone-python35 gate which was broken did indeed pass. | 22:35 |
swatson | lbragstad: I'm heading out today. I'll probably end up starting fresh soon enough. Thanks for all your help though :) | 22:36 |
*** sjain has joined #openstack-keystone | 22:38 | |
*** sjain has quit IRC | 22:39 | |
*** bjornar_ has quit IRC | 22:42 | |
samueldmq | lbragstad: dstanek: jamielennox: it'd be great to get your feedback in bug 1637682 | 22:44 |
openstack | bug 1637682 in OpenStack Identity (keystone) "[api] scoped string defined as 'unscope: {}'" [Low,In progress] https://launchpad.net/bugs/1637682 - Assigned to Samriddhi (s-jain) | 22:44 |
samueldmq | lbragstad: dstanek: jamielennox: my suspicion is that there should be more than a doc fix, see my latest comment | 22:45 |
*** sjain has joined #openstack-keystone | 22:47 | |
dstanek | samueldmq: i added my 2cents | 22:55 |
*** thorst has quit IRC | 22:56 | |
samueldmq | dstanek: thanks, that was 2boxes actually :) | 22:57 |
samueldmq | dstanek: valuable comment on the schema. I was looking at that bug, and the fix, and it was bugging my mind :p | 22:57 |
samueldmq | thanks for confirming | 22:58 |
dstanek | samueldmq: yeah, if we don't add the comment someone will eventually "fix" the validation and maybe break existing clouds | 22:58 |
samueldmq | dstanek: completely agree | 23:00 |
*** pramodrj07 has quit IRC | 23:22 | |
*** lamt has quit IRC | 23:39 | |
*** adrian_otto has quit IRC | 23:51 | |
*** thorst has joined #openstack-keystone | 23:56 | |
*** sjain has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!