Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Truenas deprecated #486

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.

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

Merged
merged 6 commits into from
Nov 7, 2024
Merged

Conversation

ctag
Copy link
Contributor

@ctag ctag commented Sep 29, 2024

Removes now deprecated installation instructions for Truenas/Truecharts.
https://truecharts.org/news/scale-deprecation/

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Remove deprecated Truenas/Truecharts installation notes.
Remove deprecated Truenas/Truecharts installation notes.
Remove deprecated Truenas/Truecharts installation notes.
Copy link

semanticdiff-com bot commented Sep 29, 2024

Review changes with  SemanticDiff

@joseluisq
Copy link
Collaborator

Thanks for the reminder. It seems like the Helm charts are still being maintained. https://github.com/truecharts/charts/tree/master/charts/stable/static-web-server

So, does this mean that TrueCharts is dead?
No, TrueCharts is not dead. We have always made Helm Charts and our Charts can be installed on any Helm-compatible OS, not just TrueNAS Scale. We continue to provide these Helm Charts as always and they can already be installed on a compatible OS.
https://truecharts.org/news/scale-faq/

Maybe we have to make this clear in the docs instead of removing the entry?

@joseluisq joseluisq added documentation Improvements or additions to documentation v2 v2 release labels Sep 29, 2024
@ctag
Copy link
Contributor Author

ctag commented Sep 29, 2024

That's good at least, but helm charts aren't directly usable as Truenas installation material. Possibly just rename that section away from Truenas Scale.

For context, installing apps on Scale through the truecharts catalog is now fully broken. See: https://www.reddit.com/r/truenas/comments/1e1dxvk/truecharts_just_nuked_everything_in_response_to/
So even though the helm charts are maintained, they aren't usable for Scale.

@joseluisq
Copy link
Collaborator

For context, installing apps on Scale through the truecharts catalog is now fully broken. See: https://www.reddit.com/r/truenas/comments/1e1dxvk/truecharts_just_nuked_everything_in_response_to/
So even though the helm charts are maintained, they aren't usable for Scale.

Fair, then let's remove any TrueNAS Scale mention and adjust the section accordingly.

Copy link
Collaborator

@joseluisq joseluisq left a comment

Choose a reason for hiding this comment

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

I just left a minor adjustment to make.
After that, we should merge the PR.

Copy link
Collaborator

@joseluisq joseluisq left a comment

Choose a reason for hiding this comment

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

Thanks!

@joseluisq joseluisq added this pull request to the merge queue Nov 7, 2024
@joseluisq joseluisq removed this pull request from the merge queue due to a manual request Nov 7, 2024
@joseluisq joseluisq merged commit cd11bd6 into static-web-server:master Nov 7, 2024
@joseluisq joseluisq added this to the v2.34.0 milestone Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation v2 v2 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants