Skip to content
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

fix(database): cgroup v2 compatibility #1963

Merged
merged 3 commits into from
Jul 14, 2021
Merged

Conversation

AmedeeBulle
Copy link
Member

The memory check in the runOracle script takes the cgroup version into
account.

Fixes #1939

Signed-off-by: Philippe Vanhaesendonck philippe.vanhaesendonck@oracle.com

The memory check in the runOracle script takes the cgroup version into
account.

Fixes oracle#1939

Signed-off-by: Philippe Vanhaesendonck <philippe.vanhaesendonck@oracle.com>
@AmedeeBulle
Copy link
Member Author

AmedeeBulle commented Jun 9, 2021

@Djelibeybi / @gvenzl

I appreciate your input on this one.
I managed to do some testing with an OL8 host with cgroup v1 and v2 and as far as I can see it works as expected.

I used memory.high rather than memory.max to keep the memory pressure reasonable.

If you think it is good to go, I'll push this fix for the other versions as well

memory.max is what the container runtime uses and we can have
memory.max < memory.high

Signed-off-by: Philippe Vanhaesendonck <philippe.vanhaesendonck@oracle.com>
gvenzl
gvenzl previously approved these changes Jun 9, 2021
Djelibeybi
Djelibeybi previously approved these changes Jun 11, 2021
@Djelibeybi
Copy link
Member

Looks good to me.

Apply previous commit to the other DB container image scripts

Signed-off-by: Philippe Vanhaesendonck <philippe.vanhaesendonck@oracle.com>
@AmedeeBulle AmedeeBulle dismissed stale reviews from Djelibeybi and gvenzl via d62e291 June 11, 2021 13:50
@AmedeeBulle
Copy link
Member Author

Applied the same changes to all scripts using /sys/fs/cgroup/memory/memory.limit_in_bytes.

Ready to be merged.

@AmedeeBulle AmedeeBulle marked this pull request as ready for review June 11, 2021 13:53
@AmedeeBulle AmedeeBulle requested a review from gvenzl June 11, 2021 13:53
Copy link
Member

@gvenzl gvenzl 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 for Single Instance files.

@Djelibeybi
Copy link
Member

Needs @psaini79 to approve for the RAC case.

@gvenzl
Copy link
Member

gvenzl commented Jul 14, 2021

Is there a chance to expedite this for Single Instance, i.e., file a PR with these changes for just the SI changes? :)

@Djelibeybi
Copy link
Member

I believe we gave @psaini79 sufficient time for review. I will merge it anyway.

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.

/sys/fs/cgroup/memory/memory.limit_in_bytes: No such file or directory
3 participants