Skip to content

Conversation

rh-tokeefe
Copy link
Contributor

@rh-tokeefe rh-tokeefe commented Apr 24, 2024

Version(s):
Service Mesh 3.0

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

Link to docs preview:
https://75094--ocpdocs-pr.netlify.app/openshift-service-mesh/latest/install/ossm-installing-openshift-service-mesh.html#about-istio-deployment

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 24, 2024

@rh-tokeefe: This pull request references OSSM-6347 which is a valid jira issue.

In response to this:

Version(s):

Issue:

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 24, 2024
@openshift-ci openshift-ci bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 24, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 24, 2024

@rh-tokeefe: This pull request references OSSM-6347 which is a valid jira issue.

In response to this:

Version(s): Service Mesh 3.0

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

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Apr 24, 2024

@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 24, 2024

@rh-tokeefe: This pull request references OSSM-6347 which is a valid jira issue.

In response to this:

Version(s):
Service Mesh 3.0

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

Link to docs preview:
https://75094--ocpdocs-pr.netlify.app/openshift-service-mesh/latest/install/ossm-installing-openshift-service-mesh.html#about-istio-deployment

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@rh-tokeefe rh-tokeefe changed the title [WIP] OSSM-6347: Deploying Istio [WIP] OSSM-6347: SM 3.0 Deploying Istio Apr 25, 2024
@rh-tokeefe rh-tokeefe force-pushed the OSSM-6347 branch 4 times, most recently from 87940eb to 5c299b6 Compare April 25, 2024 19:07

. Click *Create Project*.

. At the prompt, you must enter a name for the project in the Name field. For example, `istio-system`. The Operator deploys Istio to the project you specify. The other fields provide supplementary information and are optional.
Copy link

Choose a reason for hiding this comment

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

I think is not needed to add this information: The Operator deploys Istio to the project you specify. The other fields provide supplementary information and are optional.. In the next section, we already specify that we are going to create the resource in the namespace created before. Regarding the extra fields when you create the resource maybe is better to say this directly in the steps or in a Note in the section. Something like this:

Note: the `istio` resource definition contains supplementary information that is optional to be defined

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Made minor change here.

@fjglira
Copy link

fjglira commented Apr 26, 2024

lgtm, I only have small comments

@rh-tokeefe rh-tokeefe force-pushed the OSSM-6347 branch 3 times, most recently from 4ebb8eb to 4ac9c85 Compare April 26, 2024 19:16
@rh-tokeefe
Copy link
Contributor Author

rh-tokeefe commented Apr 29, 2024

@dgn I'm not sure what the prerequisites should be for these tasks. I'm assuming, similar to deploying the control plane in 2.x, it would be similar to this:

  • The Red Hat OpenShift Service Mesh Operator must be installed.

  • You are logged in to the OpenShift Container Platform web console as cluster-admin.

Is there anything else? I'm not sure if different permissions are needed for creating the project and creating the resource, but I would think not.

Also, for creating the resource, I can add an additional prereq that indicates that the project to which the resource gets deployed must already exist.

Can you think of anything else as far as prereqs?

Copy link

@dgn dgn left a comment

Choose a reason for hiding this comment

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

lgtm, just one comment

@dgn
Copy link

dgn commented Apr 29, 2024

@dgn I'm not sure what the prerequisites should be for these tasks. I'm assuming, similar to deploying the control plane in 2.x, it would be similar to this:

* The Red Hat OpenShift Service Mesh Operator must be installed.

* You are logged in to the OpenShift Container Platform web console as cluster-admin.

Those prerequisites are good!

Is there anything else? I'm not sure if different permissions are needed for creating the project and creating the resource, but I would think not.

Also, for creating the resource, I can add an additional prereq that indicates that the project to which the resource gets deployed must already exist.

Don't think that's needed as we explicitly explain how to create the projects

Can you think of anything else as far as prereqs?

Nope, LGTM

@rh-tokeefe rh-tokeefe force-pushed the OSSM-6347 branch 3 times, most recently from c9c665d to 95ad184 Compare April 30, 2024 15:00
@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 30, 2024

@rh-tokeefe: This pull request references OSSM-6347 which is a valid jira issue.

In response to this:

Version(s):
Service Mesh 3.0

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

Link to docs preview:
https://75094--ocpdocs-pr.netlify.app/openshift-service-mesh/latest/install/ossm-installing-openshift-service-mesh.html#about-istio-deployment

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@rh-tokeefe rh-tokeefe changed the title [WIP] OSSM-6347: SM 3.0 Deploying Istio OSSM-6347: SM 3.0 Deploying Istio Apr 30, 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 Apr 30, 2024
@gabriel-rh
Copy link
Contributor

n this example, `istio-system' is the name of the project.

Looks like a mismatch

Copy link

openshift-ci bot commented May 1, 2024

@rh-tokeefe: 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/test-infra repository. I understand the commands that are listed here.

Copy link
Contributor

@gabriel-rh gabriel-rh left a comment

Choose a reason for hiding this comment

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

LGTM

@gabriel-rh gabriel-rh merged commit 97e862b into openshift:service-mesh-docs-main May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. 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.

6 participants