search

openshift / origin-server

OpenShift 2 (deprecated)
889 stars 516 forks source link

[TO_STAGE] Add tc support for multiple IFs via TRAFFIC_CONTORL_DEVS #6310

Closed joelsmith closed 8 years ago

joelsmith commented 8 years ago

Add a new configuration item, TRAFFIC_CONTORL_DEVS, which is a space-delimited list of interfaces that should be tc-managed.

Update code so that tc changes are applied across all devices that are to be traffic-controlled.

Also hard-code limits to very high numbers for the loopback interface if it is to be managed.

These commits have already been merged to master in #6298

wshearn commented 8 years ago

[merge]

openshift-bot commented 8 years ago

Online Merge Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/merge_pull_requests_generic/1197/) (Image: devenv-stage_1196)

joelsmith commented 8 years ago

[Test] again, please

openshift-bot commented 8 years ago

Evaluated for online test up to a51dee49cd0379886c79cdcb09da4b7a58de857d

Miciah commented 8 years ago

@joelsmith, looks like we need https://www.github.com/openshift/rhc/pull/703 in stage.

Miciah commented 8 years ago

Oh, looks like stage is already has that commit. The solution isn't that simple...

openshift-bot commented 8 years ago

Linked pull request 703 in repo 'rhc' is not mergeable

openshift-bot commented 8 years ago

Online Test Results: FAILURE (https://ci.dev.openshift.redhat.com/jenkins/job/test_pull_requests_generic/688/)

joelsmith commented 8 years ago

@Miciah, the master PR, #6298 failed the same tests during its first test run, then was re-tested (automatically?) by the bot over the weekend, and the tests passed that time, so it merged.

openshift-bot commented 8 years ago

Evaluated for online merge up to a51dee49cd0379886c79cdcb09da4b7a58de857d