{"push_id":1454430728,"size":29,"distinct_size":29,"ref":"refs/heads/devel","head":"dab2bf7e134cb618ad436f4db587901fa766abd7","before":"ead002225548c88f400451b765935d3f516191ae","commits":[{"sha":"43785aa24622377a11ddef9bd5f70e734782ced7","author":{"name":"Matt Clay","email":"31e7b35f7f9f5d76c1e2ef239c79b97264b2f6d0@mystile.com"},"message":"Add --all option to ansible-doc.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/43785aa24622377a11ddef9bd5f70e734782ced7"},{"sha":"f9c4158ff848e3f02bc62444ec8a10f6b5cb3e49","author":{"name":"Matt Clay","email":"31e7b35f7f9f5d76c1e2ef239c79b97264b2f6d0@mystile.com"},"message":"Remove core/extras dirs from old PRs during CI.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/f9c4158ff848e3f02bc62444ec8a10f6b5cb3e49"},{"sha":"40ddbe026d74082d84a312d00ad2b367f4596b00","author":{"name":"Juan Antonio Valiño García","email":"f495724f38bc4b897e85d8feb953947fa055ad3f@edu.xunta.es"},"message":"Fixes #18663. Bad handling of existing config in dellos9 module. (#18664)\n\n* Fixes #18663. Bad handling of existing config in dellos9 module.\r\n\r\nThe dellos9 module doesn't build correctly the internal\r\nstructures used to represent the existing config of the managed\r\nnetwork device. This leads to apply changes every time the\r\nplaybook is run, even if the existing config is the same that the\r\none you are trying to push into the device.\r\n\r\nProbably this problem exist also in the dellos6 and dellos10\r\nmodules, but I only fixed it in the dellos9 module.\r\n\r\nThe fix modifies two methods. The first one is `get_config`,\r\nwhere the return clause didn't work correctly when the flow\r\ndoesn't enter in the `if` block. In that case the `contents`\r\nvariable is not an array an this should be handled.\r\n\r\nThe second fix is in the `get_sublevel_config` method. In this\r\ncase the indentation whitespaces of the parents should be rebuild\r\nbecause further functions and methods required it to handle\r\ncorrectly comparisons used to check if changes should be pushed\r\ninto device.\r\n\r\n* Fixes #18663 for dellos10 module with the same patches as dellos9.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/40ddbe026d74082d84a312d00ad2b367f4596b00"},{"sha":"05e5474c52490c2fed77bb8b055294997fc2341f","author":{"name":"Patrick Ogenstad","email":"38df6a931a205814ca498263d467ca19b4fbd715@ogenstad.com"},"message":"Disable CLI errors when typing enable (#18531)","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/05e5474c52490c2fed77bb8b055294997fc2341f"},{"sha":"9e5d4de49aa947585a9ee8987cb0da90680c034a","author":{"name":"ikelos","email":"02b2005dc09a492c70817be6918ce3c73028b25a@gmail.com"},"message":"Fix overwrite parameter in module_utils.junos (#18671)\n\nThe overwrite parameter is forcibly set to false, meaning a module\r\npassing that parameter will have no effect.  The overwrite facility\r\nis necessary to ensure that conflicting options can be written the\r\nconfiguration (which, in replace mode, they cannot).\r\n\r\nThis change ensures that if overwrite is set, it will not be changed\r\nto False in the logic.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/9e5d4de49aa947585a9ee8987cb0da90680c034a"},{"sha":"a65e34ce7744048af8f3cb04595a1c748cac92c5","author":{"name":"Clark Boylan","email":"02b3cf6f5dc14874cd01a8b2e94a4ea86ceae055@gmail.com"},"message":"Fix synchronize retries (#18535)\n\n* Fix synchronize retries\r\n\r\nThe synchronize module munges its task args on every invocation of\r\nrun(). This was problematic because the munged data was not fit for use\r\nby a second pass of the synchronize module. Correct this by using a copy\r\nof the task args on every invocation of run() so that the original args\r\nare not affected.\r\n\r\nLocal testing using this playbook seems to confirm that things work as\r\nexpected:\r\n\r\n  - hosts: all\r\n  tasks:\r\n    - delay: 2\r\n      register: task_result\r\n      retries: 1\r\n      until: task_result.rc == 0\r\n      synchronize:\r\n        dest: /tmp/out\r\n        mode: pull\r\n        src: /tmp/nonexistent/\r\n\r\nfixes #18281\r\n\r\n* Update synchroncization fixture assertions\r\n\r\nWhen we started operating on a copy of the task args the test assertions\r\nwere no longer asserting things about the munged state but of the\r\npristine state. Convert the copy of task args to a class member so that\r\nit can be compared against later in testing and update the assertions to\r\ncheck this munged copy.\r\n* Shuffle objects around for cleaner testing\r\n\r\nAttach the temporary args dict to the task rather than the action as\r\nthis makes updating the existing tests cleaner.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/a65e34ce7744048af8f3cb04595a1c748cac92c5"},{"sha":"5bf07454cb1efe4ab69f40608a51b106404d4100","author":{"name":"Robin Roth","email":"db336c104b64ce860ba2801579d1223a35f78f47@rroth.de"},"message":"Test apache2 force (#18500)\n\n* Use native yaml for apache2 test\r\n\r\n* Test removal of default modules with force\r\n\r\na2enmod on debian has `-f`, but not on SUSE (runs there without force).\r\nTherefore don't test that option on SUSE.\r\nThe docs already specify that the option is intended for Debian systems\r\nonly.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/5bf07454cb1efe4ab69f40608a51b106404d4100"},{"sha":"4b27d08643d91a799122e4085861277dba5004de","author":{"name":"Vlad Glagolev","email":"619d2467d9b4170e3622e91e65d647ae03ac0b41@vaygr.net"},"message":"dropped 'BOOLEANS' choices, converted env vars to str","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/4b27d08643d91a799122e4085861277dba5004de"},{"sha":"ef391a11ec536d0efb1b59e8139735b27b67c67f","author":{"name":"Andrea Tartaglia","email":"b4a5ea001e047040e8c9075b67a268293138da2c@andreatartaglia.com"},"message":"Removed dict.iteritems() in modules. (#18859)\n\nThis is for py3 compatibility, addressed in #18506","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/ef391a11ec536d0efb1b59e8139735b27b67c67f"},{"sha":"a28d898c290f50f815b986e71e88fcd577b899cb","author":{"name":"René Moser","email":"3b8e10480754ba93523db5dc4626a5f001e54bb7@renemoser.net"},"message":"cloudstack: cs_securitygroup_rule: fix impossible to set icmp_code and icmp_type to 0 (#19258)","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/a28d898c290f50f815b986e71e88fcd577b899cb"},{"sha":"bdc4fa6b99bde21f25611cec65f804f268c4e581","author":{"name":"John R Barker","email":"febb44f0a3536761749f6c82b901d79bfa74cce8@johnrbarker.com"},"message":"Add group_vars/ops.yaml (#19288)\n\nAuthentication details for ops modules. Was missed during porting","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/bdc4fa6b99bde21f25611cec65f804f268c4e581"},{"sha":"a8af6c6baffefc5a0f5e43b14941b548c7832b56","author":{"name":"Toshio Kuratomi","email":"cbdeb5b67e5fbe73365734a4941de7f6991b359a@gmail.com"},"message":"Fix some SSL errors in mail.py causing SMTP Syntax Errors (Rebase of https://github.com/ansible/ansible-modules-extras/pull/708 ) (#19253)\n\n* Rebase of https://github.com/ansible/ansible-modules-extras/pull/708\r\n\r\n708 was full of extraneous merge commits interwoven with commits to\r\nimplement the feature. In the end the only way I could clean this up\r\nin reasonable time was to just take a regular diff between the PR and\r\nthe base.  This lost the history of intermediate commits but I've\r\npreserved attribution to @dayton967 via git's --author field.\r\n\r\nAlthough I preserved the logic of the PR, there were a few additional\r\nthings that I cleaned up:\r\n\r\n* Fixed import of email.mime.multipart\r\n* Used the argspec to set port and timeout to integers instead of having\r\n  ad hoc code inside of the module.\r\n* Used argspec's choices for secure instead of ad hoc code inside of the\r\n  module.\r\n* Removed some unused variables\r\n* Made secure_state a python boolean instead of using 0 and 1\r\n* Used secure with string comparisons instead of turning it into an\r\n  integer code.  This is much more readable.\r\n* Fixed catching of SMTPExceptions (SMTPException wasn't imported\r\n  directly so it needed to use the smtplib namespace.)","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/a8af6c6baffefc5a0f5e43b14941b548c7832b56"},{"sha":"51491c990444be4153d8d4e07ced00f8af348ddd","author":{"name":"Toshio Kuratomi","email":"cbdeb5b67e5fbe73365734a4941de7f6991b359a@gmail.com"},"message":"Remove itervalues (not available on py3)","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/51491c990444be4153d8d4e07ced00f8af348ddd"},{"sha":"95ef9fd9f9dea1f11c1077c5fddd9e7a630e1dc1","author":{"name":"Ondra Machacek","email":"df2c132f19097877156e910314f2de200ec8176f@gmail.com"},"message":"cloud: ovirt: Add ovirt_tags and ovirt_tags_facts modules (#19295)","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/95ef9fd9f9dea1f11c1077c5fddd9e7a630e1dc1"},{"sha":"b915e790d378ddee612ffbd0f9af1c6ad8e2938d","author":{"name":"John R Barker","email":"febb44f0a3536761749f6c82b901d79bfa74cce8@johnrbarker.com"},"message":"veos-dut-01 is down, so use veos01 (#19299)","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/b915e790d378ddee612ffbd0f9af1c6ad8e2938d"},{"sha":"84544ee8fd69b1345b969d89335f49c085a4fecb","author":{"name":"Lumír 'Frenzy' Balhar","email":"31f215f826fe4e0c73a04482693263f47163ad55@gmail.com"},"message":"Fix AST nodes for Python 3 and enable dependent test_uri (#18597)\n\n* Enable tests on python 3 for uri\r\n\r\n* Added one more node type to SAFE_NODES into safe_eval module.\r\n\r\nast.USub represents unary operators. This is necessary for\r\nparsing some unusual but still valid JSON files during testing\r\nwith Python 3.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/84544ee8fd69b1345b969d89335f49c085a4fecb"},{"sha":"6ffd494efb146714fc4b297706ade5823118a6b1","author":{"name":"Michael Scherer","email":"f3d8dddad83def0ebeaea5b3b8be8824ed854b37@redhat.com"},"message":"Cleanup imports for user\n\nSome class use shutil, who was not imported, and we want to\navoid '*' for future refactoring.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/6ffd494efb146714fc4b297706ade5823118a6b1"},{"sha":"db8719cf8bcb51fa76bb0111a75f2f2c9e494b87","author":{"name":"Michael Scherer","email":"f3d8dddad83def0ebeaea5b3b8be8824ed854b37@redhat.com"},"message":"Add support for NetBSD\n\nWhile I only checked on NetBSD 7.0, it seems to use the same\nrc system as FreeBSD so it should be working for now.","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/db8719cf8bcb51fa76bb0111a75f2f2c9e494b87"},{"sha":"f9460f3c8056e34c6c25201dcc17487744d0e651","author":{"name":"Michael Scherer","email":"f3d8dddad83def0ebeaea5b3b8be8824ed854b37@zarb.org"},"message":"Use type='path' instead of 'str' for deploy_helper","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/f9460f3c8056e34c6c25201dcc17487744d0e651"},{"sha":"054a3fccf808cc9746e773818dfcfbc4fbca7eb7","author":{"name":"Michael Scherer","email":"f3d8dddad83def0ebeaea5b3b8be8824ed854b37@redhat.com"},"message":"Cleanup import for sysctl module","distinct":true,"url":"https://api.github.com/repos/mattclay/ansible/commits/054a3fccf808cc9746e773818dfcfbc4fbca7eb7"}]}
{"actor":{"display_login":"mattclay"}}