# Failed logic with when statement

**URL:** <https://forum.ansible.com/t/failed-logic-with-when-statement/27305>\
**Category:** Ansible Project\
**Created:** [April 17, 2018, 8:21pm UTC](https://forum.ansible.com/t/failed-logic-with-when-statement/27305 "2018-04-17T20:21:35Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![John\_Harmon](https://avatars.discourse-cdn.com/v4/letter/j/edb3f5/32.png) [@John\_Harmon](https://forum.ansible.com/u/John_Harmon)\
**Post date:** [April 17, 2018, 8:21pm UTC](https://forum.ansible.com/t/failed-logic-with-when-statement/27305/1 "2018-04-17T20:21:35Z")

</div>

I have a variable ‘ntp\_timezone’ set to UTC by default. I am trying to override it in certain situation, but this ALWAYS overrides it. I think something with my logic is wrong and could use another pair of eyes:  
**NOTE: ansible-rhel7 has a current hostname of ansible-rhel7ebs**

`

- debug:  
var: ntp\_timezone  
verbosity: 2

- name: Set timezone variable for non-UTC servers (Overriding UTC as default)  
set\_fact:  
ntp\_timezone: America/Denver  
when: item not in ansible\_hostname | lower  
with\_items:

- agl

- ascp

- dmtr

- ebs

- vtx

- debug:  
var: ntp\_timezone  
verbosity: 2

`

**Results:**

`  
TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:27  
ok: [ansible-rhel7] =\> {  
“ansible\_hostname”: “ansible-rhel7ebs”  
}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:27  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

TASK [ntp : Set timezone variable for non-UTC servers (Overriding UTC as default)] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:31  
ok: [ansible-rhel7] =\> (item=agl) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “agl”}  
ok: [ansible-rhel7] =\> (item=ascp) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “ascp”}  
ok: [ansible-rhel7] =\> (item=dmtr) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “dmtr”}  
skipping: [ansible-rhel7] =\> (item=ebs) =\> {“changed”: false, “item”: “ebs”, “skip\_reason”: “Conditional result was False”}  
ok: [ansible-rhel7] =\> (item=vtx) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “vtx”}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:42  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “America/Denver”  
}  
META: ending play

`

---

<div class="post-metadata">

**Author:** ![John\_Harmon](https://avatars.discourse-cdn.com/v4/letter/j/edb3f5/32.png) [@John\_Harmon](https://forum.ansible.com/u/John_Harmon)\
**Post date:** [April 17, 2018, 8:41pm UTC](https://forum.ansible.com/t/failed-logic-with-when-statement/27305/2 "2018-04-17T20:41:36Z")

</div>

Ah, I think I see my problem… it works, but then gets checked against vtx and gets set… I will have to try something else

---

<div class="post-metadata">

**Author:** ![John\_Harmon](https://avatars.discourse-cdn.com/v4/letter/j/edb3f5/32.png) [@John\_Harmon](https://forum.ansible.com/u/John_Harmon)\
**Post date:** [April 17, 2018, 8:45pm UTC](https://forum.ansible.com/t/failed-logic-with-when-statement/27305/3 "2018-04-17T20:45:10Z")

</div>

I was wrong… I changed the order and it made no difference. I also have a similar test in another playbook that has always worked just fine… so still looking for info from anybody that can help me.

Reordered:

`  
ask path: /etc/ansible/roles/ntp/tasks/main.yml:27  
ok: [ansible-rhel7] =\> {  
“ansible\_hostname”: “ansible-rhel7ebs”  
}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:30  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

TASK [ntp : Set timezone variable for non-UTC servers (Overriding UTC as default)] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:34  
ok: [ansible-rhel7] =\> (item=agl) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “agl”}  
ok: [ansible-rhel7] =\> (item=ascp) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “ascp”}  
ok: [ansible-rhel7] =\> (item=dmtr) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “dmtr”}  
ok: [ansible-rhel7] =\> (item=vtx) =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false, “item”: “vtx”}  
skipping: [ansible-rhel7] =\> (item=ebs) =\> {“changed”: false, “item”: “ebs”, “skip\_reason”: “Conditional result was False”}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:45  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “America/Denver”  
}

`

---

<div class="post-metadata">

**Author:** ![John\_Harmon](https://avatars.discourse-cdn.com/v4/letter/j/edb3f5/32.png) [@John\_Harmon](https://forum.ansible.com/u/John_Harmon)\
**Post date:** [April 17, 2018, 8:46pm UTC](https://forum.ansible.com/t/failed-logic-with-when-statement/27305/4 "2018-04-17T20:46:55Z")

</div>

With it as the only option it does work… so now I can see my flawed logic.

`  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

TASK [ntp : Set timezone variable for non-UTC servers (Overriding UTC as default)] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:34  
skipping: [ansible-rhel7] =\> (item=ebs) =\> {“changed”: false, “item”: “ebs”, “skip\_reason”: “Conditional result was False”}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:45  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

`

---

<div class="post-metadata">

**Author:** ![John\_Harmon](https://avatars.discourse-cdn.com/v4/letter/j/edb3f5/32.png) [@John\_Harmon](https://forum.ansible.com/u/John_Harmon)\
**Post date:** [April 17, 2018, 9:00pm UTC](https://forum.ansible.com/t/failed-logic-with-when-statement/27305/5 "2018-04-17T21:00:45Z")

</div>

OK, Here are my working results should anybody else come looking and find this:

**Play:**

`

- name: Set timezone variable for non-UTC servers (Overriding UTC as default)  
set\_fact:  
ntp\_timezone: America/Denver  
when: ansible\_hostname|lower is not search (“agl|ascp|ebs|dmtr|vtx”)  
`

**Result with ebs in name:**

`  
ok: [ansible-rhel7] =\> {  
“ansible\_hostname”: “ansible-rhel7ebs”  
}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:30  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

TASK [ntp : Set timezone variable for non-UTC servers (Overriding UTC as default)] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:34  
skipping: [ansible-rhel7] =\> {“changed”: false, “skip\_reason”: “Conditional result was False”}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:39  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

`

**Result WITHOUT ebs in name:**

`  
ok: [ansible-rhel7] =\> {  
“ansible\_hostname”: “ansible-rhel7”  
}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:30  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “UTC”  
}

TASK [ntp : Set timezone variable for non-UTC servers (Overriding UTC as default)] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:34  
ok: [ansible-rhel7] =\> {“ansible\_facts”: {“ntp\_timezone”: “America/Denver”}, “changed”: false}

TASK [ntp : debug] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
task path: /etc/ansible/roles/ntp/tasks/main.yml:39  
ok: [ansible-rhel7] =\> {  
“ntp\_timezone”: “America/Denver”  
}

`

Hope it helps somebody