1304789 - Autophone - administration docs and tools

Status: RESOLVED FIXED

(Testing Graveyard :: Autophone, defect)

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: bug-1304789-v1.patch

These are actually live on autophone-{1,2,3} if you want to apply and test. Don't forget to set up your .profile as described in USAGE.md.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8793839 [details] [diff] [review]
bug-1304789-v1.patch

Review of attachment 8793839 [details] [diff] [review]:
-----------------------------------------------------------------

hey, great stuff here!

::: USAGE.md
@@ +811,5 @@
> +
> +Once the Autophone processes are shut down on each server, you can ssh
> +to the server, change to the autophone directory and pull the changes
> +from the github repository. Note that currently you must deploy the
> +code changes as the bclary user.

can you add a note why we need the bclary user for some commands?  he is cool, but others might not know that tools were installed under his user name.

@@ +854,5 @@
> +
> +You should see it in the output:
> +
> +    nexus-4-4=device
> +    nexus-4-7=device

we really should stop testing on nexus 4 :)

::: ap-assignments.py
@@ +26,5 @@
> +build_types = ['opt', 'debug']
> +
> +device_times = {
> +    'nexus-4': {
> +        'runtestsremote.py Cw':        1,

lots of magic in this file- possibly a future followup bug to document the magic better?

::: ap-pushes.py
@@ +32,5 @@
> +
> +if args.repo in 'mozilla-beta,mozilla-aurora,mozilla-release':
> +    pushes_base_url += 'releases/'
> +elif args.repo != 'mozilla-central':
> +    pushes_base_url += 'integration/'

does this work for try server?

Comment 3

[Bob Clary [:bc] (inactive)]

(In reply to Joel Maher ( :jmaher) from comment #2)
> Comment on attachment 8793839 [details] [diff] [review]
> bug-1304789-v1.patch
> 
> Review of attachment 8793839 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> hey, great stuff here!
> 
> ::: USAGE.md
> @@ +811,5 @@
> > +
> > +Once the Autophone processes are shut down on each server, you can ssh
> > +to the server, change to the autophone directory and pull the changes
> > +from the github repository. Note that currently you must deploy the
> > +code changes as the bclary user.
> 
> can you add a note why we need the bclary user for some commands?  he is
> cool, but others might not know that tools were installed under his user
> name.
> 

sure.

> @@ +854,5 @@
> > +
> > +You should see it in the output:
> > +
> > +    nexus-4-4=device
> > +    nexus-4-7=device
> 
> we really should stop testing on nexus 4 :)
> 
> ::: ap-assignments.py
> @@ +26,5 @@
> > +build_types = ['opt', 'debug']
> > +
> > +device_times = {
> > +    'nexus-4': {
> > +        'runtestsremote.py Cw':        1,
> 
> lots of magic in this file- possibly a future followup bug to document the
> magic better?
> 

Yeah. I started to re-write it but the re-write was much more complicated and I dropped it for now.

> ::: ap-pushes.py
> @@ +32,5 @@
> > +
> > +if args.repo in 'mozilla-beta,mozilla-aurora,mozilla-release':
> > +    pushes_base_url += 'releases/'
> > +elif args.repo != 'mozilla-central':
> > +    pushes_base_url += 'integration/'
> 
> does this work for try server?

No, but I'll add it. Not sure how useful it will be for Autophone users but it will be more complete.

Comment 4

[Bob Clary [:bc] (inactive)]

https://github.com/mozilla/autophone/commit/f49b0eac81c5bb23b82da9827dc93ded037e1498

deployed 2016-09-22 ~11:45

Comment 5

[Bob Clary [:bc] (inactive)]

github doesn't support markdown extras: https://github.com/mozilla/autophone/commit/5ea7b2674a9b33eb5902f55054a28e44f78c3234

Comment 6

[Bob Clary [:bc] (inactive)]

follow up: https://github.com/mozilla/autophone/commit/fc84e5bf98b9a07db6ca83066fe0992ec92b7cf1