Story #15356

[Composer] Use cluster config

Added by Tom Morris 2 months ago. Updated 15 days ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
07/31/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
1.0

Description

Fetch the safe clusterwide config to bootstrap the app. Update app to expect clusterwide config.

Composer should go to a well known endpoint relative to the app's base URL, which should then redirect to or proxy for the public clusterwide config exported by controller (#15000).


Subtasks

Task #15485: Review 15356-use-cluster-configResolvedPeter Amstutz


Related issues

Related to Arvados - Story #13648: [Epic] Use one cluster configuration file for all componentsNew

Associated revisions

Revision dde5e605
Added by Peter Amstutz 24 days ago

Merge branch '15356-composer-config' refs #15356

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

Revision ee1a7e40 (diff)
Added by Peter Amstutz 19 days ago

run-tests.sh fixups refs #15356

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

History

#1 Updated by Tom Morris 2 months ago

  • Story points set to 1.0
  • Target version set to Arvados Future Sprints

#2 Updated by Tom Clegg 2 months ago

  • Related to Story #13648: [Epic] Use one cluster configuration file for all components added

#3 Updated by Tom Morris about 1 month ago

  • Target version changed from Arvados Future Sprints to 2019-07-31 Sprint

#4 Updated by Tom Morris about 1 month ago

  • Assigned To set to Peter Amstutz

#5 Updated by Peter Amstutz 25 days ago

15356-use-cluster-config @ commit:3fef11e150bb04f7d449b3a1f31250d52a7757c8 (:composer.git)

#6 Updated by Peter Amstutz 25 days ago

  • Status changed from New to In Progress

#7 Updated by Peter Amstutz 25 days ago

  • Target version changed from 2019-07-31 Sprint to 2019-08-14 Sprint

#8 Updated by Lucas Di Pentima 25 days ago

This LGTM, the only nit would be that I'm not sure if we should enforce https on composer's api_host config.

#9 Updated by Peter Amstutz 24 days ago

Lucas Di Pentima wrote:

This LGTM, the only nit would be that I'm not sure if we should enforce https on composer's api_host config.

We don't support non-HTTPS for the API server and never have. This is intended to be consistent with ARVADOS_API_HOST which also assumes HTTPS.

#10 Updated by Lucas Di Pentima 24 days ago

A couple additional comments:

#11 Updated by Peter Amstutz 24 days ago

Lucas Di Pentima wrote:

A couple additional comments:

15356-composer-config @ f1d3a442ea2220c9d6965431f7c878312f56bd66

#12 Updated by Lucas Di Pentima 24 days ago

15356-composer-config LGTM, thanks!

#13 Updated by Peter Amstutz 24 days ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF