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

fix(plugins.output.sensu): Update go-sensu to v2.12.0 #10247

Merged
merged 1 commit into from
Dec 16, 2021
Merged

fix(plugins.output.sensu): Update go-sensu to v2.12.0 #10247

merged 1 commit into from
Dec 16, 2021

Conversation

SVilgelm
Copy link
Contributor

@SVilgelm SVilgelm commented Dec 8, 2021

Update the version of go-sensu from v2.9.0 to v2.12.0 to get rid of the github.com/dgrijalva/jwt-go as indirect dependency

@telegraf-tiger
Copy link
Contributor

telegraf-tiger bot commented Dec 8, 2021

Thanks so much for the pull request!
🤝 ✒️ Just a reminder that the CLA has not yet been signed, and we'll need it before merging. Please sign the CLA when you get a chance, then post a comment here saying !signed-cla

feat(plugins.output.sensu): Update go-sensu to v2.12.0
605c6aa 
Update the version of go-sensu from v2.9.0 to  v2.12.0 to get rid of the `github.com/dgrijalva/jwt-go` as indirect dependency
@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Dec 8, 2021
@telegraf-tiger
Copy link
Contributor

telegraf-tiger bot commented Dec 8, 2021

Thanks so much for the pull request!
🤝 ✒️ Just a reminder that the CLA has not yet been signed, and we'll need it before merging. Please sign the CLA when you get a chance, then post a comment here saying !signed-cla

@SVilgelm
Copy link
Contributor Author

SVilgelm commented Dec 8, 2021

!signed-cla

@SVilgelm
Copy link
Contributor Author

SVilgelm commented Dec 8, 2021 •
edited
Loading

Actually the github.com/dgrijalva/jwt-go blocks the upgrade of telegraf to latest version

@SVilgelm
Copy link
Contributor Author

@powersj Hi, would you be able to check this PR?
For some weird reason we got into the situation, when we cannot update the telegraf, due to github.com/dgrijalva/jwt-go is listed as an indirect dependency.
It would be really helpful if this PR can be merged

@sspaink sspaink changed the title feat(plugins.output.sensu): Update go-sensu to v2.12.0 fix(plugins.output.sensu): Update go-sensu to v2.12.0 Dec 16, 2021
@powersj powersj merged commit 52ed68d into influxdata:master Dec 16, 2021
@SVilgelm SVilgelm deleted the update-go-sensu branch December 16, 2021 16:09
@SVilgelm
Copy link
Contributor Author

Thank you!

reimda pushed a commit that referenced this pull request Dec 16, 2021
@SVilgelm @reimda
fix: Update go-sensu to v2.12.0 (#10247)
aebc130 
(cherry picked from commit 52ed68d)
reimda pushed a commit that referenced this pull request Dec 16, 2021
@SVilgelm @reimda
fix: Update go-sensu to v2.12.0 (#10247)
fbb9323 
(cherry picked from commit 52ed68d)
powersj pushed a commit to powersj/telegraf that referenced this pull request Jan 21, 2022
@SVilgelm @powersj
fix: Update go-sensu to v2.12.0 (influxdata#10247)
5f64985
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@powersj powersj powersj approved these changes

@sspaink sspaink sspaink approved these changes

Assignees
No one assigned
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SVilgelm @sspaink @powersj