Skip to content
This repository was archived by the owner on Jun 18, 2021. It is now read-only.

ansible: Add system update playbook#258

Merged
sbesson merged 1 commit into
ome:masterfrom
rleigh-codelibre:ansible-system-update
Apr 12, 2017
Merged

ansible: Add system update playbook#258
sbesson merged 1 commit into
ome:masterfrom
rleigh-codelibre:ansible-system-update

Conversation

@rleigh-codelibre
Copy link
Copy Markdown

@rleigh-codelibre rleigh-codelibre commented Apr 10, 2017

roles/system-base-update: Update base operating system; will update FreeBSD, Linux (Debian/Ubuntu, RedHat/CentOS), MacOS X and Windows systems

roles/system-package-update: Update packages using package manager on systems with a package manager; will update FreeBSD (pkgng), Linux (apt, yum) and MacOS X (homebrew). Similar to the base update for apt and yum, though apt is more conservative here (safe upgrade only, no new kernel packages etc.).

system-update: New playbook which runs both the above roles.

Testing:

You'll need to add to ansible.cfg:

[defaults]
allow_world_readable_tmpfiles = True

to allow unprivileged user switching on MacOS X (deficiency in ACL support in ansible for MacOS X AFAICT).

ansible-playbook -l ci-jenkins-osx  -i ~/code/management_tools/ansible/inventory/ci-hosts ~/code/infrastructure/ansible/system-update.yml --vault-password-file=~/code/credentials/ci/vault
ansible-playbook -l ci-jenkins-freebsd  -i ~/code/management_tools/ansible/inventory/ci-hosts ~/code/infrastructure/ansible/system-update.yml --vault-password-file=~/code/credentials/ci/vault

The Windows and Linux variants will become supported as we move those nodes over to ansible.

roles/system-base-update: Update base operating system; will
update FreeBSD, Linux (Debian/Ubuntu, RedHat/CentOS), MacOS X
and Windows systems

roles/system-package-update: Update packages using package manager
on systems with a package manager; will update FreeBSD (pkgng),
Linux (apt, yum) and MacOS X (homebrew).  Similar to the base
update for apt and yum, though apt is more conservative here
(safe upgrade only, no new kernel packages etc.).

system-update: New playbook which runs both the above roles.
@manics
Copy link
Copy Markdown
Member

manics commented Apr 10, 2017

There's already a Centos 7 upgrade role: https://github.com/openmicroscopy/ansible-role-upgrade-distpackages. Could this be extended?

In practice when would you use system-base-update instead of system-package-update? Could these roles be combined?

@rleigh-codelibre
Copy link
Copy Markdown
Author

rleigh-codelibre commented Apr 10, 2017

The CentOS 7 role could be combined with one or both of the roles here.

The separation exists in the two roles added here because on some systems e.g. MacOS X with homebrew, there's a big difference between updating the base system and updating homebrew; they are separate actions. We might want to update homebrew more frequently than the base system (the playbook doesn't do this, but the possibility is there for other playbooks to do so). Same thing with FreeBSD and the ports. This separation doesn't exist with the apt and yum package management.

@snoopycrimecop
Copy link
Copy Markdown
Member

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1361. See the console output for more details.
Possible conflicts:

@snoopycrimecop
Copy link
Copy Markdown
Member

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1362. See the console output for more details.
Possible conflicts:

@rleigh-codelibre
Copy link
Copy Markdown
Author

@manics While we can certainly move this out and merge it with the existing role, it might be helpful to stage it all in this repo first. This is just one step of what will be many steps to get all this working on 5+ separate platforms. I don't think splitting it up at the get go will help us do that in an efficient or timely manner. Could we get to the point where everything is functional before splitting it?

If that's OK, could we merge this so we can start staging all the dependent pieces and get some machines up and running on openstack? I'd like to be able to get some done this week.

@@ -0,0 +1,2 @@
---
# Defaults
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There's no need to commit empty files

@sbesson
Copy link
Copy Markdown
Member

sbesson commented Apr 12, 2017

Discussed earlier today with @manics and @kennethgillen. This PR makes sense as a first step and should be considered as development roles/playbooks for now. Merging assuming #258 (comment) can be addressed in the following PR.

@sbesson sbesson merged commit 7dc0dbe into ome:master Apr 12, 2017
@rleigh-codelibre rleigh-codelibre deleted the ansible-system-update branch April 12, 2017 20:39
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.

4 participants