Merge lp:~milo/linaro-ci-dashboard/android-build-app-models into lp:linaro-ci-dashboard

[Milo Casagrande]

Proposing a first branch to be merged with the master dashboard project.

In this branch:
- a new app was added: android_build
- the data model for an android specific build has been defined: there are still some TODOs in the code, that are up for comment
- added preliminary URLs for the new application, that are based on the already present work
- enabled the new application in settings.py
- added migration data for the data structure
- fixed a couple of errors on Makefile and README

We wanted to get a first review and check that we are on the right track with it.

[Georgy Redkozubov (gesha)]

AndroidLoop model misses MAKE_TARGETS and EXTERNAL_TARBALL fields. They are present in the spec on wiki and used in current build scripts.
Other thoughts are about:
 "Besides being interpreted by build scripts as described below, all configuration variables are also exported to the environment, so they may affect build procedure for particular target (for good or for bad, please make sure you don't select variable names which clash with well-known variables used by make or autotools)."
My guess is that we need field to hold all other variables that are not predefined.

review: Needs Fixing

[Milo Casagrande (milo)]

On Wed, Aug 1, 2012 at 9:40 AM, Georgy Redkozubov
<email address hidden> wrote:
> Review: Needs Fixing
>
> AndroidLoop model misses MAKE_TARGETS and EXTERNAL_TARBALL fields. They are present in the spec on wiki and used in current build scripts.
> Other thoughts are about:
> "Besides being interpreted by build scripts as described below, all configuration variables are also exported to the environment, so they may affect build procedure for particular target (for good or for bad, please make sure you don't select variable names which clash with well-known variables used by make or autotools)."
> My guess is that we need field to hold all other variables that are not predefined.

Yes, indeed.
I also thought it was possible to create primay keys based on multiple
columns with Django, but it is not possible, so I will have to
simplify the models a little bit (basically removing some of the
foreign keys we are using now).

--
Milo Casagrande
Infrastructure Engineer
Linaro.org <www.linaro.org> │ Open source software for ARM SoCs

[Milo Casagrande (milo)]

I pushed a new version of the data model.
I simplified it a little bit: now there are only AndroidLoop and Repository as real models. All the necessary info are inside AndroidLoop.

I added a new field in AndroidLoop, user_defined_values, that is set as a TextField in Django, so we should get a text area for users to fill in with their own values. I also added the missing bits.

For the repository model, now it is a ManytoMany relation with a unique value composed of repository+branch, so that we do not have duplicates in that table. I aslo check for uniqueness of those values before saving, otherwise we will have a ValueError.

[lp:~milo/linaro-ci-dashboard/android-build-app-models]

18. By Milo Casagrande on 2012-08-13

Merged lp:~milo/linaro-ci-dashboard/better-html.

[Stevan Radaković (stevanr)]

Hi Milo, thanks for working on this !!1 :)

I have a couple of general comments for AndroidLoop class.
I really don't think we should make a separate application for it right now, and even if we have some cool reasons to do so, then the code should be re-factored so that the IntegrationLoop has a separate application for itself as well. Then what about the base Loop class? (I think you get the point)
The other thing is that all the WhateverLoop classes should extend from the Loop class. If they have something in common, then the method (or property) in question should be moved to the base Loop class as well.

Also you mention the Repository model in the comments above, but I can't seem to locate it in the diff. Can you elaborate please? :)

On to the code implementation suggestions:

...
+class AndroidLoop(IntegrationLoop):
+ """Defines the base class for an Android integration loop."""
+ class Meta:
+ app_label = 'android_build'
+
+ # The default file name for the manifest file.
+ DEFAULT_MANIFEST_FILE = 'default.xml'
+ DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
+ DEFAULT_BRANCH = 'master'
+ DEFAULT_BINUTILS = '2.20.1'
+ DEFAULT_GCC = '4.4.0'
+ DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
+ DEFAULT_BRANCH = 'master'

We have double constants? And furthermore, isn't the config file better place for these, even if android specific?

+from django.db import models
+from django.conf import settings
+
+class JenkinsConfig(models.Model):
+ class Meta:
+ app_label = 'jenkinsserver'
+
+ config_template = models.CharField(max_length=255)
+ config = models.CharField(max_length=255, blank=True)
+
+ def save(self, *args, **kwargs):
+ super(JenkinsConfig, self).save(*args, **kwargs)

I support the implementation of this model and providing the (super)user possibility to control jenkins configuration from our app. I don't see tht need of the config-template tho, except for development purposes, and in that case we can put it in the settings file as a hardcoded path (since we still don't have different setups for production and developtment databases).

I'd like to keep the review open, I'll have some few more comments for the AndroidLoop fields later today or tomorrow.

review: Needs Fixing (code)

[Milo Casagrande (milo)]

Hello Stevan!

Thanks for taking a look!

On Mon, Aug 13, 2012 at 5:31 PM, Stevan Radaković
<email address hidden> wrote:
>
> I have a couple of general comments for AndroidLoop class.
> I really don't think we should make a separate application for it right now, and even if we have some cool reasons to do so, then the code should be re-factored so that the IntegrationLoop has a separate application for itself as well. Then what about the base Loop class? (I think you get the point)

Yeah... this was one of our "problems" at the beginning of the work:
deciding if we had to create a separate app, or work on top of the
existing one.
We chose the former.

My main concern was for maintanability (smaller code base for each
Loop). Even after a brief talk with Danilo we decided to keep the
separate app for the moment. if we decide to use the same app for
everything, it shouldn't be that hard to move everything inside.

> The other thing is that all the WhateverLoop classes should extend from the Loop class. If they have something in common, then the method (or property) in question should be moved to the base Loop class as well.

Good point, I actually didn't remember if we had to use Loop or IntegrationLoop.

> Also you mention the Repository model in the comments above, but I can't seem to locate it in the diff. Can you elaborate please? :)

That was in the initial model, but I later removed it and stored
everything inside AndroidLoop.

> On to the code implementation suggestions:
>
> ...
> +class AndroidLoop(IntegrationLoop):
> + """Defines the base class for an Android integration loop."""
> + class Meta:
> + app_label = 'android_build'
> +
> + # The default file name for the manifest file.
> + DEFAULT_MANIFEST_FILE = 'default.xml'
> + DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
> + DEFAULT_BRANCH = 'master'
> + DEFAULT_BINUTILS = '2.20.1'
> + DEFAULT_GCC = '4.4.0'
> + DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
> + DEFAULT_BRANCH = 'master'
>
> We have double constants? And furthermore, isn't the config file better place for these, even if android specific?

Hmmm... wrong merge here, I didn't even see it.
For storing them in the config file, I have no idea. In this case you
intend config.xml or some Django config?
I honestly do not know which is the best way to store default values
for database entries in Django...

> +from django.db import models
> +from django.conf import settings
> +
> +class JenkinsConfig(models.Model):
> + class Meta:
> + app_label = 'jenkinsserver'
> +
> + config_template = models.CharField(max_length=255)
> + config = models.CharField(max_length=255, blank=True)
> +
> + def save(self, *args, **kwargs):
> + super(JenkinsConfig, self).save(*args, **kwargs)
>
> I support the implementation of this model and providing the (super)user possibility to control jenkins configuration from our app. I don't see tht need of the config-template tho, except for development purposes, and in that case we can put it in the settings file as a hardcoded path (since we still don't have different setups for production and dev...

[lp:~milo/linaro-ci-dashboard/android-build-app-models]

19. By Milo Casagrande on 2012-08-13

Fixed problem from review.

20. By Milo Casagrande on 2012-08-13

Removed null.

21. By Milo Casagrande on 2012-08-14

Fixed typo, update migrations.

22. By Milo Casagrande on 2012-08-14

Fixed wordings of fields.

23. By Milo Casagrande on 2012-08-14

Added new fields.

24. By Milo Casagrande on 2012-08-14

Updated migrations.

25. By Milo Casagrande on 2012-08-14

Added better name to fields.

[Stevan Radaković (stevanr)]

On 08/13/2012 06:04 PM, Milo Casagrande wrote:
> Hello Stevan!
>
> Thanks for taking a look!
>
> On Mon, Aug 13, 2012 at 5:31 PM, Stevan Radaković
> <email address hidden> wrote:
>> I have a couple of general comments for AndroidLoop class.
>> I really don't think we should make a separate application for it right now, and even if we have some cool reasons to do so, then the code should be re-factored so that the IntegrationLoop has a separate application for itself as well. Then what about the base Loop class? (I think you get the point)
> Yeah... this was one of our "problems" at the beginning of the work:
> deciding if we had to create a separate app, or work on top of the
> existing one.
> We chose the former.
>
> My main concern was for maintanability (smaller code base for each
> Loop). Even after a brief talk with Danilo we decided to keep the
> separate app for the moment. if we decide to use the same app for
> everything, it shouldn't be that hard to move everything inside.

Though I have nothing against the reasons why you decided to make this a
separate application (pros and cons, at the moment maintainability
really does sound better), I can't agree with the implementation,
because now we have mixed ways of doing things. As I pointed out, if
we're separating loops into applications, then part of your change
should be a re-factoring of the IntegrationLoop, thus bringing it into
separate application as well. If we leave it as it is, we will have
mixed and even more unmaintainable solution, with developers wandering
where the IntegrationLoop is and why all loop derivatives have their own
app, and integration one does not. There are some new currents in the
implementation discussion (drop frontend etc.) so maybe we should
organize a quick session with danilo and discuss this a bit more before
making any decisions since this is still the core of our app and we will
have to carry these changes as our burden or our comfort through future
developing :)

>> The other thing is that all the WhateverLoop classes should extend from the Loop class. If they have something in common, then the method (or property) in question should be moved to the base Loop class as well.
> Good point, I actually didn't remember if we had to use Loop or IntegrationLoop.
>
>> Also you mention the Repository model in the comments above, but I can't seem to locate it in the diff. Can you elaborate please? :)
> That was in the initial model, but I later removed it and stored
> everything inside AndroidLoop.
>
>> On to the code implementation suggestions:
>>
>> ...
>> +class AndroidLoop(IntegrationLoop):
>> + """Defines the base class for an Android integration loop."""
>> + class Meta:
>> + app_label = 'android_build'
>> +
>> + # The default file name for the manifest file.
>> + DEFAULT_MANIFEST_FILE = 'default.xml'
>> + DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
>> + DEFAULT_BRANCH = 'master'
>> + DEFAULT_BINUTILS = '2.20.1'
>> + DEFAULT_GCC = '4.4.0'
>> + DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
>> + DEFAULT_BRANCH = 'master'
>>
>> We have double constants? And furtherm...

[Milo Casagrande (milo)]

Hello Stevan!

On Tue, Aug 14, 2012 at 1:53 PM, Stevan Radaković
<email address hidden> wrote:
>
> Though I have nothing against the reasons why you decided to make this a
> separate application (pros and cons, at the moment maintainability
> really does sound better), I can't agree with the implementation,
> because now we have mixed ways of doing things. As I pointed out, if
> we're separating loops into applications, then part of your change
> should be a re-factoring of the IntegrationLoop, thus bringing it into
> separate application as well.

Indeed, but we didn't want to do that without discussing further the approach.
Whatever we decide, refactoring wouldn't be a big problem: code is
already self contained that shouldn't be hard to do.

> If we leave it as it is, we will have
> mixed and even more unmaintainable solution, with developers wandering
> where the IntegrationLoop is and why all loop derivatives have their own
> app, and integration one does not. There are some new currents in the
> implementation discussion (drop frontend etc.) so maybe we should
> organize a quick session with danilo and discuss this a bit more before
> making any decisions since this is still the core of our app and we will
> have to carry these changes as our burden or our comfort through future
> developing :)

Totally agree with you.
Bare in mind that me and gesha are for splitting it up into separate apps. :-)

I was looking around and I saw also use of "sub-apps": a single Django
app, with multile apps inside. This might even be our case.
But let's schedule a meeting to discuss about it.

> Well I dunno putting string directly in the default section for a field
> is a common practice, but you can also use settings.py in django, and
> getting the value as i.e. "settings.ANDROID_DEFAULT_REPO"...

I thought about settings.py, but didn't really want to use it for
that, more for a locality point of view, keeping fields related to a
single app within itself, and keeping settings.py only for general
stuff.

> Ok, on to the model fields:
>
> +class AndroidLoop(IntegrationLoop):
> + """Defines the base class for an Android integration loop."""
> + class Meta:
> + app_label = 'android_build'
> +
> + # The default file name for the manifest file.
> + DEFAULT_MANIFEST_FILE = 'default.xml'
> + DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
> + DEFAULT_BRANCH = 'master'
> + DEFAULT_BINUTILS = '2.20.1'
> + DEFAULT_GCC = '4.4.0'
> + DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
> + DEFAULT_BRANCH = 'master'
> +
> + # BUILD_TYPE
> + build_type = models.CharField(max_length=100,
> choices=ANDROID_BUILD_TYPES)
> + # MANIFEST_REPO
> + manifest_repo = models.CharField(max_length=255, default=DEFAULT_REPO)
> + # MANIFEST_BRANCH
> + manifest_branch = models.CharField(max_length=50,
> default=DEFAULT_BRANCH)
>
> Are we missing a revision field here as well? I guess users will
> sometimes want something other then :head: revision for their builds...

Consider that our model is a 1:1 representation of what we have in
place now. We didn't really invent anything here, it is...

=== modified file 'Makefile'
--- Makefile	2012-07-19 08:00:53 +0000
+++ Makefile	2012-08-14 08:45:23 +0000
@@ -10,8 +10,10 @@
 create-migration: syncdb
 	python dashboard/manage.py schemamigration frontend --auto || \
 			echo "No changes to frontend models."
-	python dashboard/manage.py schemamigration jenkins --auto || \
+	python dashboard/manage.py schemamigration jenkinsserver --auto || \
 			echo "No changes to jenkins models."
+	python dashboard/manage.py schemamigration android_build --auto || \
+			echo "No changes to android_build models."
 	python dashboard/manage.py migrate
 
 migrate: syncdb

=== modified file 'README'
--- README	2012-07-24 21:34:26 +0000
+++ README	2012-08-14 08:45:23 +0000
@@ -46,7 +46,7 @@
 Whenever any of the models are changed, it is strongly advised to run
 South' schemamigration command:
 
-  $ make migration
+  $ make migrate
 
 
 Tests

=== added directory 'dashboard/android_build'
=== added file 'dashboard/android_build/__init__.py'
=== added directory 'dashboard/android_build/migrations'
=== added file 'dashboard/android_build/migrations/0001_initial.py'
--- dashboard/android_build/migrations/0001_initial.py	1970-01-01 00:00:00 +0000
+++ dashboard/android_build/migrations/0001_initial.py	2012-08-14 08:45:23 +0000
@@ -0,0 +1,99 @@
+# encoding: utf-8
+import datetime
+from south.db import db
+from south.v2 import SchemaMigration
+from django.db import models
+
+class Migration(SchemaMigration):
+
+    def forwards(self, orm):
+        
+        # Adding model 'AndroidLoop'
+        db.create_table('android_build_androidloop', (
+            ('loop_ptr', self.gf('django.db.models.fields.related.OneToOneField')(to=orm['frontend.Loop'], unique=True, primary_key=True)),
+            ('build_type', self.gf('django.db.models.fields.CharField')(max_length=100)),
+            ('manifest_repo', self.gf('django.db.models.fields.CharField')(default='git://android.git.kernel.org/platform/manifest.git', max_length=255)),
+            ('manifest_branch', self.gf('django.db.models.fields.CharField')(default='master', max_length=50)),
+            ('repo_quiet', self.gf('django.db.models.fields.BooleanField')(default=False)),
+            ('manifest_filename', self.gf('django.db.models.fields.CharField')(default='default.xml', max_length=255)),
+            ('make_targets', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('make_jobs', self.gf('django.db.models.fields.PositiveIntegerField')(default=0)),
+            ('ramdisk_size', self.gf('django.db.models.fields.PositiveIntegerField')(default=0)),
+            ('external_tarball', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('build_fs_image', self.gf('django.db.models.fields.BooleanField')(default=False)),
+            ('fs_image_size', self.gf('django.db.models.fields.CharField')(default='2G', max_length=20, blank=True)),
+            ('user_defined_values', self.gf('django.db.models.fields.TextField')(blank=True)),
+            ('toolchain_url', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('target_product', self.gf('django.db.models.fields.CharField')(max_length=50, blank=True)),
+            ('target_tools_prefix', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('binutils_version', self.gf('django.db.models.fields.CharField')(default='2.20.1', max_length=25)),
+            ('binutils_url', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('gcc_version', self.gf('django.db.models.fields.CharField')(default='4.4.0', max_length=25)),
+            ('gcc_url', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('sysroot_ndk_url', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('lava_submit', self.gf('django.db.models.fields.BooleanField')(default=False)),
+            ('lava_test_plan', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+            ('lava_submit_fatal', self.gf('django.db.models.fields.BooleanField')(default=True)),
+            ('lava_android_binaries', self.gf('django.db.models.fields.BooleanField')(default=True)),
+        ))
+        db.send_create_signal('android_build', ['AndroidLoop'])
+
+
+    def backwards(self, orm):
+        
+        # Deleting model 'AndroidLoop'
+        db.delete_table('android_build_androidloop')
+
+
+    models = {
+        'android_build.androidloop': {
+            'Meta': {'object_name': 'AndroidLoop', '_ormbases': ['frontend.Loop']},
+            'binutils_url': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'binutils_version': ('django.db.models.fields.CharField', [], {'default': "'2.20.1'", 'max_length': '25'}),
+            'build_fs_image': ('django.db.models.fields.BooleanField', [], {'default': 'False'}),
+            'build_type': ('django.db.models.fields.CharField', [], {'max_length': '100'}),
+            'external_tarball': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'fs_image_size': ('django.db.models.fields.CharField', [], {'default': "'2G'", 'max_length': '20', 'blank': 'True'}),
+            'gcc_url': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'gcc_version': ('django.db.models.fields.CharField', [], {'default': "'4.4.0'", 'max_length': '25'}),
+            'lava_android_binaries': ('django.db.models.fields.BooleanField', [], {'default': 'True'}),
+            'lava_submit': ('django.db.models.fields.BooleanField', [], {'default': 'False'}),
+            'lava_submit_fatal': ('django.db.models.fields.BooleanField', [], {'default': 'True'}),
+            'lava_test_plan': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'loop_ptr': ('django.db.models.fields.related.OneToOneField', [], {'to': "orm['frontend.Loop']", 'unique': 'True', 'primary_key': 'True'}),
+            'make_jobs': ('django.db.models.fields.PositiveIntegerField', [], {'default': '0'}),
+            'make_targets': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'manifest_branch': ('django.db.models.fields.CharField', [], {'default': "'master'", 'max_length': '50'}),
+            'manifest_filename': ('django.db.models.fields.CharField', [], {'default': "'default.xml'", 'max_length': '255'}),
+            'manifest_repo': ('django.db.models.fields.CharField', [], {'default': "'git://android.git.kernel.org/platform/manifest.git'", 'max_length': '255'}),
+            'ramdisk_size': ('django.db.models.fields.PositiveIntegerField', [], {'default': '0'}),
+            'repo_quiet': ('django.db.models.fields.BooleanField', [], {'default': 'False'}),
+            'sysroot_ndk_url': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'target_product': ('django.db.models.fields.CharField', [], {'max_length': '50', 'blank': 'True'}),
+            'target_tools_prefix': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'toolchain_url': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'user_defined_values': ('django.db.models.fields.TextField', [], {'blank': 'True'})
+        },
+        'frontend.loop': {
+            'Meta': {'object_name': 'Loop'},
+            'config': ('django.db.models.fields.related.ForeignKey', [], {'to': "orm['jenkinsserver.JenkinsConfig']"}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
+            'name': ('django.db.models.fields.CharField', [], {'unique': 'True', 'max_length': '200'}),
+            'server': ('django.db.models.fields.related.ForeignKey', [], {'to': "orm['jenkinsserver.JenkinsServer']"})
+        },
+        'jenkinsserver.jenkinsconfig': {
+            'Meta': {'object_name': 'JenkinsConfig'},
+            'config': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
+            'config_template': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'})
+        },
+        'jenkinsserver.jenkinsserver': {
+            'Meta': {'object_name': 'JenkinsServer'},
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
+            'password': ('django.db.models.fields.CharField', [], {'max_length': '100'}),
+            'url': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
+            'username': ('django.db.models.fields.CharField', [], {'max_length': '100'})
+        }
+    }
+
+    complete_apps = ['android_build']

=== added file 'dashboard/android_build/migrations/__init__.py'
=== added directory 'dashboard/android_build/models'
=== added file 'dashboard/android_build/models/__init__.py'
--- dashboard/android_build/models/__init__.py	1970-01-01 00:00:00 +0000
+++ dashboard/android_build/models/__init__.py	2012-08-14 08:45:23 +0000
@@ -0,0 +1,1 @@
+from android_loop import AndroidLoop

=== added file 'dashboard/android_build/models/android_loop.py'
--- dashboard/android_build/models/android_loop.py	1970-01-01 00:00:00 +0000
+++ dashboard/android_build/models/android_loop.py	2012-08-14 08:45:23 +0000
@@ -0,0 +1,132 @@
+# Copyright (C) 2012 Linaro
+#
+# This file is part of linaro-ci-dashboard.
+#
+# linaro-ci-dashboard is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# linaro-ci-dashboard is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with linaro-ci-dashboard.  If not, see <http://www.gnu.org/licenses/>.
+
+from django.db import models
+from frontend.models.loop import Loop
+from jenkinsserver.models.jenkins_config import JenkinsConfig
+from jenkinsserver.models.jenkins_server import JenkinsServer
+
+
+# The internal names for the different build type, and a user visible name
+BUILD_ANDROID = (r'build-android', 'Android')
+BUILD_ANDROID_RESTRICTED = (r'build-android-restricted', 'Android Restricted')
+BUILD_ANDROID_TOOLCHAIN = (r'build-android-toolchain', 'Android Toolchain')
+BUILD_ANDROID_TOOLCHAIN_LINARO = (r'build-android-toolchain-linaro',
+                                    'Android Toolchain Linaro')
+# All the supported build types.
+ANDROID_BUILD_TYPES = (
+    BUILD_ANDROID,
+    BUILD_ANDROID_RESTRICTED,
+    BUILD_ANDROID_TOOLCHAIN,
+    BUILD_ANDROID_TOOLCHAIN_LINARO,
+    )
+
+# Inetrnal name of the board, and a user visible name
+BEAGLEBOARD = (r'beagleboard', 'Beagleboard')
+PANDABOARD = (r'panda', 'Pandaboard')
+SNOWBALL = (r'snowball', 'Snowball')
+# The supported boards for the build.
+TARGET_BOARDS = (BEAGLEBOARD, PANDABOARD, SNOWBALL)
+
+
+class AndroidLoop(Loop):
+    """Defines the base class for an Android integration loop."""
+    class Meta:
+        app_label = 'android_build'
+
+    # The default file name for the manifest file.
+    DEFAULT_MANIFEST_FILE = 'default.xml'
+    DEFAULT_REPO = 'git://android.git.kernel.org/platform/manifest.git'
+    DEFAULT_BRANCH = 'master'
+    DEFAULT_BINUTILS = '2.20.1'
+    DEFAULT_GCC = '4.4.0'
+    DEFAULT_IMAGE_SIZE = '2G'
+
+    # BUILD_TYPE
+    build_type = models.CharField(max_length=100, choices=ANDROID_BUILD_TYPES)
+    # MANIFEST_REPO
+    manifest_repo = models.CharField(max_length=255, default=DEFAULT_REPO)
+    # MANIFEST_BRANCH
+    manifest_branch = models.CharField(max_length=50, default=DEFAULT_BRANCH)
+    # REPO_QUIET
+    repo_quiet = models.BooleanField(default=False)
+    # MANIFEST_FILENAME
+    manifest_filename = models.CharField(max_length=255,
+                                         default=DEFAULT_MANIFEST_FILE)
+    # MAKE_TARGETS
+    make_targets = models.CharField(max_length=255, blank=True)
+    # MAKE_JOBS
+    make_jobs = models.PositiveIntegerField(default=0)
+    # RAMDISK_SIZE
+    ramdisk_size = models.PositiveIntegerField(default=0)
+    # EXTERNAL_TARBALL
+    external_tarball = models.CharField(max_length=255, blank=True)
+    # BUILD_FS_IMAGE
+    build_fs_image = models.BooleanField(default=False,
+                                         verbose_name='Build filesystem image')
+    # FS_IMAGE_SIZE
+    fs_image_size = models.CharField(max_length=20,
+                                     blank=True,
+                                     verbose_name='Filesystem image size',
+                                     default=DEFAULT_IMAGE_SIZE)
+    # Users might want to define their own values.
+    user_defined_values = models.TextField(blank=True)
+
+    # Data needed for build of type build-android.
+    toolchain_url = models.CharField(max_length=255, blank=True)
+    target_product = models.CharField(max_length=50, choices=TARGET_BOARDS,
+                                        blank=True)
+    target_tools_prefix = models.CharField(max_length=255, blank=True)
+
+    # Data needed for build of type build-android-toolchain.
+    binutils_version = models.CharField(max_length=25,
+                                        default=DEFAULT_BINUTILS)
+    binutils_url = models.CharField(max_length=255, blank=True)
+
+    gcc_version = models.CharField(max_length=25, default=DEFAULT_GCC)
+    gcc_url = models.CharField(max_length=255, blank=True)
+
+    # TODO need to check the default value for this one.
+    sysroot_ndk_url = models.CharField(max_length=255, blank=True)
+
+    # Data needed for LAVA integration.
+    lava_submit = models.BooleanField(default=False)
+    # TODO need to know the values here
+    lava_test_plan = models.CharField(max_length=255, blank=True)
+    lava_submit_fatal = models.BooleanField(default=True, blank=True)
+    lava_android_binaries = models.BooleanField(default=True, blank=True)
+
+    def save(self, *args, **kwargs):
+        # TODO: Create encoded config
+        jenkins_config = JenkinsConfig()
+        jenkins_config.save()
+        self.config = jenkins_config
+        self.server = JenkinsServer.objects.get(id=1)
+        super(AndroidLoop, self).save(*args, **kwargs)
+        # We do not want a failure in remote server to affect the
+        # CI Loop flow.
+        try:
+            self.server.create_or_update_integration_loop(self)
+        except:
+            # TODO: Log error.
+            pass
+
+    def schedule_build(self):
+        # Every time before the build is scheduled, update the
+        # loop config on the remote server.
+        self.server.create_or_update_integration_loop(self)
+        super(AndroidLoop, self).schedule_build()

=== added directory 'dashboard/android_build/templates'
=== added file 'dashboard/android_build/templates/__init__.py'
=== added directory 'dashboard/android_build/tests'
=== added file 'dashboard/android_build/tests/__init__.py'
=== added file 'dashboard/android_build/urls.py'
--- dashboard/android_build/urls.py	1970-01-01 00:00:00 +0000
+++ dashboard/android_build/urls.py	2012-08-14 08:45:23 +0000
@@ -0,0 +1,41 @@
+# Copyright (C) 2012 Linaro
+#
+# This file is part of linaro-ci-dashboard.
+#
+# linaro-ci-dashboard is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# linaro-ci-dashboard is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with linaro-ci-dashboard.  If not, see <http://www.gnu.org/licenses/>.
+
+from django.conf.urls.defaults import patterns, include, url
+from frontend.views.home_page_view import HomePageView
+from frontend.views.index_view import IndexView
+from frontend.views.integration_loop_create_view \
+  import IntegrationLoopCreateView
+from frontend.views.integration_loop_update_view \
+  import IntegrationLoopUpdateView
+from frontend.views.integration_loop_detail_view \
+  import IntegrationLoopDetailView
+from frontend.views.loop_build_view import LoopBuildView
+
+urlpatterns = patterns('',
+    url(r'^$', IndexView.as_view(), name='Index'),
+    url(r'^index/$', IndexView.as_view(), name='Index'),
+    url(r'^home/$', HomePageView.as_view(), name='Home'),
+    url(r'^android/create/$',
+        IntegrationLoopCreateView.as_view(), name='AndroidLoopCreate'),
+    url(r'^android/detail/(?P<slug>[\w|\W]+)/$',
+        IntegrationLoopDetailView.as_view(), name='AndroidLoopDetail'),
+    url(r'^android/update/(?P<slug>[\w|\W]+)/$',
+        IntegrationLoopUpdateView.as_view(), name='AndroidLoopUpdate'),
+    url(r'^loop/build/(?P<slug>[\w|\W]+)/$',
+        LoopBuildView.as_view(), name='LoopBuild'),
+)

=== added directory 'dashboard/android_build/views'
=== added file 'dashboard/android_build/views/__init__.py'
=== modified file 'dashboard/frontend/migrations/0001_initial.py'
--- dashboard/frontend/migrations/0001_initial.py	2012-07-25 12:20:01 +0000
+++ dashboard/frontend/migrations/0001_initial.py	2012-08-14 08:45:23 +0000
@@ -13,6 +13,7 @@
             ('id', self.gf('django.db.models.fields.AutoField')(primary_key=True)),
             ('name', self.gf('django.db.models.fields.CharField')(unique=True, max_length=200)),
             ('server', self.gf('django.db.models.fields.related.ForeignKey')(to=orm['jenkinsserver.JenkinsServer'])),
+            ('config', self.gf('django.db.models.fields.related.ForeignKey')(to=orm['jenkinsserver.JenkinsConfig'])),
         ))
         db.send_create_signal('frontend', ['Loop'])
 
@@ -61,7 +62,8 @@
             'Meta': {'object_name': 'Loop'},
             'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
             'name': ('django.db.models.fields.CharField', [], {'unique': 'True', 'max_length': '200'}),
-            'server': ('django.db.models.fields.related.ForeignKey', [], {'to': "orm['jenkinsserver.JenkinsServer']"})
+            'server': ('django.db.models.fields.related.ForeignKey', [], {'to': "orm['jenkinsserver.JenkinsServer']"}),
+            'config': ('django.db.models.fields.related.ForeignKey', [], {'to': "orm['jenkinsserver.JenkinsConfig']"})
         },
         'frontend.loopbuild': {
             'Meta': {'ordering': "['-id']", 'object_name': 'LoopBuild'},
@@ -78,6 +80,12 @@
             'password': ('django.db.models.fields.CharField', [], {'max_length': '100'}),
             'url': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
             'username': ('django.db.models.fields.CharField', [], {'max_length': '100'})
+        },
+        'jenkinsserver.jenkinsconfig': {
+            'Meta': {'object_name': 'JenkinsConfig'},
+            'config_template': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
+            'config': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'})
         }
     }
 

=== modified file 'dashboard/frontend/models/loop.py'
--- dashboard/frontend/models/loop.py	2012-07-26 13:25:06 +0000
+++ dashboard/frontend/models/loop.py	2012-08-14 08:45:23 +0000
@@ -19,6 +19,7 @@
 from django.contrib.auth.models import User
 from django.db import models
 from jenkinsserver.models.jenkins_server import JenkinsServer
+from jenkinsserver.models.jenkins_config import JenkinsConfig
 
 
 class Loop(models.Model):
@@ -27,6 +28,7 @@
 
     name = models.CharField(max_length=200, unique=True)
     server = models.ForeignKey(JenkinsServer)
+    config = models.ForeignKey(JenkinsConfig)
 
     def schedule_build(self):
         '''Add a LoopBuild record for this Loop.

=== added file 'dashboard/jenkinsserver/migrations/0002_auto__add_jenkinsconfig.py'
--- dashboard/jenkinsserver/migrations/0002_auto__add_jenkinsconfig.py	1970-01-01 00:00:00 +0000
+++ dashboard/jenkinsserver/migrations/0002_auto__add_jenkinsconfig.py	2012-08-14 08:45:23 +0000
@@ -0,0 +1,42 @@
+# encoding: utf-8
+import datetime
+from south.db import db
+from south.v2 import SchemaMigration
+from django.db import models
+
+class Migration(SchemaMigration):
+
+    def forwards(self, orm):
+        
+        # Adding model 'JenkinsConfig'
+        db.create_table('jenkinsserver_jenkinsconfig', (
+            ('id', self.gf('django.db.models.fields.AutoField')(primary_key=True)),
+            ('config_template', self.gf('django.db.models.fields.CharField')(max_length=255)),
+            ('config', self.gf('django.db.models.fields.CharField')(max_length=255, blank=True)),
+        ))
+        db.send_create_signal('jenkinsserver', ['JenkinsConfig'])
+
+
+    def backwards(self, orm):
+        
+        # Deleting model 'JenkinsConfig'
+        db.delete_table('jenkinsserver_jenkinsconfig')
+
+
+    models = {
+        'jenkinsserver.jenkinsconfig': {
+            'Meta': {'object_name': 'JenkinsConfig'},
+            'config_template': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
+            'config': ('django.db.models.fields.CharField', [], {'max_length': '255', 'blank': 'True'}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'})
+        },
+        'jenkinsserver.jenkinsserver': {
+            'Meta': {'object_name': 'JenkinsServer'},
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
+            'password': ('django.db.models.fields.CharField', [], {'max_length': '100'}),
+            'url': ('django.db.models.fields.CharField', [], {'max_length': '255'}),
+            'username': ('django.db.models.fields.CharField', [], {'max_length': '100'})
+        }
+    }
+
+    complete_apps = ['jenkinsserver']

=== added file 'dashboard/jenkinsserver/models/jenkins_config.py'
--- dashboard/jenkinsserver/models/jenkins_config.py	1970-01-01 00:00:00 +0000
+++ dashboard/jenkinsserver/models/jenkins_config.py	2012-08-14 08:45:23 +0000
@@ -0,0 +1,29 @@
+# Copyright (C) 2012 Linaro
+#
+# This file is part of linaro-ci-dashboard.
+#
+# linaro-ci-dashboard is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# linaro-ci-dashboard is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with linaro-ci-dashboard.  If not, see <http://www.gnu.org/licenses/>.
+
+from django.db import models
+from django.conf import settings
+
+class JenkinsConfig(models.Model):
+    class Meta:
+        app_label = 'jenkinsserver'
+
+    config_template = models.CharField(max_length=255)
+    config = models.CharField(max_length=255)
+
+    def save(self, *args, **kwargs):
+        super(JenkinsConfig, self).save(*args, **kwargs)

=== modified file 'dashboard/settings.py'
--- dashboard/settings.py	2012-07-24 21:28:00 +0000
+++ dashboard/settings.py	2012-08-14 08:45:23 +0000
@@ -139,6 +139,7 @@
     'django_openid_auth',
     'jenkinsserver',
     'frontend',
+    'android_build',
     'south',
     # Uncomment the next line to enable the admin:
     # 'django.contrib.admin',

=== modified file 'dashboard/urls.py'
--- dashboard/urls.py	2012-07-19 19:58:21 +0000
+++ dashboard/urls.py	2012-08-14 08:45:23 +0000
@@ -30,4 +30,5 @@
     url(r'^js/(?P<path>.*)$', 'django.views.static.serve',
      {'document_root': settings.JS_PATH}),
     url(r'^', include('dashboard.frontend.urls')),
+    url(r'^', include('dashboard.android_build.urls')),
 )