Skip to content
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

Remove dcs.edcs.network-server configuration #6048

Open
4 tasks done
johanstokking opened this issue Feb 3, 2023 · 0 comments
Open
4 tasks done

Remove dcs.edcs.network-server configuration #6048

johanstokking opened this issue Feb 3, 2023 · 0 comments
Labels
bump/major Needs new major version for release technical debt Not necessarily broken, but could be done better/cleaner

Comments

@johanstokking
Copy link
Member

Summary

Remove dcs.edcs.network-server configuration

Current Situation

This configuration contains only two deprecated fields: home-ns-id and hostname.

Why do we need this? Who uses it, and when?

We don't need them anymore as they are superseded by dcs.edcs.ns-id and dcs.edcs.as-id respectively.

Proposed Implementation

Remove configuration and remove falling back to these deprecated settings.

Contributing

  • I can help by doing more research.
  • I can help by implementing the feature after the proposal above is approved.
  • I can help by testing the feature before it's released.

Code of Conduct

@johanstokking johanstokking added technical debt Not necessarily broken, but could be done better/cleaner bump/major Needs new major version for release labels Feb 3, 2023
@github-actions github-actions bot added the needs/triage We still need to triage this label Feb 3, 2023
@johanstokking johanstokking removed the needs/triage We still need to triage this label Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump/major Needs new major version for release technical debt Not necessarily broken, but could be done better/cleaner
Projects
None yet
Development

No branches or pull requests

1 participant