lp:~danilo/landscape-charm/api-endpoint

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~landscape/landscape-charm/trunk at revision 316

🤖 Landscape Builder: Approve (test results) on 2015-06-19

Free Ekanayaka (community): Approve on 2015-06-18

Adam Collard (community): Approve on 2015-06-18

Diff: 56 lines (+16/-5)

2 files modified

lib/tests/test_templates.py (+6/-0)
templates/service.conf (+10/-5)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1463955: API endpoint is localhost when LDS is deployed with the new charm

Medium

Fix Released

Link a bug report

Related blueprints

319. By Данило Шеган on 2015-06-18

Remove /api from the root-url for API: it must not be there for API to work.

318. By Данило Шеган on 2015-06-18

Do not append slash if already there in the configured root_url for the api root-url.

317. By Данило Шеган on 2015-06-18

Append /api to API root-url.

316. By Данило Шеган on 2015-06-18

Set [api] and [package-upload] root-urls.

315. By Данило Шеган on 2015-06-16

Merge license-file-plain-text [f=1455198] [r=bjornt,tealeg,landscape-builder] [a=Данило Шеган]
Allow license files to be put in as plain text as well

After some discussion, we've decided to support both plain text and base64-encoded data for license-file charm config option (along with URLs to fetch the license from).

This means that we need to base64-decode only when really needed, and base64 module is not helping there.

314. By Björn Tillenius on 2015-06-16

Merge pause-action-fail [f=1458515] [r=landscape-builder,chad.smith,benji] [a=Björn Tillenius]
Make the 'resume' action fail if not all services started.

The 'lsctl start' call returns 0 even if some services didn't start. Now
we check 'lsctl status' as well and flag an error if not all services
are started.

313. By Данило Шеган on 2015-06-15

Merge admin-user-action [f=1462387] [r=landscape-builder,free.ekanayaka,benji] [a=Данило Шеган]
Create bootstrap action that creates an LDS account and adds an admin user

Introduce "bootstrap" action that takes "admin-name", "admin-email" and "admin-password" parameters and creates a "standalone" LDS account with the appropriate admin user.

At the moment, we are using 'schema --create-lds-account-only', though we should move to not using this in the future. This script doesn't report nicely if you've already called 'bootstrap' once.

312. By David Britton on 2015-06-11

s/pep8/flake8/ in the Makefile [trivial] [r=benji]

311. By Björn Tillenius on 2015-06-11

Merge leader-destroyed-integration-tests [f=1464161] [r=free.ekanayaka,adam-collard,landscape-builder] [a=Björn Tillenius]
Add integration tests for when the landscape-server leader dies.

I added tests to ensuring that all services are up and running,
including package-upload which runs only on the leader.

310. By Björn Tillenius on 2015-06-11

Merge integration-tests-no-hardcode-unit [f=1464134] [r=adam-collard,landscape-builder,free.ekanayaka] [a=Björn Tillenius]
Don't hardcode the unit ids in the integration tests.

The tests assumed that landscape-server/0 was always there. This is not
always true, especially not when we'll add a layer that destroys that
unit.

I also changed it so that error output is included from the 'juju ssh'
calls. Error output from the cron scripts was already included, but if
the actualy 'juju ssh' call failed, all you saw that the command exited
with error code 1, but you didn't see what the actual problem was.