Support from Odoo 8.0 to odoo 10.0 #88

archetipo · 2016-09-12T13:38:10Z

I made the following changes to support buildout for odoo 10 .
odoo master , should be merged with this PR.

gracinet · 2016-09-16T16:44:17Z

Thanks for the contribution!
That is too much of a change to be released with 1.9.2 (which has been waiting for much too long), and besides, it'll need some adaptation of the unit tests. So, I will consider the PR for next release (don't know yet if that'd be 1.9.3 or 2.0.0)

StefanRijnhart · 2016-10-03T20:05:37Z

I get the same error as in the failing tests, when running bin/buildout

buildout/10.0/anybox.recipe.odoo/anybox/recipe/odoo/server.py", line 105, in _create_default_config
    from openerp.tools.config import configmanager
ImportError: No module named openerp.tools.config

StefanRijnhart · 2016-10-03T20:10:22Z

Sorry, actual error is

buildout/10.0/anybox.recipe.odoo/anybox/recipe/odoo/runtime/session.py", line 19, in <module>
    from odoo.tools import config
ImportError: No module named tools

which does share the same import problem as https://travis-ci.org/anybox/anybox.recipe.odoo/jobs/163332958#L2703

StefanRijnhart · 2016-10-03T20:17:35Z

Problem above was caused by inouk.recipe.odoo_cmd which messes up the path with its bin/odoo.py. Sorry for the noise.

StefanRijnhart · 2016-10-03T20:26:08Z

However, on 8.0 I do get

buildout/8.0/anybox.recipe.odoo/anybox/recipe/odoo/server.py", line 107, in _create_default_config
    from odoo.tools.config import configmanager
ImportError: No module named tools.config

and in this case it is Odoo's own buildout/8.0/parts/odoo/odoo.py which was wisely moved out of the way in 10.0. Can this be solved by juggling paths in the recipe or should this change be made backwards compatible in some way?

petrus-v · 2016-10-04T06:47:45Z

@archetipo, Thanks a lot for this works!
cc @gracinet,

After some discussion on #87 the outcomes is that we would prefer maintain only one branch to be able to work from Odoo 8.0 and greater, do you mind to try/catch odoo import? Do you see any other breaking changes that would be hard to accomplished?

archetipo · 2016-10-04T08:42:29Z

@StefanRijnhart thanks
Now you can do the buildout for 10

@petrus-v thanks

the differences between 8/9 and 10 are many.
odoo.py has totally changed.
odoo/openerp/base now is odoo/odoo/base
openerp-server became odoo-bin
__openerp__.py the files have become __manifiest__.py

imo, it would be easier than the 2.0 versions of anybox are for versions of odoo 10+, and versions of anybox from 1.9.x and minors are for OpenERP 9-8 ...

what do you think?

petrus-v · 2016-10-04T09:32:23Z

anybox/recipe/odoo/base.py

@@ -1604,7 +1605,7 @@ def finalize_addons_paths(self, check_existence=True):
                            "please use addons lines with type 'local' "
                            "instead." % (self.name, opt_key))

-        base_addons = join(self.openerp_dir, 'openerp', 'addons')
+        base_addons = join(self.odoo_dir, 'openerp', 'addons')


I guess here will be:

base_addons = join(self.odoo_dir, 'odoo', 'addons')

@petrus-v yes, thank you!

coveralls · 2016-10-04T13:34:24Z

Coverage increased (+0.03%) to 86.555% when pulling ccfb0d2 on archetipo:fix_for_odoo10 into 3edb72b on anybox:master.

petrus-v · 2016-12-16T13:30:08Z

@archetipo

Hum, thinking... Ok, let's try as you say!

Depending of how many errors I get while doing my test plan I may change my point of view to automate integration test on our buildbot. if so, I'll create a dedicated branch in this repo called dev-odoo_8-9-10_compat (as much persons can get involved to that branch). And I'll give to you and @StefanRijnhart write access to allow you merge PR to that branch. Then I'll do the builders to launch test integration on that branch against odoo 8 / 9 / 10 (that require some change in master branch to set it).

Thanks! Odoo8to10

coveralls · 2016-12-16T14:33:29Z

Coverage decreased (-0.3%) to 86.128% when pulling 93d308e on archetipo:fix_for_odoo10 into f14b801 on anybox:master.

it do not sent properties likes force_demo/status/update_modules so instanciate a we needs to instaniate new registry while installing module. In any odoo version we were already getting a new registry as install step occured while loading the registry.

Using odoo 10.0 RegistryManager.get behavior change it do not sent properties likes force_demo/status/update_modules so instanciate a we needs to instaniate new registry while installing module. In any odoo version we were already getting a new registry as install step occured while loading the registry.

coveralls · 2016-12-21T15:11:48Z

Coverage decreased (-0.3%) to 86.128% when pulling 9573201 on archetipo:fix_for_odoo10 into f14b801 on anybox:master.

This PR should make compatible this branch with the future of the anybox.recipe.odoo: anybox/anybox.recipe.odoo#88

llacroix · 2016-12-22T20:55:05Z

I hope this pr gets merged sooner than later. I have to commit the changes in my repo to prevent buildout to fetch from pypi.

petrus-v · 2016-12-23T08:48:14Z

@llacroix my plan is (as mentioned earlier: #88 (comment)) to finalized testing integration and non regressions as soon as possible, to be able to merge this branch in the beginning of January.

Regards,

rvalyi · 2016-12-23T13:23:16Z

sorry if this is a dumb question, but will v7 still be supported after this merge?

petrus-v · 2016-12-23T14:19:49Z

@rvalyi, No we don't expect to maintain v7 with this anybox.recipe.odoo (a.r.Odoo)! You should use anybox.recipe.openerp (a.r.Openerp) instead which expect to be used for this purpose! a.r.Openerp may have some feature not implemented that are in a.r.Odoo, if so, you can open an issue on that repo to backport those features from a.r.Odoo and explicitly tell if you start to work on it.

petrus-v · 2016-12-23T14:20:48Z

While testing odoo v8 deployment using this branch I figure out longpolling wasn't working, I'll fix that!

fix longpolling (gevent) odoo 8 and 9

coveralls · 2016-12-23T21:46:57Z

Coverage decreased (-0.3%) to 86.179% when pulling fb4cf16 on archetipo:fix_for_odoo10 into f14b801 on anybox:master.

fix install-all parameter on odoo10 / rename files openerp -> odoo

coveralls · 2017-01-03T20:12:16Z

Coverage decreased (-0.3%) to 86.097% when pulling 9e0b6d1 on archetipo:fix_for_odoo10 into f14b801 on anybox:master.

petrus-v · 2017-05-03T07:58:59Z

As announced(https://odoo-community.org/groups/contributors-15/contributors-56573?mode=thread&date_begin=&date_end=) I'm merging this PR.

archetipo force-pushed the fix_for_odoo10 branch 2 times, most recently from 49083c2 to 4aa8f17 Compare September 22, 2016 07:58

archetipo force-pushed the fix_for_odoo10 branch 3 times, most recently from dfe3bb3 to d417165 Compare October 4, 2016 06:37

archetipo force-pushed the fix_for_odoo10 branch 9 times, most recently from c7d0a8b to b4396fa Compare October 4, 2016 08:26

archetipo force-pushed the fix_for_odoo10 branch 2 times, most recently from 547ca9a to 3739f3a Compare October 4, 2016 08:58

petrus-v reviewed Oct 4, 2016

View reviewed changes

archetipo force-pushed the fix_for_odoo10 branch 5 times, most recently from b7d15a4 to 249fd0c Compare October 4, 2016 12:47

Merge pull request #8 from petrus-v/odoo8to10

93d308e

Thanks! Odoo8to10

Pierre Verkest added 2 commits December 21, 2016 16:02

petrus-v changed the title ~~Fix for odoo10~~ Prepare Odoo 8 to odoo 10.0 Dec 21, 2016

petrus-v changed the title ~~Prepare Odoo 8 to odoo 10.0~~ Support from Odoo 8.0 to odoo 10.0 Dec 21, 2016

petrus-v pushed a commit to anybox/anybox.buildbot.odoo that referenced this pull request Dec 21, 2016

Prepare support for aro future branch

659e9fe

This PR should make compatible this branch with the future of the anybox.recipe.odoo: anybox/anybox.recipe.odoo#88

petrus-v mentioned this pull request Dec 21, 2016

Prepare support for aro future branch anybox/anybox.buildbot.odoo#13

Closed

petrus-v pushed a commit to petrus-v/anybox.buildbot.odoo that referenced this pull request Dec 21, 2016

Prepare support for aro future branch

ca7498f

This PR should make compatible this branch with the future of the anybox.recipe.odoo: anybox/anybox.recipe.odoo#88

petrus-v mentioned this pull request Dec 21, 2016

Prepare support for aro future branch anybox/anybox.buildbot.odoo#14

Merged

Pierre Verkest added 2 commits December 23, 2016 22:37

fix longpolling for odoo8

1d6f676

Merge pull request #10 from petrus-v/odoo8to10

fb4cf16

fix longpolling (gevent) odoo 8 and 9

Pierre Verkest added 3 commits December 23, 2016 23:02

fix install-all parameter

0ed6d9e

rename start_openerp.py to start_odoo.py

d16dd51

Merge pull request #11 from petrus-v/odoo8to10

9e0b6d1

fix install-all parameter on odoo10 / rename files openerp -> odoo

bealdav mentioned this pull request Mar 14, 2017

Odoo v10 --dev=all broken restart #99

Open

petrus-v mentioned this pull request Mar 16, 2017

odoo v10 compat #100

Closed

petrus-v merged commit e37e0ac into anybox:master May 3, 2017

rvalyi added a commit to akretion/voodoo-template that referenced this pull request May 16, 2017

v10 support is merged in recipe anybox/anybox.recipe.odoo#88

10380d2

rvalyi mentioned this pull request Jun 2, 2017

New release? #74

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Support from Odoo 8.0 to odoo 10.0 #88

Support from Odoo 8.0 to odoo 10.0 #88

archetipo commented Sep 12, 2016 •

edited

Loading

gracinet commented Sep 16, 2016

StefanRijnhart commented Oct 3, 2016

StefanRijnhart commented Oct 3, 2016

StefanRijnhart commented Oct 3, 2016

StefanRijnhart commented Oct 3, 2016

petrus-v commented Oct 4, 2016

archetipo commented Oct 4, 2016

petrus-v Oct 4, 2016

archetipo Oct 4, 2016

coveralls commented Oct 4, 2016 •

edited

Loading

petrus-v commented Dec 16, 2016

coveralls commented Dec 16, 2016 •

edited

Loading

coveralls commented Dec 21, 2016 •

edited

Loading

llacroix commented Dec 22, 2016

petrus-v commented Dec 23, 2016

rvalyi commented Dec 23, 2016

petrus-v commented Dec 23, 2016

petrus-v commented Dec 23, 2016

coveralls commented Dec 23, 2016 •

edited

Loading

coveralls commented Jan 3, 2017 •

edited

Loading

petrus-v commented May 3, 2017

Support from Odoo 8.0 to odoo 10.0 #88

Support from Odoo 8.0 to odoo 10.0 #88

Conversation

archetipo commented Sep 12, 2016 • edited Loading

gracinet commented Sep 16, 2016

StefanRijnhart commented Oct 3, 2016

StefanRijnhart commented Oct 3, 2016

StefanRijnhart commented Oct 3, 2016

StefanRijnhart commented Oct 3, 2016

petrus-v commented Oct 4, 2016

archetipo commented Oct 4, 2016

petrus-v Oct 4, 2016

Choose a reason for hiding this comment

archetipo Oct 4, 2016

Choose a reason for hiding this comment

coveralls commented Oct 4, 2016 • edited Loading

petrus-v commented Dec 16, 2016

coveralls commented Dec 16, 2016 • edited Loading

coveralls commented Dec 21, 2016 • edited Loading

llacroix commented Dec 22, 2016

petrus-v commented Dec 23, 2016

rvalyi commented Dec 23, 2016

petrus-v commented Dec 23, 2016

petrus-v commented Dec 23, 2016

coveralls commented Dec 23, 2016 • edited Loading

coveralls commented Jan 3, 2017 • edited Loading

petrus-v commented May 3, 2017

archetipo commented Sep 12, 2016 •

edited

Loading

coveralls commented Oct 4, 2016 •

edited

Loading

coveralls commented Dec 16, 2016 •

edited

Loading

coveralls commented Dec 21, 2016 •

edited

Loading

coveralls commented Dec 23, 2016 •

edited

Loading

coveralls commented Jan 3, 2017 •

edited

Loading