-
Notifications
You must be signed in to change notification settings - Fork 100
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
Remove SUSE Liberty Linux Base Media section #3120
base: master
Are you sure you want to change the base?
Conversation
For Liberty Linux this is not necessary according to Ricardo. Signed-off-by: Dominik Gedon <[email protected]>
Signed-off-by: Dominik Gedon <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure if you need the media or not.
We tested the SLL migration with the liberate formula without the media and it worked. As far as I know, it's only needed if we want to install a new system. |
But that chapter is not exclusivly for liberty migration. Customers may want to perform a new installation. We remove now important Informationen |
Ok, I'll take a closer look! @Others - do not hesitate to share your thoughts and proposals to solve this dilemma. |
Maybe, a better place for client installation instructions is in the Autoinstallation chapter: https://documentation.suse.com/suma/5.0/en/suse-manager/client-configuration/autoinst-distributions.html. In the Now we must decide how we should fix the raised issue for the currents releases (5.x and 4.3.x) and then how we want to structure the chapters of the Client Config Guide in general. |
There is a lot of useful information here that is being removed that is important to understand the overall picture around Liberty. I don't think we need to merge this now and rather handle this later when we have some more time. I would suggest to change it back to draft so it doesn't get merged by mistake. |
modules/specialized-guides/pages/salt/salt-formula-liberate.adoc
Outdated
Show resolved
Hide resolved
Converted back to DRAFT. |
That is correct. That is why I asked Ricardo again here: https://suse.slack.com/archives/C02D12TNYLS/p1719324925029689:
|
Michael has a good point. I will not harm the customer by keeping this in here, and someone may need it. Lets keeps this section, and close this PR so as not merged and not risk merging it. We can remove it in the future If needed. |
I'll move my comment #3120 (comment) to our team board. Next time we can discuss how we can further improve the Client Config Guide. Thanks to Ornela, it looks nice and structure is generally good. But at some places, we either could add more details, exchange some sections, or need better intro texts (book, chapters, sections). |
Description
This PR
Target branches
Backport targets (edit as needed):
Links