Page MenuHomeFreeBSD
Differential D59267

name: add cni-dnsname
ClosedPublic
Actions

Authored by adrian on Sat, Aug 29, 7:29 PM.
Tags
Referenced Files
Unknown Object (File)
Fri, Sep 18, 6:45 PM
Unknown Object (File)
Fri, Sep 18, 1:36 PM
Unknown Object (File)
Thu, Sep 17, 7:24 AM
Unknown Object (File)
Thu, Sep 17, 7:15 AM
Unknown Object (File)
Tue, Sep 15, 7:44 AM
Unknown Object (File)
Mon, Sep 14, 9:20 PM
Unknown Object (File)
Sun, Sep 13, 10:21 AM
Unknown Object (File)
Sat, Sep 12, 8:26 AM
View All 53 Files
Subscribers
arrowd
lwhsu

Details

Reviewers
arrowd
Group Reviewers
Ports Committers
Commits
R11:98dc52b70391: name: add cni-dnsname
Summary

this is a CNI plugin for podman containers which allows for
dynamic registration of containers into a local dns zone.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

adrian requested review of this revision.Sat, Aug 29, 7:29 PM
adrian created this revision.
Harbormaster completed remote builds in B76325: Diff 185419.Sat, Aug 29, 7:29 PM
lwhsu added a subscriber: lwhsu.Sun, Aug 30, 12:50 PM

Would it be more suitable to be in dns category?

adrian added a comment.Mon, Aug 31, 2:44 AM
In D59267#1359826, @lwhsu wrote:

Would it be more suitable to be in dns category?

not sure; its part of container infra rather than DNS infra and we don't specifically have a container infra top level directory. So I put it where podman lives (ie sysutils).

arrowd added a subscriber: arrowd.Sat, Sep 5, 12:23 PM
arrowd added inline comments.
sysutils/cni-dnsname/Makefile
5

Should be sysutils, possibly followed by net.

sysutils/cni-dnsname/distinfo
6

Re-run make makesum after changing the category:

=> go/sysutils_cni-dnsname/cni-dnsname-v1.3.1/v1.3.1.mod is not in /usr/ports/sysutils/cni-dnsname/distinfo.                       
=> Either /usr/ports/sysutils/cni-dnsname/distinfo is out of date, or
=> go/sysutils_cni-dnsname/cni-dnsname-v1.3.1/v1.3.1.mod is spelled incorrectly.
adrian updated this revision to Diff 185936.Sat, Sep 5, 4:34 PM

feedback from arrowd

Harbormaster completed remote builds in B76571: Diff 185936.Sat, Sep 5, 4:34 PM
adrian marked 2 inline comments as done.Sat, Sep 5, 4:34 PM

fixed, thanks!

adrian added projects: Contributor Reviewers (ports), Ports Committers.Sat, Sep 5, 4:35 PM
adrian added a reviewer: Ports Committers.
arrowd accepted this revision.Sat, Sep 5, 5:21 PM

LGTM
Macro lgtm: LGTM LGTM

LGTM

This revision is now accepted and ready to land.Sat, Sep 5, 5:21 PM
Closed by commit R11:98dc52b70391: name: add cni-dnsname (authored by adrian). · Explain WhySat, Sep 5, 5:36 PM
This revision was automatically updated to reflect the committed changes.
adrian added a commit: R11:98dc52b70391: name: add cni-dnsname.

Revision Contents
Changeset List

Diff 185939

View Options

sysutils/Makefile

Loading...
View Options

sysutils/cni-dnsname/Makefile

Loading...
View Options

sysutils/cni-dnsname/distinfo

Loading...
View Options

sysutils/cni-dnsname/files/patch-plugins_meta_dnsname_config.go

Loading...
View Options

sysutils/cni-dnsname/files/patch-plugins_meta_dnsname_files.go

Loading...
View Options

sysutils/cni-dnsname/files/patch-plugins_meta_dnsname_files__test.go

Loading...
View Options

sysutils/cni-dnsname/pkg-descr

Loading...