Merge lp://qastaging/~ahasenack/landscape-charm/new-dense-target-experiment into lp://qastaging/~landscape/landscape-charm/trunk

Proposed by Andreas Hasenack
Status: Merged
Approved by: Andreas Hasenack
Approved revision: 203
Merged at revision: 201
Proposed branch: lp://qastaging/~ahasenack/landscape-charm/new-dense-target-experiment
Merge into: lp://qastaging/~landscape/landscape-charm/trunk
Diff against target: 72 lines (+35/-9)
2 files modified
README.md (+29/-9)
config/landscape-deployments.yaml (+6/-0)
To merge this branch: bzr merge lp://qastaging/~ahasenack/landscape-charm/new-dense-target-experiment
Reviewer Review Type Date Requested Status
Free Ekanayaka (community) Approve
Geoff Teale (community) Approve
Review via email: mp+228736@code.qastaging.launchpad.net

Commit message

New deployer target called "landscape-dense", that deploys a working LDS on the bootstrap node on providers other than MAAS.

Description of the change

This adds a new deployer target called "landscape-dense". It's just like "landscape-dense-maas", but works on other providers where containers don't get routable IP addresses.

The trick is simple: we only need the frontend (apache2) to be reachable from the "outside". So I co-locate apache2 on the bootstrap node, and deploy the rest on containers.

I wondered if we need to keep the maas-only target. It's cleaner, in the sense that no hulk-smashing is done. But landscape-dense works everywhere. Discuss! :)

To post a comment you must log in.
Revision history for this message
Geoff Teale (tealeg) wrote :

+1 Tested on scapestack, works a treat!

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Thanks, but to really test this you need to deploy the bundle on an environment that is not MAAS, because on MAAS the containers get routable IP addresses :)

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

+1

As mentioned at standup, Juju is going to implement containers routing for non-MAAS providers, so this code might eventually become obsolete.

review: Approve

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