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

Drop tree.service config parameter #511

Open
KirillovDenis opened this issue Jun 6, 2022 · 2 comments
Open

Drop tree.service config parameter #511

KirillovDenis opened this issue Jun 6, 2022 · 2 comments
Labels
blocked Can't be done because of something config Configuration format update or breaking change I3 Minimal impact S3 Minimally significant tree-service Implementing tree-service features U4 Nothing urgent

Comments

@KirillovDenis
Copy link
Contributor

Currently, the tree service in node listens the same port as the object service so we can drop the tree.service config param.

But we have to think about something like a pool for tree-service clients in s3. To start we can use the first node endpoint from the peers config section

@alexvanin
Copy link
Contributor

I expect that tree-service will be a part of NeoFS API one day, then it will be a part of the SDK Client and SDK Pool, so the issue will be solved. Until then, I think manual tree service endpoint specification is better than having random picking from the list of pool endpoints.

@alexvanin
Copy link
Contributor

Blocked by SDK adoption.

@roman-khimov roman-khimov added U4 Nothing urgent config Configuration format update or breaking change S3 Minimally significant I3 Minimal impact and removed triage U3 Regular labels Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Can't be done because of something config Configuration format update or breaking change I3 Minimal impact S3 Minimally significant tree-service Implementing tree-service features U4 Nothing urgent
Projects
None yet
Development

No branches or pull requests

4 participants