~juju-gui/juju-gui/trunk

Viewing all changes in revision 1149.

  • Committer: Gary Poster
  • Date: 2013-10-23 16:32:36 UTC
  • mfrom: (1147.3.5 exportBugs)
  • Revision ID: gary.poster@canonical.com-20131023163236-1bsxj6jsorvr8hlz
Fix export and import bugs

Our support for exporting and importing config values and expose flags was buggy.  This fixes four discrete but related bugs.

 - Exporting config values would exclude those that had false-y default values.
 - We did not export the expose flag.
 - Importing config values did not work.
 - We called the expose flag "exposed", which is not what the actual delpoyer expects.

To qa, open up the sandbox, deploy apache with "enable_modules" set to something or other, and expose it.  Then export the environment.  When you look at the service in the file, you should see "expose: true" and "enable_modules: mod_proxy" or whatever value you set.  Then reload the GUI and drag the file into the sandbox.  When the service has loaded, you should see that it is exposed, and when you look at the config in the inspector, you should see your value in the "enable_modules" field.

R=jeff.pihach
CC=
https://codereview.appspot.com/15400052

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: