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

add redis rmp builder #3

Merged
merged 1 commit into from
Sep 1, 2021
Merged

add redis rmp builder #3

merged 1 commit into from
Sep 1, 2021

Conversation

wy65701436
Copy link
Contributor

@wy65701436 wy65701436 commented Aug 31, 2021

In the arm env, redis needs to set the --with-lg-page=16 --with-lg-hugepage=21 when to build jemalloc
For details, needs to refer to: redis/docker-library-redis#254

Note:

  1. The rmp build must be executed on an arm based environment.
  2. The make check in the redis source is disabled as the new parameters will fail the jemalloc test, it needs more investigation.

Signed-off-by: Wang Yan wangyan@vmware.com

In the arm env, redis needs to set the --with-lg-page=16 --with-lg-hugepage=21 when to build jemalloc
For details, needs to refer to: redis/docker-library-redis#254

Note:
1. The rmp build must be executed on an arm based environment.
2. The make check in the redis source is disabled as the new parameters will fail the jemalloc test, it needs more investigation.

Signed-off-by: Wang Yan <wangyan@vmware.com>
Copy link
Collaborator

@Jeremy-boo Jeremy-boo left a comment

Choose a reason for hiding this comment

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

lgtm,thanks

@Jeremy-boo Jeremy-boo merged commit 7072b53 into goharbor:main Sep 1, 2021
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