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

Jump to bottom

[release-3.11] Bug 1554293 - logging-eventrouter event not formatted correctly in El… #10242

Merged
merged 1 commit into from Oct 12, 2018
Merged

[release-3.11] Bug 1554293 - logging-eventrouter event not formatted correctly in El… #10242

merged 1 commit into from Oct 12, 2018

Conversation

openshift-cherrypick-robot
Copy link

This is an automated cherry-pick of #10207

/assign vrutkovs

@nhosoi
Bug 1554293 - logging-eventrouter event not formatted correctly in El…
a7a4a04 
…asticsearch when using MUX

If openshift_logging_install_eventrouter is true, set TRANSFORM_EVENTS true in MUX, as well.
@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Sep 26, 2018
Merged
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 26, 2018
@vrutkovs
Copy link
Member

/test gcp-major-upgrade

@vrutkovs
Copy link
Member

/retest

@vrutkovs
Copy link
Member

Flake
/test gcp-major-upgrade

@vrutkovs
Copy link
Member

vrutkovs commented Oct 1, 2018

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 1, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

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

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

The pull request process is described here

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

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 1, 2018
@openshift-bot
Copy link

/retest

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

1 similar comment
@openshift-bot
Copy link

/retest

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

@jcantrill
Copy link
Contributor

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 1, 2018
@jcantrill
Copy link
Contributor

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 1, 2018
@jcantrill
Copy link
Contributor

/test logging

@openshift-bot
Copy link

/retest

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

2 similar comments
@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@richm
Copy link
Contributor

richm commented Oct 1, 2018

/test logging

@openshift-bot
Copy link

/retest

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

5 similar comments
@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

3 similar comments
@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@richm
Copy link
Contributor

richm commented Oct 2, 2018

/test logging

@openshift-bot
Copy link

/retest

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

@richm
Copy link
Contributor

richm commented Oct 2, 2018

@vrutkovs any idea why ci/openshift-jenkins/logging — Error starting Jenkins job. ?

@openshift-bot
Copy link

/retest

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

3 similar comments
@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@openshift-bot
Copy link

/retest

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

@vrutkovs
Copy link
Member

vrutkovs commented Oct 3, 2018

ci/openshift-jenkins/logging job is broken, I'll hold until .z release and then merge manually
/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 3, 2018
@nhosoi
Copy link
Contributor

nhosoi commented Oct 6, 2018

/retest

@nhosoi nhosoi mentioned this pull request Oct 6, 2018
Merged
@nhosoi
Copy link
Contributor

nhosoi commented Oct 6, 2018

/retest

1 similar comment
@nhosoi
Copy link
Contributor

nhosoi commented Oct 8, 2018

/retest

@nhosoi
Copy link
Contributor

nhosoi commented Oct 8, 2018

/test logging

@jcantrill
Copy link
Contributor

believe this is blocked on openshift-eng/aos-cd-jobs#1610

@nhosoi
Copy link
Contributor

nhosoi commented Oct 8, 2018

/retest

1 similar comment
@nhosoi
Copy link
Contributor

nhosoi commented Oct 8, 2018

/retest

@nhosoi
Copy link
Contributor

nhosoi commented Oct 8, 2018

/test logging

@jcantrill
Copy link
Contributor

jcantrill commented Oct 9, 2018 •
edited

The logging job is building master images... not sure why... investigating

@jcantrill
Copy link
Contributor

blocked by openshift/origin-aggregated-logging#1389

@jcantrill
Copy link
Contributor

/test logging

@vrutkovs
Copy link
Member

/hold cancel
/retest
/refresh
/skip

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 11, 2018
@nhosoi
Copy link
Contributor

nhosoi commented Oct 11, 2018

/test e2e-aws

@openshift-bot
Copy link

/retest

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

@openshift-merge-robot openshift-merge-robot merged commit d351c5f into openshift:release-3.11 Oct 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jcantrill jcantrill Awaiting requested review from jcantrill

@richm richm Awaiting requested review from richm

Assignees

@vrutkovs vrutkovs

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@openshift-cherrypick-robot @vrutkovs @openshift-ci-robot @openshift-bot @jcantrill @richm @nhosoi @openshift-merge-robot