)]}'
{"manifests/repo/redhat/redhat.pp":[{"author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"change_message_id":"57db96e235c91c4688df987b3e7e84c0410fd0ca","unresolved":false,"context_lines":[{"line_number":148,"context_line":"  }"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"  exec { \u0027yum_refresh\u0027:"},{"line_number":151,"context_line":"    command     \u003d\u003e \u0027/usr/bin/yum clean all \u0026\u0026 /usr/bin/yum install -y applydeltarpm\u0027,"},{"line_number":152,"context_line":"    refreshonly \u003d\u003e true,"},{"line_number":153,"context_line":"  } -\u003e Package \u003c||\u003e"},{"line_number":154,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba3cc151_3318fb74","line":151,"updated":"2015-07-02 00:09:59.000000000","message":"the package name is \"deltarpm\"","commit_id":"c536b06d1941d8bf6e70c90dc32af3b3d64577e6"},{"author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"change_message_id":"962da4980002556a20c54b6acb12b14af86d7627","unresolved":false,"context_lines":[{"line_number":148,"context_line":"  }"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"  exec { \u0027yum_refresh\u0027:"},{"line_number":151,"context_line":"    command     \u003d\u003e \u0027/usr/bin/yum clean all \u0026\u0026 /usr/bin/yum install -y deltarpm\u0027,"},{"line_number":152,"context_line":"    refreshonly \u003d\u003e true,"},{"line_number":153,"context_line":"  } -\u003e Package \u003c||\u003e"},{"line_number":154,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ba3cc151_33bf9bab","line":151,"updated":"2015-07-02 00:34:40.000000000","message":"I believe the yum install shouldn\u0027t be in the yum_refresh resource, what about being called before hand? . Otherwise it\u0027s going to be triggered for every refresh after a new repo is added, not a big deal but not ideal either.\n\nThat said, I\u0027m not sure exactly where that should go since I\u0027m not familiar with the infra stuff, but I would say wherever the RH box is provisioned? \nDo you know where that would be?","commit_id":"1ec627e2647e10700e6907f210960f5d4c7b8449"},{"author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"change_message_id":"becda42970f5666825823203cbb496a689bdc784","unresolved":false,"context_lines":[{"line_number":148,"context_line":"  }"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"  exec { \u0027yum_refresh\u0027:"},{"line_number":151,"context_line":"    command     \u003d\u003e \u0027/usr/bin/yum install -y deltarpm \u0026\u0026 /usr/bin/yum -y update \u0026\u0026 /usr/bin/yum clean all\u0027,"},{"line_number":152,"context_line":"    refreshonly \u003d\u003e true,"},{"line_number":153,"context_line":"  } -\u003e Package \u003c||\u003e"},{"line_number":154,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":5,"id":"ba3cc151_16e365a5","line":151,"updated":"2015-07-02 01:06:10.000000000","message":"\"yum clean expire-cache\" instead of \"yum clean all\" could be a time saver: \n\n\"Eliminate the local data saying when the metadata and mirrorlists were downloaded for each repo. This means  yum  will revalidate the cache for each repo. next time it is used. However if the cache is still valid, nothing significant was deleted.\"","commit_id":"0ac0dbdda2e18a00fc062b01ca927f9c0fd5837f"}]}