Story #15356

[Composer] Use cluster config

Added by Tom Morris 12 months ago. Updated 4 months 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
Release relationship:
Auto

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 componentsResolved

Associated revisions

Revision dde5e605
Added by Peter Amstutz 10 months ago

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

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

Revision ee1a7e40 (diff)
Added by Peter Amstutz 10 months ago

run-tests.sh fixups refs #15356

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

Revision 1312bde0 (diff)
Added by Peter Amstutz 9 months ago

Fix Composer login, use https, refs #15356

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

History

#1 Updated by Tom Morris 12 months ago

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

#2 Updated by Tom Clegg 12 months ago

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

#3 Updated by Tom Morris 11 months ago

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

#4 Updated by Tom Morris 11 months ago

  • Assigned To set to Peter Amstutz

#5 Updated by Peter Amstutz 10 months ago

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

#6 Updated by Peter Amstutz 10 months ago

  • Status changed from New to In Progress

#7 Updated by Peter Amstutz 10 months ago

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

#8 Updated by Lucas Di Pentima 10 months 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 10 months 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 10 months ago

A couple additional comments:

#11 Updated by Peter Amstutz 10 months ago

Lucas Di Pentima wrote:

A couple additional comments:

15356-composer-config @ f1d3a442ea2220c9d6965431f7c878312f56bd66

#12 Updated by Lucas Di Pentima 10 months ago

15356-composer-config LGTM, thanks!

#13 Updated by Peter Amstutz 10 months ago

  • Status changed from In Progress to Resolved

#14 Updated by Peter Amstutz 4 months ago

  • Release set to 22

Also available in: Atom PDF