[immich] Extra environment variables are not passed through to the main container #22477
Closed
2 tasks done
Labels
bug
Something isn't working
Comments
13 tasks
stavros-k
added a commit
that referenced
this issue
May 26, 2024
…d from values (#22478) **Description** <!-- Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. --> ⚒️ Fixes #22477 ⚒️ Fixes #21749 Any configuration for `main` container that was added to values, it was "overwritten" instead of merged **⚙️ Type of change** - [ ] ⚙️ Feature/App addition - [x] 🪛 Bugfix - [ ]⚠️ Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] 🔃 Refactor of current code **🧪 How Has This Been Tested?** <!-- Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration --> **📃 Notes:** <!-- Please enter any other relevant information here --> **✔️ Checklist:** - [x] ⚖️ My code follows the style guidelines of this project - [x] 👀 I have performed a self-review of my own code - [ ] #️⃣ I have commented my code, particularly in hard-to-understand areas - [ ] 📄 I have made corresponding changes to the documentation - [x]⚠️ My changes generate no new warnings - [ ] 🧪 I have added tests to this description that prove my fix is effective or that my feature works - [x] ⬆️ I increased versions for any altered app according to semantic versioning - [x] I made sure the title starts with `feat(chart-name):`, `fix(chart-name):` or `chore(chart-name):` **➕ App addition** If this PR is an app addition please make sure you have done the following. - [ ] 🖼️ I have added an icon in the Chart's root directory called `icon.png` --- _Please don't blindly check all the boxes. Read them and only check those that apply. Those checkboxes are there for the reviewer to see what is this all about and the status of this PR with a quick glance._
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
chart Name
immich
Operating System
TrueNAS SCALE 24.04.0
Deployment Method
TrueNAS SCALE charts
Chart Version
15.13.6
Kubernetes Events
chartlication Logs
Chart Configuration
Describe the bug
I need to add an extra CA bundle for Immich to make HTTPS requests to my OIDC provider which has a self-signed certificate.
I discovered that Node.js (and hence the Immich server) makes use of a bundled CA certificates bundle. In order to include a custom CA certificate, it is recommended to use the
NODE_EXTRA_CA_CERTSenvironment variable.However, after confirming in a Discord support thread, it was discovered that the extra environment variables which are set in the TrueNAS SCALE UI are not passed through to the main (immich-server) container.
To Reproduce
NODE_EXTRA_CA_CERTSenvironment variable pointing to an extra CA certificate bundleNODE_EXTRA_CA_CERTSenvironment variable in the main server podExpected Behavior
NODE_EXTRA_CA_CERTSenvironment variable is set in the main (immich-server) container to the value supplied in the app configurationScreenshots
Additional Context
@ksimm1 attempted to set an extra environment variable for a brand new app install and faced the same issue.
I've read and agree with the following
[Chart-Name]The text was updated successfully, but these errors were encountered: