Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rethink how settings get applied to conda-store servers/workers #1005

Open
soapy1 opened this issue Nov 27, 2024 · 0 comments · May be fixed by #1044
Open

Rethink how settings get applied to conda-store servers/workers #1005

soapy1 opened this issue Nov 27, 2024 · 0 comments · May be fixed by #1044
Labels
area: configuration ⚙️ area: documentation 📖 Improvements or additions to documentation area: user experience 👩🏻‍💻 Items impacting the end-user experience needs: investigation 🔎 Someone in the team needs to look into this issue before scoping type: maintenance 🛠

Comments

@soapy1
Copy link
Contributor

soapy1 commented Nov 27, 2024

Context

Currently there are some issues with how conda store server/workers are configured and how settings are persisted. There are 2 places where settings come from:

  • the configuration file (provided at application startup)
  • the settings db table - contains only a subset of settings to apply globally or to a namespace/environment

However, it is not entirely clear which of these get precedent when there is overlap, or how changes to settings in one place affect the other.

This leads to unpredictable beheviour, for example when you are running the service in a cloud environment. Sometimes redeploying a worker with new settings does not work, etc.

It would be nice to have:

  • a way to configure conda-store servers/workers with a yaml/config language type file
  • a clear understanding of which settings take precedent from each config source
  • a unified way of checking settings in conda-store

Value and/or benefit

This change will:

  • make it more clear to users where their config is coming from and how to change their deployment
  • make the code for retrieving settings in conda-store-server more clear
  • (hopefully) lay some ground work for enabling per environment/namespace settings
@soapy1 soapy1 added area: configuration ⚙️ area: documentation 📖 Improvements or additions to documentation area: user experience 👩🏻‍💻 Items impacting the end-user experience needs: investigation 🔎 Someone in the team needs to look into this issue before scoping type: maintenance 🛠 labels Nov 27, 2024
@soapy1 soapy1 moved this from New 🚦 to TODO 📬 in conda-store 🐍 Nov 27, 2024
@soapy1 soapy1 moved this from TODO 📬 to In Progress 🏗 in conda-store 🐍 Dec 10, 2024
@soapy1 soapy1 mentioned this issue Jan 1, 2025
7 tasks
@soapy1 soapy1 moved this from In Progress 🏗 to In review 👀 in conda-store 🐍 Jan 9, 2025
@soapy1 soapy1 linked a pull request Jan 9, 2025 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: configuration ⚙️ area: documentation 📖 Improvements or additions to documentation area: user experience 👩🏻‍💻 Items impacting the end-user experience needs: investigation 🔎 Someone in the team needs to look into this issue before scoping type: maintenance 🛠
Projects
Status: In review 👀
Development

Successfully merging a pull request may close this issue.

1 participant