Skip to content

Conversation

@slotdawg
Copy link
Contributor

Kasten K10 rebranded in January of 2024 to Veeam Kasten, with a new logo. I have updated our partner chart entry to reflect this new name as well as update the product logo.

@slotdawg slotdawg requested a review from a team as a code owner May 29, 2025 21:14
ChartMetadata:
kubeVersion: '>= 1.17.0-0'
icon: https://docs.kasten.io/_static/logo-kasten-k10-blue-white.png
icon: https://kastenmarketplace.s3.us-west-2.amazonaws.com/veeam-kasten-logo-square_white_bg.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi @slotdawg, it is good to set this here. However, when bin/partner-charts-ci update runs, it does not update the icon if it finds one at the assets/icons/k10.png. This file exists, so the icon will not be updated with only these changes.

Please update assets/icons/k10.png in this PR with the one you want to use - that will make existing and future versions of this chart use the proper icon. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps the whole chart needs to be moved from kasten/k10 to veeam/kasten. @slotdawg would this be more correct with the new branding?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@troytop it would be more correct with the new branding, however our helm chart still uses the "k10" moniker (i.e. k10-8.0.0 is our latest chart and it's referred to as kasten/k10 in our documentation). So long story short, unless you guys disagree, I think we should keep the chart/folder structure as-is to align with "k10" in the product itself, versus "veeam kasten" for the branding.

Copy link
Contributor

Choose a reason for hiding this comment

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

Works for me to leave it where it is. As @adamkpickering says though, we'll need you to update assets/icons/k10.png

@slotdawg
Copy link
Contributor Author

slotdawg commented Jun 3, 2025

All good - PR has been updated with the new logo :-) 91540bf

@adamkpickering adamkpickering merged commit bc5a51b into rancher:main-source Jun 5, 2025
1 check passed
j-travis pushed a commit to kasmtech/partner-charts that referenced this pull request Nov 4, 2025
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