| Age | Commit message (Collapse) | Author |
|---|
|
|
|
|
|
- Push config dir logic out of module and use host variables instead.
- Backup master config with ansible utility.
- Add error handling for the upgrade config module.
- Add verbose option to installer.
- Return details on what we changed when upgrading config.
- Cleanup use of first master.
- Don't install upgrade rpms to check what version we'll upgrade to.
|
|
|
|
TODO: We desparately need tests cases for:
- interactive with no config file
- interactive with config file and all installed hosts
- interactive with config file and no installed hosts
- interactive with config file and some installed some uninstalled hosts
- unattended with config file and all installed hosts (with and without --force)
- unattended with config file and no installed hosts (with and without --force)
- unattended with config file and some installed some uninstalled hosts (with and without --force)
|
|
Previously the output was a little confusing. We didn't display anything about
the uninstalled hosts.
|
|
all cases
|
|
|
|
Previously we were writing out a inventory like this:
~~~
[OSEv3:children]
masters
nodes
[OSEv3:vars]
ansible_ssh_user=root
deployment_type=openshift-enterprise
ansible_connection=local
[masters]
ose3-master.example.com openshift_hostname=ose3-master.example.com
[nodes]
ose3-master.example.com openshift_hostname=ose3-master.example.com
ose3-node1.example.com openshift_hostname=ose3-node1.example.com
ose3-node2.example.com openshift_hostname=ose3-node2.example.com
~~~
The problem with that is now all the hosts are consider local connections. In
addition our sudo check wasn't working as expected. We would check that we
have sudo, but the playbooks were not running with root privileges. When
gathering facts you'd hit:
~~~
__main__.OpenShiftFactsFileWriteError: Could not create fact file: /etc/ansible/facts.d/openshift.fact, error: [Errno 13] Permission denied: '/etc/ansible/facts.d/openshift.fact'
~~~
Instead the test for locale connections needs to be per host. Anytime we're not running as root we need `ansible_become` set:
~~~
ose3-master.example.com openshift_hostname=ose3-master.example.com ansible_connection=local ansible_become=true
~~~
|
|
https://bugzilla.redhat.com/show_bug.cgi?id=1274201#c13
|
|
Removing the full call to config resulted in rpms not getting upgraded. Config
was doing a yum update of everything, which picks up the
atomic-openshift-master obsoleting openshift-master. The actual yum call
changed here would not. Instead we switch to a direct call to yum which
correctly picks up the obsoletes and updates to atomic-openshift packages.
|
|
|
|
|
|
|
|
atomic-openshift-installer: Add default openshift-ansible-playbook
|
|
atomic-openshift-installer: Correct inaccurate prompt
|
|
|
|
This adds a default value to the openshift-ansible-playbook directory and also
removes the requirement that it be writable.
|
|
|
|
Update to check both hostname and public_hostname.
Remove ansible_sudo=no as I failed to notice we were already checking
if ansible_ssh_user == 'root' and setting it there.
|
|
This adds a check to see if the host the installer is running on is one of
the hosts to be installed and sets i
ansible_connection=local
ansible_sudo=no
in the inventory file.
|
|
atomic-openshift-installer: Text improvements
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Improvements to some of the installer text based on suggestions from
the doc team.
|
|
|
|
|
|
|
|
enterprise is being phased out in favor of openshift-enterprise, you need to
specify where you wish to go.
|
|
Because we're now installing from an rpm, we have a good idea where to find the
default playbooks and shouldn't require the user to tell us.
|
|
|
|
|
|
|
|
add atomic-openshift-utils to openshift-ansible.spec file
no need for redundent spec file
remove dependencies that are not needed
|
