Skip to content

Conversation

@mrnicegyu11
Copy link
Member

@mrnicegyu11 mrnicegyu11 commented Jul 8, 2025

What do these changes do?

Re-iteration of mrnicegyu11#1

Emerged as a needed refactoring to facilitate #1041 (comment)

Sets the traefik-config using env-vars instead of CLI arguments, avoids the need for per-deployment-target docker-compose files (except osparc.local)

Bonus:
  • Removes env-var REDIS_EXTERNAL_HOST
  • Removes docker compose yaml version specification
  • (in osparc-config) adds osparc.local/cert_domains.json
  • Bugfix: sets TRACING_OPENTELEMETRY_COLLECTOR_SAMPLING_PERCENTAGE to 100 everywhere

Related issue/s

Related PR/s

https://git.speag.com/oSparc/osparc-ops-deployment-configuration/-/merge_requests/1482

Checklist

  • I tested and it works

mrnicegyu11 and others added 29 commits October 15, 2024 16:18
Merge remote-tracking branch 'upstream/main'
…oundation#979)

* Introduce longhorn chart

* Further longhorn configuration

* Longhorn: further settings configuration

* Fix longhorn configuration bugs

Extra: introduce longhorn pv vales for portainer

* Add comment for deletion longhorn

* Further longhorn configuration

* Add README.md for Longhorn wit FAQ

* Update Longhorn readme

* Update readme

* Futher LH configuration

* Update LH's Readme

* Update Longhorn Readme

* Improve LH's Readme

* LH: Reduce reserved default disk space to 5%

Since we use a dedicated disk for LH, we can go ahead with 5%

* Use values to set Longhorn storage class

* Update LH's Readme

* LH Readme: add requirements reference

* PR Review: bring back portainer s3 pv

* LH: decrease portinaer volume size
Merge remote-tracking branch 'upstream/main'
@mrnicegyu11 mrnicegyu11 self-assigned this Jul 8, 2025
@mrnicegyu11 mrnicegyu11 added observability alerting/monitoring refactoring labels Jul 8, 2025
@mrnicegyu11 mrnicegyu11 added this to the Engage milestone Jul 8, 2025
@mrnicegyu11 mrnicegyu11 changed the title Refactor: ops-traefik (2nd attempt) Refactor: ops-traefik configuration (2nd attempt) Jul 8, 2025
@mrnicegyu11 mrnicegyu11 marked this pull request as ready for review July 8, 2025 13:31
@mrnicegyu11 mrnicegyu11 requested a review from YuryHrytsuk as a code owner July 8, 2025 13:31
@mrnicegyu11
Copy link
Member Author

this was successfully tested on osparc.local

@mrnicegyu11 mrnicegyu11 merged commit 8b54592 into ITISFoundation:main Jul 22, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

observability alerting/monitoring refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants