lp:~lihuiguo/landscape-charm/registration-relation

Branch merges

Propose for merging

No branches dependent on this one.

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

🤖 Landscape Builder: Approve (test results) on 2021-10-29

Simon Poirier (community): Approve on 2021-10-29

BootStack Reviewers: Pending requested 2021-10-04

Diff: 189 lines (+121/-1)

8 files modified

config.yaml (+4/-0)
hooks/application-dashboard-relation-changed (+8/-0)
hooks/application-dashboard-relation-joined (+8/-0)
hooks/config-changed (+0/-1)
lib/relations/application_dashboard.py (+62/-0)
lib/relations/tests/test_application_dashboard.py (+35/-0)
lib/services.py (+2/-0)
metadata.yaml (+2/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1945732: [feature] add registration relation to register landscape to Homer dashboard

Undecided

Fix Released

Link a bug report

Related blueprints

414. By Linda Guo <email address hidden> on 2021-10-29

removed hooks/application_dashboard_relation.py

413. By Linda Guo <email address hidden> on 2021-10-29

Used data provider for application-dashboard relation

412. By Linda Guo <email address hidden> on 2021-10-25

Added unit test

411. By Linda Guo <email address hidden> on 2021-10-25

1. Read url from root-url config
2. Set scheme to https
3. Updated unit test

410. By Linda Guo <email address hidden> on 2021-10-08

Update typo

409. By Linda Guo <email address hidden> on 2021-10-08

Return empty string if JUJU_CHARM_DIR is not set

408. By Linda Guo <email address hidden> on 2021-10-07

Add site-name config option

407. By Linda Guo <email address hidden> on 2021-10-05

Changed relation name to application-dashboard

406. By Linda Guo <email address hidden> on 2021-10-04

Add registration relation

By adding registration relation in landscape charm,
we can register landscape in Homer dashboard after
creating relation between landscape charm and Homer charm

405. By Simon Poirier on 2021-07-29

Merge juju-deploy-err [f=] [r=maxiberta] [a=Simon Poirier]
Replace juju-deployer by plain "juju deploy" in integration tests to allow testing on bionic.