Skip to content

Conversation

@gwynnemonahan
Copy link
Contributor

@gwynnemonahan gwynnemonahan commented Oct 15, 2024

OSSM 3.0 TP1

This content was not required for TP 1 go live, but OSSM 3.0 is still in TP1 status so this applies to TP1.

Merge to: https://github.com/openshift/openshift-docs/tree/service-mesh-docs-main

Cherry pick: to https://github.com/openshift/openshift-docs/tree/service-mesh-docs-3.0.0tp1

OSSM-8153 OSSM 3.x Concepts

This PR is part of the standalone doc set for the OpenShift Service Mesh project. Kathryn is aware that this content applies for a product that is part of a Technology Preview release. The project is seeking feedback from early adopters.

Version(s):

Technology Preview

OSSM 3.0 is moving to stand alone format and will not be cherry-picked back to OCP core branches.

Issue:
https://issues.redhat.com/browse/OSSM-8153

Link to docs preview:
https://83560--ocpdocs-pr.netlify.app/openshift-service-mesh/latest/about/ossm-about-concepts-assembly.html

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 15, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 15, 2024

🤖 Fri Nov 01 17:27:43 - Prow CI generated the docs preview:

https://83560--ocpdocs-pr.netlify.app/openshift-service-mesh/latest/about/ossm-about-concepts-assembly.html

@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 15, 2024
@gwynnemonahan gwynnemonahan changed the title OSSM 3.0 TP1 OSSM-8153 [DOC] OSSM 3.x Concepts [WIP] OSSM 3.0 TP1 OSSM-8153 [DOC] OSSM 3.x Concepts Oct 15, 2024
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 15, 2024
@gwynnemonahan gwynnemonahan changed the title [WIP] OSSM 3.0 TP1 OSSM-8153 [DOC] OSSM 3.x Concepts OSSM 3.0 TP1 OSSM-8153 [DOC] OSSM 3.x Concepts Oct 22, 2024
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 22, 2024
Comment on lines 72 to 79
== `IstioRevision` resource

The `IstioRevision` is a cluster-wide resource and the lowest-level API {SMProduct} provides. It is usually not created by the user, but by the operator itself. Its schema closely resembles that of the `Istio` resource - but instead of representing the state of a control plane you want to be present in your cluster, it represents a revision of that control plane.

A revision of the control plane you want to be present in your cluster is an instance of Istio with a specific version and revision name, and its revision name can be used to add workloads or entire namespaces to the mesh. For example: by using the `istio.io/rev=<REVISION_NAME>` label.

You can think of the relationship between the `Istio` and `IstioRevision` resources as similar to the relationship between Kubernetes' `ReplicaSet` and Pod: a `ReplicaSet` can be created by users and results in the automatic creation of Pods, which will trigger the instantiation of your containers.

Similarly, users create an `Istio` resource which instructs the {SMProduct} Operator to create a matching `IstioRevision`, which then in turn triggers the creation of the Istio control plane. To do that, the {SMProduct} Operator will copy all of your relevant configuration from the `Istio` resource to the `IstioRevision` resource.
Copy link

Choose a reason for hiding this comment

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

This section does a good job of explaining how the two resources relate to each other!

@longmuir
Copy link

I think we should try to draw more of a line around what is part of the OpenShift Service Mesh and what is an integration (not part of the service mesh offering). Simply re-ordering the content may achieve this.

Parts of Service Mesh:

  • Service Mesh resources
  • Kiali (note, Kiali has a CRDs called Kiali and OSSMConsole which we support - should we include them here too?)

Nots part of Service Mesh (integrations):

  • OpenShift Distributed tracing *
  • OpenShift Monitoring *
  • cert-manager
  • Argo Rollouts

*tracing and monitoring fall under OpenShift Observability, so I would probably put these together. It may also be that they have other features, such as the Cluster Observability operator that can be used with service mesh in the future, though the metrics, logs and tracing integrations described today will remain.

@gwynnemonahan
Copy link
Contributor Author

  • Kiali (note, Kiali has a CRDs called Kiali and OSSMConsole which we support - should we include them here too?)

It is part of the Kiali section https://83560--ocpdocs-pr.netlify.app/openshift-service-mesh/latest/about/ossm-about-concepts-assembly.html#ossm-about-concepts-kiali_ossm-about-concepts-assembly

@gwynnemonahan
Copy link
Contributor Author

Parts of Service Mesh:

  • Service Mesh resources
  • Kiali (note, Kiali has a CRDs called Kiali and OSSMConsole which we support - should we include them here too?)

Nots part of Service Mesh (integrations):

  • OpenShift Distributed tracing *
  • OpenShift Monitoring *
  • cert-manager
  • Argo Rollouts

These have been clarified in ossm-about-concepts-assembly.adoc

Copy link

@nrfox nrfox left a comment

Choose a reason for hiding this comment

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

Minor nit but otherwise LGTM

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.

@gwynnemonahan Looks good, just a few minor comments/suggestions.

@gwynnemonahan gwynnemonahan force-pushed the OSSM-8153 branch 2 times, most recently from 1abe14c to 54cadf0 Compare October 29, 2024 14:52
@gwynnemonahan
Copy link
Contributor Author

/retest

@gwynnemonahan
Copy link
Contributor Author

/label peer-review-needed

In the spirit of expediency, happy to jump on a call and review/update together.

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Oct 29, 2024
@gwynnemonahan
Copy link
Contributor Author

Rebased evening of 10/29/2024

+
Integrating {DTProductName} with {SMProductName} is made of up two parts: {TempoName} and {OTELName}.
+
{TempoName}:: Provides distributed tracing to monitor and troubleshoot transactions in complex distributed systems. It is based on the open source link:https://grafana.com/oss/tempo/[Grafana Tempo] project.
Copy link
Contributor

Choose a reason for hiding this comment

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

Per doc guidelines: When pointing to a link that exists external to Red Hat’s customer portal documentation, add the link attribute, and use the <document_heading_name> (<document_source>) format.

@mburke5678
Copy link
Contributor

@gwynnemonahan Mostly formatting suggestions, a few typo-level things. Otherwise, looks great!!

@mburke5678 mburke5678 added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR peer-review-needed Signifies that the peer review team needs to review this PR labels Oct 31, 2024
@gwynnemonahan
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Oct 31, 2024
@opayne1 opayne1 added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Nov 1, 2024
@opayne1 opayne1 added this to the Continuous Release milestone Nov 1, 2024
@opayne1
Copy link
Contributor

opayne1 commented Nov 1, 2024

@gwynnemonahan could you squash your commits? Thanks!

@gwynnemonahan
Copy link
Contributor Author

@opayne1 , my bad. Commit squashed, and also rebased.

@openshift-ci
Copy link

openshift-ci bot commented Nov 1, 2024

@gwynnemonahan: all tests passed!

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

Copy link
Contributor

@opayne1 opayne1 left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 1, 2024
@opayne1 opayne1 merged commit aa7c1f9 into openshift:service-mesh-docs-main Nov 1, 2024
2 checks passed
@opayne1
Copy link
Contributor

opayne1 commented Nov 1, 2024

/cherrypick service-mesh-docs-3.0.0tp1

@openshift-cherrypick-robot

@opayne1: new pull request created: #84371

In response to this:

/cherrypick service-mesh-docs-3.0.0tp1

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-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. merge-review-in-progress Signifies that the merge review team is reviewing this PR peer-review-done Signifies that the peer review team has reviewed this PR 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.

10 participants