Closed Bug 860750 Opened 9 years ago Closed 9 years ago

Autophone - Add support for multiple architectures


(Testing :: Autophone, defect)

Not set


(Not tracked)



(Reporter: bc, Assigned: bc)




(1 file)

In bug 821954 we decided to defer adding multiple architecture support for Autophone. We now have the need to support x86 phones. Mark prefers to have a single instance of Autophone deal with multiple architectures so that is what I'll work towards.
I've been considering ways to get abi information about phones. I can either do it from Autophone by 
shelling out to getprop or could have do the getprop and report it in the registration url. Putting it in SUTAgent seems reasonable. Thoughts?
Depends on: 860986
Attached patch patch v1Splinter Review
This isn't quite what I would have done if starting from scratch, which would have involved a server in autophone that received job commands which were dispatched to each worker to get their own jobs matching the criteria.

Instead, this adds armv6, x86 architectures to the pulse listener in and to so that all arm, armv6 and x86 builds which match the criteria are returned. and filter out incompatible architectures before sending the job to the worker.

I also added some logging to which sends log messages to the base autophone.log.
Attachment #739088 - Flags: review?(mcote)
Comment on attachment 739088 [details] [diff] [review]
patch v1

Review of attachment 739088 [details] [diff] [review]:

Looks good.

@@ +434,5 @@
> +            if 'armv6' not in build_url:
> +                incompatible_job = True
> +        else:
> +            if 'x86' in build_url or 'armv6' in build_url:
> +                incompatible_job = True

Can we refactor this into a function that can be used by both autophone and worker?  I worry that we'll want to update this some day and forget to make the change in both locations.
Attachment #739088 - Flags: review?(mcote) → review+
Blocks: 866195
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.