Subfactories

Bug #1201099 reported by Georges Racinet
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenERP buildbot configurator
Fix Released
Undecided
Unassigned

Bug Description

Over time, our builds have become more and more pluggable.
We have three method registries inside the main configurator class. docstring extract:

       buildout_dl_steps : name -> method returning the main buildout config
                           file name and a list of steps to
                           construct the buildout configuration slave-side.
       post_dl_steps: name-> method returning the main buildout config file
                             name and a list of steps to be inserted
                             between the buildout conf retrieval and the main
                             buildout run.
       post_buildout_steps : name -> method returning a list of steps to be
                             added after buildout has run and testing db is
                             done

post_buildout_steps now come with three builtin sets of steps :

 - standard (uses bin/test_openerp -i)
 - nose (install modules, run nosetests)
 - functional (start openerp on a reserved port, run functional testing command)

We have two more to make independent of anybox internal configuration and include in the standard distribution:

 - static-analysis (runs flake8)
 - packaging (use all packaging oriented features of anybox.recipe.openerp, cook, test and upload a tarball)

(these are currently added from our master.cfg files)

The configurator class is now way too big, we must extract these registries in a separated 'subfactories' package

Related branches

Revision history for this message
Georges Racinet (gracinet) wrote :

Done !
static-analysis has been published in the same round

Changed in anybox.buildbot.openerp:
status: New → Fix Committed
Revision history for this message
Georges Racinet (gracinet) wrote :

Released with 0.9

Changed in anybox.buildbot.openerp:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.