Merge lp://staging/~arges/openstack-charm-testing/configure-changes into lp://staging/openstack-charm-testing

Proposed by Chris J Arges
Status: Superseded
Proposed branch: lp://staging/~arges/openstack-charm-testing/configure-changes
Merge into: lp://staging/openstack-charm-testing
Diff against target: 440 lines (+207/-176)
10 files modified
README (+5/-8)
configure (+18/-102)
configure-ctsstack (+0/-9)
configure-dellstack (+0/-14)
configure-ppc64el (+0/-43)
profiles/ctsstack (+9/-0)
profiles/default (+103/-0)
profiles/dellstack (+14/-0)
profiles/kernelstack (+15/-0)
profiles/ppc64el (+43/-0)
To merge this branch: bzr merge lp://staging/~arges/openstack-charm-testing/configure-changes
Reviewer Review Type Date Requested Status
Ryan Beisner Needs Fixing
James Page Pending
Review via email: mp+261448@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2015-06-09.

Description of the change

Please merge this after kernelstack changes.
This changes the configuration system completely. Please test serverstack/etc deploys before merging.

To post a comment you must log in.
Revision history for this message
Ryan Beisner (1chb1n) wrote :

FYI, feeding this to UOSCI to make sure there is no breakage of automation, will report back in ~24hrs.

Revision history for this message
Ryan Beisner (1chb1n) wrote :

The configure script rearrangement caused a jenkins job failure in generating tempest templates in UOSCI.

I believe this is because the ./configure script is just echoing the command that we intend to run.

See inline comment.

review: Needs Fixing
106. By Chris J Arges

configure: simplify configure system

Put all configurations in 'profiles' subdir.
Updated configure script.
Update documentation.

Revision history for this message
Chris J Arges (arges) wrote :

> The configure script rearrangement caused a jenkins job failure in generating
> tempest templates in UOSCI.
>
> I believe this is because the ./configure script is just echoing the command
> that we intend to run.
>
> See inline comment.

Checked in the incorrect commit, due to lack of bzr-foo. Resubmitting.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: