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

Fixes #36723 - Remove Fedora/Red Hat Atomic related code #9823

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Sep 6, 2023

As far as I can see this has been EOL for a few years now. Red Hat Atomic went EOL in 2021.

@theforeman-bot
Copy link
Member

Issues: #36723

@stejskalleos
Copy link
Contributor

@nofaralfasi

@@ -146,8 +143,6 @@ def get_rhsm_mac(interface)
end

def distribution_to_puppet_os(name)
return REDHAT_ATOMIC_HOST_OS if name == REDHAT_ATOMIC_HOST_DISTRO_NAME

name = name.downcase
Copy link
Member Author

Choose a reason for hiding this comment

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

I noticed this block could be cleaner using a case statement: #9838

@nofaralfasi
Copy link
Contributor

Hi @ekohl, can you please resolve the merge conflicts to I can rebase this PR onto it?

@stejskalleos
Copy link
Contributor

@ekohl any updates here? it's blocking the #9837 which is for upcoming release and would be nice to have it ready for testing ASAP

As far as I can see this has been EOL for a few years now. Red Hat
Atomic went EOL in 2021.
Copy link
Contributor

@stejskalleos stejskalleos left a comment

Choose a reason for hiding this comment

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

🍏 LGTM

@stejskalleos stejskalleos merged commit 4e205ba into theforeman:develop Oct 4, 2023
9 checks passed
@stejskalleos
Copy link
Contributor

Thanks @ekohl

@ekohl ekohl deleted the 36723-drop-atomic branch October 4, 2023 08:59
@ekohl
Copy link
Member Author

ekohl commented Oct 4, 2023

Release note: theforeman/foreman-documentation#2486

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants