deepseek-ai/3FS
1
0
Fork 0
mirror of https://github.com/deepseek-ai/3FS.git synced 2026-11-05 00:47:47 +00:00
Code Issues 166 Projects Packages Wiki Activity

[PR #143] Resolve mgmtd address name #308

New issue
Open
opened 2026-10-26 09:48:32 +00:00 by bsakaguchi · 0 comments
bsakaguchi commented 2026-10-26 09:48:32 +00:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/deepseek-ai/3FS/pull/143
Author: @huww98
Created: 3/11/2025
Status: 🔄 Open

Base: main ← Head: named-address

📝 Commits (1)

  • 1c00672 Resolve mgmtd address name

📊 Changes

12 files changed (+147 additions, -24 deletions)

View changed files

📝 benchmarks/storage_bench/StorageBench.cc (+2 -2)
📝 benchmarks/storage_bench/StorageBench.h (+1 -1)
📝 src/client/mgmtd/MgmtdClient.cc (+15 -1)
📝 src/client/mgmtd/MgmtdClient.h (+2 -1)
➕ src/common/utils/NamedAddress.cc (+71 -0)
➕ src/common/utils/NamedAddress.h (+38 -0)
📝 tests/client/ClientWithConfig.h (+1 -1)
📝 tests/client/ServerWithConfig.h (+1 -1)
📝 tests/client/TestMgmtdClient.cc (+12 -13)
📝 tests/lib/UnitTestFabric.cc (+2 -2)
📝 tests/meta/MetaTestBase.h (+1 -1)
📝 tests/migration/TestMigrationService.cc (+1 -1)

📄 Description

Name resolution is very convenient for service discovery. We can just write the domain name in the config file and let external component to manage the DNS, e.g. Kubernetes coredns.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/deepseek-ai/3FS/pull/143 **Author:** [@huww98](https://github.com/huww98) **Created:** 3/11/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `named-address` --- ### 📝 Commits (1) - [`1c00672`](https://github.com/deepseek-ai/3FS/commit/1c00672322f7d6ed0f2a807af923c3059e5062da) Resolve mgmtd address name ### 📊 Changes **12 files changed** (+147 additions, -24 deletions) <details> <summary>View changed files</summary> 📝 `benchmarks/storage_bench/StorageBench.cc` (+2 -2) 📝 `benchmarks/storage_bench/StorageBench.h` (+1 -1) 📝 `src/client/mgmtd/MgmtdClient.cc` (+15 -1) 📝 `src/client/mgmtd/MgmtdClient.h` (+2 -1) ➕ `src/common/utils/NamedAddress.cc` (+71 -0) ➕ `src/common/utils/NamedAddress.h` (+38 -0) 📝 `tests/client/ClientWithConfig.h` (+1 -1) 📝 `tests/client/ServerWithConfig.h` (+1 -1) 📝 `tests/client/TestMgmtdClient.cc` (+12 -13) 📝 `tests/lib/UnitTestFabric.cc` (+2 -2) 📝 `tests/meta/MetaTestBase.h` (+1 -1) 📝 `tests/migration/TestMigrationService.cc` (+1 -1) </details> ### 📄 Description Name resolution is very convenient for service discovery. We can just write the domain name in the config file and let external component to manage the DNS, e.g. Kubernetes coredns. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
SF-Zhou-patch-1
No results found.
Labels
Clear labels   
pull-request
Mirrored from GitHub Pull Request

No labels
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
deepseek-ai/3FS#308
No description provided.