-
Notifications
You must be signed in to change notification settings - Fork 30
✨ Computational backend: set container limits as labels ⚠️ (devops checks on grafana dashboards!) 🚨 #4453
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
Merged
sanderegg
merged 52 commits into
ITISFoundation:master
from
sanderegg:comp_backend/change-label-names
Jul 6, 2023
Merged
✨ Computational backend: set container limits as labels ⚠️ (devops checks on grafana dashboards!) 🚨 #4453
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
6e31cf3
change label
sanderegg cc87d44
needs separation of PR concerns
sanderegg 5253058
ongoing
sanderegg d017805
remove oputdated comment
sanderegg 5232080
new docker label dictionary
sanderegg eece8d2
doc
sanderegg 5697c2c
refactor
sanderegg fb44b7c
properly pass the labels so they are applied onto the user services
sanderegg 493c054
properly pass the labels so they are applied on the computational con…
sanderegg 32c1fb6
fix test
sanderegg e3f3bb5
fix test of dask client
sanderegg 925ac92
fix test
sanderegg cced2f8
linter
sanderegg d6d9286
ruff
sanderegg b7d27db
ruffing
sanderegg 0afab8e
cleanup
sanderegg 9ab10bb
setting labels on dy-sidecar
sanderegg ac5d0c1
set labels on dy-proxy
sanderegg b1c71a5
fix tests
sanderegg 7c00aab
ruff
sanderegg 0c8370b
cleaning up
sanderegg 195df12
cleaning
sanderegg 66e03bb
using the new label
sanderegg 8e93b7b
cleaning up labels
sanderegg d22dc43
cleaning
sanderegg d5b4da1
ruff
sanderegg 6d65934
cleaning
sanderegg 80cd859
typo
sanderegg d075344
fixed labels
sanderegg cf2f399
bit of renaming
sanderegg 074bc8e
more renaming
sanderegg 0a4a41c
fix tests
sanderegg 954f607
linter
sanderegg cf1e0e3
renaming
sanderegg 3180893
service image is already defined within the docker service/container
sanderegg 5c1db73
removed port label and replaced how it is extracted
sanderegg 534519b
added a warning to remove the backwards compatible code
sanderegg 63b78b2
use one status
sanderegg 5c2c4fb
simplify export
sanderegg a648609
typo
sanderegg efbf006
adapted director-v0
sanderegg ea8bc53
replaced key by service_key and version by service_version
sanderegg 07cc3fc
fix
sanderegg e9392dc
test fixed
sanderegg af45b9a
use function
sanderegg b7dfd16
fixing tests
sanderegg d315476
fix issues after renaming
sanderegg 611cf43
@GitHK review: add parenthesis back
sanderegg 0c04800
@GitHK review: revert change
sanderegg 413a3e8
@GitHK review: use 1e9
sanderegg d43b23c
@GitHK review: make it readable again
sanderegg 1396d73
@GitHK review: use 1e9
sanderegg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.