Add Discover local servers#3059
Conversation
e6c5135 to
bae4bac
Compare
|
@SeBBBe, the release notes also mention "additional options for saving the output to a file in different formats and providing a token which can be used for bulk registration into Fleet Manager," as well as some limitations and configurations. Do you think they refer to the registration code as a token, and if so, is it true that the code is always one regardless of the number of servers? What if I want to monitor 3 out of my 5 local servers? Here are the quotes from the draft Release notes: "The server now includes the Neo4j discovery service which can be used to identify running Neo4j deployments on the network. See neo4j-admin section for details of how to use the client. See DOCS for details of limitations and configuration." "Added client utility for the Neo4j discovery service which can be used to identify running Neo4j deployments on the network. Basic usage is ‘neo4j-admin fleet discover’ which will provide a list to the terminal of discovered services. Additional options allow saving to a file in different formats and providing a token which can be used for bulk registration into Fleet Manager." |
I think it actually doesn't. The default output from the command is pretty print + the code. But it's also possible to get json or csv output, either to file or in the terminal. It doesn't seem like all the other commands have the complete list of optional parameters documented in the ops manual, so I didn't find the appropriate place this should be added. Or maybe I'm wrong? The help output from the command itself explains it pretty well:
Yes, this is accurate. The choice of which servers to monitor will be made in the Aura Console after the full code is pasted (UI flow: neo4j/docs-aura#991) |
|
@SeBBBe, thank you so much for the clarifications. I'll update the PR and create a separate page for the command. |
35957f6 to
0c89ba0
Compare
NataliaIvakina
left a comment
There was a problem hiding this comment.
I think everything looks good!
The test fails because of the missing periods in the code
|
The test failure should be fixed when we merge this PR https://github.com/neo-technology/neo4j/pull/37510. |
42d5e77 to
8dee238
Compare
26bb5f6 to
be6890f
Compare
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
No description provided.