*** tuanla____ has joined #openstack-chef | 03:11 | |
*** higgins has quit IRC | 04:16 | |
*** higgins has joined #openstack-chef | 04:20 | |
openstackgerrit | Christoph Albers proposed openstack/cookbook-openstack-dashboard stable/pike: FWaaS Dashboard fix https://review.openstack.org/554554 | 09:09 |
---|---|---|
openstackgerrit | Merged openstack/cookbook-openstack-dashboard master: FWaaS Dashboard fix https://review.openstack.org/554275 | 09:55 |
openstackgerrit | Merged openstack/cookbook-openstack-dashboard master: Update lbaas url for Queens, logging handler https://review.openstack.org/550890 | 09:55 |
*** tuanla____ has quit IRC | 10:01 | |
scas | frickler: as always, thank you for keeping me honest. i was following keystone documentation to make sure i had things "right", but there is no "right" here, when taking into context the other deployment projects | 12:59 |
scas | one could presume that keystone, nova, etc. documentation is The Convention to follow, but it isn't even loosely enforced | 13:01 |
scas | in any case, i'm reverting back to our known convention of keystone-main. revised patch is under local testing currently | 13:17 |
frickler | scas: I did a bit more digging, it seems keystone itself it still a bit inconsistent, calling the outside script keystone-wsgi-public but leaving the application name 'main' on the inside: https://github.com/openstack/keystone/blob/3624c2f5ad20fb3795576b35fd07ceedcc2642ba/keystone/server/wsgi.py#L59-L60 | 13:19 |
scas | yeah. i saw a similar story myself | 13:20 |
frickler | scas: I did somehow think however that the external name was still *main, however that has been changed some time ago (3y) https://github.com/openstack/keystone/commit/3624c2f5ad20fb3795576b35fd07ceedcc2642ba | 13:20 |
frickler | scas: so that would indeed be an argument in favor of changing the chef side, too | 13:21 |
scas | i knew it *used* to be called main, and that recent times call it public | 13:21 |
scas | i presumed "public" was the convention people were lazily getting to | 13:21 |
frickler | ya, lots of folks don't use the "internal" any more it seems | 13:22 |
scas | it seemed kind of confusing to me when i would read over the keystone docs to get a better handle on the wsgi configs until uwsgi can be supported in chef | 13:22 |
frickler | scas: so now I'll leave it to you to decide how to proceed, I'd be fine with either variant | 13:22 |
scas | in my research, chef was the one that was the holdout, but nobody is consistent. even chef has a mixture of public and main with the age of the cookbooks | 13:24 |
scas | it works, in any case, and results in what suggests to be a functioning openstack | 13:24 |
scas | whether it's named main or admin, chef doesn't seem to care, and provides openstack as the outcome | 13:26 |
scas | i have no personal preference, but was following what i thought to be some semblance of prescribed conventions. i have this weird thing where i like for things to look consistent. i had to stop myself from renaming all the api_os_compute refereces to nova_api or api_compute | 13:28 |
openstackgerrit | Samuel Cassiba proposed openstack/cookbook-openstack-identity master: Rename keystone-main service https://review.openstack.org/552275 | 13:49 |
scas | i'm leaving it as keystone-public, fixed up the comments and provided context for the change from keystone itself (thanks again) | 13:50 |
scas | i forgot the spec fix. more caffeine needed | 13:50 |
openstackgerrit | Samuel Cassiba proposed openstack/cookbook-openstack-identity master: Rename keystone-main service https://review.openstack.org/552275 | 13:51 |
scas | looking at devstack and its uwsgi configuration, keystone is split two ways: /identity (public/main/whatever) and /identity_admin | 14:08 |
scas | if devstack is any point of reference, that is the intended path | 14:09 |
frickler | scas: actually with keystone v3, the identity-admin part isn't needed/used anymore. I think I should propose a patch to drop that from devstack, so others can follow ;) | 14:41 |
scas | :D | 14:41 |
scas | that makes it even more streamlined | 14:42 |
-openstackstatus- NOTICE: zuul.o.o has been restarted to pick up latest code base and clear memory usage. Both check / gate queues were saved, be sure to check your patches and recheck when needed. | 14:50 | |
scas | i'll update the dashboard cookbook on supermarket a bit later today | 15:16 |
-openstackstatus- NOTICE: Gerrit will be temporarily unreachable as we restart it to complete the rename of some projects. | 15:48 | |
openstackgerrit | Merged openstack/cookbook-openstack-dashboard stable/pike: FWaaS Dashboard fix https://review.openstack.org/554554 | 15:58 |
scas | with the uwsgi implementation of having the keystone endpoint live at /identity, it seems like the endpoint handling lwrp code could possibly be deprecated | 17:58 |
*** os-chef-bot has quit IRC | 18:17 | |
*** os-chef-bot has joined #openstack-chef | 18:53 | |
*** openstackgerrit has quit IRC | 19:34 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!