Skip to content
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

OSSMDOC-337: Document how to access logs from Kiali. #39931

Merged
merged 1 commit into from Jan 6, 2022

Conversation

JStickler
Copy link
Contributor

@JStickler JStickler commented Dec 16, 2021

This PR

  • Rearranges the modules for a better flow.
  • Updates some topic headings for improved usability.
  • Updates the topics for accessing the Kiali and Jaeger consoles, renames the files, and updates the links in both the Metrics, Traces, and Logs assembly and the Troubleshooting assembly.
  • Replaces the “Accessing metrics and tracing data from the CLI” topic with a revised “Discovering console addresses“ topic
  • Replaces the “Viewing service mesh data” topic with the revised “Accessing the Kiali console” topic
  • Adds a NEW topic – Viewing metrics in the Kiali console
  • Adds a NEW topic – Viewing logs in the Kiali console
  • Adds the distributed tracing variables to the document attributes file.
  • Renames ossm-dist-trace to ossm-distr-tracing
  • Updates where modules are used, fixes a couple of typos, removes some extra spaces.

Preview is here ->https://deploy-preview-39931--osdocs.netlify.app/openshift-enterprise/latest/service_mesh/v2x/ossm-observability.html

Eng review - jshaughn
QE review - pbajjuri20, mattmahoneyrh
Peer review - stevsmit

@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 16, 2021
@netlify
Copy link

netlify bot commented Dec 16, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: 2a0b6e3

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/61d734a52ac3b60008ffbe91

😎 Browse the preview: https://deploy-preview-39931--osdocs.netlify.app

@JStickler
Copy link
Contributor Author

FYI @longmuir
@yxun
@tsze-redhat

Copy link

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JStickler Definite improvements. Some comments for you to look through.

modules/ossm-kiali-accessing-console.adoc Show resolved Hide resolved
modules/ossm-kiali-viewing-logs.adoc Outdated Show resolved Hide resolved
modules/ossm-kiali-viewing-logs.adoc Show resolved Hide resolved
modules/ossm-kiali-viewing-metrics.adoc Outdated Show resolved Hide resolved
modules/ossm-kiali-viewing-metrics.adoc Outdated Show resolved Hide resolved
modules/ossm-observability-cli.adoc Outdated Show resolved Hide resolved
modules/ossm-observability-visual.adoc Show resolved Hide resolved
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 22, 2021
@pbajjuri20
Copy link

LGTM!

@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 4, 2022
@JStickler JStickler force-pushed the OSSMDOC-337 branch 5 times, most recently from 6f44f61 to ca96a9e Compare January 5, 2022 19:32
modules/ossm-config-external-jaeger.adoc Outdated Show resolved Hide resolved
modules/ossm-jaeger-accessing-console.adoc Show resolved Hide resolved
modules/ossm-jaeger-accessing-console.adoc Outdated Show resolved Hide resolved
modules/ossm-kiali-viewing-logs.adoc Show resolved Hide resolved
modules/ossm-kiali-viewing-metrics.adoc Show resolved Hide resolved
modules/ossm-observability-cli.adoc Outdated Show resolved Hide resolved
modules/ossm-observability-visual.adoc Show resolved Hide resolved
modules/ossm-observability-visual.adoc Outdated Show resolved Hide resolved
modules/ossm-observability-visual.adoc Outdated Show resolved Hide resolved
modules/ossm-tutorial-bookinfo-install.adoc Show resolved Hide resolved
@JStickler JStickler merged commit 80fc599 into openshift:main Jan 6, 2022
@JStickler
Copy link
Contributor Author

/cherry-pick enterprise-4.6

@JStickler
Copy link
Contributor Author

/cherry-pick enterprise-4.7

@JStickler
Copy link
Contributor Author

/cherry-pick enterprise-4.8

@JStickler
Copy link
Contributor Author

/cherry-pick enterprise-4.9

@JStickler
Copy link
Contributor Author

/cherry-pick enterprise-4.10

@openshift-cherrypick-robot

@JStickler: new pull request created: #40312

In response to this:

/cherry-pick enterprise-4.6

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-cherrypick-robot

@JStickler: new pull request created: #40313

In response to this:

/cherry-pick enterprise-4.7

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-cherrypick-robot

@JStickler: new pull request created: #40314

In response to this:

/cherry-pick enterprise-4.8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-cherrypick-robot

@JStickler: new pull request created: #40315

In response to this:

/cherry-pick enterprise-4.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-cherrypick-robot

@JStickler: new pull request created: #40316

In response to this:

/cherry-pick enterprise-4.10

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.6 branch/enterprise-4.7 branch/enterprise-4.8 branch/enterprise-4.9 branch/enterprise-4.10 service-mesh Label for all Service Mesh PRs size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants