networkservicemesh / networkservicemesh/api

Do we need to add the possibility to confgure static DNS entries?

Open
#92 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

question
Dominant language
Go
Stars
18
Forks
22
Avg merge
27m
Merged PRs (30d)
3

Description

Description

Currently, we support DNSConfigs related to the concrete DNS servers.
https://github.com/networkservicemesh/api/blob/master/pkg/api/networkservice/connectioncontext.proto#L66-L71.

Probably it can be used to add an additional config into https://github.com/networkservicemesh/api/blob/master/pkg/api/networkservice/connectioncontext.proto#L73
That will represent static DNS entry.

message StaticEntry {
    string domain = 1;
    string ip = 2;
}
message DNSContext {
    repeated DNSConfig configs = 1;
  + repeated StaticEntry static_entries = 2;
}

It can be used to configure DNS static entries on the client-side. Probably it is useful for external applications that using DNS domains inside.

Question

Is it make sense?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading pkg/api/networkservice/connectioncontext.proto around lines 66–73, including DNSConfig and DNSContext. Clarify whether static DNS entries are needed and what schema and compatibility requirements are accepted; the issue is done when the project has a decided API change or a documented decision not to make one.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
api, networking
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.