summaryrefslogtreecommitdiffstats
path: root/roles/openshift_cli
Commit message (Collapse)AuthorAgeFilesLines
* bug: container_binary_sync no longer moves upon symlinksSteve Milner2017-08-151-0/+5
| | | | | | | | | | | | | | | | | With origin 1.5, /usr/local/bin/oc was a symlink to /usr/local/bin/openshift. During the container_binary_sync updated versions of both binaries are copied to the host. First openshift is copied to /usr/local/bin/openshift followed by copying oc to /usr/local/bin/oc. Since oc is a symlink back to /usr/local/bin/openshift the end result was everything linked to oc. This change adds a check before copying a binary. If the destination is a symlink then said symlink is removed before copying the new binary over. Fixed #4965 Reference: https://github.com/openshift/openshift-ansible/issues/4965
* cli_image: do not require Docker when using CRI-OGiuseppe Scrivano2017-08-032-25/+83
| | | | | | Use atomic to copy the CLI binaries to the host. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* Remove vim configuration from Python filesRodolfo Carvalho2017-05-091-2/+0
| | | | | | | In a project where contributors are free to use whatever editor they want and we have linting tools that verify the proper formatting of Python files, it should not be required to have a vim-specific line in Python files.
* update tests and flake8/pylint fixesJason DeTiberus2016-11-291-3/+3
|
* Refactor to use Ansible package moduleRussell Teague2016-11-171-2/+2
| | | | | The Ansible package module will call the correct package manager for the underlying OS.
* Switch from "oadm" to "oc adm" and fix bug in binary sync.Devan Goodwin2016-10-191-1/+6
| | | | | | | | Found bug syncing binaries to containerized hosts where if a symlink was pre-existing, but pointing to the wrong destination, it would not be corrected. Switched to using oc adm instead of oadm.
* Skip the docker role in early upgrade stages.Devan Goodwin2016-09-291-0/+1
| | | | | | | | This improves the situation further and prevents configuration changes from accidentally triggering docker restarts, before we've evacuated nodes. Now in two places, we skip the role entirely, instead of previous implementation which only skipped upgrading the installed version. (which did not catch config issues)
* Copy openshift binaries instead of using wrapper script.Devan Goodwin2016-07-213-44/+135
| | | | | | | | | | | | | | For containerized masters, we previously create a wrapper script to run the docker CLI image and clean up afterward, but this approach is much slower than having the binary on the native system. Instead we now use an ansible module to handle the logic of syncing the various OpenShift binaries and symlinks for client tools out onto the host. The module will correctly report changed if modifications were needed. Substantial speed improvement for containerized installs which requires many openshift/oc commands.
* Stop reporting changes when docker pull is already up to date.Devan Goodwin2016-07-201-0/+2
|
* Merge remote-tracking branch 'upstream/master' into upgrade33Devan Goodwin2016-07-141-0/+3
|\
| * Move the bash completion into the cli role. Only add when not containerizedTim Bielawa2016-07-141-0/+4
| |
* | Remove all debug used during devel of openshift_version.Devan Goodwin2016-07-081-3/+0
| |
* | More version fixes.Devan Goodwin2016-06-161-1/+1
| |
* | Always populate openshift_image_tag and openshift_pkg_version.Devan Goodwin2016-06-151-2/+1
| | | | | | | | | | Allows the use of arbitrary tags, precise control over containers and rpms, and likely mixed environments.
* | Merge branch 'master' into upgrade33Devan Goodwin2016-06-102-3/+7
|\ \ | |/
| * Fix Docker 1.10 problems with empty tags and trailing :Devan Goodwin2016-06-062-4/+8
| | | | | | | | | | | | | | | | | | | | Docker 1.10 is no longer tolerant of commands like "docker pull myimage:" when we do not have an image tag in play. Adjust all occurrences with one that only includes the : if a version is defined. Adjust the containerized CLI wrappers for a similar problem.
* | Refactor openshift_version behavior.Devan Goodwin2016-05-301-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | Very early in playbooks we must init the openshift_version for each host. First we determine it for the master, logic now is pushed into the openshift_docker role which we run only on first master via openshift_cli. Facts are reloaded leaving us with a first master with openshift.common.version fact we can then re-use on all other hosts. The correct version of docker should be installed as well. We then set openshift_version for all other hosts by re-using the master fact.
* | Work towards determining openshift_version when unspecified.Devan Goodwin2016-05-251-0/+1
| | | | | | | | | | | | | | openshift_docker role was largely useless now, but also almost did what we needed. (deps ordering still needs to be changed) Remove defaulting of openshift_version.
* | Set openshift_version in config playbooks for first master.Devan Goodwin2016-05-253-7/+3
| | | | | | | | | | | | | | Starting to remove openshift.docker.openshift_version fact usage. openshift_version should no longer contain a leading 'v' for containerized installs, just a version number.
* | Debug output.Devan Goodwin2016-05-251-0/+6
|/
* Fix 'recursive loop detected in template string' for upgrading variable.Andrew Butcher2016-05-161-1/+1
|
* Do not upgrade containers to latest avail during a normal config run.Devan Goodwin2016-05-121-1/+1
|
* The openshift_docker role must set the version facts for containerized installsBrenton Leanhardt2016-04-141-1/+1
| | | | | | QE found that for fresh installs we were basing the docker version facts of the images that could be pulled prior to configuring /etc/sysconfig/docker. This is an edge case but something we need to fix.
* Merge pull request #1695 from brenton/bz1323123Brenton Leanhardt2016-04-061-0/+0
|\ | | | | Fixing bugs 1322788 and 1323123
| * Bug 1322788 - The IMAGE_VERSION wasn't added to atomic-openshift-master-api ↵Brenton Leanhardt2016-04-011-0/+0
| | | | | | | | and atomic-openshift-master-controllers
* | Downgrade to docker 1.8.2 if installing OSE < 3.2Scott Dodson2016-04-051-1/+0
|/
* Bug 1320829 - Ensure docker installed for factsJason DeTiberus2016-03-312-7/+2
| | | | | | - gather facts requiring docker only if docker is present and running - Update reference to etcd role in playbooks/common/openshift-etcd/config.yml to use openshift_etcd
* Move common common facts to openshift_factsJason DeTiberus2016-03-152-2/+2
| | | | | | | - Prevents roles that need common facts from needing to require openshift_common, which pulls in the openshift binary. - Add dependency on openshift_facts to os_firewall, since it uses openshift.common facts
* Docker role refactorJason DeTiberus2016-03-142-6/+7
| | | | | | | | | | | | | | | - refactors the docker role to push generic config into docker role and wrap openshift specific variables into an openshift_docker role and it's dependent openshift_docker_facts role - adds support for setting --confirm-def-push flag (Resolves https://github.com/openshift/openshift-ansible/issues/1014) - moves docker related facts from common/node roles to a new docker role - renames cli_docker_* role varialbes to openshift_docker-* (maintaining backward compatibility) - update role dependencies to pull in openshift_docker conditionally based on is_containerized - remove playbooks/common/openshift-docker since the docker role is now conditionally included
* reverting back to pre-pulling the master imageBrenton Leanhardt2016-03-081-0/+5
|
* Support openshift_image_tagBrenton Leanhardt2016-03-072-1/+3
| | | | | | | | | | | | | This is the containerized openshift_pkg_version equivalent. Originally I was hoping to reuse openshift_pkg_version for containerized installs but the fact that it's very coupled to yum made that pretty ugly. However, I did opt to rely on the previously existing 'openshift_version' variable. Containerized and RPM installs can both use that variable and it will be set appropriately if either openshift_pkg_version or openshift_image_tag are set. I suspect someday containerized installs will be the only option and I didn't can to have thinkgs like openshift_pkg_version and openshift_image_tag in the playbooks anymore the necessary.
* Correctly set the image tag for containerized installs (and upgrades)Brenton Leanhardt2016-03-031-24/+3
|
* Updating the containerized cli wrapper to work for both docker 1.8 and 1.9Brenton Leanhardt2016-03-031-5/+18
|
* First past at the upgrade processBrenton Leanhardt2016-03-032-7/+11
|
* replace yum with dnfSeth Jennings2016-01-261-1/+1
|
* Merge pull request #1097 from sdodson/containersBrenton Leanhardt2016-01-111-16/+1
|\ | | | | Always pull images
| * Set the cli image to match osm_image in openshift_cli roleScott Dodson2016-01-111-0/+1
| |
| * Always pull docker imagesScott Dodson2016-01-071-16/+0
| |
* | Add a banner to CLI wrapper instructing users that it's only for bootstrappingScott Dodson2016-01-071-1/+15
|/ | | | Also remove -a STDIN,STDOUT,STDERR that's the default
* Fix multi-word arguments & cli wrapper stdin plumbingScott Dodson2016-01-061-8/+1
| | | | | Fixes https://bugzilla.redhat.com/1295677 Fixes `echo "foo" | oc create -f -`
* Add some guards to wait for images to be pulled before moving onScott Dodson2015-12-161-0/+16
|
* Add openshift_cli roleScott Dodson2015-12-153-0/+64
This role ensures that the clients package is installed or if it's a containerized install places a wrapper script in /usr/local/bin/openshift and symlinks for oc, oadm, and kubectl.
generated by cgit v1.2.1 (git 2.18.0) at 2024-05-05 11:55:34 +0000