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

Yum install on el #93

Closed
PumpkinSeed opened this issue Feb 19, 2016 · 7 comments
Closed

Yum install on el #93

PumpkinSeed opened this issue Feb 19, 2016 · 7 comments
Labels

Comments

@PumpkinSeed
Copy link

When I use the nginx installer on EL the code disable all repo and the it can download the dependency packages like openssl... etc.

yum: name={{ item }} state=present disablerepo='*' enablerepo={{ "nginx," if nginx_official_repo else "" }}{{ yum_epel_repo }},{{ yum_base_repo }}

But if I allow him to use all repo it will work without problem:

yum: name={{ item }} state=present

@jdauphant jdauphant added the bug label Feb 20, 2016
@jdauphant
Copy link
Owner

Maybe @jrobeson @etcet @victorluft or @mtotheikle can help on this issue and suggestion a solution.
An MR will be welcomed

Best

@basnijholt
Copy link

Having the same issue, trying to install nginx in this Dockerimage: https://hub.docker.com/r/ansible/centos7-ansible/

PumpkinSeed's suggestion fixed it for me.

@aries1980
Copy link

If the Nginx repo maintainers follow the same version pattern as EL, I see no issue enabling the default repos.

@jdauphant
Copy link
Owner

We can try something like that:

yum: name={{ item }} state=present enablerepo={{ "nginx," if nginx_official_repo else "" }}

or

yum: name={{ item }} state=present enablerepo={{ "nginx," if nginx_official_repo else "" }}{{ yum_epel_repo }},{{ yum_base_repo }}

If you have a solution that keep the possibility to use the official repo, it could be very nice.
A MR will be welcomed :)

@PumpkinSeed
Copy link
Author

I tried the first:

yum: name={{ item }} state=present enablerepo={{ "nginx," if nginx_official_repo else "" }}

It is working correctly, thanks.

@jdauphant
Copy link
Owner

@PumpkinSeed Thanks, I can't test it on my side. Can you send a pull request with the tested solution ?

@PumpkinSeed
Copy link
Author

@jdauphant Yes, I will do it asap.

jdauphant added a commit that referenced this issue Mar 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants