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

Refactor repo handling #660

Merged
merged 1 commit into from
Aug 16, 2018
Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Aug 13, 2018

  • Drop use of the 'stable' repository
  • Disallow releases/ as part of the repository
  • Add foreman-rails repository
  • Stop including foreman::params
  • Remove anchor pattern
  • Move to https

Will need #659 for the acceptance tests to pass.

Note that the foreman-rails repository assumes Foreman 1.17+ so that'll become the new minimum version. I'm going to continue with removing some compatibility with older versions in follow up PRs.

@ekohl
Copy link
Member Author

ekohl commented Aug 13, 2018

I'll need to split out a few more things, work on tests and reorder a bit. Then I'll need to verify if there are more version specific workarounds we can get rid of. Also thinking about removing the explicit failures about an unsupported OS and setting values to undef so people can override where needed.

@ekohl ekohl changed the title [WIP] Refactor repo handling Refactor repo handling Aug 15, 2018
@mmoll
Copy link
Contributor

mmoll commented Aug 15, 2018

tests fail.

@ekohl ekohl force-pushed the refactor-repo-handling branch 3 times, most recently from 34803fb to d6df712 Compare August 15, 2018 18:44
@mmoll
Copy link
Contributor

mmoll commented Aug 15, 2018

this needs a rebase :)

* Drop use of the 'stable' repository
* Disallow releases/ as part of the repository
* Add foreman-rails repository
* Stop including foreman::params
* Remove anchor pattern
* Move to https
@ekohl
Copy link
Member Author

ekohl commented Aug 16, 2018

Updated to fix a bug in the yum handling. Debian nightlies are still broken so can't promise we don't have regressions there.

@mmoll mmoll merged commit dca5b67 into theforeman:master Aug 16, 2018
@mmoll
Copy link
Contributor

mmoll commented Aug 16, 2018

merged, bedankt @ekohl!

@ekohl ekohl deleted the refactor-repo-handling branch August 16, 2018 16:02
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.

None yet

4 participants