Juju Charms Collection
openstack-dashboard package

Merge lp://qastaging/~springfield-team/charms/precise/openstack-dashboard/trunk into lp://qastaging/~openstack-charmers-archive/charms/precise/openstack-dashboard/trunk

Precise Pangolin (12.04)
trunk
Merge into trunk

Proposed by ChingWei Chang on 2014-05-02

Status:	Superseded
Proposed branch:	lp://qastaging/~springfield-team/charms/precise/openstack-dashboard/trunk
Merge into:	lp://qastaging/~openstack-charmers-archive/charms/precise/openstack-dashboard/trunk
Diff against target:	127 lines (+46/-3) 6 files modified config.yaml (+3/-0) hooks/horizon_contexts.py (+11/-1) hooks/horizon_utils.py (+9/-0) templates/havana/local_settings.py (+7/-1) templates/icehouse/_40_router.py (+10/-0) templates/icehouse/local_settings.py (+6/-1)
To merge this branch:	bzr merge lp://qastaging/~springfield-team/charms/precise/openstack-dashboard/trunk
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Shiv Prasad Rao (community)		Needs Resubmitting on 2014-07-22
James Page	2014-05-02	Needs Fixing on 2014-05-21
Marco Ceppi	2014-05-02	Pending
Review via email: mp+218170@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2014-04-29.

This proposal has been superseded by a proposal from 2014-07-28.

Description of the change

This merge proposal is prepared for supporting Cisco Nexus 1000V dashboard for its related extension tab/panel.
config.yaml
New parameter for supporting Cisco N1KV specific policy profile profile-support.

hooks/horizon_contexts.py
Context logic to retrieve profile-support

templates/havana/local_settings.py
For profile-support setting

Revision history for this message

Marco Ceppi (marcoceppi) wrote on 2014-04-30: Posted in a previous version of this proposal

Leaving for openstack-charmers, thanks!!

review: Abstain

Revision history for this message

ChingWei Chang (cwchang) wrote on 2014-05-02:

We have merged with latest Icehouse merge to charm store now.
Please re-review this version

Revision history for this message

James Page (james-page) wrote on 2014-05-21:

Thanks for this proposal.

I think you need to tweak the way that the profile_support is handled. Specifically

1) config.yaml

Don't provide a default on "None" - the default is null which translates into None in python once parsed.

Also lets drop "-support" from the configuration key - its really just the profile.

Some comments in the description as to which values work would be nice as well.

2) local_settings.py

Rather than dropping the value directly into local settings, I'd prefer to see a conditional code block in the template that sets the profile_support value if its not None, and defaults the the original code if its None.

3) hooks/horizon_contexts.py

Please validate that the profile-support config is a supported value - 'cisco' or None should be right I think.

Thanks!

review: Needs Fixing

lp://qastaging/~springfield-team/charms/precise/openstack-dashboard/trunk updated on 2014-07-22

32. By Shiv Prasad Rao on 2014-06-13: Bug fix for havana deployment
33. By Shiv Prasad Rao on 2014-07-22: Chaning profile-support to profile
changes to address review comments
34. By Shiv Prasad Rao on 2014-07-22: sync revision 31 from lp:charms/openstack-dashboard

Revision history for this message

Shiv Prasad Rao (shivrao) wrote on 2014-07-22:

Addressed the comments:

1) changed the config parameter to "profile" in config.yaml. It was originally created as profile-support because the parameter in the local_setting is profile-support.

2) Added conditional code block in local_settings.py

3) added validation for supported value. Currently it is only cisco.

review: Needs Resubmitting

lp://qastaging/~springfield-team/charms/precise/openstack-dashboard/trunk updated on 2014-08-01