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

(RHEL-55245) fix(dracut-install): copy xattr when use clone ioctl #98

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

pvalena
Copy link
Contributor

@pvalena pvalena commented Nov 27, 2024

When use clone ioctl to copy a file, the extended attributes of files are
missing, which is inconsistent with the result by using the cp command.
This commit add the process to copy extended attributes after clone_file().

Signed-off-by: Huaxin Lu [email protected]

(cherry picked from commit 1cf0db26e43fe4c6173acdb8047f16666ebf070a)

Resolves: RHEL-55245

When use clone ioctl to copy a file, the extended attributes of files are
missing, which is inconsistent with the result by using the cp command.
This commit add the process to copy extended attributes after clone_file().

Signed-off-by: Huaxin Lu <[email protected]>

(cherry picked from commit 1cf0db26e43fe4c6173acdb8047f16666ebf070a)

Resolves: RHEL-55245
@pvalena pvalena added the bug Something isn't working label Nov 27, 2024
@pvalena pvalena self-assigned this Nov 27, 2024
@github-actions github-actions bot changed the title fix(dracut-install): copy xattr when use clone ioctl (RHEL-55245) fix(dracut-install): copy xattr when use clone ioctl Nov 27, 2024
Copy link

Commit validation

Tracker - RHEL-55245

The following commits need an inspection

commit note
d8fd529 - fix(dracut-install): copy xattr when use clone ioctl Missing upstream reference ‼️

Tracker validation

Failed

🔴 Tracker RHEL-55245 is missing severity

Success

🟢 Tracker RHEL-55245 has set desired product: rhel-9.5
🟢 Tracker RHEL-55245 has set desired component: dracut
🟢 Tracker RHEL-55245 has been approved


Pull Request validation

Failed

🔴 Failed or pending checks - test (centos:stream9, 35)[failure],test (centos:stream9, 36)[failure],test (centos:stream9, 31)[failure],test (centos:stream9, 30)[failure],test (centos:stream9, 15)[failure],test (centos:stream9, 14)[failure],test (centos:stream9, 03)[failure],centos-9-stream (centos:stream9, 98)[cancelled],centos-9-stream (centos:stream9, 41)[cancelled],centos-9-stream (centos:stream9, 40)[cancelled],centos-9-stream (centos:stream9, 21)[cancelled],centos-9-stream (centos:stream9, 20)[cancelled],centos-9-stream (centos:stream9, 17)[cancelled],centos-9-stream (centos:stream9, 16)[cancelled],centos-9-stream (centos:stream9, 13)[cancelled],centos-9-stream (centos:stream9, 12)[cancelled],centos-9-stream (centos:stream9, 11)[cancelled],centos-9-stream (centos:stream9, 10)[cancelled],centos-9-stream (centos:stream9, 02)[cancelled],centos-9-stream (centos:stream9, 01)[cancelled],basic (centos:stream9, 04)[failure],testing-farm:centos-stream-9-x86_64[in_progress],rpm-build:centos-stream-9-aarch64[in_progress],rpm-build:centos-stream-9-x86_64[in_progress],test (centos:stream9, 98)[in_progress],test (centos:stream9, 41)[in_progress],test (centos:stream9, 21)[in_progress],test (centos:stream9, 20)[in_progress],test (centos:stream9, 17)[in_progress],test (centos:stream9, 16)[in_progress],test (centos:stream9, 13)[in_progress],test (centos:stream9, 12)[in_progress],test (centos:stream9, 11)[in_progress],test (centos:stream9, 10)[in_progress],test (centos:stream9, 01)[in_progress],lint-shell[in_progress]
🔴 Review - Missing review from a member (1 required)

@pvalena
Copy link
Contributor Author

pvalena commented Nov 27, 2024

LGTM. No relevant changes in upstream since.

@pvalena pvalena merged commit e3bba58 into redhat-plumbers:main Nov 27, 2024
22 of 44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant