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.

Sign up for GitHub

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

docs note: azurerm_cognitive_account, not all kind support storage block #24191

Merged
merged 1 commit into from
Jan 24, 2024
Merged

docs note: azurerm_cognitive_account, not all kind support storage block #24191

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions website/docs/r/cognitive_account.html.markdown
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,8 @@ A `storage` block supports the following:

* `identity_client_id` - (Optional) The client ID of the managed identity associated with the storage resource.

~> **NOTE:** Not all `kind` support a `storage` block. For example the `kind` `OpenAI` does not support it.
Copy link
Member

Choose a reason for hiding this comment

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

@zioproto is there some MSFT documentation that we can link here with a definitive list of what cognitive account kinds support storage? If not then I think we should rephrase this to be more specific

Suggested change
~> **NOTE:** Not all `kind` support a `storage` block. For example the `kind` `OpenAI` does not support it.
~> **NOTE:** Cognitive Accounts with `kind` `OpenAI` do not support a `storage` block.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@felixwa Are you able to help me with this ask ? Is it just OpenAI that does not support this ? Thanks

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@stephybun it seems there is not a MSFT documentation page to link that publishes this definitive list. Should we just keep the generic sentence ?

Not all `kind` support a `storage` block.

I added the OpenAI example because it is very popular and a lot of customers ask about what the storage block is for.

Please advice what changes are necessary to get this merged. Thanks


## Attributes Reference

In addition to the Arguments listed above - the following Attributes are exported:
Expand Down
Loading