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

node pools: replicate from authoritative region #17456

Merged
merged 2 commits into from
Jun 12, 2023
Merged

Conversation

tgross
Copy link
Member

@tgross tgross commented Jun 7, 2023

Upserts and deletes of node pools are forwarded to the authoritative region, just like we do for namespaces, quotas, ACL policies, etc. Replicate node pools from the authoritative region.

Copy link
Contributor

@lgfa29 lgfa29 left a comment

Choose a reason for hiding this comment

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

LGTM! Some minor comments that are more questions 😅

go s.replicateNamespaces(stopCh)
go s.replicateNodePools(stopCh)
Copy link
Contributor

Choose a reason for hiding this comment

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

While definitively not ideal in a real cluster do namespaces and node pools require ACLs to be enabled? I was bitten by this a few time while testing things and not seeing replication happening 😅

Copy link
Member Author

@tgross tgross Jun 12, 2023

Choose a reason for hiding this comment

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

Technically no, because the replication token will just be the zero value empty string. But if you check out the federation tutorial and ACL tutorial it really looks like we consider federation and replication as two different problems. You can have federation without replication and that just forwards requests around. But the purpose of replication seems entirely focused on ACLs. Namespaces are arguably ACL objects, and in the ENT case so are Node Pools. So I think we should probably keep the behavior we have here with namespaces.

nomad/leader.go Show resolved Hide resolved
Upserts and deletes of node pools are forwarded to the authoritative region,
just like we do for namespaces, quotas, ACL policies, etc. Replicate node pools
from the authoritative region.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/node-pools Issues related to node pools type/enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants