Skip to content
This repository has been archived by the owner on Sep 16, 2023. It is now read-only.

[Feature Request] Handle region-specific configurations as region parameters #56

Open
avouacr opened this issue Oct 6, 2022 · 7 comments
Assignees

Comments

@avouacr
Copy link
Member

avouacr commented Oct 6, 2022

Right now, region-specific configurations (e.g. proxy adresses, certificates..) are applied through init scripts run at services startup. These settings should be instead passed as parameters of the region, so that they can be directly injected through the chart of the service.

This change would be especially useful when working with an Onyxia instance not open to the internet, as it is not possible/easy to fetch the region init script.. which precisely configures regional proxies/certificates settings (chicken-egg problem).

@alexisdondon
Copy link

alexisdondon commented Oct 13, 2022

How should it be handle in the pojo region? I mean which property shoud we add in https://github.com/InseeFrLab/onyxia-api/blob/be4705924e3d073b38745e31349aeb7e0af0acdb/onyxia-model/src/main/java/fr/insee/onyxia/model/region/Region.java

For example at root of region should we add:

network : { httpProxy: someValue, httpsProxy: someValue, noProxy someValue],
certificates : {authorities: [ "linktoAn AUthorities", "otherLink"]

@alexisdondon
Copy link

just to keep trace :
images-datascience could be build as root with var_env that run script to delete sudo right.
or
images-datascience could be build as non root with var_env giving sudo right.

In this case to allow certificate injection when non root we could think about giving:

onyxia ALL(ALL:ALL) NOPASSWD:/usr/sbin/update-ca-certificates

@alexisdondon
Copy link

We need before update the ui to handle this new parameters InseeFrLab/onyxia#409

@alexisdondon
Copy link

This is possible from release 2.2.26 of onyxia-web and release v0.17 of onyxia-api

We could try the injection in helm charts catalogs

@alexisdondon
Copy link

working on adding a path to a bundle ca InseeFrLab/onyxia-api#194

@alexisdondon
Copy link

alexisdondon commented Jan 3, 2023

on my way to add pathToCaBundle and cacrts un api and after i will depreciate crts old field in the api

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants