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

salaboy commented Aug 19, 2025

Copy link
Copy Markdown
Collaborator

@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

@salaboy salaboy left a comment

Copy link
Copy Markdown
Collaborator

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

luigirende commented Aug 21, 2025

Copy link
Copy Markdown
Contributor Author

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

@salaboy

salaboy commented Aug 21, 2025

Copy link
Copy Markdown
Collaborator

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

salaboy commented Aug 21, 2025

Copy link
Copy Markdown
Collaborator

@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