<feed xmlns='http://www.w3.org/2005/Atom'>
<title>csa/devops/ansible-patches/openshift.git/roles/openshift_node_facts, branch ands</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/'/>
<entry>
<title>Remove openshift_node_facts role</title>
<updated>2017-12-18T19:33:43+00:00</updated>
<author>
<name>Michael Gugino</name>
<email>mgugino@redhat.com</email>
</author>
<published>2017-12-18T19:33:43+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=23b283e22982c032db341b7b4d81a384f0ec0b71'/>
<id>23b283e22982c032db341b7b4d81a384f0ec0b71</id>
<content type='text'>
This commit removes the remainder of openshift_node_facts
role.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit removes the remainder of openshift_node_facts
role.
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove openshift_node_facts part 1</title>
<updated>2017-12-14T16:01:53+00:00</updated>
<author>
<name>Michael Gugino</name>
<email>mgugino@redhat.com</email>
</author>
<published>2017-12-12T22:54:20+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=42d3824ba0d5a1e2cff3dc3a179cf02e735bc5da'/>
<id>42d3824ba0d5a1e2cff3dc3a179cf02e735bc5da</id>
<content type='text'>
This commit removes some items from openshift_facts
for the openshit_node role.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit removes some items from openshift_facts
for the openshit_node role.
</pre>
</div>
</content>
</entry>
<entry>
<title>Implement container runtime role</title>
<updated>2017-12-01T17:32:39+00:00</updated>
<author>
<name>Michael Gugino</name>
<email>mgugino@redhat.com</email>
</author>
<published>2017-11-16T19:56:14+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=5120f8e90c0178ac7f6d911159ceb278dd87b4c9'/>
<id>5120f8e90c0178ac7f6d911159ceb278dd87b4c9</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>remove schedulable from openshift_facts</title>
<updated>2017-11-27T21:58:07+00:00</updated>
<author>
<name>Michael Gugino</name>
<email>mgugino@redhat.com</email>
</author>
<published>2017-11-27T21:15:28+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=b9010d93f61600364b37b7fda772bca71f0a6bd3'/>
<id>b9010d93f61600364b37b7fda772bca71f0a6bd3</id>
<content type='text'>
This commit removes schedulable from openshift_facts in
favor of detecting whether or not a host is a master
via inventory.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit removes schedulable from openshift_facts in
favor of detecting whether or not a host is a master
via inventory.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix broken debug_level</title>
<updated>2017-10-11T15:45:04+00:00</updated>
<author>
<name>Michael Gugino</name>
<email>mgugino@redhat.com</email>
</author>
<published>2017-10-10T17:10:23+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=bb1d5f45255d60a88a3220ea7e7b724a073465b1'/>
<id>bb1d5f45255d60a88a3220ea7e7b724a073465b1</id>
<content type='text'>
Currently, debug_level is documented as a way to change
the debug output level for both masters and nodes.

debug_level does not currently have any effect.

This commit removes debug_level from openshift_facts
and properly sets openshift_master_debug_level and
openshift_node_debug_level to the value of debug_level
specified in the inventory.

This commit also reorganizes some set_fact tasks
needed during master upgrades to put all work-around
set-facts for undefined variables in one place, allowing
for easier cleanup in the future.  This includes an
entry for openshift_master_debug_level.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1500164
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Currently, debug_level is documented as a way to change
the debug output level for both masters and nodes.

debug_level does not currently have any effect.

This commit removes debug_level from openshift_facts
and properly sets openshift_master_debug_level and
openshift_node_debug_level to the value of debug_level
specified in the inventory.

This commit also reorganizes some set_fact tasks
needed during master upgrades to put all work-around
set-facts for undefined variables in one place, allowing
for easier cleanup in the future.  This includes an
entry for openshift_master_debug_level.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1500164
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove oo_option facts.</title>
<updated>2017-09-29T14:57:21+00:00</updated>
<author>
<name>Andrew Butcher</name>
<email>abutcher@redhat.com</email>
</author>
<published>2017-09-14T17:17:07+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=b09f52797efe0dd5f4888672935bbf3b501dea37'/>
<id>b09f52797efe0dd5f4888672935bbf3b501dea37</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Rename filter_plugins to unique names</title>
<updated>2017-09-27T14:45:53+00:00</updated>
<author>
<name>Russell Teague</name>
<email>rteague@redhat.com</email>
</author>
<published>2017-09-27T14:18:23+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=4dcc5042e6a5ab70de755e2d9d3bdab7e37d45f5'/>
<id>4dcc5042e6a5ab70de755e2d9d3bdab7e37d45f5</id>
<content type='text'>
Filter plugin modules should have unique names to not conflict with core
Ansible modules.  (Ansible 2.4)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Filter plugin modules should have unique names to not conflict with core
Ansible modules.  (Ansible 2.4)
</pre>
</div>
</content>
</entry>
<entry>
<title>move (and rename) get_dns_ip filter into openshift_node_facts</title>
<updated>2017-09-12T19:34:19+00:00</updated>
<author>
<name>Joel Diaz</name>
<email>jdiaz@redhat.com</email>
</author>
<published>2017-09-12T19:11:37+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=3ebc0b591f9445b8fde65bda3f76666bad2c8ca3'/>
<id>3ebc0b591f9445b8fde65bda3f76666bad2c8ca3</id>
<content type='text'>
Due to a combination of unexpected behavior when resolving filter plugins in Ansible and recent changes to filter_plugins/openshift_node.py, when trying to run a current version of the openshift_node_facts role on a system with an older version of the openshift-ansible-filter-plugins RPM, the role will fail.

  File "/usr/share/ansible_plugins/filter_plugins/openshift_node.py", line 30, in get_dns_ip
    if bool(hostvars['openshift']['common']['use_dnsmasq']):
KeyError: 'use_dnsmasq'

It's not possible for us to have our current RPM version (presently openshift-ansible-filter-plugins-3.5.120-1.git.0.c60f69a.el7.noarch) and a newer RPM installed and run both current openshift-ansible code and older checked-out/vendored releases (for older clusters/releases).

Since only the openshift_node_facts role uses the get_dns_ip filter, move the functionality into a role-specific filter plugin.

In addition, rename the filter plugin to 'node_get_dns_ip' because Ansible is prefering the get_dns_ip from the RPM (ie /usr/share/ansible_plugins/filter_plugins/openshift_node.py) over the role-specific filter plugin of the same name. Ansile prefers the filter plugins in /usr/share/ansible_plugins/filter_plugins/* even when /etc/ansible/ansible.cfg is set to something like:

filter_plugins     = filter_plugins:/usr/share/ansible_plugins/filter_plugins:filter_plugins   &lt;--- yes, 'filter_plugins' before and after /usr/share/ansible_plugins/filter_plugins (ansible 2.3.1.0)

Renaming the filter plugin function ensures that roles that depend on the old implementation can continue to use it through what the older RPM provides, and the new role can use it's role-specific filter plugin without any variable namespace colisions.

Lastly, remove filter_plugins/openshift_node.py since it is now self-contained in roles/openshift_node_facts.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Due to a combination of unexpected behavior when resolving filter plugins in Ansible and recent changes to filter_plugins/openshift_node.py, when trying to run a current version of the openshift_node_facts role on a system with an older version of the openshift-ansible-filter-plugins RPM, the role will fail.

  File "/usr/share/ansible_plugins/filter_plugins/openshift_node.py", line 30, in get_dns_ip
    if bool(hostvars['openshift']['common']['use_dnsmasq']):
KeyError: 'use_dnsmasq'

It's not possible for us to have our current RPM version (presently openshift-ansible-filter-plugins-3.5.120-1.git.0.c60f69a.el7.noarch) and a newer RPM installed and run both current openshift-ansible code and older checked-out/vendored releases (for older clusters/releases).

Since only the openshift_node_facts role uses the get_dns_ip filter, move the functionality into a role-specific filter plugin.

In addition, rename the filter plugin to 'node_get_dns_ip' because Ansible is prefering the get_dns_ip from the RPM (ie /usr/share/ansible_plugins/filter_plugins/openshift_node.py) over the role-specific filter plugin of the same name. Ansile prefers the filter plugins in /usr/share/ansible_plugins/filter_plugins/* even when /etc/ansible/ansible.cfg is set to something like:

filter_plugins     = filter_plugins:/usr/share/ansible_plugins/filter_plugins:filter_plugins   &lt;--- yes, 'filter_plugins' before and after /usr/share/ansible_plugins/filter_plugins (ansible 2.3.1.0)

Renaming the filter plugin function ensures that roles that depend on the old implementation can continue to use it through what the older RPM provides, and the new role can use it's role-specific filter plugin without any variable namespace colisions.

Lastly, remove filter_plugins/openshift_node.py since it is now self-contained in roles/openshift_node_facts.
</pre>
</div>
</content>
</entry>
<entry>
<title>Move node facts to new openshift_node_facts role.</title>
<updated>2017-07-27T18:25:44+00:00</updated>
<author>
<name>Andrew Butcher</name>
<email>abutcher@redhat.com</email>
</author>
<published>2017-07-27T17:41:32+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.ru.suren.me/cgit/csa/devops/ansible-patches/openshift.git/commit/?id=6bf9371c6cb52b2727dd52bc38f2b8b3a01d67d8'/>
<id>6bf9371c6cb52b2727dd52bc38f2b8b3a01d67d8</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
</feed>
