)]}'
{"manifests/repo/debian/debian.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"74e783e88b286d7bf85462164a67ad59de4d1204","unresolved":true,"context_lines":[{"line_number":75,"context_line":"      ensure_packages([\u0027extrepo\u0027,], {\u0027ensure\u0027 \u003d\u003e \u0027present\u0027})"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"      exec { \"/usr/bin/extrepo enable openstack_${lowercase_release}\":"},{"line_number":78,"context_line":"        command     \u003d\u003e \"/usr/bin/extrepo enable openstack_${lowercase_release}\","},{"line_number":79,"context_line":"        logoutput   \u003d\u003e \u0027on_failure\u0027,"},{"line_number":80,"context_line":"        tries       \u003d\u003e 3,"},{"line_number":81,"context_line":"        try_sleep   \u003d\u003e 1,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"80d97a94_5cde97a7","line":78,"updated":"2021-05-10 15:09:15.000000000","message":"Alternatively is to just set path \u003d\u003e [\u0027/bin\u0027, \u0027/usr/bin\u0027]","commit_id":"a53486c1fff2183b31efed391c1a4f7e07fc658b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"52ace7928337dca683d9733ade3a1358ee6e47de","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        logoutput   \u003d\u003e \u0027on_failure\u0027,"},{"line_number":80,"context_line":"        tries       \u003d\u003e 3,"},{"line_number":81,"context_line":"        try_sleep   \u003d\u003e 1,"},{"line_number":82,"context_line":"        refreshonly \u003d\u003e false,"},{"line_number":83,"context_line":"        require     \u003d\u003e Package[\u0027extrepo\u0027],"},{"line_number":84,"context_line":"      }"},{"line_number":85,"context_line":"      if $package_require {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"f0f95579_9a20ce65","line":82,"range":{"start_line":82,"start_character":23,"end_line":82,"end_character":28},"updated":"2021-05-16 13:07:44.000000000","message":"Note this change makes this exec resource always executed. To ensure idempotency of this resource at puppet layer, you should define a condition by unless or onlyif to skip this resource if the repo is already enabled.","commit_id":"6f5d9f9b8e706dbe951ef5c32509eb0613fbe45c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6ee942af7b21936d149b407ebe9cdc7177daa859","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        logoutput   \u003d\u003e \u0027on_failure\u0027,"},{"line_number":80,"context_line":"        tries       \u003d\u003e 3,"},{"line_number":81,"context_line":"        try_sleep   \u003d\u003e 1,"},{"line_number":82,"context_line":"        refreshonly \u003d\u003e false,"},{"line_number":83,"context_line":"        require     \u003d\u003e Package[\u0027extrepo\u0027],"},{"line_number":84,"context_line":"      }"},{"line_number":85,"context_line":"      if $package_require {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"2f690291_ef85eef6","line":82,"range":{"start_line":82,"start_character":23,"end_line":82,"end_character":28},"in_reply_to":"37d5d9db_dd4da22e","updated":"2021-05-17 10:02:24.000000000","message":"It sounds good.","commit_id":"6f5d9f9b8e706dbe951ef5c32509eb0613fbe45c"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"8ca3249ac56eb225c68e2f32a1e841d01ef0967c","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        logoutput   \u003d\u003e \u0027on_failure\u0027,"},{"line_number":80,"context_line":"        tries       \u003d\u003e 3,"},{"line_number":81,"context_line":"        try_sleep   \u003d\u003e 1,"},{"line_number":82,"context_line":"        refreshonly \u003d\u003e false,"},{"line_number":83,"context_line":"        require     \u003d\u003e Package[\u0027extrepo\u0027],"},{"line_number":84,"context_line":"      }"},{"line_number":85,"context_line":"      if $package_require {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"37d5d9db_dd4da22e","line":82,"range":{"start_line":82,"start_character":23,"end_line":82,"end_character":28},"in_reply_to":"f0f95579_9a20ce65","updated":"2021-05-16 21:39:31.000000000","message":"I decided to use:\ncreates \u003d\u003e \"/etc/apt/sources.list.d/extrepo_openstack_${lowercase_release}.sources\"\n\nwhich should be as good as we can do. Normally, we should as well check if /etc/apt/sources.list.d/extrepo_openstack_${lowercase_release}.sources has \"Enabled: yes\", but if that\u0027s not the case, it means someone has done something by hand to do it, so we may as well ignore this, and give freedom to our users to do what they feel like.","commit_id":"6f5d9f9b8e706dbe951ef5c32509eb0613fbe45c"}]}