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.

Sign up for GitHub

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

Jump to bottom

ControlHub : Add logging-related dependencies to RPM specfile #28

Closed
tswilliams opened this issue Apr 10, 2017 · 1 comment
Closed

ControlHub : Add logging-related dependencies to RPM specfile #28

tswilliams opened this issue Apr 10, 2017 · 1 comment
Assignees
@tswilliams
Labels
bug
Milestone
v2.5.1 release

Comments

@tswilliams
Copy link
Collaborator

The ControlHub's logs are routed via syslog and have a log rotation configuration file. So, in order to be able to provide logs to debug any problems whilst not filling up the disk with logs, the ControlHub requires both syslog (provided by the rsyslog RPM on RHEL 6 & 7) and logrotate

However, this dependency is not currently encoded in the controlhub RPM. This oversight was not picked up earlier since the rsyslog and logrotate RPMs are already installed on almost all machines; however, I noticed that the log files were not being on the docker images used for testing - since they (deliberately) have a minimal setup.
@tswilliams tswilliams added the bug label Apr 10, 2017
@tswilliams tswilliams added this to the v2.5.1 release milestone Apr 10, 2017
@tswilliams tswilliams self-assigned this Apr 10, 2017
tswilliams added a commit that referenced this issue Apr 10, 2017
@tswilliams
ControlHub : Add logging-related dependencies to RPM specfile; fixes …
1d356f2 
…issue #28
@tswilliams tswilliams mentioned this issue Apr 11, 2017
Merged
@tswilliams
Copy link
Collaborator Author

All done. Closing

jhegeman pushed a commit to jhegeman/ipbus-software that referenced this issue Dec 6, 2021
@dmnewbold @timelike
Adding script. Closes ipbus#28
f28b61b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tswilliams tswilliams

Labels
bug
Projects
None yet
Milestone
v2.5.1 release
Development

No branches or pull requests
1 participant
@tswilliams