-
Notifications
You must be signed in to change notification settings - Fork 157
4553 - GCS and Azure inference logging #536
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
base: main
Are you sure you want to change the base?
4553 - GCS and Azure inference logging #536
Conversation
Signed-off-by: cory-johannsen <[email protected]>
Signed-off-by: cory-johannsen <[email protected]>
…cj/4553-inference-logging
✅ Deploy Preview for elastic-nobel-0aef7a ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docs/model-serving/predictive-inference/logger/blob-storage-logger.md
Outdated
Show resolved
Hide resolved
docs/model-serving/predictive-inference/logger/blob-storage-logger.md
Outdated
Show resolved
Hide resolved
| When specifying the logger configuration you must specify the bucket url for the data that will be stored. The URL protocol is used to determine the cloud storage: | ||
| - `s3://` - S3-compatible | ||
| - `abfs://` - Azure |
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.
Is this the correct url prefix?
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.
Azure supports multiple url prefixes; adfs (azure directory file store) is a supported option. The default for azure urls is https, but that is indistinguishable from a standard HTTPS upstream. We need a way to clearly identify the Azure URLs. I am happy to consider alternatives.
Signed-off-by: cory-johannsen <[email protected]>
Proposed Changes
Fixes #4553