-
Notifications
You must be signed in to change notification settings - Fork 231
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
Bug 1506099 - fluentd downstream vendor record_modifier #745
Bug 1506099 - fluentd downstream vendor record_modifier #745
Conversation
746a63d
to
1f55de5
Compare
/retest |
/lgtm |
/test json-file |
/retest |
/test logging |
1 similar comment
/test logging |
/retest |
1 similar comment
/retest |
looks like the failing test is always the same: I would like to take some time and try to find you why, because this PR doesn't change anything in upstream images or tests, only in downstream image and that is not used or built for CI tests AFAIK. Maybe PR #716 will resolve the issue when it merges. |
Vendoring the missing record_modifier to downstream image. The openshift#622 PR introduced record_modifier to upstream Dockerfile but not to the downstream. openshift#622
1f55de5
to
7f6b96e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. |
Vendoring the missing record_modifier to downstream image. The #622 PR
introduced record_modifier to upstream Dockerfile but not to the downstream.
This is porting forward the #742