[PR #115] [MERGED] worker docker config also uses env variables #552

New issue

Closed

opened 2026-02-25 21:32:13 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:32:13 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ciur/papermerge/pull/115
Author: @jessecureton
Created: 9/9/2020
Status: ✅ Merged
Merged: 9/9/2020
Merged by: @ciur

Base: stable/1.4.x ← Head: jessecureton/worker-env-vars

📝 Commits (1)

e33a7f5 worker docker config also uses env variables

📊 Changes

1 file changed (+7 additions, -5 deletions)

View changed files

📝 docker/1.4/config/worker.production.py (+7 -5)

📄 Description

Description

Fixes #114, this sets the docker worker config to also use environment variables like the app config already does.

Test Plan

[x] cd docker/1.4 && docker-compose up - ensure that the defaults still yield a functional instance
[x] cd docker/1.4 && docker build -t papermerge-worker:dev -f worker.dockerfile - deploy compiled image to a running nomad cluster and ensure it picks up appropriate environment variables.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ciur/papermerge/pull/115 **Author:** [@jessecureton](https://github.com/jessecureton) **Created:** 9/9/2020 **Status:** ✅ Merged **Merged:** 9/9/2020 **Merged by:** [@ciur](https://github.com/ciur) **Base:** `stable/1.4.x` ← **Head:** `jessecureton/worker-env-vars` --- ### 📝 Commits (1) - [`e33a7f5`](https://github.com/ciur/papermerge/commit/e33a7f521f030fd0ca161ac663e6525016ac249d) worker docker config also uses env variables ### 📊 Changes **1 file changed** (+7 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `docker/1.4/config/worker.production.py` (+7 -5) </details> ### 📄 Description ### Description Fixes #114, this sets the docker worker config to also use environment variables like the app config already does. ### Test Plan [x] `cd docker/1.4 && docker-compose up` - ensure that the defaults still yield a functional instance [x] `cd docker/1.4 && docker build -t papermerge-worker:dev -f worker.dockerfile` - deploy compiled image to a running nomad cluster and ensure it picks up appropriate environment variables. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>