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

feat(swagger): extends sm agent /node_info with storage_size. #4170

Merged
merged 2 commits into from
Dec 17, 2024

Conversation

VAveryanov8
Copy link
Collaborator

@VAveryanov8 VAveryanov8 commented Dec 16, 2024

This extends /node_info agent response with storage_size - total disk size in bytes. It's needed for the backup manifest and should be calculated on the agent side.

Refs: #4130

p.s. I've realized that exposing DataDirectory is not enough, because storage_size can be calculated only on agent side, but backup manifest is created on the SM side.


Please make sure that:

  • Code is split to commits that address a single change
  • Commit messages are informative
  • Commit titles have module prefix
  • Commit titles have issue nr. suffix

This extends `/node_info` agent response with `storage_size` - total disk size
in bytes. It's needed for the backup manifest and should be calculated
on the agent side.

Refs: #4130
@VAveryanov8 VAveryanov8 changed the title feat(swagger): extends sm agent /node_info' with storage_size`. feat(swagger): extends sm agent /node_info with storage_size. Dec 16, 2024
@VAveryanov8 VAveryanov8 marked this pull request as ready for review December 16, 2024 16:19
@VAveryanov8 VAveryanov8 merged commit cafa851 into master Dec 17, 2024
51 checks passed
@VAveryanov8 VAveryanov8 deleted the va/extend-backup-manifest-part-2 branch December 17, 2024 16:11
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.

2 participants