Page MenuHomeFreeBSD
Differential D50847

net/containernetworking-plugins: use correct family during rule generation
ClosedPublic
Actions

Authored by dch on Sat, Jun 14, 5:09 PM.
Tags
None
Referenced Files
F121856522: D50847.id157055.diff
Mon, Jun 30, 1:30 AM
F121856513: D50847.id157055.diff
Mon, Jun 30, 1:30 AM
Unknown Object (File)
Fri, Jun 27, 4:52 PM
Unknown Object (File)
Wed, Jun 25, 8:43 PM
Unknown Object (File)
Tue, Jun 24, 5:43 PM
Unknown Object (File)
Mon, Jun 23, 3:10 PM
Unknown Object (File)
Fri, Jun 20, 7:35 AM
Unknown Object (File)
Fri, Jun 20, 7:20 AM
View All 16 Files
Subscribers
dfr

Details

Reviewers
dfr
dch
Summary

PR: 287196
Tested by: Bretton Vine <bv@honeyguide.eu>
Sponsored by: SkunkWerks, GmbH

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 64883
Build 61766: arc lint + arc unit

Event Timeline

dch requested review of this revision.Sat, Jun 14, 5:09 PM
dch created this revision.
Harbormaster completed remote builds in B64865: Diff 157024.Sat, Jun 14, 5:09 PM
dch updated this revision to Diff 157053.Sat, Jun 14, 10:34 PM

fix family alignment in rule generation and skip cross-family rules

Harbormaster completed remote builds in B64882: Diff 157053.Sat, Jun 14, 10:34 PM
dch updated this revision to Diff 157054.Sat, Jun 14, 10:53 PM

tidy up patch

Harbormaster completed remote builds in B64883: Diff 157054.Sat, Jun 14, 10:53 PM
dch updated this revision to Diff 157055.Sat, Jun 14, 11:00 PM

fix map error

Harbormaster completed remote builds in B64884: Diff 157055.Sat, Jun 14, 11:00 PM
dfr accepted this revision.Mon, Jun 16, 2:55 PM
dfr added a subscriber: dfr.

Looks good to me and works in my testing. It would be helpful if you could also make a pull request for github.com/dfr/plugins which is the upstream for this port (my fork of the CNI plugins).

This revision is now accepted and ready to land.Mon, Jun 16, 2:55 PM
dch accepted this revision.Tue, Jun 17, 10:05 AM

Let's upstream this first and then I can just bump the port.

What is the branch from your https://github.com/dfr/plugins ?
There's no matching branch compared to what the ports tree fetches

{F120368428}

dfr added a comment.Tue, Jun 17, 12:37 PM
In D50847#1161816, @dch wrote:

Let's upstream this first and then I can just bump the port.

What is the branch from your https://github.com/dfr/plugins ?
There's no matching branch compared to what the ports tree fetches

{F120368428}

It's the freebsd branch - the port is fetching the freebsd-v0.5 tag which somehow wasn't pushed to the freebsd branch (it was present in my local git repo). Not sure how it was fetchable from github - I think I must have pushed the tag up to github but not the branch. It should make sense now that I have pushed the branch.

dch closed this revision.Wed, Jun 18, 10:58 PM

fixed in d2fd973c7d34

Revision Contents
Changeset List

PathSize
net/
containernetworking-plugins/
2 lines
files/
61 lines

Diff 157054

View Options

net/containernetworking-plugins/Makefile

Loading...
View Options

net/containernetworking-plugins/files/patch-plugins_freebsd_meta_portmap_portmap.go

Loading...