Re: [PATCH] Removing branched composes

2017-07-07 Thread Patrick Uiterwijk 
On Fri, Jul 7, 2017 at 8:50 PM, Mohan Boddu  wrote:
> As per Kevin's comment, just commenting the lines.
> And, here's the new diff:

+1

Patrick
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org

Re: [PATCH] Removing branched composes

2017-07-07 Thread Kevin Fenzi 
On 07/07/2017 12:50 PM, Mohan Boddu wrote:
> As per Kevin's comment, just commenting the lines. 
> And, here's the new diff:

+1

kevin




signature.asc
Description: OpenPGP digital signature
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org

Re: [PATCH] Removing branched composes

2017-07-07 Thread Mohan Boddu 
As per Kevin's comment, just commenting the lines. 
And, here's the new diff:

diff --git a/roles/releng/files/branched b/roles/releng/files/branched
index 26fee6c..64192dd 100644
--- a/roles/releng/files/branched
+++ b/roles/releng/files/branched
@@ -1,4 +1,4 @@
 # branched compose
-MAILTO=releng-c...@lists.fedoraproject.org
-15 7 * * * root TMPDIR=`mktemp -d /tmp/branched.XX` && cd $TMPDIR && git 
clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && git checkout f26 
&& LANG=en_US.UTF-8 ./nightly.sh && sudo -u ftpsync 
/usr/local/bin/update-fullfiletimelist -l 
/pub/fedora-secondary/update-fullfiletimelist.lock -t /pub fedora 
fedora-secondary
-15 18 * * * root TMPDIR=`mktemp -d /tmp/branched-modular.XX` && cd $TMPDIR 
&& git clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && git 
checkout f26 && LANG=en_US.UTF-8 ./nightly-modular.sh
+#MAILTO=releng-c...@lists.fedoraproject.org
+#15 7 * * * root TMPDIR=`mktemp -d /tmp/branched.XX` && cd $TMPDIR && git 
clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && git checkout f26 
&& LANG=en_US.UTF-8 ./nightly.sh && sudo -u ftpsync 
/usr/local/bin/update-fullfiletimelist -l 
/pub/fedora-secondary/update-fullfiletimelist.lock -t /pub fedora 
fedora-secondary
+#15 18 * * * root TMPDIR=`mktemp -d /tmp/branched-modular.XX` && cd 
$TMPDIR && git clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && 
git checkout f26 && LANG=en_US.UTF-8 ./nightly-modular.sh
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org

Re: [PATCH] Removing branched composes

2017-07-07 Thread Kevin Fenzi 
On 07/07/2017 11:38 AM, mbo...@redhat.com wrote:
> From: Mohan Boddu 
> 
> Signed-off-by: Mohan Boddu 
> ---
>  roles/releng/files/branched | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/roles/releng/files/branched b/roles/releng/files/branched
> index 26fee6c..0cebe18 100644
> --- a/roles/releng/files/branched
> +++ b/roles/releng/files/branched
> @@ -1,4 +1 @@
>  # branched compose
> -MAILTO=releng-c...@lists.fedoraproject.org
> -15 7 * * * root TMPDIR=`mktemp -d /tmp/branched.XX` && cd $TMPDIR && git 
> clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && git checkout 
> f26 && LANG=en_US.UTF-8 ./nightly.sh && sudo -u ftpsync 
> /usr/local/bin/update-fullfiletimelist -l 
> /pub/fedora-secondary/update-fullfiletimelist.lock -t /pub fedora 
> fedora-secondary
> -15 18 * * * root TMPDIR=`mktemp -d /tmp/branched-modular.XX` && cd 
> $TMPDIR && git clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && 
> git checkout f26 && LANG=en_US.UTF-8 ./nightly-modular.sh
> 

I'd say just comment them (#) instead of removing entirely... this makes
it easier for us to re-add them when the next branched starts.

Otherwise +1, we need to stop branched composes.

kevin




signature.asc
Description: OpenPGP digital signature
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org

[PATCH] Removing branched composes

2017-07-07 Thread mboddu 
From: Mohan Boddu 

Signed-off-by: Mohan Boddu 
---
 roles/releng/files/branched | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/roles/releng/files/branched b/roles/releng/files/branched
index 26fee6c..0cebe18 100644
--- a/roles/releng/files/branched
+++ b/roles/releng/files/branched
@@ -1,4 +1 @@
 # branched compose
-MAILTO=releng-c...@lists.fedoraproject.org
-15 7 * * * root TMPDIR=`mktemp -d /tmp/branched.XX` && cd $TMPDIR && git 
clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && git checkout f26 
&& LANG=en_US.UTF-8 ./nightly.sh && sudo -u ftpsync 
/usr/local/bin/update-fullfiletimelist -l 
/pub/fedora-secondary/update-fullfiletimelist.lock -t /pub fedora 
fedora-secondary
-15 18 * * * root TMPDIR=`mktemp -d /tmp/branched-modular.XX` && cd $TMPDIR 
&& git clone https://pagure.io/pungi-fedora.git && cd pungi-fedora && git 
checkout f26 && LANG=en_US.UTF-8 ./nightly-modular.sh
-- 
1.8.3.1
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org