search

sdnfv / openNetVM

A high performance container-based NFV platform from GW and UCR.
http://sdnfv.github.io/onvm/
Other
263 stars 136 forks source link

[Bug Fix] Fix Typo in Stats Header #142

Closed koolzz closed 5 years ago

koolzz commented 5 years ago

The header says rx_drop and tx_drop but its actually rx and tx stat, this pr fixes that.

Summary:

Yeaaaaa, can't believe I didn't notice this one during release review. First bug of 19.05, kudos to @Grace-Liu for helping find this one

Usage: Just test the stats output modes.

This PR includes
Resolves issues
Breaking API changes
Internal API changes
Usability improvements
Bug fixes 👍
New functionality
New NF/onvm_mgr args
Changes to starting NFs
Dependency updates
Web stats updates

Merging notes:

TODO before merging :

Test Plan:

Can someone look through the args and confirm everything else is in the right spot

Review:

Pls review

onvm commented 5 years ago

In response to PR creation

CI Message

Your results will arrive shortly

koolzz commented 5 years ago

@kevindweb can you review this one?

kevindweb commented 5 years ago

@onvm testing with another branch

onvm commented 5 years ago

@onvm testing with another branch

CI Message

Another CI run in progress, adding request to the end of the list

kevindweb commented 5 years ago

@onvm test over here

onvm commented 5 years ago

@onvm test over here

CI Message

Another CI run in progress, adding request to the end of the list

kevindweb commented 5 years ago

@onvm sorry for so many emails 2

onvm commented 5 years ago

@onvm sorry for so many emails 2

CI Message

Another CI run in progress, adding request to the end of the list

kevindweb commented 5 years ago

@onvm sfes

onvm commented 5 years ago

@onvm sfes

CI Message

Another CI run in progress, adding request to the end of the list

onvm commented 5 years ago

@onvm sorry for so many emails 2

CI Message

Your results will arrive shortly

kevindweb commented 5 years ago

@onvm queue me up

kevindweb commented 5 years ago

@onvm ignore me

kevindweb commented 5 years ago

@onvm are we going to work

onvm commented 5 years ago

@onvm are we going to work

CI Message

Duplicate request already waiting, ignoring message

kevindweb commented 5 years ago

@onvm test

onvm commented 5 years ago

@onvm test

CI Message

Another CI run in progress, adding request to the end of the list

onvm commented 5 years ago

@onvm are we going to work

CI Message

Your results will arrive shortly

koolzz commented 5 years ago

@onvm whats the deal with CI rn

onvm commented 5 years ago

@onvm whats the deal with CI rn

CI Message

Your results will arrive shortly