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

Update fluent-plugin-kubernetes_metadata_filter to 2.11.1 #10

Conversation

@rsicart
Copy link
Contributor Author

rsicart commented Jul 5, 2022

Hey @monotek, could you please take a look?

Thanks in advance!

Copy link
Owner

@monotek monotek left a comment

Choose a reason for hiding this comment

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

Please rebase and update all other gems too.

@rsicart rsicart force-pushed the rsi/update-fluent-plugin-kubernetes-metadata-filter-version branch from be26abe to 2e54182 Compare July 5, 2022 12:17
@rsicart rsicart force-pushed the rsi/update-fluent-plugin-kubernetes-metadata-filter-version branch from 2e54182 to 7e5cfdc Compare July 5, 2022 14:59
@rsicart
Copy link
Contributor Author

rsicart commented Jul 5, 2022

Thanks for the feedback,

rebase and update done!

@rsicart
Copy link
Contributor Author

rsicart commented Jul 5, 2022

I see some dependency errors in docker-build job:

...
#15 13.29   Activated fluentd-1.15.0
#15 13.29   which does not match conflicting dependency (< 1.15, >= 0.14.0)
#15 13.29 
#15 13.29   Conflicting dependency chains:
#15 13.29     fluentd (= 1.15.0), 1.15.0 activated
#15 13.29 
#15 13.29   versus:
#15 13.29     fluent-plugin-kubernetes_metadata_filter (= 2.11.1), 2.11.1 activated, depends on
#15 13.29     fluentd (< 1.15, >= 0.14.0)
...

Installing those gems manually does not produce errors.

Should I revert fluentd version to 1.14.6?

Copy link
Owner

@monotek monotek left a comment

Choose a reason for hiding this comment

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

Should I revert fluentd version to 1.14.6?

Sure, lets try :)

@monotek monotek merged commit 0566c62 into monotek:master Jul 5, 2022
@rsicart
Copy link
Contributor Author

rsicart commented Jul 5, 2022

Thank you! I was AFK!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants