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-59871) udev: Handle PTP device symlink properly on udev action 'change' #42

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

dtardon
Copy link
Member

@dtardon dtardon commented Sep 24, 2024

PTP device symlink creation rules are currently executed only when the udev action is 'add'. If a user reloads the rules and runs the udevadm trigger command to reapply changes, the symlink may be deleted, which can prevent the chronyd service from restarting properly.

Signed-off-by: Chengen Du [email protected]
(cherry picked from commit 6bd12be3fa7761f190e17efdbdbff4440da7528b)

Resolves: RHEL-59871

PTP device symlink creation rules are currently executed only when the
udev action is 'add'. If a user reloads the rules and runs the udevadm
trigger command to reapply changes, the symlink may be deleted, which
can prevent the chronyd service from restarting properly.

Signed-off-by: Chengen Du <[email protected]>
(cherry picked from commit 6bd12be3fa7761f190e17efdbdbff4440da7528b)

Resolves: RHEL-59871
@github-actions github-actions bot changed the title udev: Handle PTP device symlink properly on udev action 'change' (RHEL-59871) udev: Handle PTP device symlink properly on udev action 'change' Sep 24, 2024
Copy link

github-actions bot commented Sep 24, 2024

Commit validation

Tracker - RHEL-59871

The following commits meet all requirements

commit upstream
8db4d72 - udev: Handle PTP device symlink properly on udev action 'change' systemd/systemd@6bd12be

Tracker validation

Success

🟢 Tracker RHEL-59871 has set desired product: CentOS Stream 10
🟢 Tracker RHEL-59871 has set desired component: systemd
🟢 Tracker RHEL-59871 has been approved


Pull Request validation

Success

🟢 CI - All checks have passed
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved


Auto Merge

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟢 Pull Request meet requirements, mergeable_state is clean
🟢 Pull Request has correct target branch main
🟢 Pull Request was merged

Copy link
Member

@jamacku jamacku left a comment

Choose a reason for hiding this comment

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

LGTM

@jamacku jamacku added this to the RHEL-10.0 milestone Oct 1, 2024
@github-actions github-actions bot merged commit ab07d07 into redhat-plumbers:main Oct 1, 2024
19 checks passed
@dtardon dtardon deleted the RHEL-59871-ptp-change branch October 1, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants