Closed lzchen closed 5 months ago
Learn Build status updates of commit c9790c4:
File | Status | Preview URL | Details |
---|---|---|---|
docs-ref-services/preview/monitor-opentelemetry-exporter-readme.md | :warning:Warning | View (azure-python-preview) | Details |
Missing required attribute: 'title'. Add a title string to show in search engine results.
Invalid value for 'ms.service': 'monitor'.
Link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python#telemetry-types' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Missing required attribute: 'description'.
Absolute link 'https://learn.microsoft.com/azure/azure-monitor/app/opentelemetry-enable?tabs=python' will be broken in isolated environments. Replace with a relative link.
Absolute link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python' will be broken in isolated environments. Replace with a relative link.
Absolute link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python#telemetry-types' will be broken in isolated environments. Replace with a relative link.
Absolute link 'https://learn.microsoft.com/azure/azure-monitor/app/azure-ad-authentication?tabs=python' will be broken in isolated environments. Replace with a relative link.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
Learn Build status updates of commit 95b1e05:
File | Status | Preview URL | Details |
---|---|---|---|
docs-ref-services/preview/monitor-opentelemetry-exporter-readme.md | :warning:Warning | View (azure-python-preview) | Details |
Missing required attribute: 'title'. Add a title string to show in search engine results.
Invalid value for 'ms.service': 'monitor'.
Link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python#telemetry-types' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Missing required attribute: 'description'.
Absolute link 'https://learn.microsoft.com/azure/azure-monitor/app/opentelemetry-enable?tabs=python' will be broken in isolated environments. Replace with a relative link.
Absolute link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python' will be broken in isolated environments. Replace with a relative link.
Absolute link 'https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-overview?tabs=python#telemetry-types' will be broken in isolated environments. Replace with a relative link.
Absolute link 'https://learn.microsoft.com/azure/azure-monitor/app/azure-ad-authentication?tabs=python' will be broken in isolated environments. Replace with a relative link.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
@scbedd can this be reviewed?
Hey @lzchen these changes look good, but unfortunately what will happen is on the very next release, your changes will be wiped out!
The readme file you changed in this repo is auto-populated from this readme file when we release a new version of the azure-monitor-opentelemetry-exporter
package.
I will gladly sign off this review after you merge a PR to the repo w/ these same changes.
CC @danieljurek @JimSuplizio
@scbedd
Thanks for the headsup. I am already making the same changes in that README file in this [pr](https://github.com/Azure/azure-sdk-for-python/pull/34462. If this doc is indeed populated from that README file then I can close this PR.
Add excerpt to help users decide on whether to use the distro or exporter. Also added data collection basics as a prerequisite read.