This message was deleted.
# generala
adamant-kite-43734
08/30/2024, 7:57 PMThis message was deleted.
f
future-fountain-82544
08/30/2024, 8:01 PMHere's an example of what I wound up seeing while doing an upgrade...
future-fountain-82544
08/30/2024, 8:04 PMI know the mistake I made in the upgrade process. It just seems like it's a very easy thing to do so I wanted to bring it to the attention of the chart maintainers.
And fwiw, those screenshots are with the the longhorn/charts added as a repo in the Rancher UI.
future-fountain-82544
08/30/2024, 8:17 PMI do know that the Rancher chart devs did some work to ensure that
longhorn-crdlonghornb
bland-article-62755
08/31/2024, 2:44 AMIf anything I'd open it with a PR against the longhorn charts Repo to add wording on the Readme as an adequate warning to use the official packaged charts if you're using Rancher.
s
stocky-account-63046
09/02/2024, 7:46 AMUnfortunately it's a known issue. In some situations where there are multiple similar charts in different repos, the one wrong is selected https://github.com/rancher/dashboard/issues/9613
f
future-fountain-82544
09/03/2024, 1:58 PMI wound up posting our details on the rancher/dashboard issue.
future-fountain-82544
09/03/2024, 2:01 PMI might still open a README update PR, but I feel like that's not going to be a huge help because it doesn't get in the user's face during the upgrade process. They have to wander off the shortest-path to actually see a message like that.
b
bland-article-62755
09/03/2024, 2:29 PMI haven't been around that long, but you're the only user I've heard about installing from the repo and overriding the built in charts and sources, so I feel like that's where the wandering off the path actually occurred. It sounds like mixing the charts, not just upgrading is what broke things. ¯\_(ツ)_/¯
f
future-fountain-82544
09/03/2024, 2:35 PMYup - mixing charts is indeed what happened, but my point is that the Rancher UI encourages it
future-fountain-82544
09/03/2024, 2:37 PMAnd maybe I am the only idiot that clicked through the UI to do an upgrade and thought "huh, the versions look different -- guess that's OK, Rancher has my back" but maybe I should just not expect that to be the case.
b
bland-article-62755
09/03/2024, 2:41 PMI don't know if the proper logic can exist to give the warning you're looking for in the UI, but you could potentially head off the problem of chart mixing in the README.md in the chart repository and avoid it that way.
f
future-fountain-82544
09/03/2024, 2:53 PMThat's a fair statement.
And for what it's worth, I'm not "overriding the built in charts and sources". I installed Longhorn using that project's instructions and the Rancher UI offered an upgrade which I walked through. I didn't do anything to override a source. I just followed the UI cues like a normal person would.
b
bland-article-62755
09/03/2024, 3:00 PMYeah I totally get that. At that point the Longhorn Project is a third party repo and not the cultivated built in system repo for Rancher (where they do testing and verify upgrades etc). I have no idea how to you program the logic for adequate warnings with third party repos.
f
future-fountain-82544
09/03/2024, 3:11 PMGiuseppe Leo mentioned this PR: https://github.com/rancher/dashboard/pull/10180
s
stocky-account-63046
09/03/2024, 3:15 PMWe're considering scaling back the feature, given increased chance of false positives (users are now more familiar with helm, adding more repos, there are more alternative repos containing common charts, etc). Unfortunately that won't be until at least 2.10 is out
f
future-fountain-82544
09/03/2024, 3:15 PMthat's understandable
5 Views