Repository navigation
RLIMIT support - part 3 - #868
NicolasFussberger wants to merge 6 commits into
Conversation
|
Documentation preview for this pull request is available at: |
171c58b to
30f35ba
Compare
| // Scheduling priority for the component's initial thread. | ||
| scheduling_priority:int32 = null; // required | ||
|
|
||
| // to be removed - only here for compatibility but values will be ignored |
There was a problem hiding this comment.
Do you think this removal should be mentioned in a release note? I guess it's not really a breaking changes since it wasn't used but a users build could fail if they did set this.
There was a problem hiding this comment.
I guess it would also be good to increase the schema version due to this change?
There was a problem hiding this comment.
Yeah think we should maybe start honoring that. Might also be good to keep git tags for the config? so that external tools can reference the raw schema file?
e.g. something like cfg-schema-vX
so then a link like this would never change:
https://raw.githubusercontent.com/eclipse-score/lifecycle/refs/tags/cfg-schema-v1/score/launch_manager/src/daemon/src/configuration/config_schema/launch_manager.schema.json
There was a problem hiding this comment.
Could be an idea. I guess at the current stage it might be enough to use the release tag?
I assume we will keep changing the schema for each upcoming release
There was a problem hiding this comment.
Yeah that would work too.
30f35ba to
cb9a1b5
Compare
76a5ed0 to
85de26b
Compare
This is the last PR for adding rlimit support.
It makes the rlimits configurable via json and adds a component/feature-integration test for all rlimits.
See previous PRs:
#847
#845