YannickB / odoo-hosting

Other
64 stars 50 forks source link

[REF] Volumes management #220

Closed YannickB closed 7 years ago

YannickB commented 7 years ago

Hello all,

This is the PR for #219. As announced, this is a heavy refactor of the container structure, to remove the data/files/exec we used until now.

-Add new clouder.volume model. The clouder.volume is linked to service but not removed when service is unlink (unless explicit) -Bind/Postfix/Proxy/Postgres/Odoo templates are simplified and don't use data/files/exec anymore -SSH template is now a new application type -Resolve some bug with raise_error having wrong number of argument -Service now know they have to create a container thanks to an explicit checkbox, remove dummy field.

Also I officially drop the support for Odoo 8. You can still deploy Odoo 8 instance with Clouder, but you can't install Clouder module on Odoo 8 anymore. The reason is mainly the use of odoo tag in xml file.

Some image use my yannickburon docker hub repo, until the PR is merged and I can create the new image on official repo.

codecov-io commented 7 years ago

Codecov Report

Merging #220 into master will increase coverage by 0.24%. The diff coverage is 27.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
+ Coverage   32.92%   33.17%   +0.24%     
==========================================
  Files          50       52       +2     
  Lines        3836     3901      +65     
==========================================
+ Hits         1263     1294      +31     
- Misses       2573     2607      +34
Impacted Files Coverage Δ
clouder/models/service_link.py 58.53% <ø> (+3.99%) :arrow_up:
clouder/models/base_metadata.py 41.66% <ø> (ø) :arrow_up:
clouder/models/service_metadata.py 41.66% <ø> (ø) :arrow_up:
clouder/models/base_option.py 83.33% <ø> (ø) :arrow_up:
clouder/models/base.py 23.09% <ø> (ø) :arrow_up:
clouder/models/base_link.py 57.14% <ø> (ø) :arrow_up:
clouder/models/service_option.py 85.71% <ø> (ø) :arrow_up:
clouder/models/image.py 42.1% <0%> (ø) :arrow_up:
clouder/models/template_one_2_many.py 30.3% <0%> (-0.95%) :arrow_down:
clouder/models/model.py 22.61% <0%> (-0.11%) :arrow_down:
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 59dd6ea...55d2412. Read the comment docs.

lasley commented 7 years ago

@tedsalmon please review

YannickB commented 7 years ago

Thanks @lasley ! Ready for review