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

[enterprise-4.6] Issue in file service_mesh/v2x/ossm-security.adoc #27619

Closed
nicop311 opened this issue Nov 24, 2020 · 5 comments · Fixed by #30936
Closed

[enterprise-4.6] Issue in file service_mesh/v2x/ossm-security.adoc #27619

nicop311 opened this issue Nov 24, 2020 · 5 comments · Fixed by #30936
Assignees
Labels
issue-burndown Indicates an issue worked on during an "GitHub Issue Burndown" session service-mesh Label for all Service Mesh PRs

Comments

@nicop311
Copy link

nicop311 commented Nov 24, 2020

Hi, there might be a wrong path for the location of the certificates when "Adding an external certificate authority key and certificate".

Suggestion: Replace /etc/certs/root-cert.pem by /var/run/secrets/istio/root-cert.pem in the doc.

Which section(s) is the issue in?

  • Direct link
  • Path : Documentation > OpenShift Container Platform > [4.6] > Service Mesh > Service Mesh 2.x > Security > Adding an external certificate authority key and certificate > Verifying your certificates

What needs fixing?

The path of the volume where the root-cert.pem certificate is stored on the istio-proxy containers of the Bookinfo application seems to be wrong.

In the doc (as of 2020-11-24), we can see that the path is /etc/certs/root-cert.pem. However in my OpenShift 4.6 testbed, the path is /var/run/secrets/istio/root-cert.pem.

The path /var/run/secrets/istio/root-cert.pem is configured in the following file: on a istio-proxy (envoy) container /etc/istio/proxy/envoy-rev0.json.

$ more /etc/istio/proxy/envoy-rev0.json

[...]
              "validation_context": {
                "trusted_ca": {
                  "filename": "./var/run/secrets/istio/root-cert.pem"
                },
                "match_subject_alt_names": [{"exact":"istiod-full-install.istio-system.svc"}]
              }
[...]

My testbed software version

Red Hat OpenShift

$ oc version

# result
Client Version: 4.6.4
Server Version: 4.6.4
Kubernetes Version: v1.19.0+9f84db3

Red Hat Service Mesh, Maistra

$ oc get operator servicemeshoperator.openshift-operators -o yaml
[...]
servicemeshoperator.v2.0.0.2
[...]

Bookinfo: I take the example from Maistra Github, with no modification.

git clone https://github.com/maistra/istio.git
cd istio

git branch

# result
* [maistra-2](https://issues.redhat.com/browse/maistra-2).0
oc apply -f istio/samples/bookinfo/platform/kube/bookinfo.yaml
[...]etc....[...]
@openshift-bot
Copy link

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci-robot openshift-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Feb 22, 2021
@openshift-bot
Copy link

Stale issues rot after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle rotten
/remove-lifecycle stale

@openshift-ci-robot openshift-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Mar 24, 2021
@jeana-redhat jeana-redhat added issue-burndown Indicates an issue worked on during an "GitHub Issue Burndown" session and removed lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. labels Mar 25, 2021
@jeana-redhat
Copy link
Contributor

Hi @nicop311 - does this apply to 4.6 only, or to subsequent versions as well?

@JStickler
Copy link
Contributor

@jeana-redhat, there were quite a few changes in 4.6, but it should be the same for subsequent versions.

@JStickler JStickler added the service-mesh Label for all Service Mesh PRs label Mar 25, 2021
@jeana-redhat
Copy link
Contributor

thanks, Julie!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue-burndown Indicates an issue worked on during an "GitHub Issue Burndown" session service-mesh Label for all Service Mesh PRs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants