Bug#777664: [Pkg-salt-team] Bug#777664: Bug#777664: salt-minion: Make log file readable by adm group

2015-02-18 Thread Benjamin Drung
Am Sonntag, den 15.02.2015, 00:03 +1100 schrieb Joe Healy: Hi Ben, On Wed, Feb 11, 2015 at 9:18 PM, Benjamin Drung benjamin.dr...@profitbricks.com wrote: I'll make these changes. You are partially right - I/we do need a sponsor, but I haven't asked for one yet as I want to make

Bug#777664: [Pkg-salt-team] Bug#777664: Bug#777664: salt-minion: Make log file readable by adm group

2015-02-14 Thread Joe Healy
Hi Ben, On Wed, Feb 11, 2015 at 9:18 PM, Benjamin Drung benjamin.dr...@profitbricks.com wrote: I'll make these changes. You are partially right - I/we do need a sponsor, but I haven't asked for one yet as I want to make sure the latest addition to the packaging (salt-api) is ok

Bug#777664: [Pkg-salt-team] Bug#777664: salt-minion: Make log file readable by adm group

2015-02-12 Thread Benjamin Drung
Am Mittwoch, den 11.02.2015, 23:28 +1100 schrieb Joe Healy: On Wed, Feb 11, 2015 at 8:43 PM, Benjamin Drung benjamin.dr...@profitbricks.com wrote: A patch against your master branch is attached. debian/patches/make-log-file-group-readable.patch is a

Bug#777664: [Pkg-salt-team] Bug#777664: salt-minion: Make log file readable by adm group

2015-02-11 Thread Joe Healy
Thanks very much for this Ben, I'll make these changes. You are partially right - I/we do need a sponsor, but I haven't asked for one yet as I want to make sure the latest addition to the packaging (salt-api) is ok before I do (hence the -3...). I suspect the latest version is fine and ready for

Bug#777664: salt-minion: Make log file readable by adm group

2015-02-11 Thread Benjamin Drung
Package: salt-minion Version: 2014.1.13+ds-2 Severity: normal Tags: patch The adm group is good for viewing all the log files in /var/log and therefore should get read access to /var/log/salt/minion. That is done by creating the log file with permission 640 and setting the setgid permission bit

Bug#777664: [Pkg-salt-team] Bug#777664: salt-minion: Make log file readable by adm group

2015-02-11 Thread Benjamin Drung
Hi Joe, Am Mittwoch, den 11.02.2015, 21:06 +1100 schrieb Joe Healy: Thanks very much for this Ben, I'll make these changes. You are partially right - I/we do need a sponsor, but I haven't asked for one yet as I want to make sure the latest addition to the packaging (salt-api) is ok before

Bug#777664: [Pkg-salt-team] Bug#777664: salt-minion: Make log file readable by adm group

2015-02-11 Thread Joe Healy
On Wed, Feb 11, 2015 at 9:18 PM, Benjamin Drung benjamin.dr...@profitbricks.com wrote: PS: Are you aware of the lintian warnings 'script-not-executable' [1]. Is the deploy directory needed in the package? At least some of it is needed for salt-cloud. The bootstrap script [1] is copied to

Bug#777664: [Pkg-salt-team] Bug#777664: salt-minion: Make log file readable by adm group

2015-02-11 Thread Benjamin Drung
Am Mittwoch, den 11.02.2015, 21:46 +1100 schrieb Joe Healy: On Wed, Feb 11, 2015 at 9:18 PM, Benjamin Drung benjamin.dr...@profitbricks.com wrote: PS: Are you aware of the lintian warnings 'script-not-executable' [1]. Is the deploy directory

Bug#777664: [Pkg-salt-team] Bug#777664: salt-minion: Make log file readable by adm group

2015-02-11 Thread Joe Healy
On Wed, Feb 11, 2015 at 8:43 PM, Benjamin Drung benjamin.dr...@profitbricks.com wrote: A patch against your master branch is attached. debian/patches/make-log-file-group-readable.patch is a backport of the patch that was accepted upstream (in their develop branch). Should this also be