-
Notifications
You must be signed in to change notification settings - Fork 107
5.0 - Added SLES 16 and openSUSE Leap 16 to support matrix #4553
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
base: manager-5.0
Are you sure you want to change the base?
Conversation
juliogonzalez
left a comment
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 only see Leap SLE16 being added, outside of the matrixes, and only for MLM. Not for Uyuni.
We don't have, for example, the name of the channels for Leap 16 on MLM. Not sure if Leap 16 s in SCC or not.
In any case I can't validate this PR fully, as I did not try neither SLE16 nor Leap 16 on MLM, and on Uyuni I did only Leap 16.0 on Alpha (and support on Uyuni is still missing as Abid knows).
Matrix looks to be correct, the other stuff I can't tell. Maybe Orion knows more as they did the sumatoolbox changes, etc.
In any case be aware the changes for Uyuni are missing. Uyuni doesn't use SCC for Leap, and doesn't use SCC for SLE client tools (but things are not ready yet, so doing it for MLM for now is OK, as long as @admd has a card for Uyuni later).
It's available in SCC. |
Then such documentation seems to be missing here? |
|
Also remember: SCC is for MLM, so you need either ifs, or manage different adocs. I can't remember how this is is done nowadays. For the channels, someone needs to look at SCC or talk to Orion. I didn't test Leap 16.0 from SCC :) |
| | {check} | ||
| | {check} | ||
| | {check} | ||
| | {check} |
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.
Product migration isn't supported for now so you can cross it.
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.
Will do.
| | {check} | ||
| | {check} | ||
| | {check} | ||
| | {check} |
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.
We don't have scap content for it available yet either so change them to question mark
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.
Will do.
Description
Adding new clients to support matrix.
Target branches
Links