Closed Bug 1304789 Opened 4 years ago Closed 4 years ago

Autophone - administration docs and tools

Categories

(Testing :: Autophone, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

Attachments

(1 file)

No description provided.
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.
Attachment #8793839 - Flags: review?(jmaher)
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?
Attachment #8793839 - Flags: review?(jmaher) → review+
(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.
https://github.com/mozilla/autophone/commit/f49b0eac81c5bb23b82da9827dc93ded037e1498

deployed 2016-09-22 ~11:45
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.