)]}'
{"docker/sensu/sensu-server/Dockerfile.j2":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"2070c6747f58c046c6ca812050bd816b8ec53934","unresolved":false,"context_lines":[{"line_number":26,"context_line":"# NOTE(mgoddard): mailer 4+ requires Ruby 2.3+, which is not available on"},{"line_number":27,"context_line":"# CentOS/RHEL7. See https://bugs.launchpad.net/kolla/+bug/1820594."},{"line_number":28,"context_line":"{% set sensu_plugins \u003d ["},{"line_number":29,"context_line":"    \u0027mailer:3.0.0\u0027,"},{"line_number":30,"context_line":"    \u0027slack\u0027"},{"line_number":31,"context_line":"] %}"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5fc1f717_ac97f2eb","line":29,"updated":"2019-03-18 12:48:05.000000000","message":"Please use the pessimistic version constraint operator (~\u003e) and pin it to the latest major version supporting the ruby version available in Centos/RHEL7.\n\nThis should read:\n\n  \u0027mailer\u0027:\"~\u003e3\",\n\nSee how I did in sensu-client to deal with the same sort of issue, https://review.openstack.org/#/c/571175/4/docker/sensu/sensu-client/Dockerfile.j2","commit_id":"1b85a51acad40413c331ad65a9980d21d079a633"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"035bef478e88eb0d2f91fa01b9da3983a9c5169b","unresolved":false,"context_lines":[{"line_number":26,"context_line":"# NOTE(mgoddard): mailer 4+ requires Ruby 2.3+, which is not available on"},{"line_number":27,"context_line":"# CentOS/RHEL7. See https://bugs.launchpad.net/kolla/+bug/1820594."},{"line_number":28,"context_line":"{% set sensu_plugins \u003d ["},{"line_number":29,"context_line":"    \u0027mailer:3.0.0\u0027,"},{"line_number":30,"context_line":"    \u0027slack\u0027"},{"line_number":31,"context_line":"] %}"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5fc1f717_4caf6e29","line":29,"in_reply_to":"5fc1f717_ac97f2eb","updated":"2019-03-18 12:51:31.000000000","message":"Oops, placed the quote at the wrong place:\n\n  \u0027mailer:\"~\u003e3\"\u0027,","commit_id":"1b85a51acad40413c331ad65a9980d21d079a633"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"2070c6747f58c046c6ca812050bd816b8ec53934","unresolved":false,"context_lines":[{"line_number":27,"context_line":"# CentOS/RHEL7. See https://bugs.launchpad.net/kolla/+bug/1820594."},{"line_number":28,"context_line":"{% set sensu_plugins \u003d ["},{"line_number":29,"context_line":"    \u0027mailer:3.0.0\u0027,"},{"line_number":30,"context_line":"    \u0027slack\u0027"},{"line_number":31,"context_line":"] %}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"RUN sensu-install --plugins {{ sensu_plugins | customizable(\u0027plugins\u0027) | join (\u0027,\u0027) }}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"5fc1f717_6c9d8a08","line":30,"range":{"start_line":30,"start_character":5,"end_line":30,"end_character":10},"updated":"2019-03-18 12:48:05.000000000","message":"Same, we should pin this one too.","commit_id":"1b85a51acad40413c331ad65a9980d21d079a633"}]}
