)]}'
{"docker/services/octavia-worker.yaml":[{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"e31e36e6135899539a4b531e56449fffda7fdd82","unresolved":false,"context_lines":[{"line_number":138,"context_line":"            - python2-neutronclient"},{"line_number":139,"context_line":"            - python2-openstackclient"},{"line_number":140,"context_line":"            - openssl"},{"line_number":141,"context_line":"          when: {get_param: EnablePackageInstall}"},{"line_number":142,"context_line":"      upgrade_tasks:"},{"line_number":143,"context_line":"        - name: Stop and disable octavia_worker service"},{"line_number":144,"context_line":"          when: step|int \u003d\u003d 2"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f96bb07_bacb4551","line":141,"updated":"2018-01-23 13:45:07.000000000","message":"Doesn\u0027t the parameter \"EnablePackageInstall\" need to be defined in this template for this to work?","commit_id":"3ed6fb9599606247ee730ac99ebbc20296cc89c6"},{"author":{"_account_id":25899,"name":"Or Idgar","email":"oidgar@redhat.com","username":"oidgar"},"change_message_id":"76aa6ef8ed8f6e7bce1f43459745205508d6b62b","unresolved":false,"context_lines":[{"line_number":138,"context_line":"            - python2-neutronclient"},{"line_number":139,"context_line":"            - python2-openstackclient"},{"line_number":140,"context_line":"            - openssl"},{"line_number":141,"context_line":"          when: {get_param: EnablePackageInstall}"},{"line_number":142,"context_line":"      upgrade_tasks:"},{"line_number":143,"context_line":"        - name: Stop and disable octavia_worker service"},{"line_number":144,"context_line":"          when: step|int \u003d\u003d 2"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f96bb07_1d5d93e8","line":141,"in_reply_to":"7f96bb07_1a7039dc","updated":"2018-01-23 14:02:18.000000000","message":"I thought that it propagates anyway. my bad.","commit_id":"3ed6fb9599606247ee730ac99ebbc20296cc89c6"},{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"c5971065f9e265e6f3d6b9ff5ae48e3cfa0a9efe","unresolved":false,"context_lines":[{"line_number":138,"context_line":"            - python2-neutronclient"},{"line_number":139,"context_line":"            - python2-openstackclient"},{"line_number":140,"context_line":"            - openssl"},{"line_number":141,"context_line":"          when: {get_param: EnablePackageInstall}"},{"line_number":142,"context_line":"      upgrade_tasks:"},{"line_number":143,"context_line":"        - name: Stop and disable octavia_worker service"},{"line_number":144,"context_line":"          when: step|int \u003d\u003d 2"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f96bb07_1a7039dc","line":141,"in_reply_to":"7f96bb07_bacb4551","updated":"2018-01-23 13:47:55.000000000","message":"oh, thought it was defined there. Good catch.","commit_id":"3ed6fb9599606247ee730ac99ebbc20296cc89c6"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"b0d2beef535b7c5b5bc92e88a3e6f8c0b7902aed","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    description: Parameters specific to the role"},{"line_number":38,"context_line":"    type: json"},{"line_number":39,"context_line":"  EnablePackageInstall:"},{"line_number":40,"context_line":"    default: false"},{"line_number":41,"context_line":"    description: Set to true to enable package installation at deploy time"},{"line_number":42,"context_line":"    type: boolean"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f96bb07_61766909","line":40,"updated":"2018-01-23 17:22:10.000000000","message":"the format of this variable description doesn\u0027t quite match the others and is causing the template validation/pep8 job to fail. The others define it as \u0027false\u0027.","commit_id":"3f46a817884031510918b62637453421ba54274f"},{"author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"change_message_id":"2489a6fd540c9c23b20db2a39f00cfd6e5463f34","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    description: Parameters specific to the role"},{"line_number":38,"context_line":"    type: json"},{"line_number":39,"context_line":"  EnablePackageInstall:"},{"line_number":40,"context_line":"    default: false"},{"line_number":41,"context_line":"    description: Set to true to enable package installation at deploy time"},{"line_number":42,"context_line":"    type: boolean"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f96bb07_94e63fe8","line":40,"in_reply_to":"7f96bb07_61766909","updated":"2018-01-24 08:03:28.000000000","message":"Yeah the pep8 failure looks real","commit_id":"3f46a817884031510918b62637453421ba54274f"}]}