jvictorsoto / tc-wrapper

NodeJS wrapper for tc command (traffic control on linux)
MIT License
12 stars 3 forks source link

refactor to better limit ingress #2

Open mrbar42 opened 7 years ago

mrbar42 commented 7 years ago

might worth bumping a major version as the underlying limit mechanism had changed.

image

mrbar42 commented 7 years ago

@jvictorsoto ping

jvictorsoto commented 6 years ago

@mrbar42 Sorry, I had notifications off in this account. I saw that you did another commit on your branch, add to the pull request that commit too for reviewing it.

mrbar42 commented 6 years ago

The rest of the commits are still in progress. this can be reviewed meanwhile

jvictorsoto commented 6 years ago

@mrbar42 Ok, I'll review it this weekend. Love the e2e tests using docker btw.