)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c76484d6249b0e9174913ab2e198aa2eb8205b6c","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"manifests/backend/rbd.pp attempts to write in /etc/init, when"},{"line_number":10,"context_line":"upstart has long gone. The result is that it just fails to run."},{"line_number":11,"context_line":"So we change the param ceph_init_override to /etc/default/ceph."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7666c775f4d269f214d3f853e56d0eb076834164"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3f4c43b2_3257185b","line":11,"range":{"start_line":11,"start_character":45,"end_line":11,"end_character":62},"updated":"2020-04-19 15:11:57.000000000","message":"Sorry for my last incorrect comment, but this should be /etc/defaut/cinder-volume because we need to set the env parameter for cinder-volume.","commit_id":"7fbf21d6ecc9e279c66fc308b59cdcb47c174de0"}],"manifests/backend/rbd.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"fe7173ac655bc94b8bb05e12b6cc7f124e210dfb","unresolved":false,"context_lines":[{"line_number":164,"context_line":"  }"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"  # Creates an empty file if it doesn\u0027t yet exist"},{"line_number":167,"context_line":"  ensure_resource(\u0027file\u0027, $::cinder::params::ceph_init_override, {\u0027ensure\u0027 \u003d\u003e \u0027present\u0027})"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"  file_line { \"set initscript env ${name}\":"},{"line_number":170,"context_line":"    line   \u003d\u003e $override_line,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ffb9cba7_294aed54","side":"PARENT","line":167,"updated":"2019-04-22 22:41:35.000000000","message":"This is /etc/sysconfig on rhel which is still valid.  The case block before this are actually for this contents as well.  The tests would need to get updated as well if we don\u0027t wan to configure this on Debian based systems though I\u0027m wondering if there is a new path for the CEPH_ARGS","commit_id":"ce80da23581b6385d685acd576a0274a4b717edb"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f670ee9c049e31b9c16942a982fa57c2d83aaefe","unresolved":false,"context_lines":[{"line_number":151,"context_line":""},{"line_number":152,"context_line":"  case $::osfamily {"},{"line_number":153,"context_line":"    \u0027Debian\u0027: {"},{"line_number":154,"context_line":"      $override_line    \u003d \"env CEPH_ARGS\u003d\\\"--id ${rbd_user}\\\"\""},{"line_number":155,"context_line":"      $override_match   \u003d \u0027^env CEPH_ARGS\u003d\u0027"},{"line_number":156,"context_line":"    }"},{"line_number":157,"context_line":"    \u0027RedHat\u0027: {"},{"line_number":158,"context_line":"      $override_line    \u003d \"export CEPH_ARGS\u003d\\\"--id ${rbd_user}\\\"\""}],"source_content_type":"text/x-puppet","patch_set":6,"id":"3f4c43b2_af38e59f","line":155,"range":{"start_line":154,"start_character":0,"end_line":155,"end_character":43},"updated":"2020-04-19 12:18:56.000000000","message":"This should be also cleaned up.","commit_id":"51df00093aa45da6e5982554ee829f92e1cea3d6"}]}