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

[Helm,Gateway] devopsPassword and statusPassword wrong values.yaml path #1670

Closed
mladBlum opened this issue Jun 27, 2023 · 1 comment · Fixed by #1671
Closed

[Helm,Gateway] devopsPassword and statusPassword wrong values.yaml path #1670

mladBlum opened this issue Jun 27, 2023 · 1 comment · Fixed by #1671
Assignees
@thjaeckle
Labels
bug Helm chart
Milestone
3.3.2

Comments

@mladBlum
Copy link
Contributor

In templates/gateway-secret.yaml the devopsPassword and statusPassword are read from .Values.gateway.devopsPassword and .Values.gateway.statusPassword.

In the deployment/helm/ditto/values.yaml these two values are located at gateway.config.devops.devopsPassword and gateway.config.devops.statusPassword

As a result, the two passwords are always regenerated and the passed passwords are not used.
thjaeckle added a commit that referenced this issue Jun 27, 2023
@thjaeckle
#1670 fix wrong config path for pulling devops and status password fr…
02966ed 
…om helm values

Signed-off-by: Thomas Jäckle <thomas.jaeckle@beyonnex.io>
@thjaeckle thjaeckle mentioned this issue Jun 27, 2023
Merged
@thjaeckle
Copy link
Member

Thanks for the hint, will fix in #1671

@thjaeckle thjaeckle added the bug label Jun 27, 2023
@thjaeckle thjaeckle added this to the 3.3.1 milestone Jun 27, 2023
thjaeckle added a commit that referenced this issue Jun 27, 2023
@thjaeckle
#1670 fix wrong config path for pulling devops and status password fr…
a6253fc 
…om helm values

Signed-off-by: Thomas Jäckle <thomas.jaeckle@beyonnex.io>
@thjaeckle thjaeckle self-assigned this Jun 28, 2023
thjaeckle added a commit that referenced this issue Jun 28, 2023
@thjaeckle
Merge pull request #1671 from eclipse-ditto/bugfix/1670-devops-passwo…
54e25dd 
…rd-helm-chart

#1670 fix wrong config path for pulling devops and status password from helm values
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thjaeckle thjaeckle

Labels
bug Helm chart
Projects
Ditto Planning
Status: Done
Milestone
3.3.2
2 participants
@thjaeckle @mladBlum