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

Add epel-release and extra repos for RHEL #225

Merged
merged 1 commit into from
May 27, 2019

Conversation

mamedin
Copy link
Contributor

@mamedin mamedin commented Sep 11, 2018

RHEL doesn't include the epel-release package on its official repositories,
so it is needed to install the epel-releaserpm package from the fedoraproject
homepage.

It is recommended to also enable the optional and extras repositories since
EPEL packages may depend on packages from these repositories.

Connects to archivematica/Issues#155
Connects to #209

@mamedin mamedin self-assigned this Sep 11, 2018
@mamedin mamedin added this to the 1.8.0 milestone Sep 11, 2018
Copy link
Member

@hakamine hakamine left a comment

Choose a reason for hiding this comment

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

I added a couple of comments. Re epel-release, the stable/1.6.x branch of the role installed it as a task in common.yml when ansible_os_family == "RedHat", but stable/1.7.x doesn't have this task. Not sure if this was removed on purpose or unintentionally.

tasks/main.yml Show resolved Hide resolved
@mamedin mamedin force-pushed the dev/add-epel-release-for-RHEL branch from 7c86f7a to d6c86a5 Compare September 12, 2018 09:51
@mamedin
Copy link
Contributor Author

mamedin commented Sep 12, 2018

I removed the always tag, it is ready for CR again

@sevein
Copy link
Member

sevein commented Sep 19, 2018

@mamedin, should we merge this?

@mamedin
Copy link
Contributor Author

mamedin commented Sep 19, 2018

I tested on a RHEL, but it was an evaluation version. I think RHEL includes a set of repository depending of the license and type (developer, workcenter, etc)

In the next days I'll test ansible deployments on licensed RHEL servers, so I think we can wait a week before merging this PR.

@nickwilkinson nickwilkinson removed this from the 1.8.0 milestone Oct 22, 2018
RHEL doesn't include the `epel-release` package on its official repositories,
so it is needed to install the `epel-release`rpm package from the fedoraproject
homepage.

It is recommended to also enable the optional and extras repositories since
EPEL packages may depend on packages from these repositories.
@mamedin mamedin force-pushed the dev/add-epel-release-for-RHEL branch from d6c86a5 to 823d46a Compare April 12, 2019 16:22
@mamedin mamedin changed the base branch from stable/1.7.x to stable/1.9.x April 12, 2019 16:23
@mamedin mamedin requested a review from hakamine April 12, 2019 16:23
@mamedin
Copy link
Contributor Author

mamedin commented Apr 12, 2019

Rebased and changed base branch to stable/1.9.x.

This morning a client confirmed this change is needed for RHEL.

@mamedin mamedin merged commit 519e022 into stable/1.9.x May 27, 2019
@mamedin mamedin deleted the dev/add-epel-release-for-RHEL branch May 27, 2019 16:03
@mamedin
Copy link
Contributor Author

mamedin commented May 27, 2019

Cherry-picked to qa/1.x: a36e49c

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

Successfully merging this pull request may close these issues.

4 participants