Skip to content

Fix template chart to work with spiffe and with dapr 1.14.4#57

Merged
salaboy merged 4 commits into
dapr:mainfrom
luigirende:feature/dapr-1.14.4
Aug 22, 2025
Merged

Fix template chart to work with spiffe and with dapr 1.14.4#57
salaboy merged 4 commits into
dapr:mainfrom
luigirende:feature/dapr-1.14.4

Conversation

@luigirende
Copy link
Copy Markdown
Contributor

This PR change the chart template to fix the issue #56 and to be compatible with dapr 1.14.4. In fact the actual version of the chart when use the image tag to 1.14.4 there is an issue about the certs.

Signed-off-by: Luigi Rende <luigi.rende@assistdigital.com>
Comment thread chart/dapr-shared/templates/helmhooks.yaml Outdated
@salaboy
Copy link
Copy Markdown
Collaborator

salaboy commented Aug 19, 2025

@luigirende thanks for submitting this PR, we should update the charts to use the latest version too. Are you aiming to use 1.14.4?

@salaboy salaboy self-requested a review August 19, 2025 07:45
Copy link
Copy Markdown
Collaborator

@salaboy salaboy left a comment

Choose a reason for hiding this comment

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

@luigirende I am happy to merge this, if we remove the istio label.

Signed-off-by: luigirende <luigirende@gmail.com>
@luigirende
Copy link
Copy Markdown
Contributor Author

luigirende commented Aug 21, 2025

@salaboy removed the useless label about istio, sorry it has been a typo

@salaboy
Copy link
Copy Markdown
Collaborator

salaboy commented Aug 21, 2025

Thanks! I will merge and test

@luigirende
Copy link
Copy Markdown
Contributor Author

@luigirende thanks for submitting this PR, we should update the charts to use the latest version too. Are you aiming to use 1.14.4?

@salaboy I'm using this version for 1.15.3 version too

@salaboy
Copy link
Copy Markdown
Collaborator

salaboy commented Aug 21, 2025

@luigirende can you update this PR to include the version that you are using? merging for 1.14.x doesn't make too much sense

Signed-off-by: salaboy <Salaboy@gmail.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
@salaboy salaboy dismissed their stale review August 22, 2025 11:40

changes made

@salaboy salaboy merged commit aa40672 into dapr:main Aug 22, 2025
5 checks passed
@akotsarelation
Copy link
Copy Markdown
Contributor

Thanks @luigirende and @salaboy! Looks like the Helm unit-tests are failing and preventing the Helm chart from being published. I've created #61 to fix.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants