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
Update storage.md with Oracle Cloud Infrastructure provider #5778
Update storage.md with Oracle Cloud Infrastructure provider #5778
Conversation
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.
Thanks! Can you sign up the DCO?
d0fcb38
to
f55a41b
Compare
Sorry, forgot about it. |
docs/storage.md
Outdated
|
||
#### API Signing Key | ||
|
||
The default API signing key authentication provider leverages same [configuration as the OCI CLI](https://docs.oracle.com/en-us/iaas/Content/API/Concepts/cliconcepts.htm) which is usually stored in at `$HOME/.oci/config` or via variable names starting with the string `OCI_CLI`. You can also use environment variables that start with `TF_VAR`. If the same configuration is found in multiple places the provider will prefer the first one. |
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.
You can also use environment variables that start with
TF_VAR
.
This sounds funny to me. What does Thanos's OCI storage implementation have to do with Teratogen
Terraform?
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.
@squat
Where did you see Teratogen? Can you clarify? Thx!
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'll assume @squat had an autocorrect mishap, and he meant simply Terraform :), I assume he refers to why we're mentioning the TF_VAR
here.
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.
Signed-off-by: aaron.tam <aaron.tam@oracle.com>
f55a41b
to
555045a
Compare
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.
This looks good to me now, thanks @aarontams!
Changes
Issue: #5592
This PR sync up https://github.com/thanos-io/thanos/blob/main/docs/storage.md and https://github.com/thanos-io/objstore/blob/main/README.md. See issue for details.
Verification
This is just documentation sync up. No test is needed.