lp://staging/~tealeg/landscape-client/use-configobj

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/~landscape/landscape-client/trunk at revision 723

Chris Glass (community): Approve on 2013-09-16

Free Ekanayaka (community): Approve on 2013-09-11

Diff: 351 lines (+110/-37)

7 files modified

debian/control (+1/-0)
landscape/broker/tests/test_server.py (+5/-1)
landscape/configuration.py (+11/-8)
landscape/deployment.py (+42/-16)
landscape/tests/test_configuration.py (+11/-4)
landscape/tests/test_deployment.py (+37/-0)
landscape/ui/model/configuration/tests/test_mechanism.py (+3/-8)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1221659: landscape client removes comments from config files when modifying them

Low

Fix Released

Link a bug report

Related blueprints

729. By Geoff Teale on 2013-09-16

Fix up failing landscape-client-ui tests that were skipped, but thankfully picked up by Tarmac! *yay tarmac*

728. By Geoff Teale on 2013-09-16

Merge forwards

727. By Geoff Teale on 2013-09-16

lint

726. By Geoff Teale on 2013-09-16

Remove code duplication and empty subclass implementation of _get_config_object.

725. By Geoff Teale on 2013-09-16

All unit tests passing again, post review tasks.u

724. By Geoff Teale on 2013-09-12

add python-configobj to debian/control dependencies for landscape-client package.

723. By Geoff Teale on 2013-09-10

Merge forwards and resolve conflicts.

722. By Geoff Teale on 2013-09-10

lint

721. By Geoff Teale on 2013-09-10

Get ConfigObj instances from a utility function to avoid duplication of set-up.

720. By Geoff Teale on 2013-09-10

Add test to confirm that comments are maintained when we modify and write
back to the same configuration file.