Skip to content

HDDS-15977. Remove mountpoint-s3 - #98

Open
Eason09053360 wants to merge 2 commits into
apache:masterfrom
Eason09053360:HDDS-15977
Open

HDDS-15977. Remove mountpoint-s3#98
Eason09053360 wants to merge 2 commits into
apache:masterfrom
Eason09053360:HDDS-15977

Conversation

@Eason09053360

@Eason09053360 Eason09053360 commented Aug 15, 2026

Copy link
Copy Markdown

What changes were proposed in this pull request?

Remove mountpoint-s3 and csc from the runner image. HDDS-15876 removed ozone-csi, so neither the S3 FUSE mount support nor the CSI command line client is used by anything in the project.

  • Drop the MOUNTPOINT_S3_VERSION build arg and the RUN layer that downloads and installs the mount-s3 RPM.
  • Drop the fuse package from the dnf install list — mountpoint-s3 was its only consumer.
  • Drop the golang builder stage that installs gocsi/csc, and the COPY that puts csc into the image.

What is the link to the Apache JIRA?

https://issues.apache.org/jira/browse/HDDS-15977

How was this patch tested?

Relies on the CI image build on this PR.

@adoroszlai adoroszlai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Eason09053360 for the patch. Please enable workflows in your fork.

It looks like the task description was incomplete. We should also remove csc:

FROM golang:1.26-bookworm AS go
RUN go install github.com/rexray/gocsi/csc@v1.2.2

# CSI / k8s dependency
COPY --from=go /go/bin/csc /usr/bin/csc

@Eason09053360

Copy link
Copy Markdown
Author

Thanks @Eason09053360 for the patch. Please enable workflows in your fork.

Thanks for reviewing ! enabled now

It looks like the task description was incomplete. We should also remove csc:

FROM golang:1.26-bookworm AS go
RUN go install github.com/rexray/gocsi/csc@v1.2.2

# CSI / k8s dependency
COPY --from=go /go/bin/csc /usr/bin/csc

Sure, I will take a look at it later :)

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