Skip to content
This repository has been archived by the owner on Dec 26, 2020. It is now read-only.

Oracle support #5

Merged
merged 1 commit into from
Apr 28, 2015
Merged

Oracle support #5

merged 1 commit into from
Apr 28, 2015

Conversation

rndmh3ro
Copy link
Member

  • Add check for Oracle operating systems
  • Add minus sign to remove whitespace

@chris-rock
Copy link
Member

Great work. I try to get it running on Oracle Linux, but ansible installations fails as you mentioned yesterday. How do you test this?

- Add check for Oracle operating systems

- Add minus sign to remove whitespace
@chris-rock
Copy link
Member

@rndmh3ro Should add support CentOS as well?

@rndmh3ro
Copy link
Member Author

CentOS is already supported, because it is in the "Redhat"-os-family. Oracle Linux, while a clone of Redhat, decided to be in a separate os-family:

-----> Execute command on default-ubuntu-1204.ago",                                                                   o" 
        "ansible_os_family": "Debian", 
-----> Execute command on default-ubuntu-1404.
        "ansible_os_family": "Debian", 
-----> Execute command on default-centos-64.
        "ansible_os_family": "RedHat", 
-----> Execute command on default-centos-65.
        "ansible_os_family": "RedHat", 
-----> Execute command on default-oracle-64.
        "ansible_os_family": "Oracle Linux", 
-----> Execute command on default-oracle-65.
        "ansible_os_family": "Oracle Linux", 

Now we can check the OS with one OR and one AND:

{% elif (ansible_os_family == 'RedHat' or ansible_os_family == 'Oracle Linux') and ansible_distribution_major_version <= '6' -%}

Or I could change it to make it clearer, what operating-systems are used, but then I have to use 2 ORs and one AND:

{% elif (ansible_distribution == 'RedHat' or ansible_distribution == 'Oracle Linux' or ansible_distribution == 'CentOS') and ansible_distribution_major_version <= '6' -%}

What do you prefer?

@chris-rock
Copy link
Member

I think the first one fits well. Thanks for the clarification.

chris-rock added a commit that referenced this pull request Apr 28, 2015
@chris-rock chris-rock merged commit 0c9f129 into master Apr 28, 2015
@chris-rock chris-rock deleted the oracle_tests branch April 28, 2015 14:30
@rndmh3ro rndmh3ro restored the oracle_tests branch April 28, 2015 14:54
@rndmh3ro
Copy link
Member Author

I found an inconsistency with the version-checking of the OS. I'm going to create a new PR.

@chris-rock
Copy link
Member

+1

@rndmh3ro
Copy link
Member Author

Great work. I try to get it running on Oracle Linux, but ansible installations fails as you mentioned yesterday. How do you test this?

Right now I don't know how to test Oracle fully automatically.
The problem is that Ansible requires PyYAML. This package is in one of the oracle-repositories that is disabled by default. That's why the installation fails.
To work-around this, I have to run bundle exec create oracle to create the machines, then execute the following command to install PyYAML:
bundle exec kitchen exec default-oracle-65 -c "sudo sed -i 's/enabled=0/enabled=1/g' /etc/yum.repos.d/public-yum-ol6.repo && sudo yum install -y PyYAML"
Then the machine can be converged.

@chris-rock
Copy link
Member

Thanks for pointing this out!

rndmh3ro pushed a commit that referenced this pull request Aug 16, 2016
fix mysql restart not happening because of missing os specific variable
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants