Skip to content

Conversation

giuseppe
Copy link
Member

follow-up for #1574

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the support-io-weight-device branch from e00739b to 0319f1b Compare October 17, 2024 21:16
Copy link

podman system tests failed. @containers/packit-build please check.

@giuseppe giuseppe force-pushed the support-io-weight-device branch from 0319f1b to 0a571fa Compare October 17, 2024 21:20
Copy link
Collaborator

@flouthoc flouthoc left a comment

Choose a reason for hiding this comment

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

LGTM

@giuseppe giuseppe marked this pull request as draft October 18, 2024 12:06
@giuseppe giuseppe force-pushed the support-io-weight-device branch from 0a571fa to 7d24aa3 Compare October 18, 2024 15:07
@giuseppe giuseppe marked this pull request as ready for review October 18, 2024 15:09
@giuseppe giuseppe force-pushed the support-io-weight-device branch 2 times, most recently from e86b5c4 to 30fda6b Compare October 18, 2024 15:25
@giuseppe giuseppe marked this pull request as draft October 18, 2024 15:47
it is a preparation for the next commit

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
If the "io.(bfq)weight" is specified in the format "$DEVICE $VALUE",
pass the information to systemd using IODeviceWeight instead of
IOWeight, that is only used for the default value.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
writing to io.weight, at least, fails with EINVAL the write attempts
with multiple lines.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the support-io-weight-device branch from 30fda6b to c4a65aa Compare October 18, 2024 15:50
@giuseppe giuseppe marked this pull request as ready for review October 18, 2024 15:50
@giuseppe
Copy link
Member Author

@flouthoc can you please take another look and merge if it is good for you?

Copy link
Collaborator

@flouthoc flouthoc left a comment

Choose a reason for hiding this comment

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

LGTM

@flouthoc flouthoc merged commit f7557b7 into containers:main Oct 19, 2024
53 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants