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

fix(rpm): Ensure telegraf is installed after useradd #14654

Merged
merged 3 commits into from
Jan 31, 2024

Conversation

powersj
Copy link
Contributor

@powersj powersj commented Jan 30, 2024

Summary

Enables SuSE support by ensuring telegraf is installed after useradd.

Checklist

  • No AI generated code was used in this PR

Related issues

resolves #3833

@powersj
Copy link
Contributor Author

powersj commented Jan 30, 2024

@Snelius30,

20-30mins after I post this message, some new artifacts should get posted to this PR. Could you please try them and confirm that the RPM now works successfully on SuSE? Can you also let me know what version of SuSE you are using to test?

Thanks!

@powersj powersj changed the title Fix/3833 fix(rpm): Ensure telegraf is installed after useradd Jan 30, 2024
@powersj powersj added the waiting for response waiting for response from contributor label Jan 30, 2024
@powersj powersj self-assigned this Jan 30, 2024
@telegraf-tiger
Copy link
Contributor

@Snelius30
Copy link
Contributor

@Snelius30,

20-30mins after I post this message, some new artifacts should get posted to this PR. Could you please try them and confirm that the RPM now works successfully on SuSE? Can you also let me know what version of SuSE you are using to test?

I have tested latest artifact telegraf-1.30.0~468a7c41-0.x86_64.rpm. It works! OS: OpenSuSe Tumbleweed 15.4. Thank you for ur job!

@telegraf-tiger telegraf-tiger bot removed the waiting for response waiting for response from contributor label Jan 31, 2024
@powersj powersj marked this pull request as ready for review January 31, 2024 13:51
@powersj powersj assigned srebhan and unassigned powersj Jan 31, 2024
@powersj powersj added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Jan 31, 2024
Copy link
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Awesome. Thanks @powersj!

@srebhan srebhan added area/packaging fix pr to fix corresponding bug labels Jan 31, 2024
@srebhan srebhan merged commit ff8e28b into influxdata:master Jan 31, 2024
26 of 27 checks passed
@github-actions github-actions bot added this to the v1.29.4 milestone Jan 31, 2024
hhiroshell pushed a commit to hhiroshell/telegraf that referenced this pull request Feb 1, 2024
powersj added a commit that referenced this pull request Feb 20, 2024
@powersj powersj modified the milestones: v1.29.4, v1.29.5 Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/packaging fix pr to fix corresponding bug ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Telegraf dependency to shadow-utils not appropriate for Suse 12.3
3 participants