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

dynamic host volumes: send register to client for fingerprint #24802

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

tgross
Copy link
Member

@tgross tgross commented Jan 7, 2025

When we register a volume without a plugin, we need to send a client RPC so that the node fingerprint can be updated. The registered volume also needs to be written to client state so that we can restore the fingerprint after a restart.

@tgross tgross added this to the 1.10.0 milestone Jan 7, 2025
@tgross tgross force-pushed the dhv-register-client-rpc branch from 12bac9b to 4e59820 Compare January 7, 2025 20:42
@tgross tgross force-pushed the dhv-register-client-rpc branch from 4e59820 to 21158f4 Compare January 7, 2025 21:14
@tgross tgross force-pushed the dhv-register-client-rpc branch from 21158f4 to fa258c8 Compare January 7, 2025 21:22
@tgross tgross force-pushed the dhv-register-client-rpc branch from fa258c8 to c8bfea0 Compare January 7, 2025 21:36
@tgross tgross force-pushed the dhv-register-client-rpc branch from c8bfea0 to a10b429 Compare January 7, 2025 21:48
@tgross tgross force-pushed the dhv-register-client-rpc branch from a10b429 to 6ac78c6 Compare January 7, 2025 21:57
@tgross tgross marked this pull request as ready for review January 7, 2025 21:59
@tgross tgross requested review from a team as code owners January 7, 2025 21:59
@tgross tgross force-pushed the dhv-register-client-rpc branch from 6ac78c6 to 64726b5 Compare January 8, 2025 13:45
gulducat
gulducat previously approved these changes Jan 8, 2025
Copy link
Member

@gulducat gulducat left a comment

Choose a reason for hiding this comment

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

lgtm!

client/host_volume_endpoint_test.go Outdated Show resolved Hide resolved
client/hostvolumemanager/host_volumes.go Outdated Show resolved Hide resolved
client/hostvolumemanager/host_volumes.go Outdated Show resolved Hide resolved
client/hostvolumemanager/host_volumes.go Outdated Show resolved Hide resolved
client/hostvolumemanager/host_volumes_test.go Outdated Show resolved Hide resolved
client/hostvolumemanager/host_volumes_test.go Outdated Show resolved Hide resolved
client/structs/host_volumes.go Outdated Show resolved Hide resolved
command/volume_delete_host_test.go Show resolved Hide resolved
demo/hostvolume/no-plugin.volume.hcl Outdated Show resolved Hide resolved
tgross and others added 3 commits January 8, 2025 14:00
When we register a volume without a plugin, we need to send a client RPC so that
the node fingerprint can be updated. The registered volume also needs to be
written to client state so that we can restore the fingerprint after a restart.
@tgross tgross force-pushed the dhv-register-client-rpc branch from ba35c05 to 252f99d Compare January 8, 2025 19:05
Copy link
Member

@gulducat gulducat left a comment

Choose a reason for hiding this comment

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

looks great!

@tgross tgross merged commit 4a65b21 into main Jan 8, 2025
32 checks passed
@tgross tgross deleted the dhv-register-client-rpc branch January 8, 2025 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants