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

inotify-tools is unmaintained #605

Open
elfosardo opened this issue Dec 18, 2024 · 3 comments · May be fixed by #607
Open

inotify-tools is unmaintained #605

elfosardo opened this issue Dec 18, 2024 · 3 comments · May be fixed by #607
Labels
triage/accepted Indicates an issue is ready to be actively worked on.

Comments

@elfosardo
Copy link
Member

inotify-tools last release is from september 2023 and last update in the repo is 7 months old (May 2024), it's not present in EPEL10 and not updated in EPEL9
we need to find a replacement or rewrite inotifywait mechanism in some way
we use inotifywait from inotify-tools in multiple places:
https://github.com/metal3-io/ironic-image/blob/main/scripts/runironic#L16
https://github.com/metal3-io/ironic-image/blob/main/scripts/runhttpd#L76
https://github.com/metal3-io/ironic-image/blob/main/scripts/runhttpd#L84
https://github.com/metal3-io/ironic-image/blob/main/scripts/runlogwatch.sh#L6

also we should not accept changes that use inotifywait anymore

@metal3-io-bot metal3-io-bot added the needs-triage Indicates an issue lacks a `triage/foo` label and requires one. label Dec 18, 2024
@Rozzii
Copy link
Member

Rozzii commented Dec 18, 2024

/triage accepted
Has been discussed on 2024.12.18 Community meeting.

@metal3-io-bot metal3-io-bot added triage/accepted Indicates an issue is ready to be actively worked on. and removed needs-triage Indicates an issue lacks a `triage/foo` label and requires one. labels Dec 18, 2024
@tuminoid
Copy link
Member

I quickly checked alternatives and very few options in EPEL in general. fswatch, watchman etc all missing in EPEL9 as well. incron is there but it is completely different usage and bad fit for containers.

I might be easiest to just make a custom Python snippet to simulate inotify-tools. We already use similar for processing jinja2 templates.

@dtantsur
Copy link
Member

Bonus points for avoiding EPEL entirely (python3-inotify seems to be in the main repo).

@elfosardo elfosardo linked a pull request Dec 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/accepted Indicates an issue is ready to be actively worked on.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants