summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2017-09-18Merge pull request #5431 from sosiouxme/20170915-system-container-cwdOpenShift Merge Robot
Automatic merge from submit-queue update system container cwd This changes the cwd for the system container to be the base of the openshift-ansible content. This way the playbook can be specified as a relative path, and in the future when we drop the symlinks for various plugins and rely on cwd to find them, this will still work. Looking through the Dockerfile side of things I noticed that the run script changes directories to WORK_DIR which is the content base, so this change brings the two methods closer together. I was looking for anything that actually wrote to the current directory (which is $HOME at the beginning of the run script) and found one, the vault password. It seemed slightly more robust to write that to a temporary location instead so I tacked on a commit to do that as well.
2017-09-16Merge pull request #5334 from juanluisvaladas/move-sysctlOpenShift Merge Robot
Automatic merge from submit-queue Move sysctl.conf customizations to a separate file Move them from /etc/sysctl.conf to /etc/sysctl.d/99-openshift.conf This is a good idea becuase: 1- /etc/sysctl.conf is evaluated later, so it can easily be overwritten by previous customizations 2- It's likely that there is an agent like puppet monitoring this file 3- It's easier to know what's being changed by OpenShift
2017-09-15Merge pull request #5345 from smarterclayton/firewallOpenShift Merge Robot
Automatic merge from submit-queue Add `openshift_node_open_ports` to allow arbitrary firewall exposure It should be possible for an admin to define an arbitrary set of ports to be exposed on each node that will relate to the cluster function. This adds a new global variable for the node that supports Array(Object{'service':<name>,'port':<port_spec>,'cond':<boolean>}) which is the same format accepted by the firewall role. @sdodson as discussed, open to alternatives. I used this from origin-gce with openshift_node_open_ports: - service: Router stats port: 1936/tcp - service: Open node ports port: 9000-10000/tcp - service: Open node ports port: 9000-10000/udp Which then allows me to set firewall rules appropriately. Alternatives considered: * Simpler external format (have to parse inputs) * Additional parameter to role - felt ugly
2017-09-15Merge pull request #5407 from sdodson/bz1490739OpenShift Merge Robot
Automatic merge from submit-queue Only attempt to start iptables on hosts in the current batch If os_firewall role is called from within a play that uses serial then it was attempting to start iptables on hosts that may not have had iptables installed on them yet. So limit the hosts to the current batch. According to the ansible docs on plays where serial is unused this is the same as ansible_play_hosts. See http://docs.ansible.com/ansible/latest/playbooks_variables.html Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1490739
2017-09-15installer image: use tmp file for vaultpassLuke Meyer
2017-09-15system container: use ansible root as cwdLuke Meyer
2017-09-15Merge pull request #5427 from adelton/issue-2454-2Scott Dodson
No conversion to boolean and no quoting for include_granted_scopes.
2017-09-15Merge pull request #5425 from mtnbikenc/fix-openshift-nfsScott Dodson
1491657 Correct firewall install for openshift-nfs
2017-09-15No conversion to boolean and no quoting for include_granted_scopes.Jan Pazdziora
2017-09-15Correct firewall install for openshift-nfsRussell Teague
2017-09-14Merge pull request #5401 from sosiouxme/20170913-retries-subsetLuke Meyer
add retries on repoquery
2017-09-14Merge pull request #5382 from mtnbikenc/add-lb-configScott Dodson
Add loadbalancer config entry point
2017-09-14Merge pull request #5420 from ashcrow/update-versions-in-masterScott Dodson
inventory: Update versions to 3.7
2017-09-14Merge pull request #5360 from sdodson/embedded-etcdScott Dodson
Clarify requirement of having etcd group
2017-09-14inventory: Update versions to 3.7Steve Milner
Updated the versions in inventory to point to 3.7 by default since 3.6 has already been released. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1491592 Signed-off-by: Steve Milner <smilner@redhat.com>
2017-09-14Merge pull request #5410 from jmencak/tuned-after-node-restartScott Dodson
Setup tuned after the node has been restarted.
2017-09-14Merge pull request #5396 from abutcher/etcd-migrate-openshift-ipScott Dodson
Use openshift.common.ip rather than ansible_default_ipv4 in etcd migration playbook.
2017-09-14Merge pull request #5367 from abutcher/etcd-scaleup-master-client-urlsScott Dodson
Bug 1490304: Etcd scale-up playbook should add new member to etcdClientInfo of master-config.yaml
2017-09-14Merge pull request #5381 from ewolinetz/37_bz1487648_OpenShift Bot
Merged by openshift-bot
2017-09-14Merge pull request #5389 from tremble/storage_migrationScott Dodson
Don't assume storage_migration control variables are already boolean
2017-09-14Merge pull request #5043 from juanvallejo/jvallejo/add-inventory-generator-imageScott Dodson
Merge inventory-generator with origin-ansible image
2017-09-14Merge pull request #5399 from tbielawa/masterScott Dodson
Scaffold out the entire build defaults hash
2017-09-14Setup tuned after the node has been restarted.Jiri Mencak
2017-09-13Only attempt to start iptables on hosts in the current batchScott Dodson
If os_firewall role is called from within a play that uses serial then it was attempting to start iptables on hosts that may not have had iptables installed on them yet. So limit the hosts to the current batch. According to the ansible docs on plays where serial is unused this is the same as ansible_play_hosts. See http://docs.ansible.com/ansible/latest/playbooks_variables.html Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1490739
2017-09-13Merge pull request #5395 from sdodson/bz1480442Scott Dodson
Add IMAGE_VERSION to the image stream tag source
2017-09-13Merge pull request #5151 from jarrpa/gluster-portsScott Dodson
GlusterFS: Correct firewall port names
2017-09-13Merge pull request #5315 from jarrpa/glusterfs-fixes-3.7Scott Dodson
GlusterFS: Various fixes
2017-09-13Merge pull request #5394 from sosiouxme/20170912-health-role-bz1487548OpenShift Bot
Merged by openshift-bot
2017-09-13Merge pull request #5370 from jmencak/tuned-openshift-ansible-bad_merge-fixOpenShift Bot
Merged by openshift-bot
2017-09-13Merge pull request #5405 from ewolinetz/issue_5403Scott Dodson
Removing setting of pod presets
2017-09-13Merge pull request #5359 from mgugino-upstream-stage/version-docker-auth-configOpenShift Bot
Merged by openshift-bot
2017-09-13Removing setting of pod presetsEric Wolinetz
2017-09-13Merge pull request #5402 from ashcrow/fix-cri-o-image-nameScott Dodson
cri-o: Fix Fedora image name
2017-09-13cri-o: Fix Fedora image nameSteve Milner
Signed-off-by: Steve Milner <smilner@redhat.com>
2017-09-13add retry on repoquery_cmdLuke Meyer
2017-09-13add retries to repoquery moduleLuke Meyer
2017-09-13inventory generate: fix config docLuke Meyer
2017-09-13inventory generate: remove refs to openshift_cluster_userLuke Meyer
2017-09-13inventory generate: always use kubeconfig, no loginLuke Meyer
2017-09-13Scaffold out the entire build defaults hashTim Bielawa
Some functions called later may expect sub-keys to exist which will not with the current default empty-dict.
2017-09-13Merge pull request #5379 from mtnbikenc/fix-jinja2-openshift_hostedOpenShift Bot
Merged by openshift-bot
2017-09-13Use openshift.common.ip rather than ansible_default_ipv4 in etcd migration ↵Andrew Butcher
playbook.
2017-09-13Add IMAGE_VERSION to the image stream tag sourceScott Dodson
2017-09-13Merge pull request #5386 from ashcrow/fix-wording-in-upgrade-noticeOpenShift Bot
Merged by openshift-bot
2017-09-13Add loadbalancer config entry pointRussell Teague
2017-09-13Merge pull request #5321 from mtnbikenc/proposal-processScott Dodson
[Proposal] OpenShift-Ansible Proposal Process
2017-09-13Merge pull request #5377 from ingvagabund/double-required-disk-space-etcd-backupOpenShift Bot
Merged by openshift-bot
2017-09-13Don't assume storage_migration control variables are already booleanMark Chappell
openshift_upgrade_pre_storage_migration_enabled openshift_upgrade_pre_storage_migration_fatal openshift_upgrade_post_storage_migration_enabled openshift_upgrade_post_storage_migration_enabled because the 4 variables are not already defaulted/evaluated there's no guarantee that they're actually the boolean type at this point. Example: if they're passed in on the command line
2017-09-12Merge pull request #5365 from sosiouxme/20170908-disconnected-image-checkOpenShift Bot
Merged by openshift-bot
2017-09-12Merge pull request #5343 from mtnbikenc/improve-jinja2-searchOpenShift Bot
Merged by openshift-bot