diff options
Diffstat (limited to 'zarb-ml/mageia-sysadm/2010-November/000323.html')
| -rw-r--r-- | zarb-ml/mageia-sysadm/2010-November/000323.html | 121 |
1 files changed, 121 insertions, 0 deletions
diff --git a/zarb-ml/mageia-sysadm/2010-November/000323.html b/zarb-ml/mageia-sysadm/2010-November/000323.html new file mode 100644 index 000000000..f5e7062cd --- /dev/null +++ b/zarb-ml/mageia-sysadm/2010-November/000323.html @@ -0,0 +1,121 @@ +<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN"> +<HTML> + <HEAD> + <TITLE> [Mageia-sysadm] [186] Add review board to isntall + </TITLE> + <LINK REL="Index" HREF="index.html" > + <LINK REL="made" HREF="mailto:mageia-sysadm%40mageia.org?Subject=Re%3A%20%5BMageia-sysadm%5D%20%5B186%5D%20Add%20review%20board%20to%20isntall&In-Reply-To=%3C20101107142906.GD2381%40sisay.ephaone.org%3E"> + <META NAME="robots" CONTENT="index,nofollow"> + <META http-equiv="Content-Type" content="text/html; charset=us-ascii"> + <LINK REL="Previous" HREF="000315.html"> + <LINK REL="Next" HREF="000309.html"> + </HEAD> + <BODY BGCOLOR="#ffffff"> + <H1>[Mageia-sysadm] [186] Add review board to isntall</H1> + <B>Michael scherer</B> + <A HREF="mailto:mageia-sysadm%40mageia.org?Subject=Re%3A%20%5BMageia-sysadm%5D%20%5B186%5D%20Add%20review%20board%20to%20isntall&In-Reply-To=%3C20101107142906.GD2381%40sisay.ephaone.org%3E" + TITLE="[Mageia-sysadm] [186] Add review board to isntall">misc at zarb.org + </A><BR> + <I>Sun Nov 7 15:29:06 CET 2010</I> + <P><UL> + <LI>Previous message: <A HREF="000315.html">[Mageia-sysadm] [186] Add review board to isntall +</A></li> + <LI>Next message: <A HREF="000309.html">[Mageia-sysadm] [187] - add bugs access to bugs database +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#323">[ date ]</a> + <a href="thread.html#323">[ thread ]</a> + <a href="subject.html#323">[ subject ]</a> + <a href="author.html#323">[ author ]</a> + </LI> + </UL> + <HR> +<!--beginarticle--> +<PRE>On Sun, Nov 07, 2010 at 09:40:19AM +0100, Dexter Morgan wrote: +><i> On Sun, Nov 7, 2010 at 3:23 AM, Michael scherer <<A HREF="https://www.mageia.org/mailman/listinfo/mageia-sysadm">misc at zarb.org</A>> wrote: +</I>><i> > On Sat, Nov 06, 2010 at 09:31:35PM +0100, <A HREF="https://www.mageia.org/mailman/listinfo/mageia-sysadm">root at mageia.org</A> wrote: +</I>><i> >> Revision: 186 +</I>><i> >> Author:   dmorgan +</I>><i> >> Date:     2010-11-06 21:31:35 +0100 (Sat, 06 Nov 2010) +</I>><i> >> Log Message: +</I>><i> >> ----------- +</I>><i> >> Add review board to isntall +</I>><i> > +</I>><i> > who asked for it ? +</I>><i> > i do not think this was discussed, and given the current type of +</I>><i> > contribution we have, I am not sure it will be suiable. At least, +</I>><i> > I think this should first be discussed with packager, no ( and also +</I>><i> > with sysadmin if we find it suitable, maintainable, etc ) +</I>><i> +</I>><i> i addedd it to not forgot, but reviewboard is something we _must_ have +</I>><i> to allow more contribution to our own projects, tools, code, ... +</I>><i> This is much better to use this than sending a patch on bugzilla. +</I> +A tool do not replace a process. If there is no one to do code review, +the tool will not be used. We cannot dictate "now, we do code review" without +discussing with others involved persons first ( ie, coders, maybe mackagers ). +Maybe they would prefer a different way of doing it ( like email, like doing it on irc, +like doing it like postgresql ). If we want to empower the community as a whole, +the community must do its own choices. + +I do think code review is a good idea. And I do like reviewboard, don't get me wrong. +But before installing it and using it, others peoples must see how it can be useful, +how it work, etc. + +This also mean to agree on procedure, ie, do we let people propose patch on bugzilla ? +Who can review ? Who has the final word to say ? + +Ie, the tool must be derivated by the workflow, not the reverse. So first a workflow +must be found, IMHO. + +And all of this requires to have the team in place, and to have someone involved +into trying to convince people of using best practices. Even if we will +likely be right in the end, what is needed is that procedure like this are feel +welcomed by contributers, rather than imposed. This is indeed a more lengthy process, yes +and this is frustrating, yes. And this also may end differently that what we would +want, yes. + +><i> +</I>><i> what do you mean suitable ? +</I>><i> Projects like kde use it since a long time so they have already tested +</I>><i> and we would know if there were any maintainance issue +</I> +Well, transifex is also used since a long time by other projects ( Fedora for +example ), and no one raised the maintenace issues here afaik, +despites them being know ( at least in the pre 1.0 version ). + +We also need to check if it support ldap and others things : +- users in ldap, +- dynamic acls based on ldap, ( ie, let all people in the sysadm team review patch for sysadm ) +- a non webbased config (for puppet integration, or we would lose auditing and the advantage of vcs ). + +1 seems ok. 2 and 3, rather not :/ + +( and this make me think this may not be the case for transifex etheir, and that we +should have checked ). +-- +Michael scherer +</PRE> + + + +<!--endarticle--> + <HR> + <P><UL> + <!--threads--> + <LI>Previous message: <A HREF="000315.html">[Mageia-sysadm] [186] Add review board to isntall +</A></li> + <LI>Next message: <A HREF="000309.html">[Mageia-sysadm] [187] - add bugs access to bugs database +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#323">[ date ]</a> + <a href="thread.html#323">[ thread ]</a> + <a href="subject.html#323">[ subject ]</a> + <a href="author.html#323">[ author ]</a> + </LI> + </UL> + +<hr> +<a href="https://www.mageia.org/mailman/listinfo/mageia-sysadm">More information about the Mageia-sysadm +mailing list</a><br> +</body></html> |