)]}'
{"deployment/ceph-ansible/ceph-rgw.yaml":[{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"30b5e7ef30ecedfa275216570951d6c70d6258c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ffb9cba7_9b40ea37","line":110,"updated":"2019-04-23 18:48:31.000000000","message":"Will this be as easy as replacing [1] with [2]?\n\n[1] https://github.com/openstack/puppet-ceph/blob/master/manifests/rgw/keystone/auth.pp\n[2] https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/service_identity.pp","commit_id":"8baf4b30abb47fec28c2419bbe085336579d4381"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"7d25776404379997538b247a69d1f11f07a601fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7faddb67_e6709376","line":110,"in_reply_to":"7faddb67_393b4938","updated":"2019-08-07 15:48:32.000000000","message":"added Kevin on the submission because he is working on the migration of out puppet stuff to ansible and it might be worth replacing this with ansible tasks instead of just moving it to puppet-keystone and embed puppet code into tht\n\nKevin suggested we look at [1] or try to get a generic role for keystone resources and try to consume it from tripleo-ansible\n\n1. https://github.com/openstack/openstack-ansible/blob/039d043bb3de52533d0179a184623f2efe45ad9c/playbooks/ceph-rgw-keystone-setup.yml#L29-L108","commit_id":"8baf4b30abb47fec28c2419bbe085336579d4381"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"f3f09bdd3ff8fd966456f2ec2fa4995213be6401","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7faddb67_393b4938","line":110,"in_reply_to":"7faddb67_595725cb","updated":"2019-08-06 15:51:43.000000000","message":"Yeh, thanks Giulio, I\u0027ve seen it tries to use on step 3 the variables defined here [1], so we need to make some changes on puppet-ceph to get rid of them.\n\n[1] https://github.com/openstack/puppet-ceph/blob/master/manifests/rgw/keystone/auth.pp","commit_id":"8baf4b30abb47fec28c2419bbe085336579d4381"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"137cd2b21062d175c0a85a683e7a29cc2ca8131c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7faddb67_fbba4cbe","line":110,"in_reply_to":"7faddb67_e6709376","updated":"2019-08-09 06:45:14.000000000","message":"I think it\u0027s a good idea and follow the pattern of consuming services from tripleo-ansible looks good to me.\nI will look at it and we\u0027ll follow-up with the next PS.","commit_id":"8baf4b30abb47fec28c2419bbe085336579d4381"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"5487247b50094947ec43db594681b36572760e18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7faddb67_595725cb","line":110,"in_reply_to":"ffb9cba7_9b40ea37","updated":"2019-08-06 15:39:45.000000000","message":"nah because it is a define not a class wo we need to write a manifest to include it\n\nnot a lot more work though, we\u0027ll get back to it!","commit_id":"8baf4b30abb47fec28c2419bbe085336579d4381"}]}