Skip to content

libcontainer/intelrdt: use Mkdir/Remove instead of MkdirAll/RemoveAll#4840

Merged
rata merged 1 commit into
opencontainers:mainfrom
marquiz:devel/rdt-mkdir-rmdir
Aug 7, 2025
Merged

libcontainer/intelrdt: use Mkdir/Remove instead of MkdirAll/RemoveAll#4840
rata merged 1 commit into
opencontainers:mainfrom
marquiz:devel/rdt-mkdir-rmdir

Conversation

@marquiz

@marquiz marquiz commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

The more restricted Mkdir and Rmdir are sufficient in resctrl fs.

@marquiz

marquiz commented Aug 6, 2025

Copy link
Copy Markdown
Contributor Author

@rata @kolyshkin

The more restricted Mkdir and Rmdir are sufficient in resctrl fs.

Signed-off-by: Markus Lehtonen <markus.lehtonen@intel.com>
@marquiz
marquiz force-pushed the devel/rdt-mkdir-rmdir branch from 8062ba5 to 620956c Compare August 6, 2025 07:43
@kolyshkin

Copy link
Copy Markdown
Contributor

@marquiz please don't mind linter warnings, this is a golangci-lint issue (probably caused by caching), I'll report it upstream.

@kolyshkin kolyshkin 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.

LGTM, thanks!

@marquiz

marquiz commented Aug 6, 2025

Copy link
Copy Markdown
Contributor Author

please don't mind linter warnings, this is a golangci-lint issue (probably caused by caching), I'll report it upstream.

Thx, was just about to ask about this

@kolyshkin

Copy link
Copy Markdown
Contributor

@marquiz please don't mind linter warnings, this is a golangci-lint issue (probably caused by caching), I'll report it upstream.

Filed golangci/golangci-lint#5979

@rata rata left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@rata
rata merged commit 89c46a9 into opencontainers:main Aug 7, 2025
57 of 59 checks passed
@marquiz
marquiz deleted the devel/rdt-mkdir-rmdir branch August 7, 2025 19:04
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.

3 participants