New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 660787 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature

Blocking:
issue 634529
issue 642811



Sign in to add a comment

Add traffic-shaping kernel modules in M55, till network bandwidth throttling lands in M-56

Project Member Reported by kirtika@chromium.org, Oct 31 2016

Issue description

Forking this off from crbug.com/634529 (or crbug.com/642811).
The feature lands in M56. We would like to provide enterprise customers the ability to test out the feature in dev-mode. 
This will need at a minimum, the kernel CL to be included in M-55: https://chromium-review.googlesource.com/#/c/385167/

After that, we have two options:
(a) (Preferred?) Provide enterprise customers with a bunch of tc commands or a shell script to run from dev-mode crosh.
(b) Merge the shill CL into M-55 as well, and then provide a single command (dbus-send)to run from dev-mode crosh. 

Creating this bug simply because we need a bug ID for the merge-requested label. 


 

Comment 2 by snanda@chromium.org, Oct 31 2016

I might've missed it but are the customers wanting to try out this feature in M55 using the tc commands?  Why wouldn't they use dev-channel M56 instead?
Owner: kirtika@chromium.org
Status: WontFix (was: Untriaged)
My bad, I didn't realize that was an option. 
Closing as WontFix. 

Comment 4 by dimu@chromium.org, Nov 1 2016

Labels: -Merge-Request-55 Merge-Approved-55 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M55 (branch: 2883)
Project Member

Comment 5 by sheriffbot@chromium.org, Nov 4 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 6 by sheriffbot@chromium.org, Nov 7 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by kirtika@google.com, Nov 15 2016

Labels: -Hotlist-Merge-Approved -Merge-Approved-55

Sign in to add a comment