)]}'
{"environments/storage-environment.yaml":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"5fa45922a8089e0e931172886ed9edbb0e4c1ad0","unresolved":false,"context_lines":[{"line_number":66,"context_line":"  ## Ceph client key, e.g \u0027AQC+vYNXgDAgAhAAc8UoYt+OTz5uhV7ItLdwUw\u003d\u003d\u0027"},{"line_number":67,"context_line":"  # CephClientKey: \u0027\u0027"},{"line_number":68,"context_line":"  ## Ceph default features enabled when creating a block device image"},{"line_number":69,"context_line":"  # RbdDefaultFeatures: 3"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7a3c09a3_eddc4f14","line":69,"updated":"2017-01-16 16:26:15.000000000","message":"seems confusing to have it here, one could use this parameter for managed deployments but it isn\u0027t something people is expected to change ... not strong about it, just saying","commit_id":"eac9ff244683b5cb484d9ef1f9cbf6b12a4a0004"}],"puppet/services/ceph-base.yaml":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"e72716b49b0a63f6b3dcbf24892119c3be552690","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    default: false"},{"line_number":68,"context_line":"    description: Whether to deploy Ceph Storage (OSD) on the Controller"},{"line_number":69,"context_line":"    type: boolean"},{"line_number":70,"context_line":"  RbdDefaultFeatures:"},{"line_number":71,"context_line":"    default: \u0027\u0027"},{"line_number":72,"context_line":"    description: The default features enabled when creating a block device"},{"line_number":73,"context_line":"                 image. Only applies to format 2 images. Set to \u00271\u0027 for Jewel"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a3c09a3_170d503e","line":70,"updated":"2017-01-15 21:26:15.000000000","message":"this should go into ceph-external.yaml not ceph-base.yaml","commit_id":"e7249978bad39c203d38231b50cf93611d71c8e6"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"e72716b49b0a63f6b3dcbf24892119c3be552690","unresolved":false,"context_lines":[{"line_number":102,"context_line":"        ceph::profile::params::manage_repo: false"},{"line_number":103,"context_line":"        ceph::profile::params::authentication_type: cephx"},{"line_number":104,"context_line":"        ceph::profile::params::fsid: {get_param: CephClusterFSID}"},{"line_number":105,"context_line":"        ceph::profile::params::rbd_default_features: {get_param: RbdDefaultFeatures}"},{"line_number":106,"context_line":"        # FIXME(gfidente): we should not have to list the packages explicitly in the templates,"},{"line_number":107,"context_line":"        # but this has to stay until https://bugs.launchpad.net/puppet-ceph/+bug/1629933 is fixed"},{"line_number":108,"context_line":"        ceph::params::packages:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a3c09a3_37088c2d","line":105,"updated":"2017-01-15 21:26:15.000000000","message":"as per L70","commit_id":"e7249978bad39c203d38231b50cf93611d71c8e6"}]}