[Piuparts-commits] [SCM] piuparts git repository branch, bikeshed, updated. 0.51-22-g9147fc5

[Holger Levsen]

The following commit has been merged in the bikeshed branch:
commit dbbaa888042402c40061bc612e7cd6d7c3d33b4b
Author: Holger Levsen <holger at layer-acht.org>
Date:   Fri May 17 16:34:28 2013 +0200

    TODO: update plans for master-slave packages: no debconf usage, instead documentation.

diff --git a/TODO b/TODO
index 9567000..74f2178 100644
--- a/TODO
+++ b/TODO
@@ -8,16 +8,18 @@ wishlist!
 for 0.5x:
 
 - make piuparts-master + piuparts-slave packages work out-of-the-box:
-  - enable sudoers.d/piuparts-slave in p-s.deb:
-    - there should be a high prio debconf question asking the user whether
-      to enable the slave and this sudoers config - defaulting to no.
+  - do not enable sudoers.d/piuparts-slave in p-s.deb, neither the cronjobs.
+    - this is manual work one has done once after installation. we should 
+      whats needs to be done how.
   - the shipped crontabs are not suitable from cron.d/ due to missing user
-    column - discuss this with DSA
+    column -> provide instructions how to install them as piuparts(s|m) user
+    from /u/s/doc/p-(s|m)/examples
   - verify that all scripts in $user/bin/ also work from master-slave packages
     eventually enable some conjobs in the packages
-  - slave.postinst should setup ~piupartsm/.ssh/authorized_keys using triggers
+  - slave.postinst should setup '~piupartsm/.ssh/authorized_keys' using triggers
     dpkg-trigger piuparts-master-please-install-the-slave-key
     + the current method only works if -slave is configured after -master
+    - maybe do this manually again as well?
 
 - in maintainer view: sort buggy states up again
 
@@ -53,6 +55,14 @@ for 0.5x:
 
 for 0.6x:
 
+- enable unit tests again.
+
+- if it weren't for 'slave-bin/slave_cleanup', the slave would only need
+  rights to run "sudo piuparts" but nothing else. If we can clean this up,
+  the sudoers.d should recommend sudo (lsof|kill|umount) for admins.
+
+- if there were real schroot support, piuparts could be used without sudo.
+
 - generate piuparts.1.txt automatically from piuparts.py - see this blog post
   for a nice howto:
   http://andialbrecht.wordpress.com/2009/03/17/creating-a-man-page-with-distutils-and-optparse/
@@ -84,7 +94,8 @@ for 0.6x:
 - "decorate" (strike-through) bug links generated by piuparts-analyze to
   indicate resolved state (take package version into account!)
 
-* verify that find_default_debian_mirrors does something sane
+- verify that find_default_debian_mirrors does something sane
+
 - find_default_debian_mirrors: if parts[2] contains a / (think stable/updates
   for security.d.o), you can't ignore this, it will break later...
   - with distros.conf this may no longer be needed
@@ -123,9 +134,6 @@ for 0.7x and later:
 - a redirect of http://piuparts.d.o/foo to http://p.d.o/source/f/foo.html would
   be nice
 
-- monitor: master.log grows to fast and there is no mechanism to stop it
-  - use logrotate
-
 - really support multiple architectures:
   - piuparts-report should have a list of available arch and list packages only
     available on untested archs in a new state
@@ -144,7 +152,7 @@ for 0.7x and later:
 
 - not sure if it's a sensible thing to to, but provide a way to turn off
   debugging output for piuparts.py - see
-http://docs.python.org/library/logging.html
+  http://docs.python.org/library/logging.html
 
 - commandline-switches for all programms
 

-- 
piuparts git repository