Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

sys-apps/systemd: Fix the patch to update DefaultTasksMax #971

Merged
merged 1 commit into from Apr 26, 2021

Conversation

sayanchowdhury
Copy link
Member

@sayanchowdhury sayanchowdhury commented Apr 22, 2021

The upstream changed the way it handles default value, and
make the property partially dynamic which made the previous patch not work

Testing done

Local test not done yet. Jenkins CI running

@sayanchowdhury
Copy link
Member Author

Locally tested and Jenkins CI passed. Ready to review.

@sayanchowdhury sayanchowdhury marked this pull request as ready for review April 23, 2021 11:50
Copy link
Member

@dongsupark dongsupark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Nits:

The upstream changed the way the default percentage value, and
make the property partially dynamic.

Upstream ref: systemd/systemd#14007

Fixes #382

Signed-off-by: Sayan Chowdhury <sayan@kinvolk.io>
@sayanchowdhury sayanchowdhury force-pushed the sayan/fix-systemd-tasksmax-patch branch from 58b5846 to 64e28be Compare April 26, 2021 14:30
@sayanchowdhury sayanchowdhury merged commit d8210b1 into main Apr 26, 2021
@sayanchowdhury sayanchowdhury deleted the sayan/fix-systemd-tasksmax-patch branch April 26, 2021 14:31
sayanchowdhury added a commit that referenced this pull request Apr 26, 2021
sys-apps/systemd: Fix the patch to update DefaultTasksMax
sayanchowdhury added a commit that referenced this pull request Apr 26, 2021
sys-apps/systemd: Fix the patch to update DefaultTasksMax
sayanchowdhury added a commit that referenced this pull request Apr 26, 2021
sys-apps/systemd: Fix the patch to update DefaultTasksMax
@sayanchowdhury
Copy link
Member Author

Picked this commit to flatcar-2823 flatcar-2801 flatcar-2765 flatcar-lts-2605

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants