)]}'
{"manifests/keystone/authtoken.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"6b2971a90c28574ade3160ee6a35dc1078adae0e","unresolved":false,"context_lines":[{"line_number":227,"context_line":"  $token_cache_time               \u003d $::os_service_default,"},{"line_number":228,"context_line":"  # DEPRECATED PARAMETERS"},{"line_number":229,"context_line":"  $auth_uri                       \u003d undef,"},{"line_number":230,"context_line":"  $service_token_roles_required   \u003d false,"},{"line_number":231,"context_line":") {"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"  include ::neutron::deps"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"5f7c97a3_063c91df","line":230,"range":{"start_line":230,"start_character":0,"end_line":230,"end_character":42},"updated":"2018-06-29 13:53:22.000000000","message":"Please move this above L228","commit_id":"6051370fe11dcac62386326465ca412955c67965"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"eae64112976e5f8c6885d1b29b4f37ebd1a18820","unresolved":false,"context_lines":[{"line_number":275,"context_line":"    memcached_servers              \u003d\u003e $memcached_servers,"},{"line_number":276,"context_line":"    manage_memcache_package        \u003d\u003e $manage_memcache_package,"},{"line_number":277,"context_line":"    region_name                    \u003d\u003e $region_name,"},{"line_number":278,"context_line":"    token_cache_time               \u003d\u003e $token_cache_time,"},{"line_number":279,"context_line":"  }"},{"line_number":280,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"5f7c97a3_e60f1537","line":278,"updated":"2018-06-29 13:54:02.000000000","message":"Also you\u0027re not setting it here. I don\u0027t think we support this option in puppet-keystone so you\u0027ll need to patch that as well","commit_id":"6051370fe11dcac62386326465ca412955c67965"}],"manifests/server.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"a30ca682a3a1cd62f0809c4c400aa46764b87b70","unresolved":false,"context_lines":[{"line_number":394,"context_line":"    \u0027DEFAULT/network_auto_schedule\u0027:                    value \u003d\u003e $network_auto_schedule;"},{"line_number":395,"context_line":"    \u0027DEFAULT/ovs_integration_bridge\u0027:                   value \u003d\u003e $ovs_integration_bridge;"},{"line_number":396,"context_line":"    \u0027service_providers/service_provider\u0027:               value \u003d\u003e $service_providers;"},{"line_number":397,"context_line":"    \u0027keystone_authtoken/service_token_roles_required\u0027:  value \u003d\u003e $service_token_roles_required;"},{"line_number":398,"context_line":"  }"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"  if $state_path {"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"5f7c97a3_085cb3ee","line":397,"updated":"2018-06-28 18:00:25.000000000","message":"Is this specific to neutron? If not we should add this into the keystone::resource::authtoken configuration. We configure a bunch of these keystone_authtoken/* items there.  The option would be configurable via https://github.com/openstack/puppet-neutron/blob/master/manifests/keystone/authtoken.pp instead of here","commit_id":"efdadc5729b7f33bbea65b4675f1b950d816dbcc"}]}