Skip to content

Bug 1763282: fluentd pods doesn't process kubernetes events#1778

Merged
openshift-merge-robot merged 1 commit intoopenshift:release-4.2from
openshift-cherrypick-robot:cherry-pick-1770-to-release-4.2
Nov 28, 2019
Merged

Bug 1763282: fluentd pods doesn't process kubernetes events#1778
openshift-merge-robot merged 1 commit intoopenshift:release-4.2from
openshift-cherrypick-robot:cherry-pick-1770-to-release-4.2

Conversation

@openshift-cherrypick-robot
Copy link

@openshift-cherrypick-robot openshift-cherrypick-robot commented Nov 4, 2019

This is an automated cherry-pick of #1770

/assign richm

@richm
Bug 1756920: fluentd pods doesn't process kubernetes events
7aa0d8f 
https://bugzilla.redhat.com/show_bug.cgi?id=1756920
Use the new feature of the viaq plugin to only do the
event transform for records with a specific tag.
@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Nov 4, 2019
Merged
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 4, 2019

@openshift-cherrypick-robot: An error was encountered searching for bug 1756920 on the Bugzilla server at https://bugzilla.redhat.com:

did not get one bug, but 0: {[]}
Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

[release-4.2] Bug 1756920: fluentd pods doesn't process kubernetes events

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 4, 2019
@richm richm changed the title [release-4.2] Bug 1756920: fluentd pods doesn't process kubernetes events Bug 1763282 - fluentd pods doesn't process kubernetes events Nov 4, 2019
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 4, 2019

@openshift-cherrypick-robot: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

Bug 1763282 - fluentd pods doesn't process kubernetes events

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@richm richm added autoretest Please auto-retest this PR if one of the flaky tests fail component/fluentd kind/bug Categorizes issue or PR as related to a bug. release/4.2 labels Nov 4, 2019
@richm richm changed the title Bug 1763282 - fluentd pods doesn't process kubernetes events Bug 1763282: fluentd pods doesn't process kubernetes events Nov 4, 2019
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 4, 2019

@openshift-cherrypick-robot: An error was encountered searching for dependent bug 1756920 for bug 1763282 on the Bugzilla server at https://bugzilla.redhat.com:

did not get one bug, but 0: {[]}
Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

Bug 1763282: fluentd pods doesn't process kubernetes events

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@richm
Copy link
Contributor

richm commented Nov 4, 2019

/bugzilla refresh

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 4, 2019

@richm: An error was encountered searching for dependent bug 1756920 for bug 1763282 on the Bugzilla server at https://bugzilla.redhat.com:

did not get one bug, but 0: {[]}
Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@richm
Copy link
Contributor

richm commented Nov 4, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Nov 4, 2019
@JAORMX
Copy link
Contributor

JAORMX commented Nov 13, 2019

/lgtm

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 13, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JAORMX, openshift-cherrypick-robot, richm

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@richm
Copy link
Contributor

richm commented Nov 18, 2019

/bugzilla refresh

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 18, 2019

@richm: An error was encountered searching for dependent bug 1756920 for bug 1763282 on the Bugzilla server at https://bugzilla.redhat.com:

did not get one bug, but 0: {[]}
Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@stevekuznetsov stevekuznetsov added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Nov 18, 2019
@richm richm removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 18, 2019
@richm
Copy link
Contributor

richm commented Nov 18, 2019

note to cherry pick approver - this PR does have bugzilla/valid-bug, but for some reason the tide status above does not show that - if you look at the pr labels you will see

@mfojtik
Copy link

mfojtik commented Nov 21, 2019

"You are not authorized to access bug #1756920."

Can we have this fixed?

@richm
Copy link
Contributor

richm commented Nov 21, 2019 •
edited
Loading

"You are not authorized to access bug #1756920."

Can we have this fixed?

To whom are you addressing this question, and what exactly would you like to be fixed? If it is the bz permissions - we'll have to talk to the customer to see if they will allow it. Another way would be to add the openshift bugzilla bot account to the CC list on the bz(s) affected.
@mfojtik

@knobunc knobunc added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Nov 28, 2019
@openshift-bot
Copy link

openshift-bot commented Nov 28, 2019

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link

openshift-bot commented Nov 28, 2019

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 82cd5e8 into openshift:release-4.2 Nov 28, 2019
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 28, 2019

@openshift-cherrypick-robot: All pull requests linked via external trackers have merged. Bugzilla bug 1763282 has been moved to the MODIFIED state.

Details

In response to this:

Bug 1763282: fluentd pods doesn't process kubernetes events

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bparees bparees Awaiting requested review from bparees

@jcantrill jcantrill Awaiting requested review from jcantrill

1 more reviewer

@JAORMX JAORMX JAORMX approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@richm richm

@JAORMX JAORMX

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. autoretest Please auto-retest this PR if one of the flaky tests fail bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. component/fluentd kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. release/4.2 size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@openshift-cherrypick-robot @openshift-ci-robot @richm @JAORMX @mfojtik @openshift-bot @stevekuznetsov @knobunc @openshift-merge-robot