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

router: Fix ingress handling of nil rule value #12941

Merged
merged 1 commit into from
Feb 15, 2017

Conversation

marun
Copy link
Contributor

@marun marun commented Feb 13, 2017

Addresses bz#1421632

cc: @openshift/networking

Copy link
Contributor

@knobunc knobunc left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the quick fix!

@knobunc
Copy link
Contributor

knobunc commented Feb 13, 2017

[test]

@JacobTanenbaum
Copy link
Contributor

LGTM

Copy link
Contributor

@pecameron pecameron left a comment

Choose a reason for hiding this comment

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

LGTM

@knobunc
Copy link
Contributor

knobunc commented Feb 13, 2017

[merge]

@knobunc
Copy link
Contributor

knobunc commented Feb 13, 2017

[test] It looks like it took more than 3 minutes for the pods to start.

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to dcacd84

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/171/) (Base Commit: fbdb78e)

@knobunc
Copy link
Contributor

knobunc commented Feb 14, 2017

[merge] last flaked on #12784

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to dcacd84

@openshift-bot
Copy link
Contributor

openshift-bot commented Feb 14, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/205/) (Base Commit: 2a570f8) (Image: devenv-rhel7_5912)

@openshift-bot openshift-bot merged commit 5724f25 into openshift:master Feb 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants