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

Release 21.1.0 #430

Merged
merged 1 commit into from
Nov 5, 2021
Merged

Release 21.1.0 #430

merged 1 commit into from
Nov 5, 2021

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Oct 24, 2021

Pending release PRs:

@ekohl
Copy link
Member

ekohl commented Oct 29, 2021

I'd like #432 to get in before we release. I'm not sure what the expected compatibility is with Katello versions in the 21.x line though.

@ekohl
Copy link
Member

ekohl commented Oct 29, 2021

I still think we need to know if this needs modification:


Either it's redundant and we can drop the whole block or it needs to be configurable now that OSTree is back again.
(guess why I submitted #432)

@ehelms
Copy link
Member Author

ehelms commented Oct 29, 2021

I still think we need to know if this needs modification:

Either it's redundant and we can drop the whole block or it needs to be configurable now that OSTree is back again.
(guess why I submitted #432)

@jlsherrill can you comment on this for us? Is this entry being re-used and is it needed for the new ostree work?

@jlsherrill
Copy link
Contributor

jlsherrill commented Nov 2, 2021

oh all of those content_type definitions can go away now ! We figure it out dynamically

EDIT: and to be specific, the entire section can be removed

@ekohl
Copy link
Member

ekohl commented Nov 2, 2021

Great, since which version?

@jlsherrill
Copy link
Contributor

since katello 4.2 https://projects.theforeman.org/issues/31616

@ekohl
Copy link
Member

ekohl commented Nov 2, 2021

I've opened #433 and theforeman/foreman-installer#730 to reflect that.

@ehelms
Copy link
Member Author

ehelms commented Nov 5, 2021

Updated

@ehelms ehelms merged commit d786563 into theforeman:master Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants