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.

Sign up for GitHub

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

Jump to bottom

Wrong path in identity-server-deployment.yaml #239

Closed
angeloimm opened this issue Jul 2, 2020 · 1 comment · Fixed by #246
Closed

Wrong path in identity-server-deployment.yaml #239

angeloimm opened this issue Jul 2, 2020 · 1 comment · Fixed by #246
Assignees
@chirangaalwis
Labels
Priority/High Type/Bug
Milestone
5.10.0.2

Comments

@angeloimm
Copy link

angeloimm commented Jul 2, 2020 •
edited
Loading

In the file https://github.com/wso2/kubernetes-is/blob/master/advanced/is-pattern-1/templates/is/identity-server-deployment.yaml we see a wrong log path location when central logging is enabled.
We have:

{ if .Values.wso2.centralizedLogging.enabled }}
        - name: shared-logs
          mountPath: /home/wso2carbon/wso2is-5.8.0/repository/logs/

I thin it should be

{ if .Values.wso2.centralizedLogging.enabled }}
        - name: shared-logs
          mountPath: /home/wso2carbon/wso2is-5.10.0/repository/logs/

Am I correct?
@chirangaalwis chirangaalwis added this to the 5.10.0.2 milestone Jul 7, 2020
@chirangaalwis
Copy link
Member

Related issue: #234

@chirangaalwis chirangaalwis self-assigned this Aug 4, 2020
@chirangaalwis chirangaalwis linked a pull request Aug 4, 2020 that will close this issue
[v5.10.x][Logging] Test Identity Server Deployment with Kubernetes Pipeline #246
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chirangaalwis chirangaalwis

Labels
Priority/High Type/Bug
Projects
None yet
Milestone
5.10.0.2
2 participants
@angeloimm @chirangaalwis