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 <adr...@lisas.de> 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@lists.fedoraproject.org > > http://lists.fedoraproject.org/postorius/infrastructure@lists.fedoraproject.org > > > > > > -- > Stephen J Smoogen. > _______________________________________________ > infrastructure mailing list > infrastructure@lists.fedoraproject.org > http://lists.fedoraproject.org/postorius/infrastructure@lists.fedoraproject.org
pgp_fZM676Ikc.pgp
Description: PGP signature
_______________________________________________ infrastructure mailing list infrastructure@lists.fedoraproject.org http://lists.fedoraproject.org/postorius/infrastructure@lists.fedoraproject.org