jpictl: introduce `jpictl dns show` command #27

Merged

amery merged 4 commits from pr-amery-dns-show into main 1 year ago

amery commented 1 year ago

Owner

There is no content yet.

amery added 3 commits 1 year ago

b0f4be7047 dns: refactor GetRecords() to allow commands other than sync

Signed-off-by: Alejandro Mery <amery@jpi.io>

c578990f8c jpictl: refactor dns command initialization

Signed-off-by: Alejandro Mery <amery@jpi.io>

440dcde50a jpictl: introduce `jpictl dns show` command

to list dns entries

Signed-off-by: Alejandro Mery <amery@jpi.io>

amery requested review from karasz 1 year ago

karasz requested changes 1 year ago

karasz left a comment

Owner

please update the darvaza resolver version, v0.5.4 i sincompatible with core v0.10.0 as core v0.10.0 does not have Wrapf function anymore.

👍 1

amery added 1 commit 1 year ago

66178923a3 chore: update darvaza.org/resolver to support the darvaza.org/core update

Signed-off-by: Alejandro Mery <amery@jpi.io>

amery requested review from karasz 1 year ago

karasz approved these changes 1 year ago

karasz left a comment

Owner

LGTM now, thank you.

🚀 1

amery merged commit 134606207d into main 1 year ago

amery referenced this issue from a commit 1 year ago

Merge pull request 'jpictl: introduce `jpictl dns show` command' (#27)

amery deleted branch pr-amery-dns-show 1 year ago

Reviewers

karasz approved these changes 1 year ago

The pull request has been merged as 134606207d.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.