Skip to content
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

Fix Module Stream check for Orace Linux #10743

Merged
merged 1 commit into from Oct 3, 2023
Merged

Conversation

sbernhard
Copy link
Member

@sbernhard sbernhard commented Sep 18, 2023

The operatingsystem_name for Oracle is "Oracle Linux Server" and not "OracleLinux".
See e.g. https://github.com/theforeman/foreman/blob/develop/test/unit/katello/rhsm_fact_parser_test.rb#L98

@theforeman-bot
Copy link

Issues: #35906

@jeremylenz
Copy link
Member

Oh my, seems this needs a rebase! (RIP my Redmine inbox)

@sbernhard
Copy link
Member Author

Oh my, seems this needs a rebase! (RIP my Redmine inbox)

Oh sorry. My fault. Hope your inbox is still alive :-)

Yes of course. Just saw the diff after the rebase. Now it should be fine.

@sbernhard
Copy link
Member Author

Checks passed @jeremylenz :-)

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

ACK, looks good and works correctly on my test oracle box.

@chris1984 chris1984 merged commit ca25ba5 into Katello:master Oct 3, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants