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

Private Link steps #509

Merged
merged 4 commits into from
Oct 28, 2024
Merged

Private Link steps #509

merged 4 commits into from
Oct 28, 2024

Conversation

fiquick
Copy link
Contributor

@fiquick fiquick commented Oct 25, 2024

No description provided.

@fiquick
Copy link
Contributor Author

fiquick commented Oct 25, 2024

Hey @AlexicaWright 😊 what terminology should I use instead of "nodes" here?

@fiquick fiquick requested a review from AlexicaWright October 25, 2024 14:56
@fiquick
Copy link
Contributor Author

fiquick commented Oct 25, 2024

Copy link
Collaborator

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few small comments.

The private ingress then directs the connection to the Aura instance with dbid: abcd1234.
. Finally, the write transaction is received and executed within the Aura instance with dbid: abcd1234.

.Available nodes and their roles
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Available instances? They are cluster members, aren't they? But we haven't yet surfaced much of clustering terminology so probably best to steer away from that.

@fiquick fiquick requested a review from AlexicaWright October 28, 2024 10:23
@neo-technology-commit-status-publisher
Copy link
Collaborator

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

@fiquick fiquick merged commit 9a5ccfd into main Oct 28, 2024
4 checks passed
@fiquick fiquick deleted the privatelinksteps branch October 28, 2024 10:30
@fiquick
Copy link
Contributor Author

fiquick commented Oct 30, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants