Skip to content

Conversation

jsafrane
Copy link
Contributor

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1496256

There are two upstream commits here, 53135 was already merged to 3.3 - 3.7, 53989 fixes performance regression that was discovered after merge to 3.7 and earlier.

/assign @mfojtik

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 23, 2017
@openshift-merge-robot openshift-merge-robot added the vendor-update Touching vendor dir or related files label Nov 23, 2017
@mfojtik
Copy link
Contributor

mfojtik commented Nov 23, 2017

/approve
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 23, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jsafrane, mfojtik

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

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 23, 2017
@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 16538, 17430, 17432, 17433, 17442).

openshift-merge-robot added a commit that referenced this pull request Nov 23, 2017
Automatic merge from submit-queue (batch tested with PRs 16538, 17430, 17432, 17433, 17442).

Fix predicate counting

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1496256

There are two upstream commits here, 53135 was already merged to 3.3 - 3.7, 53989 fixes performance regression that was discovered after merge to 3.7 and earlier.

/assign @mfojtik
@openshift-merge-robot openshift-merge-robot merged commit 88a6a98 into openshift:master Nov 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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/L Denotes a PR that changes 100-499 lines, ignoring generated files. vendor-update Touching vendor dir or related files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants