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

Add vlan tag option for port #63

Merged
merged 4 commits into from
Jul 6, 2018
Merged

Add vlan tag option for port #63

merged 4 commits into from
Jul 6, 2018

Conversation

chenyunchen
Copy link
Contributor

No description provided.

@chenyunchen chenyunchen force-pushed the alex/AddVlanTagOption branch 4 times, most recently from cdae902 to dd67fd4 Compare July 5, 2018 10:37
@John-Lin John-Lin self-requested a review July 5, 2018 10:38
@chenyunchen chenyunchen force-pushed the alex/AddVlanTagOption branch 2 times, most recently from 753beb9 to f4272bc Compare July 5, 2018 15:42
@John-Lin
Copy link
Contributor

John-Lin commented Jul 5, 2018

Wait for protocobuf downgrade to 3.5.1 linkernetworks/network-controller#56

@codecov-io
Copy link

codecov-io commented Jul 6, 2018

Codecov Report

Merging #63 into develop will increase coverage by 0.27%.
The diff coverage is 85.71%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #63      +/-   ##
===========================================
+ Coverage    66.51%   66.78%   +0.27%     
===========================================
  Files           28       28              
  Lines          854      867      +13     
===========================================
+ Hits           568      579      +11     
- Misses         250      251       +1     
- Partials        36       37       +1
Impacted Files Coverage Δ
src/networkcontroller/network_controller.go 83.33% <85.71%> (+0.47%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78187da...1bf26f9. Read the comment docs.

@John-Lin John-Lin force-pushed the alex/AddVlanTagOption branch 2 times, most recently from 98e7d46 to 3d420ec Compare July 6, 2018 03:17
@John-Lin John-Lin force-pushed the alex/AddVlanTagOption branch from 3d420ec to a933218 Compare July 6, 2018 03:25
@John-Lin John-Lin requested review from hwchiu and John-Lin July 6, 2018 03:32
Copy link
Contributor

@hwchiu hwchiu left a comment

Choose a reason for hiding this comment

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

LGTM

@John-Lin John-Lin merged commit f586a31 into develop Jul 6, 2018
@John-Lin John-Lin deleted the alex/AddVlanTagOption branch July 6, 2018 06:00
John-Lin added a commit that referenced this pull request Jul 25, 2018
Add vlan tag option for port

Former-commit-id: 95e0847c6397ee185db5507e1d2dc4234daa31a1 [formerly f586a31]
Former-commit-id: a72496ff8b36c101b2432be1e8e9b88ea762d267
John-Lin added a commit that referenced this pull request Jul 25, 2018
Add vlan tag option for port

Former-commit-id: 95e0847c6397ee185db5507e1d2dc4234daa31a1 [formerly 95e0847c6397ee185db5507e1d2dc4234daa31a1 [formerly f586a31]]
Former-commit-id: a72496ff8b36c101b2432be1e8e9b88ea762d267
Former-commit-id: 00b8c79
John-Lin added a commit that referenced this pull request Jul 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants