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

next: Changes queue for v2017.09 release #1222

Merged
merged 24 commits into from Nov 10, 2017

Conversation

Projects
None yet
8 participants
@lukego
Member

lukego commented Sep 8, 2017

Opening next for the new merge window...

asumu and others added some commits Apr 20, 2017

Merge pull request #1202 from Igalia/snabbwall
Merge recent snabbwall changes to next for v2017.08
Fix Unix socket EOF handling
Signed-off-by: Ben Agricola <bagricola@squiz.co.uk>
@lukego

This comment has been minimized.

Show comment
Hide comment
@lukego

lukego Nov 2, 2017

Member

So! I have been away on vacation and I am back now. Should we cut these changes immediately as the 2017.11 release and move on directly towards landing the new code on wingo-next and other PRs? what say you @eugeneia @wingo and all?

Member

lukego commented Nov 2, 2017

So! I have been away on vacation and I am back now. Should we cut these changes immediately as the 2017.11 release and move on directly towards landing the new code on wingo-next and other PRs? what say you @eugeneia @wingo and all?

@eugeneia

This comment has been minimized.

Show comment
Hide comment
@eugeneia

eugeneia Nov 3, 2017

Member

I am fine with either way! 👍

Member

eugeneia commented Nov 3, 2017

I am fine with either way! 👍

@wingo

This comment has been minimized.

Show comment
Hide comment
@wingo

wingo Nov 6, 2017

Contributor

@lukego wingo-next contains pretty solid intel_mp changes from @takikawa and Nicola, only that there was one regression on i210 cards (iirc) that needs sorting out (FAILED: ./test_1g_sw_sem.snabb). I would like to get that fixed and merged this week then I will get a lwaftr upstreaming sent upstream; we just finished a development cycle and so have a bunch of things to send. If you need to make a Snabb release before merging wingo-next (#1231) that's fine too.

Contributor

wingo commented Nov 6, 2017

@lukego wingo-next contains pretty solid intel_mp changes from @takikawa and Nicola, only that there was one regression on i210 cards (iirc) that needs sorting out (FAILED: ./test_1g_sw_sem.snabb). I would like to get that fixed and merged this week then I will get a lwaftr upstreaming sent upstream; we just finished a development cycle and so have a bunch of things to send. If you need to make a Snabb release before merging wingo-next (#1231) that's fine too.

@lukego

This comment has been minimized.

Show comment
Hide comment
@lukego

lukego Nov 8, 2017

Member

@eugeneia I suggest let's ship this branch now. That will reduce the backlog of changes and also get SnabbBot happy with master thanks to @wingo's fix to the lpm4 selftest case. Then we can focus on getting wingo-next merged and releasing that.

Member

lukego commented Nov 8, 2017

@eugeneia I suggest let's ship this branch now. That will reduce the backlog of changes and also get SnabbBot happy with master thanks to @wingo's fix to the lpm4 selftest case. Then we can focus on getting wingo-next merged and releasing that.

@eugeneia eugeneia merged commit e64d1a6 into master Nov 10, 2017

2 checks passed

SnabbDoc Documentation as single HTML file
Details
davos-eugeneia/snabb-nfv-test-vanilla Linux davos 4.4.11 x86_64 Intel(R) Xeon(R) CPU E5-2603 v2 @ 1.80GHz / eugeneia/snabb-nfv-test-vanilla
Details

eugeneia added a commit that referenced this pull request Nov 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment