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

[collectd 6] contextswitch: migration to v6.0 #3812

Merged
merged 2 commits into from
Feb 10, 2023
Merged

[collectd 6] contextswitch: migration to v6.0 #3812

merged 2 commits into from
Feb 10, 2023

Conversation

manuelluis
Copy link
Contributor

@manuelluis manuelluis commented Jan 12, 2021

ChangeLog: contextswitch plugin: migration to v6.0

@manuelluis manuelluis requested a review from a team as a code owner January 12, 2021 02:15
@collectd-bot collectd-bot added this to the 6.0 milestone Jan 12, 2021
@mrunge mrunge added this to In progress in collectd 6 Jan 13, 2021
@hnez
Copy link
Contributor

hnez commented Jan 11, 2023

Hi,

In the last few weeks I have tested some collectd 6 ports of plugins to check if they are still up to date with main (e.g. there are no new commits in main that are not in the port), if they work and if they throw any errors with address sanitization enabled (compiled with -fsanitize=address) or when looking for data races with helgrind.

This plugin was one of them. And it …

  • … is up to date with main.
  • … does not show issues when used with address sanitization.
  • … looks fine to helgrind and me with regards to data races.

I would thus suggest merging it into collectd-6 and closing #3610.

Best regards
Leonard

@mrunge
Copy link
Member

mrunge commented Feb 10, 2023

thank you very much.

@mrunge mrunge merged commit d0a61a4 into collectd:collectd-6.0 Feb 10, 2023
collectd 6 automation moved this from In progress to Done Feb 10, 2023
@manuelluis manuelluis deleted the mlsr/collectd6-contextswitch branch December 14, 2023 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
collectd 6
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants