-
Notifications
You must be signed in to change notification settings - Fork 2
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
Development quantum storage #588
Conversation
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Show resolved
Hide resolved
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Outdated
Show resolved
Hide resolved
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Show resolved
Hide resolved
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Outdated
Show resolved
Hide resolved
|
||
```sh | ||
CONFIG=zstor-default.toml | ||
PASSWORD=$(openssl rand -base64 18) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Password can't be $(openssl rand -base64 18)
, I tried a few times and it failed, cause the password is too long.
WorkloadDeployError: WorkloadDeployError: Failed to deploy zdb with name zdb5 on node 159 due to: failed to create zdb namespace: failed to set password namespace 162-132876-zdb5 in 0-db: 2461494e-***: Argument too long.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could not reproduce this problem when deploying with tfcmd
. I even increased the password length to 30+ characters without issue.
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Show resolved
Hide resolved
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Outdated
Show resolved
Hide resolved
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Outdated
Show resolved
Hide resolved
|
||
### Deploy Data Zdbs | ||
|
||
This process is very similar to the deployment of the metadata back ends, with a few small changes to the scripts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is the difference between zdb metadata and zdbs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've explained a bit more about the difference between each, in terms of the options passed to tfcmd.
|
||
``` | ||
minimal_shards = 16 | ||
expected_shards = 20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this number should be lowered to 8 for this example to work.
Error: ZstorError { kind: Config, internal: Other(ConfigError { msg: "insufficient data backends, require at least 20, only found 8" }) }
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Outdated
Show resolved
Hide resolved
...mentation/system_administrators/advanced/quantum_safe_storage/quantum_safe_storage_deploy.md
Outdated
Show resolved
Hide resolved
|
||
```sh | ||
mkdir -p /mnt/qsfs | ||
mkdir -p /data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the use of this directory?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've explained what each is used for, along with adding a note that they can be changed.
Thanks for the review. Will discuss this with @scottyeager and let you know + update the PR. |
@mik-tf any updates ? |
Fixed some basic typos. Scott is checking on his side too. |
Hi @A-Harby, thanks for having a go with this and for your comments. I went ahead and resolved anything that was a simple fix. Let me know if any of my responses or changes don't make sense. |
Co-authored-by: Scott Yeager <[email protected]>
…ll_vm.md Co-authored-by: Scott Yeager <[email protected]>
synced new dev changes
Scott will create a new PR with the latest docs for QSFS. |
Related Issue
Work Done