subject:"Please review \- Infra Ansible \- move slaves from home to srv"

Re: Please review - Infra Ansible - move slaves from home to srv

2017-11-21 Thread Tim Flink

On Tue, 21 Nov 2017 12:08:07 +0100
Josef Skladanka  wrote:

> The raw diff was attached to the original email, I could have
> mentioned that, I guess. /me was not able to make gmail send
> unformatted/unwrapped text.
> Sorry for the inconvenience.

Reading the attachment is a lot better. One might hope that I would
have learned how to do the email by now but apparently that's not always
true :-/

Thanks,

Tim

> j.
> 
> On Mon, Nov 20, 2017 at 6:32 PM, Tim Flink  wrote:
> 
> > On Mon, 20 Nov 2017 10:36:03 +0100
> > Josef Skladanka  wrote:
> >  
> > > I'm not sure what is the best way to ask for review for a
> > > pagure-less project, since we don't use Phabricator any more,
> > > so... let the funmail begin:  
> >
> > The wrapped diff is hard to read but it looks pretty good to me. I
> > think that the patch should be applied in parts as we reimage the
> > client-host machines but that's more of a nitpick :)
> >
> > Tim
> >  
> > > diff --git a/inventory/host_vars/qa10.qa.fedoraproject.org
> > > b/inventory/host_vars/qa10.qa.fedoraproject.org
> > > index 297f614e3..d2119dc47 100644
> > > --- a/inventory/host_vars/qa10.qa.fedoraproject.org
> > > +++ b/inventory/host_vars/qa10.qa.fedoraproject.org
> > > @@ -9,18 +9,18 @@ gw: 10.5.124.254
> > >
> > >  short_hostname: qa10.qa
> > >  slaves:
> > > -  - { user: "{{ short_hostname }}-1", home:
> > > "/home/{{ short_hostname }}-1", dir:
> > > "/home/{{ short_hostname }}-1/slave" }
> > > -  - { user: "{{ short_hostname }}-2", home:
> > > "/home/{{ short_hostname }}-2", dir:
> > > "/home/{{ short_hostname }}-2/slave" }
> > > -  - { user: "{{ short_hostname }}-3", home:
> > > "/home/{{ short_hostname }}-3", dir:
> > > "/home/{{ short_hostname }}-3/slave" }
> > > -  - { user: "{{ short_hostname }}-4", home:
> > > "/home/{{ short_hostname }}-4", dir:
> > > "/home/{{ short_hostname }}-4/slave" }
> > > -  - { user: "{{ short_hostname }}-5", home:
> > > "/home/{{ short_hostname }}-5", dir:
> > > "/home/{{ short_hostname }}-5/slave" }
> > > -  - { user: "{{ short_hostname }}-6", home:
> > > "/home/{{ short_hostname }}-6", dir:
> > > "/home/{{ short_hostname }}-6/slave" }
> > > -  - { user: "{{ short_hostname }}-7", home:
> > > "/home/{{ short_hostname }}-7", dir:
> > > "/home/{{ short_hostname }}-7/slave" }
> > > -  - { user: "{{ short_hostname }}-8", home:
> > > "/home/{{ short_hostname }}-8", dir:
> > > "/home/{{ short_hostname }}-8/slave" }
> > > -  - { user: "{{ short_hostname }}-9", home:
> > > "/home/{{ short_hostname }}-9", dir:
> > > "/home/{{ short_hostname }}-9/slave" }
> > > -  - { user: "{{ short_hostname }}-10", home:
> > > "/home/{{ short_hostname }}-10", dir:
> > > "/home/{{ short_hostname }}-10/slave" }
> > > -  - { user: "{{ short_hostname }}-11", home:
> > > "/home/{{ short_hostname }}-11", dir:
> > > "/home/{{ short_hostname }}-11/slave" }
> > > -  - { user: "{{ short_hostname }}-12", home:
> > > "/home/{{ short_hostname }}-12", dir:
> > > "/home/{{ short_hostname }}-12/slave" }
> > > -  - { user: "{{ short_hostname }}-13", home:
> > > "/home/{{ short_hostname }}-13", dir:
> > > "/home/{{ short_hostname }}-13/slave" }
> > > -  - { user: "{{ short_hostname }}-14", home:
> > > "/home/{{ short_hostname }}-14", dir:
> > > "/home/{{ short_hostname }}-14/slave" }
> > > -  - { user: "{{ short_hostname }}-15", home:
> > > "/home/{{ short_hostname }}-15", dir:
> > > "/home/{{ short_hostname }}-15/slave" }
> > > +  - { user: "{{ short_hostname }}-1", home: "/srv/buildslaves/{{
> > > short_hostname }}-1", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-1/slave" }
> > > +  - { user: "{{ short_hostname }}-2", home: "/srv/buildslaves/{{
> > > short_hostname }}-2", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-2/slave" }
> > > +  - { user: "{{ short_hostname }}-3", home: "/srv/buildslaves/{{
> > > short_hostname }}-3", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-3/slave" }
> > > +  - { user: "{{ short_hostname }}-4", home: "/srv/buildslaves/{{
> > > short_hostname }}-4", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-4/slave" }
> > > +  - { user: "{{ short_hostname }}-5", home: "/srv/buildslaves/{{
> > > short_hostname }}-5", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-5/slave" }
> > > +  - { user: "{{ short_hostname }}-6", home: "/srv/buildslaves/{{
> > > short_hostname }}-6", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-6/slave" }
> > > +  - { user: "{{ short_hostname }}-7", home: "/srv/buildslaves/{{
> > > short_hostname }}-7", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-7/slave" }
> > > +  - { user: "{{ short_hostname }}-8", home: "/srv/buildslaves/{{
> > > short_hostname }}-8", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-8/slave" }
> > > +  - { user: "{{ short_hostname }}-9", home: "/srv/buildslaves/{{
> > > short_hostname }}-9", dir:
> > > "/srv/buildslaves/{{ short_hostname }}-9/slave" }
> > > +  - { user: "{{ short_hostname }}-10", home: "/srv/buildslaves/{{
> >

Re: Please review - Infra Ansible - move slaves from home to srv

2017-11-21 Thread Josef Skladanka

The raw diff was attached to the original email, I could have mentioned
that, I guess. /me was not able to make gmail send unformatted/unwrapped
text.
Sorry for the inconvenience.

j.

On Mon, Nov 20, 2017 at 6:32 PM, Tim Flink  wrote:

> On Mon, 20 Nov 2017 10:36:03 +0100
> Josef Skladanka  wrote:
>
> > I'm not sure what is the best way to ask for review for a pagure-less
> > project, since we don't use Phabricator any more, so... let the
> > funmail begin:
>
> The wrapped diff is hard to read but it looks pretty good to me. I
> think that the patch should be applied in parts as we reimage the
> client-host machines but that's more of a nitpick :)
>
> Tim
>
> > diff --git a/inventory/host_vars/qa10.qa.fedoraproject.org
> > b/inventory/host_vars/qa10.qa.fedoraproject.org
> > index 297f614e3..d2119dc47 100644
> > --- a/inventory/host_vars/qa10.qa.fedoraproject.org
> > +++ b/inventory/host_vars/qa10.qa.fedoraproject.org
> > @@ -9,18 +9,18 @@ gw: 10.5.124.254
> >
> >  short_hostname: qa10.qa
> >  slaves:
> > -  - { user: "{{ short_hostname }}-1", home: "/home/{{ short_hostname
> > }}-1", dir: "/home/{{ short_hostname }}-1/slave" }
> > -  - { user: "{{ short_hostname }}-2", home: "/home/{{ short_hostname
> > }}-2", dir: "/home/{{ short_hostname }}-2/slave" }
> > -  - { user: "{{ short_hostname }}-3", home: "/home/{{ short_hostname
> > }}-3", dir: "/home/{{ short_hostname }}-3/slave" }
> > -  - { user: "{{ short_hostname }}-4", home: "/home/{{ short_hostname
> > }}-4", dir: "/home/{{ short_hostname }}-4/slave" }
> > -  - { user: "{{ short_hostname }}-5", home: "/home/{{ short_hostname
> > }}-5", dir: "/home/{{ short_hostname }}-5/slave" }
> > -  - { user: "{{ short_hostname }}-6", home: "/home/{{ short_hostname
> > }}-6", dir: "/home/{{ short_hostname }}-6/slave" }
> > -  - { user: "{{ short_hostname }}-7", home: "/home/{{ short_hostname
> > }}-7", dir: "/home/{{ short_hostname }}-7/slave" }
> > -  - { user: "{{ short_hostname }}-8", home: "/home/{{ short_hostname
> > }}-8", dir: "/home/{{ short_hostname }}-8/slave" }
> > -  - { user: "{{ short_hostname }}-9", home: "/home/{{ short_hostname
> > }}-9", dir: "/home/{{ short_hostname }}-9/slave" }
> > -  - { user: "{{ short_hostname }}-10", home: "/home/{{ short_hostname
> > }}-10", dir: "/home/{{ short_hostname }}-10/slave" }
> > -  - { user: "{{ short_hostname }}-11", home: "/home/{{ short_hostname
> > }}-11", dir: "/home/{{ short_hostname }}-11/slave" }
> > -  - { user: "{{ short_hostname }}-12", home: "/home/{{ short_hostname
> > }}-12", dir: "/home/{{ short_hostname }}-12/slave" }
> > -  - { user: "{{ short_hostname }}-13", home: "/home/{{ short_hostname
> > }}-13", dir: "/home/{{ short_hostname }}-13/slave" }
> > -  - { user: "{{ short_hostname }}-14", home: "/home/{{ short_hostname
> > }}-14", dir: "/home/{{ short_hostname }}-14/slave" }
> > -  - { user: "{{ short_hostname }}-15", home: "/home/{{ short_hostname
> > }}-15", dir: "/home/{{ short_hostname }}-15/slave" }
> > +  - { user: "{{ short_hostname }}-1", home: "/srv/buildslaves/{{
> > short_hostname }}-1", dir:
> > "/srv/buildslaves/{{ short_hostname }}-1/slave" }
> > +  - { user: "{{ short_hostname }}-2", home: "/srv/buildslaves/{{
> > short_hostname }}-2", dir:
> > "/srv/buildslaves/{{ short_hostname }}-2/slave" }
> > +  - { user: "{{ short_hostname }}-3", home: "/srv/buildslaves/{{
> > short_hostname }}-3", dir:
> > "/srv/buildslaves/{{ short_hostname }}-3/slave" }
> > +  - { user: "{{ short_hostname }}-4", home: "/srv/buildslaves/{{
> > short_hostname }}-4", dir:
> > "/srv/buildslaves/{{ short_hostname }}-4/slave" }
> > +  - { user: "{{ short_hostname }}-5", home: "/srv/buildslaves/{{
> > short_hostname }}-5", dir:
> > "/srv/buildslaves/{{ short_hostname }}-5/slave" }
> > +  - { user: "{{ short_hostname }}-6", home: "/srv/buildslaves/{{
> > short_hostname }}-6", dir:
> > "/srv/buildslaves/{{ short_hostname }}-6/slave" }
> > +  - { user: "{{ short_hostname }}-7", home: "/srv/buildslaves/{{
> > short_hostname }}-7", dir:
> > "/srv/buildslaves/{{ short_hostname }}-7/slave" }
> > +  - { user: "{{ short_hostname }}-8", home: "/srv/buildslaves/{{
> > short_hostname }}-8", dir:
> > "/srv/buildslaves/{{ short_hostname }}-8/slave" }
> > +  - { user: "{{ short_hostname }}-9", home: "/srv/buildslaves/{{
> > short_hostname }}-9", dir:
> > "/srv/buildslaves/{{ short_hostname }}-9/slave" }
> > +  - { user: "{{ short_hostname }}-10", home: "/srv/buildslaves/{{
> > short_hostname }}-10", dir: "/srv/buildslaves/{{ short_hostname
> > }}-10/slave" }
> > +  - { user: "{{ short_hostname }}-11", home: "/srv/buildslaves/{{
> > short_hostname }}-11", dir: "/srv/buildslaves/{{ short_hostname
> > }}-11/slave" }
> > +  - { user: "{{ short_hostname }}-12", home: "/srv/buildslaves/{{
> > short_hostname }}-12", dir: "/srv/buildslaves/{{ short_hostname
> > }}-12/slave" }
> > +  - { user: "{{ short_hostname }}-13", home: "/srv/buildslaves/{{
> > short_hostname }}-13", dir:

Re: Please review - Infra Ansible - move slaves from home to srv

2017-11-20 Thread Tim Flink

I applied the qa11 bits and everything seems to be working.

Tim

On Mon, 20 Nov 2017 10:32:54 -0700
Tim Flink  wrote:

> On Mon, 20 Nov 2017 10:36:03 +0100
> Josef Skladanka  wrote:
> 
> > I'm not sure what is the best way to ask for review for a
> > pagure-less project, since we don't use Phabricator any more, so...
> > let the funmail begin:  
> 
> The wrapped diff is hard to read but it looks pretty good to me. I
> think that the patch should be applied in parts as we reimage the
> client-host machines but that's more of a nitpick :)
> 
> Tim
> 
> > diff --git a/inventory/host_vars/qa10.qa.fedoraproject.org
> > b/inventory/host_vars/qa10.qa.fedoraproject.org
> > index 297f614e3..d2119dc47 100644
> > --- a/inventory/host_vars/qa10.qa.fedoraproject.org
> > +++ b/inventory/host_vars/qa10.qa.fedoraproject.org
> > @@ -9,18 +9,18 @@ gw: 10.5.124.254
> > 
> >  short_hostname: qa10.qa
> >  slaves:
> > -  - { user: "{{ short_hostname }}-1", home:
> > "/home/{{ short_hostname }}-1", dir:
> > "/home/{{ short_hostname }}-1/slave" }
> > -  - { user: "{{ short_hostname }}-2", home:
> > "/home/{{ short_hostname }}-2", dir:
> > "/home/{{ short_hostname }}-2/slave" }
> > -  - { user: "{{ short_hostname }}-3", home:
> > "/home/{{ short_hostname }}-3", dir:
> > "/home/{{ short_hostname }}-3/slave" }
> > -  - { user: "{{ short_hostname }}-4", home:
> > "/home/{{ short_hostname }}-4", dir:
> > "/home/{{ short_hostname }}-4/slave" }
> > -  - { user: "{{ short_hostname }}-5", home:
> > "/home/{{ short_hostname }}-5", dir:
> > "/home/{{ short_hostname }}-5/slave" }
> > -  - { user: "{{ short_hostname }}-6", home:
> > "/home/{{ short_hostname }}-6", dir:
> > "/home/{{ short_hostname }}-6/slave" }
> > -  - { user: "{{ short_hostname }}-7", home:
> > "/home/{{ short_hostname }}-7", dir:
> > "/home/{{ short_hostname }}-7/slave" }
> > -  - { user: "{{ short_hostname }}-8", home:
> > "/home/{{ short_hostname }}-8", dir:
> > "/home/{{ short_hostname }}-8/slave" }
> > -  - { user: "{{ short_hostname }}-9", home:
> > "/home/{{ short_hostname }}-9", dir:
> > "/home/{{ short_hostname }}-9/slave" }
> > -  - { user: "{{ short_hostname }}-10", home:
> > "/home/{{ short_hostname }}-10", dir:
> > "/home/{{ short_hostname }}-10/slave" }
> > -  - { user: "{{ short_hostname }}-11", home:
> > "/home/{{ short_hostname }}-11", dir:
> > "/home/{{ short_hostname }}-11/slave" }
> > -  - { user: "{{ short_hostname }}-12", home:
> > "/home/{{ short_hostname }}-12", dir:
> > "/home/{{ short_hostname }}-12/slave" }
> > -  - { user: "{{ short_hostname }}-13", home:
> > "/home/{{ short_hostname }}-13", dir:
> > "/home/{{ short_hostname }}-13/slave" }
> > -  - { user: "{{ short_hostname }}-14", home:
> > "/home/{{ short_hostname }}-14", dir:
> > "/home/{{ short_hostname }}-14/slave" }
> > -  - { user: "{{ short_hostname }}-15", home:
> > "/home/{{ short_hostname }}-15", dir:
> > "/home/{{ short_hostname }}-15/slave" }
> > +  - { user: "{{ short_hostname }}-1", home: "/srv/buildslaves/{{
> > short_hostname }}-1", dir:
> > "/srv/buildslaves/{{ short_hostname }}-1/slave" }
> > +  - { user: "{{ short_hostname }}-2", home: "/srv/buildslaves/{{
> > short_hostname }}-2", dir:
> > "/srv/buildslaves/{{ short_hostname }}-2/slave" }
> > +  - { user: "{{ short_hostname }}-3", home: "/srv/buildslaves/{{
> > short_hostname }}-3", dir:
> > "/srv/buildslaves/{{ short_hostname }}-3/slave" }
> > +  - { user: "{{ short_hostname }}-4", home: "/srv/buildslaves/{{
> > short_hostname }}-4", dir:
> > "/srv/buildslaves/{{ short_hostname }}-4/slave" }
> > +  - { user: "{{ short_hostname }}-5", home: "/srv/buildslaves/{{
> > short_hostname }}-5", dir:
> > "/srv/buildslaves/{{ short_hostname }}-5/slave" }
> > +  - { user: "{{ short_hostname }}-6", home: "/srv/buildslaves/{{
> > short_hostname }}-6", dir:
> > "/srv/buildslaves/{{ short_hostname }}-6/slave" }
> > +  - { user: "{{ short_hostname }}-7", home: "/srv/buildslaves/{{
> > short_hostname }}-7", dir:
> > "/srv/buildslaves/{{ short_hostname }}-7/slave" }
> > +  - { user: "{{ short_hostname }}-8", home: "/srv/buildslaves/{{
> > short_hostname }}-8", dir:
> > "/srv/buildslaves/{{ short_hostname }}-8/slave" }
> > +  - { user: "{{ short_hostname }}-9", home: "/srv/buildslaves/{{
> > short_hostname }}-9", dir:
> > "/srv/buildslaves/{{ short_hostname }}-9/slave" }
> > +  - { user: "{{ short_hostname }}-10", home: "/srv/buildslaves/{{
> > short_hostname }}-10", dir: "/srv/buildslaves/{{ short_hostname
> > }}-10/slave" }
> > +  - { user: "{{ short_hostname }}-11", home: "/srv/buildslaves/{{
> > short_hostname }}-11", dir: "/srv/buildslaves/{{ short_hostname
> > }}-11/slave" }
> > +  - { user: "{{ short_hostname }}-12", home: "/srv/buildslaves/{{
> > short_hostname }}-12", dir: "/srv/buildslaves/{{ short_hostname
> > }}-12/slave" }
> > +  - { user: "{{ short_hostname }}-13", home: "/srv/buildslaves/{{
> > short_hostname }}-13", dir: "/srv/buildslaves/{{ short_hostname
> > }}-13/slave" }
> > +  - {

Re: Please review - Infra Ansible - move slaves from home to srv

2017-11-20 Thread Tim Flink

On Mon, 20 Nov 2017 10:36:03 +0100
Josef Skladanka  wrote:

> I'm not sure what is the best way to ask for review for a pagure-less
> project, since we don't use Phabricator any more, so... let the
> funmail begin:

The wrapped diff is hard to read but it looks pretty good to me. I
think that the patch should be applied in parts as we reimage the
client-host machines but that's more of a nitpick :)

Tim

> diff --git a/inventory/host_vars/qa10.qa.fedoraproject.org
> b/inventory/host_vars/qa10.qa.fedoraproject.org
> index 297f614e3..d2119dc47 100644
> --- a/inventory/host_vars/qa10.qa.fedoraproject.org
> +++ b/inventory/host_vars/qa10.qa.fedoraproject.org
> @@ -9,18 +9,18 @@ gw: 10.5.124.254
> 
>  short_hostname: qa10.qa
>  slaves:
> -  - { user: "{{ short_hostname }}-1", home: "/home/{{ short_hostname
> }}-1", dir: "/home/{{ short_hostname }}-1/slave" }
> -  - { user: "{{ short_hostname }}-2", home: "/home/{{ short_hostname
> }}-2", dir: "/home/{{ short_hostname }}-2/slave" }
> -  - { user: "{{ short_hostname }}-3", home: "/home/{{ short_hostname
> }}-3", dir: "/home/{{ short_hostname }}-3/slave" }
> -  - { user: "{{ short_hostname }}-4", home: "/home/{{ short_hostname
> }}-4", dir: "/home/{{ short_hostname }}-4/slave" }
> -  - { user: "{{ short_hostname }}-5", home: "/home/{{ short_hostname
> }}-5", dir: "/home/{{ short_hostname }}-5/slave" }
> -  - { user: "{{ short_hostname }}-6", home: "/home/{{ short_hostname
> }}-6", dir: "/home/{{ short_hostname }}-6/slave" }
> -  - { user: "{{ short_hostname }}-7", home: "/home/{{ short_hostname
> }}-7", dir: "/home/{{ short_hostname }}-7/slave" }
> -  - { user: "{{ short_hostname }}-8", home: "/home/{{ short_hostname
> }}-8", dir: "/home/{{ short_hostname }}-8/slave" }
> -  - { user: "{{ short_hostname }}-9", home: "/home/{{ short_hostname
> }}-9", dir: "/home/{{ short_hostname }}-9/slave" }
> -  - { user: "{{ short_hostname }}-10", home: "/home/{{ short_hostname
> }}-10", dir: "/home/{{ short_hostname }}-10/slave" }
> -  - { user: "{{ short_hostname }}-11", home: "/home/{{ short_hostname
> }}-11", dir: "/home/{{ short_hostname }}-11/slave" }
> -  - { user: "{{ short_hostname }}-12", home: "/home/{{ short_hostname
> }}-12", dir: "/home/{{ short_hostname }}-12/slave" }
> -  - { user: "{{ short_hostname }}-13", home: "/home/{{ short_hostname
> }}-13", dir: "/home/{{ short_hostname }}-13/slave" }
> -  - { user: "{{ short_hostname }}-14", home: "/home/{{ short_hostname
> }}-14", dir: "/home/{{ short_hostname }}-14/slave" }
> -  - { user: "{{ short_hostname }}-15", home: "/home/{{ short_hostname
> }}-15", dir: "/home/{{ short_hostname }}-15/slave" }
> +  - { user: "{{ short_hostname }}-1", home: "/srv/buildslaves/{{
> short_hostname }}-1", dir:
> "/srv/buildslaves/{{ short_hostname }}-1/slave" }
> +  - { user: "{{ short_hostname }}-2", home: "/srv/buildslaves/{{
> short_hostname }}-2", dir:
> "/srv/buildslaves/{{ short_hostname }}-2/slave" }
> +  - { user: "{{ short_hostname }}-3", home: "/srv/buildslaves/{{
> short_hostname }}-3", dir:
> "/srv/buildslaves/{{ short_hostname }}-3/slave" }
> +  - { user: "{{ short_hostname }}-4", home: "/srv/buildslaves/{{
> short_hostname }}-4", dir:
> "/srv/buildslaves/{{ short_hostname }}-4/slave" }
> +  - { user: "{{ short_hostname }}-5", home: "/srv/buildslaves/{{
> short_hostname }}-5", dir:
> "/srv/buildslaves/{{ short_hostname }}-5/slave" }
> +  - { user: "{{ short_hostname }}-6", home: "/srv/buildslaves/{{
> short_hostname }}-6", dir:
> "/srv/buildslaves/{{ short_hostname }}-6/slave" }
> +  - { user: "{{ short_hostname }}-7", home: "/srv/buildslaves/{{
> short_hostname }}-7", dir:
> "/srv/buildslaves/{{ short_hostname }}-7/slave" }
> +  - { user: "{{ short_hostname }}-8", home: "/srv/buildslaves/{{
> short_hostname }}-8", dir:
> "/srv/buildslaves/{{ short_hostname }}-8/slave" }
> +  - { user: "{{ short_hostname }}-9", home: "/srv/buildslaves/{{
> short_hostname }}-9", dir:
> "/srv/buildslaves/{{ short_hostname }}-9/slave" }
> +  - { user: "{{ short_hostname }}-10", home: "/srv/buildslaves/{{
> short_hostname }}-10", dir: "/srv/buildslaves/{{ short_hostname
> }}-10/slave" }
> +  - { user: "{{ short_hostname }}-11", home: "/srv/buildslaves/{{
> short_hostname }}-11", dir: "/srv/buildslaves/{{ short_hostname
> }}-11/slave" }
> +  - { user: "{{ short_hostname }}-12", home: "/srv/buildslaves/{{
> short_hostname }}-12", dir: "/srv/buildslaves/{{ short_hostname
> }}-12/slave" }
> +  - { user: "{{ short_hostname }}-13", home: "/srv/buildslaves/{{
> short_hostname }}-13", dir: "/srv/buildslaves/{{ short_hostname
> }}-13/slave" }
> +  - { user: "{{ short_hostname }}-14", home: "/srv/buildslaves/{{
> short_hostname }}-14", dir: "/srv/buildslaves/{{ short_hostname
> }}-14/slave" }
> +  - { user: "{{ short_hostname }}-15", home: "/srv/buildslaves/{{
> short_hostname }}-15", dir: "/srv/buildslaves/{{ short_hostname
> }}-15/slave" }
> diff --git a/inventory/host_vars/qa11.qa.fedoraproject.org
>

Please review - Infra Ansible - move slaves from home to srv

2017-11-20 Thread Josef Skladanka

I'm not sure what is the best way to ask for review for a pagure-less
project, since we don't use Phabricator any more, so... let the funmail
begin:


diff --git a/inventory/host_vars/qa10.qa.fedoraproject.org
b/inventory/host_vars/qa10.qa.fedoraproject.org
index 297f614e3..d2119dc47 100644
--- a/inventory/host_vars/qa10.qa.fedoraproject.org
+++ b/inventory/host_vars/qa10.qa.fedoraproject.org
@@ -9,18 +9,18 @@ gw: 10.5.124.254

 short_hostname: qa10.qa
 slaves:
-  - { user: "{{ short_hostname }}-1", home: "/home/{{ short_hostname
}}-1", dir: "/home/{{ short_hostname }}-1/slave" }
-  - { user: "{{ short_hostname }}-2", home: "/home/{{ short_hostname
}}-2", dir: "/home/{{ short_hostname }}-2/slave" }
-  - { user: "{{ short_hostname }}-3", home: "/home/{{ short_hostname
}}-3", dir: "/home/{{ short_hostname }}-3/slave" }
-  - { user: "{{ short_hostname }}-4", home: "/home/{{ short_hostname
}}-4", dir: "/home/{{ short_hostname }}-4/slave" }
-  - { user: "{{ short_hostname }}-5", home: "/home/{{ short_hostname
}}-5", dir: "/home/{{ short_hostname }}-5/slave" }
-  - { user: "{{ short_hostname }}-6", home: "/home/{{ short_hostname
}}-6", dir: "/home/{{ short_hostname }}-6/slave" }
-  - { user: "{{ short_hostname }}-7", home: "/home/{{ short_hostname
}}-7", dir: "/home/{{ short_hostname }}-7/slave" }
-  - { user: "{{ short_hostname }}-8", home: "/home/{{ short_hostname
}}-8", dir: "/home/{{ short_hostname }}-8/slave" }
-  - { user: "{{ short_hostname }}-9", home: "/home/{{ short_hostname
}}-9", dir: "/home/{{ short_hostname }}-9/slave" }
-  - { user: "{{ short_hostname }}-10", home: "/home/{{ short_hostname
}}-10", dir: "/home/{{ short_hostname }}-10/slave" }
-  - { user: "{{ short_hostname }}-11", home: "/home/{{ short_hostname
}}-11", dir: "/home/{{ short_hostname }}-11/slave" }
-  - { user: "{{ short_hostname }}-12", home: "/home/{{ short_hostname
}}-12", dir: "/home/{{ short_hostname }}-12/slave" }
-  - { user: "{{ short_hostname }}-13", home: "/home/{{ short_hostname
}}-13", dir: "/home/{{ short_hostname }}-13/slave" }
-  - { user: "{{ short_hostname }}-14", home: "/home/{{ short_hostname
}}-14", dir: "/home/{{ short_hostname }}-14/slave" }
-  - { user: "{{ short_hostname }}-15", home: "/home/{{ short_hostname
}}-15", dir: "/home/{{ short_hostname }}-15/slave" }
+  - { user: "{{ short_hostname }}-1", home: "/srv/buildslaves/{{
short_hostname }}-1", dir: "/srv/buildslaves/{{ short_hostname }}-1/slave" }
+  - { user: "{{ short_hostname }}-2", home: "/srv/buildslaves/{{
short_hostname }}-2", dir: "/srv/buildslaves/{{ short_hostname }}-2/slave" }
+  - { user: "{{ short_hostname }}-3", home: "/srv/buildslaves/{{
short_hostname }}-3", dir: "/srv/buildslaves/{{ short_hostname }}-3/slave" }
+  - { user: "{{ short_hostname }}-4", home: "/srv/buildslaves/{{
short_hostname }}-4", dir: "/srv/buildslaves/{{ short_hostname }}-4/slave" }
+  - { user: "{{ short_hostname }}-5", home: "/srv/buildslaves/{{
short_hostname }}-5", dir: "/srv/buildslaves/{{ short_hostname }}-5/slave" }
+  - { user: "{{ short_hostname }}-6", home: "/srv/buildslaves/{{
short_hostname }}-6", dir: "/srv/buildslaves/{{ short_hostname }}-6/slave" }
+  - { user: "{{ short_hostname }}-7", home: "/srv/buildslaves/{{
short_hostname }}-7", dir: "/srv/buildslaves/{{ short_hostname }}-7/slave" }
+  - { user: "{{ short_hostname }}-8", home: "/srv/buildslaves/{{
short_hostname }}-8", dir: "/srv/buildslaves/{{ short_hostname }}-8/slave" }
+  - { user: "{{ short_hostname }}-9", home: "/srv/buildslaves/{{
short_hostname }}-9", dir: "/srv/buildslaves/{{ short_hostname }}-9/slave" }
+  - { user: "{{ short_hostname }}-10", home: "/srv/buildslaves/{{
short_hostname }}-10", dir: "/srv/buildslaves/{{ short_hostname
}}-10/slave" }
+  - { user: "{{ short_hostname }}-11", home: "/srv/buildslaves/{{
short_hostname }}-11", dir: "/srv/buildslaves/{{ short_hostname
}}-11/slave" }
+  - { user: "{{ short_hostname }}-12", home: "/srv/buildslaves/{{
short_hostname }}-12", dir: "/srv/buildslaves/{{ short_hostname
}}-12/slave" }
+  - { user: "{{ short_hostname }}-13", home: "/srv/buildslaves/{{
short_hostname }}-13", dir: "/srv/buildslaves/{{ short_hostname
}}-13/slave" }
+  - { user: "{{ short_hostname }}-14", home: "/srv/buildslaves/{{
short_hostname }}-14", dir: "/srv/buildslaves/{{ short_hostname
}}-14/slave" }
+  - { user: "{{ short_hostname }}-15", home: "/srv/buildslaves/{{
short_hostname }}-15", dir: "/srv/buildslaves/{{ short_hostname
}}-15/slave" }
diff --git a/inventory/host_vars/qa11.qa.fedoraproject.org
b/inventory/host_vars/qa11.qa.fedoraproject.org
index de99d2ba1..47c5b702d 100644
--- a/inventory/host_vars/qa11.qa.fedoraproject.org
+++ b/inventory/host_vars/qa11.qa.fedoraproject.org
@@ -9,18 +9,18 @@ gw: 10.5.124.254

 short_hostname: qa11
 slaves:
-  - { user: "{{ short_hostname }}-1", home: "/home/{{ short_hostname
}}-1", dir: "/home/{{ short_hostname }}-1/slave" }
-  - { user: "{{ short_hostname }}-2", home: "/home/{{ short_hostname
}}-2", dir: "/home/{{

Re: Please review - Infra Ansible - move slaves from home to srv

Re: Please review - Infra Ansible - move slaves from home to srv

Re: Please review - Infra Ansible - move slaves from home to srv

Re: Please review - Infra Ansible - move slaves from home to srv

Please review - Infra Ansible - move slaves from home to srv

5 matches

Site Navigation

Mail list logo

Footer information