[MediaWiki-commits] [Gerrit] operations/puppet[production]: tools: followup to 326153

2017-08-12 Thread Rush (Code Review)
Rush has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/371633 )

Change subject: tools: followup to  326153
..


tools: followup to  326153

* py3 does not support the seek mode here
* fix config file path

Bug: T152235
Change-Id: Idc232521026def821271e371b67628dddf5213f3
---
M modules/labstore/files/logcleanup.py
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Rush: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/labstore/files/logcleanup.py 
b/modules/labstore/files/logcleanup.py
index 89d27ee..7deafc4 100644
--- a/modules/labstore/files/logcleanup.py
+++ b/modules/labstore/files/logcleanup.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python3
+#!/usr/bin/env python2
 
 from glob import glob
 from datetime import date
@@ -27,7 +27,7 @@
 
 If a '.norotate' file exists in a directory we will skip it.
 
-Defaults are stored in the config file at /etc/logcleanup.yaml
+Defaults are stored in the config file at /etc/logcleanup-config.yaml
 """
 
 
@@ -185,7 +185,7 @@
 type=str,
 help='''YAML config file, arguments specified on command line will
  override config specifed in file''',
-default='/etc/logcleanup.yaml',
+default='/etc/logcleanup-config.yaml',
 )
 
 args = argparser.parse_args()

-- 
To view, visit https://gerrit.wikimedia.org/r/371633
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idc232521026def821271e371b67628dddf5213f3
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Rush 
Gerrit-Reviewer: Rush 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits


[MediaWiki-commits] [Gerrit] operations/puppet[production]: tools: followup to 326153

2017-08-12 Thread Rush (Code Review)
Rush has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/371633 )

Change subject: tools: followup to  326153
..

tools: followup to  326153

* py3 does not support the seek mode here
* fix config file path

Bug: T152235
Change-Id: Idc232521026def821271e371b67628dddf5213f3
---
M modules/labstore/files/logcleanup.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/33/371633/1

diff --git a/modules/labstore/files/logcleanup.py 
b/modules/labstore/files/logcleanup.py
index 89d27ee..7deafc4 100644
--- a/modules/labstore/files/logcleanup.py
+++ b/modules/labstore/files/logcleanup.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python3
+#!/usr/bin/env python2
 
 from glob import glob
 from datetime import date
@@ -27,7 +27,7 @@
 
 If a '.norotate' file exists in a directory we will skip it.
 
-Defaults are stored in the config file at /etc/logcleanup.yaml
+Defaults are stored in the config file at /etc/logcleanup-config.yaml
 """
 
 
@@ -185,7 +185,7 @@
 type=str,
 help='''YAML config file, arguments specified on command line will
  override config specifed in file''',
-default='/etc/logcleanup.yaml',
+default='/etc/logcleanup-config.yaml',
 )
 
 args = argparser.parse_args()

-- 
To view, visit https://gerrit.wikimedia.org/r/371633
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc232521026def821271e371b67628dddf5213f3
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Rush 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits