# Summary of newly merged patches (since last update)

**URL:** <https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928>\
**Category:** Ansible Project\
**Created:** [September 26, 2012, 1:02am UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928 "2012-09-26T01:02:30Z")\
**Posts on this page:** 6\
**Page:** 2

<div class="post-metadata">

**Author:** ![Cosmin\_Luta](https://avatars.discourse-cdn.com/v4/letter/c/8e8cbc/32.png) [@Cosmin\_Luta](https://forum.ansible.com/u/Cosmin_Luta)\
**Post date:** [October 17, 2012, 9:08pm UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928/21 "2012-10-17T21:08:56Z")

</div>

After resetting to the specified commit id , the behaviour has changed, but it’s still not doing the right thing.

$ git reset --hard 339c05bb88a2ab6d3585c45c7a61812ed8e0af5b~1  
HEAD is now at ac16efe Fix cowsay! Moo.

$ ansible-playbook -i ./hosts.ini ./test.yaml

PLAY [foobar] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*

GATHERING FACTS \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
fatal: [foobar] =\> {‘msg’: ‘FAILED: %d format: a number is required, not str’, ‘failed’: True}

TASK: [uptime] \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
FATAL: no hosts matched or all hosts have already failed – aborting

PLAY RECAP \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*  
foobar : ok=0 changed=0 unreachable=1 failed=0

---

<div class="post-metadata">

**Author:** ![Daniel\_Hokka\_Zakriss](https://avatars.discourse-cdn.com/v4/letter/d/f05b48/32.png) [@Daniel\_Hokka\_Zakriss](https://forum.ansible.com/u/Daniel_Hokka_Zakriss)\
**Post date:** [October 17, 2012, 9:11pm UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928/22 "2012-10-17T21:11:58Z")

</div>

Michael DeHaan wrote:

> Your pull request breaks some other behaviors that are important with  
> delegated hosts, I'll review it and see what changes I may want to  
> make.

It doesn't really break any of that, as that doesn't work either as-is...

Daniel

---

<div class="post-metadata">

**Author:** ![Michael\_DeHaan](https://avatars.discourse-cdn.com/v4/letter/m/c4cdca/32.png) [@Michael\_DeHaan](https://forum.ansible.com/u/Michael_DeHaan)\
**Post date:** [October 17, 2012, 9:12pm UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928/23 "2012-10-17T21:12:55Z")

</div>

Patience, please.

I'll let you know when I've had a time to look at it properly.

---

<div class="post-metadata">

**Author:** ![Cosmin\_Luta](https://avatars.discourse-cdn.com/v4/letter/c/8e8cbc/32.png) [@Cosmin\_Luta](https://forum.ansible.com/u/Cosmin_Luta)\
**Post date:** [October 18, 2012, 8:28am UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928/24 "2012-10-18T08:28:23Z")

</div>

Ok, I’ve done a fresh checkout this morning (HEAD at 6aafb42). The error I’m getting (see the quote below - %d format etc.) is because that ansible\_ssh\_port is stored as a string instead of an integer as paramiko/ssh expect, thus the exception.

I’m not gonna send a pull request because I don’t know the implications of this change, but I’ve fixed it by changing Host.set\_variable in lib/ansible/inventory/host.py to:

def set\_variable(self, key, value):  
if key == ‘ansible\_ssh\_port’:  
value = int(value)

self.vars[key]=value

And yeah, it’s ugly. But it seems to work.

---

<div class="post-metadata">

**Author:** ![Michael\_DeHaan](https://avatars.discourse-cdn.com/v4/letter/m/c4cdca/32.png) [@Michael\_DeHaan](https://forum.ansible.com/u/Michael_DeHaan)\
**Post date:** [October 18, 2012, 11:23am UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928/25 "2012-10-18T11:23:30Z")

</div>

Hmm. Please paste the traceback, or better yet, file a ticket with  
the traceback, and I'll look at it tonight.

---

<div class="post-metadata">

**Author:** ![Cosmin\_Luta](https://avatars.discourse-cdn.com/v4/letter/c/8e8cbc/32.png) [@Cosmin\_Luta](https://forum.ansible.com/u/Cosmin_Luta)\
**Post date:** [October 18, 2012, 11:47am UTC](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928/26 "2012-10-18T11:47:25Z")

</div>

I’ve filed [https://github.com/ansible/ansible/issues/1371](https://github.com/ansible/ansible/issues/1371)

[Previous page](https://forum.ansible.com/t/summary-of-newly-merged-patches-since-last-update/11928.md?page=1)