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

feat(dockerfile): update nfs-ganesha to v5.9 #257

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Conversation

derekbit
Copy link
Member

Which issue(s) this PR fixes:

Issue longhorn/longhorn#9008

What this PR does / why we need it:

Special notes for your reviewer:

Additional documentation or context

Longhorn 9008

Signed-off-by: Derek Su <derek.su@suse.com>
@derekbit derekbit self-assigned this Jul 16, 2024
@derekbit derekbit marked this pull request as draft July 16, 2024 07:08
@derekbit derekbit marked this pull request as ready for review July 16, 2024 07:18
&& mv ntirpc-5.0 nfs-ganesha-5.7/src/libntirpc
WORKDIR /nfs-ganesha-5.7
RUN curl -L https://github.com/rancher/nfs-ganesha/archive/refs/tags/v5_20240716.tar.gz | tar zx \
&& curl -L https://github.com/nfs-ganesha/ntirpc/archive/refs/tags/v5.8.tar.gz | tar zx \
Copy link
Member

@innobead innobead Jul 16, 2024

Choose a reason for hiding this comment

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

Is ntiprc independently released from nfs-ganesha release? so there is no one in the release, so we need to copy it from ntirpc release instead?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, ntiprc is an independent release. We always need to copy it to nfs-ganesha.
I think ntiprc 5.0 is too old, so I update it together.

@derekbit derekbit merged commit 9555b77 into longhorn:master Jul 16, 2024
7 checks passed
@derekbit
Copy link
Member Author

@mergify backport v1.7.x

Copy link

mergify bot commented Jul 22, 2024

backport v1.7.x

✅ Backports have been created

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