Skip to content

Conversation

@KarstenSchnitter
Copy link
Contributor

Description

Moves the stopping of rsyslogd to post-stop phase, so that logs during stopping of bosh jobs are still forwarded to the configured syslog target.
This addresses #131.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Testing performed?

  • Unit tests
  • Integration tests
  • Acceptance tests

Checklist:

  • This PR is being made against the main branch, or relevant version branch
  • I have made corresponding changes to the documentation
  • I have added testing for my changes

Moves the stopping of rsyslogd to post-stop phase, so that logs
during stopping of bosh jobs are still forwarded to the configured
syslog target.
This addresses #131.

Signed-off-by: Karsten Schnitter <[email protected]>
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 9, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: KarstenSchnitter / name: Karsten Schnitter (48f3f50)

@ctlong
Copy link
Member

ctlong commented May 9, 2023

@KarstenSchnitter looks like you're going to have to sign the CLA to proceed. Can you please do so :)

@KarstenSchnitter KarstenSchnitter changed the title Migrate stop of resyslogd to post-stop Migrate stop of rsyslogd to post-stop May 10, 2023
@KarstenSchnitter
Copy link
Contributor Author

@ctlong signing the CLA has been done.

Copy link
Member

@Benjamintf1 Benjamintf1 left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@mkocher mkocher left a comment

Choose a reason for hiding this comment

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

seems like a good change to me! Thanks @KarstenSchnitter

@ctlong ctlong merged commit ffa8832 into cloudfoundry:main May 11, 2023
@KarstenSchnitter KarstenSchnitter deleted the stop-syslog branch May 12, 2023 20:35
@maxmoehl
Copy link
Member

Thank you all! Do you have an ETA on a new release with this fix?

@ctlong
Copy link
Member

ctlong commented May 17, 2023

We just cut a patch release!

@Benjamintf1
Copy link
Member

https://github.com/cloudfoundry/syslog-release/releases/tag/v12.1.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

5 participants