-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix required service and host resource attributes #17348
Conversation
Hi @alanwest 👋 Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days. We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes). |
✅ Deploy Preview for docs-website-netlify ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@ally-sassman I reviewed this but I don't have the context on if there's a special publishing timing, but it should be good to go! |
...lemetry-integrations/opentelemetry/best-practices/opentelemetry-best-practices-resources.mdx
Show resolved
Hide resolved
...lemetry-integrations/opentelemetry/best-practices/opentelemetry-best-practices-resources.mdx
Outdated
Show resolved
Hide resolved
…rations/opentelemetry/best-practices/opentelemetry-best-practices-resources.mdx Co-authored-by: ally sassman <42753584+ally-sassman@users.noreply.github.com>
@ally-sassman I didn't notice this, but this mistake slipped in in this commit b55a6df#diff-47a622255b43c5e4fec6922acf3c224fa6e859b745dc80297b05d8d9f8818c5eL50-R51