Update observability best practices #8897
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
google-cla
bot
added
the
cla: yes
istio-testing
added
the
size/S
ericvn
reviewed
Feb 3, 2021
| Istio no longer ships with Prometheus in its default profile. To enable Prometheus please see | ||
| the [Prometheus integration](../../../ops/integrations/prometheus/) addon documentation. | ||
|
|
||
| While the `Quick Start` installation is well-suited for small clusters and monitoring for short time horizons, |
There was a problem hiding this comment.
Not sure what Quick Start is. Maybe you are referring the the Getting Started installation. Or we could say default configuration profile.
There was a problem hiding this comment.
I was referencing https://istio.io/latest/docs/ops/integrations/prometheus/#option-1-quick-start which is mentioned in the link to the Prometheus integration. The phrasing could be better.
There was a problem hiding this comment.
Should we call it sample install ?
There was a problem hiding this comment.
We no longer install Prometheus in the default nor in the demo profile. For the users to test prometheus they have to follow the addons integration page. It's not difficult, but no longer an option for testing out the functionality easily.
There was a problem hiding this comment.
I'm wondering if this paragraph would be a better tip/info on the prometheus add-on page. We do have a very small blurb there.
There was a problem hiding this comment.
Moved to the addons page for Prometheus. Combined Frank's suggestion too.
frankbu
reviewed
Feb 4, 2021
Co-authored-by: Frank Budinsky <frankb@ca.ibm.com>
ericvn
approved these changes
Feb 17, 2021
|
In response to a cherrypick label: new pull request created: #8993 |
istio-testing
added a commit
that referenced
this pull request
Feb 25, 2021
* Update customize prom scraping instruction. (#8976) * Update customize prom scraping instruction. * Update content/en/docs/ops/integrations/prometheus/index.md Co-authored-by: Sven Mawson <sven@google.com> Co-authored-by: Sven Mawson <sven@google.com> * IstioCon blog post (#8984) Co-authored-by: Frank Budinsky <frankb@ca.ibm.com> * s/service-apis/gateway-api/ (#8988) * Update index.md * Update index.md * Revert "remove rbac instruction (#8442)" (#8990) This reverts commit a65a859. * Update observability best practices (#8897) * Update observability best practices * Fix linting issue * Try and clarify prometheus install * Update content/en/docs/ops/best-practices/observability/index.md Co-authored-by: Frank Budinsky <frankb@ca.ibm.com> * Move to observability page Co-authored-by: Frank Budinsky <frankb@ca.ibm.com> * Add doc about how to work around missing metric expiry. (#8948) * Add doc about how to work around missing metric expiry. * address comment. * lint * add spelling change * fix * Update content/en/faq/metrics-and-logs/telemetry-v1-vs-v2.md Co-authored-by: Douglas Reid <douglas-reid@users.noreply.github.com> Co-authored-by: Douglas Reid <douglas-reid@users.noreply.github.com> * Add initial security best practices documentation (#8952) * Clarify Prometheus TLS settings. (#8962) * Clarify Prometheus TLS settings. * Update content/en/docs/ops/integrations/prometheus/index.md Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> * Update content/en/docs/ops/integrations/prometheus/index.md Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> * Update content/en/docs/ops/integrations/prometheus/index.md Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> * Automator: update common-files@master in istio/istio.io@master (#8997) * Automator: update istio.io@ reference docs (#8998) * Automator: update istio.io@ reference docs (#9004) * Make attribute gen yaml file valid. (#9000) * Fix the client IP addresses for the authz ingress task (#9002) * Fix link to Configuration title (#9009) Signed-off-by: Radim Hrazdil <rhrazdil@redhat.com> * Automator: update istio.io@ reference docs (#9019) * Automator: update istio@ test reference (#9021) * Use istio-ecosystem wasm extensions repo in extensibility concept page. (#9018) * add a troubleshooting guide for multicluster (#8957) * add a troubleshooting guide for multicluster * fix meta * fix meta * address review comments * shift weight * rel link * lint * fix link * hard to tell what our mdlint customizations are... * fix mc guide link * add more context to high-level issues * cleanup phrasing * Remove fixed limitation warning (#9034) This issue no longer exists, I verified via the code and tested it myself as well. * Fixed "are is" to "are" and addressed an incorrect link (#9035) * fixing a typo * changed the link to go directly to canary upgrades page * Automator: update istio.io@ reference docs (#9036) * fix circuit breaker task (#9022) * fix circuit breaker task * gen * Add documentation for Analysis messsage IST0134 ServiceEntryAddresses… (#9020) * Add documentation for Analysis messsage IST0134 ServiceEntryAddressesRequired Signed-off-by: zufardhiyaulhaq <zufardhiyaulhaq@gmail.com> * Apply suggestions from code review Co-authored-by: Ram Vennam <rvennam@us.ibm.com> * Update content/en/docs/reference/config/analysis/ist0134/index.md Co-authored-by: Ram Vennam <rvennam@us.ibm.com> Co-authored-by: Ram Vennam <rvennam@us.ibm.com> * Add blog for zero configuration Istio (#9025) * Add blog for zero configuration Istio The intent here is to show off what Istio provides out of the box, to attempt to counteract some of the reputation Istio has gotten for being over complicated/requiring too many CRDs. * fix links * Address comments * Fix examples for newer kubectl (#9045) * Fix istio.io tests when moving to later kubectl (#9046) * Automator: update istio.io@ reference docs (#9047) * Ignore error on first kiali apply (#9048) * Ignore some errors (#9049) * Fix syntax on local rate limiting (#9044) * Add cross references to virtual machine docs (#8913) * Add cross references to virtual machine docs * Sven's suggestions * Update content/en/docs/ops/diagnostic-tools/virtual-machines/index.md Co-authored-by: Sven Mawson <sven@google.com> Co-authored-by: Sven Mawson <sven@google.com> * update authz troubleshoot common problems (#9043) * update authz troubleshoot common problems * update * Add out-of-mesh server metadata info into telemetry v2 faq. (#9017) * Add out-of-mesh server metadata info into telemetry v2 faq. * Update content/en/faq/metrics-and-logs/telemetry-v1-vs-v2.md Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> * reword Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> * Release notes for 1.7.8 (#9054) * Announce EOL for Istio 1.7 (#9005) * Release notes for 1.7.7 * Update content/en/news/releases/1.7.x/announcing-1.7.7/index.md Co-authored-by: Brian Avery <bavery@redhat.com> * update * update * Announce EOL for Istio 1.7 * delay to 02-25 * Delete index.md Co-authored-by: Brian Avery <bavery@redhat.com> Co-authored-by: Pengyuan Bian <bianpengyuan@google.com> Co-authored-by: Sven Mawson <sven@google.com> Co-authored-by: Istio Automation <istio-testing-bot@google.com> Co-authored-by: Frank Budinsky <frankb@ca.ibm.com> Co-authored-by: craigbox <craigbox@google.com> Co-authored-by: John Howard <howardjohn@google.com> Co-authored-by: jacob-delgado <jacob.delgado@volunteers.acasi.info> Co-authored-by: Douglas Reid <douglas-reid@users.noreply.github.com> Co-authored-by: Eric Van Norman <ericvn@us.ibm.com> Co-authored-by: lei-tang <32078630+lei-tang@users.noreply.github.com> Co-authored-by: Radim Hrazdil <32546791+rhrazdil@users.noreply.github.com> Co-authored-by: Steven Landow <steven@stlcomputerservices.com> Co-authored-by: Kang-Bae <59033920+Kang-Bae@users.noreply.github.com> Co-authored-by: masquee <okayanz@outlook.com> Co-authored-by: Zufar Dhiyaulhaq <zufardhiyaulhaq@gmail.com> Co-authored-by: Ram Vennam <rvennam@us.ibm.com> Co-authored-by: Ryan Baker <ryan.baker@c2fo.com> Co-authored-by: Yangmin Zhu <ymzhu@google.com> Co-authored-by: Jimmy Chen <28548492+JimmyCYJ@users.noreply.github.com> Co-authored-by: Brian Avery <bavery@redhat.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Istio no longer ships with Prometheus. Reference the Prometheus addon.
[ ] Configuration Infrastructure
[X] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure