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

(#1779813) (#1779813) Support for long unit names #139

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

msekletar
Copy link
Contributor

No description provided.

… hash at the end

Fixes #18077

(cherry picked from commit 1d0727e76fd5e9a07cc9991ec9a10ea1d78a99c7)

Resolves: #1779813
(cherry picked from commit 2ef0101e0b2813e8c99fc8f137dbaa763ca16057)

Resolves: #1779813

[msekleta: we are missing rate limiting patches in RHEL-7 so I've
renamed TEST-60 to reflect what it actually tests.]
@yuwata
Copy link
Contributor

yuwata commented Aug 17, 2022

May not important, but upstream has two follow-ups for the first patch:

@msekletar
Copy link
Contributor Author

@yuwata These patches are not important, because we never had (in RHEL-7) ratelimiting of log messages about overly log unit paths in the first place.

Copy link
Contributor

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

The code LGTM, but I'd prefer not to do this in RHEL-7...

@systemd-rhel-bot systemd-rhel-bot changed the title (#1779813) Support for long unit names (#1779813) (#1779813) Support for long unit names Aug 6, 2023
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.

None yet

5 participants