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

Disable repository management by default #472

Merged
merged 1 commit into from
Dec 17, 2018

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Dec 16, 2018

Because we're trying to move people away from the latest repository, we
set it to configure an explicit version with a default. Because this
default would need to be updated all the time, it's a bad idea.

This patch removes the $custom_repo and makes $repo undef by default.
This means users get no repository by default and need to explicitly
choose which repository to use.

Because repo management would not be of much interest to installer
users, we move it together with $gpgcheck to the advanced section.

@ekohl
Copy link
Member Author

ekohl commented Dec 16, 2018

This will need an installer migration as well.

Copy link
Member

@ehelms ehelms left a comment

Choose a reason for hiding this comment

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

Pending tests, this looks sane.

Because we're trying to move people away from the latest repository, we
set it to configure an explicit version with a default. Because this
default would need to be updated all the time, it's a bad idea.

This patch removes the $custom_repo and makes $repo undef by default.
This means users get no repository by default and need to explicitly
choose which repository to use.

Because repo management would not be of much interest to installer
users, we move it together with $gpgcheck to the advanced section.
@ekohl
Copy link
Member Author

ekohl commented Dec 17, 2018

💚 now

Copy link
Contributor

@mmoll mmoll left a comment

Choose a reason for hiding this comment

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

@ekohl merge at will

@ekohl ekohl merged commit b526c91 into theforeman:master Dec 17, 2018
@ekohl ekohl deleted the repo-management branch December 17, 2018 16:09
@mmoll
Copy link
Contributor

mmoll commented Dec 27, 2018

@ekohl the installer migration is missing currently.

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