Thanks. Pushed and tested. On Fri, Sep 11, 2015 at 02:33:22PM -0600, Stephen John Smoogen wrote: > +1 > > push time :) > > On 11 September 2015 at 14:05, Adrian Reber <adrian@xxxxxxxx> wrote: > > Can I get two +1s for this fix which renames the report_mirror files > > to have a '.conf' at the end. > > > > > > Fix the s3sync report_mirror file names > > > > The s3sync has been running successfully for some time now. > > Unfortunately the last step, which runs report_mirror to update the > > status of the s3 mirrors, failed as the report_mirror configuration > > files had wrong names (missing .conf). > > > > diff --git a/roles/s3-mirror/tasks/main.yml b/roles/s3-mirror/tasks/main.yml > > index 3da4b8d..e3cf8a4 100644 > > --- a/roles/s3-mirror/tasks/main.yml > > +++ b/roles/s3-mirror/tasks/main.yml > > @@ -14,7 +14,7 @@ > > - mirrormanager2-client > > > > - name: Create report_mirror config files > > - template: src=report_mirror.conf dest=/etc/mirrormanager-client/report_mirror_{{ item }} owner=s3-mirror group=s3-mirror mode=0400 > > + template: src=report_mirror.conf dest=/etc/mirrormanager-client/report_mirror_{{ item }}.conf owner=s3-mirror group=s3-mirror mode=0400 > > with_items: > > - s3-mirror-us-east-1 > > - s3-mirror-us-west-1 > > > > > > Adrian > > > > _______________________________________________ > > infrastructure mailing list > > infrastructure@xxxxxxxxxxxxxxxxxxxxxxx > > http://lists.fedoraproject.org/postorius/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx > > > > > > -- > Stephen J Smoogen. > _______________________________________________ > infrastructure mailing list > infrastructure@xxxxxxxxxxxxxxxxxxxxxxx > http://lists.fedoraproject.org/postorius/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
Attachment:
pgp4LYrVerOkB.pgp
Description: PGP signature
_______________________________________________ infrastructure mailing list infrastructure@xxxxxxxxxxxxxxxxxxxxxxx http://lists.fedoraproject.org/postorius/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx