-
Notifications
You must be signed in to change notification settings - Fork 167
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
[synthetics] Update TLS certificate rule doc to include Synthetics #4750
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status 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.
I left a few minor editing suggestions, otherwise LGTM.
Co-authored-by: Arianna Laudazzi <[email protected]>
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.
🦭 👍 🔥
…4750) * update tls rule doc to include synthetics * apply suggestions from code review Co-authored-by: Arianna Laudazzi <[email protected]> --------- Co-authored-by: Arianna Laudazzi <[email protected]> (cherry picked from commit a18b7e9)
…4750) * update tls rule doc to include synthetics * apply suggestions from code review Co-authored-by: Arianna Laudazzi <[email protected]> --------- Co-authored-by: Arianna Laudazzi <[email protected]> (cherry picked from commit a18b7e9)
…4750) * update tls rule doc to include synthetics * apply suggestions from code review Co-authored-by: Arianna Laudazzi <[email protected]> --------- Co-authored-by: Arianna Laudazzi <[email protected]> (cherry picked from commit a18b7e9)
…4750) (#4756) * update tls rule doc to include synthetics * apply suggestions from code review Co-authored-by: Arianna Laudazzi <[email protected]> --------- Co-authored-by: Arianna Laudazzi <[email protected]> (cherry picked from commit a18b7e9) Co-authored-by: Colleen McGinnis <[email protected]>
…4750) (#4755) * update tls rule doc to include synthetics * apply suggestions from code review Co-authored-by: Arianna Laudazzi <[email protected]> --------- Co-authored-by: Arianna Laudazzi <[email protected]> (cherry picked from commit a18b7e9) Co-authored-by: Colleen McGinnis <[email protected]>
…4750) (#4757) * update tls rule doc to include synthetics * apply suggestions from code review Co-authored-by: Arianna Laudazzi <[email protected]> --------- Co-authored-by: Arianna Laudazzi <[email protected]> (cherry picked from commit a18b7e9) Co-authored-by: Colleen McGinnis <[email protected]>
Description
Updates the TLS certificate rule doc to include both the Uptime TLS rule and the Synthetics TLS certificate rule, and updates the format to match the Monitor status rule doc.
Documentation sets edited in this PR
Check all that apply.
docs/en/observability/*
)docs/en/serverless/*
)docs/en/integrations/*
)Related issue
Closes #4745
Checklist
Follow-up tasks
Select one.