{"push_id":585901611,"size":312,"distinct_size":1,"ref":"refs/heads/add_binary_functionality_to_copy","head":"28044dc525f5b4f920d451095dfbf8c33430cf50","before":"cefee98b47f962c83b75db60c231409e415dc7f3","commits":[{"sha":"b3e84f2dd42afbc35cd0ed349320d5fb4f3ab2ec","author":{"email":"55dc6d280caa15fa9212e2ea3868a7a53560ac2b@users.noreply.github.com","name":"Phillip Holmes"},"message":"Route53 fix - forcing zone_in, record_in to lower case\n\nIt turns out the Route53 API cares if the zone and record specified in the playbook are lower case or not when deleting a record. If you use a variable to name your servers and care about case, using that same proper case name will cause Route53 DNS delete requests to fail.\r\n\r\nThe change requested adds .lower() to the module.params.get for both zone and record when used in the underlying code.\r\n\r\nBoth zone and record are mandatory variables, and as such a more complicated implementation is not needed, as they must always be specified when using this module see lines 169 and 170 for the required state).\r\n\r\nIf you use lowercase names (or don't use a name variable and share it between a tag and DNS entries) then you will never see this issue.\r\n\r\nTested/Confirmed as an issue in Ansible 1.6.6 and above.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/b3e84f2dd42afbc35cd0ed349320d5fb4f3ab2ec"},{"sha":"7402827950b873d35ff7d25174e57d71bff5c598","author":{"email":"55dc6d280caa15fa9212e2ea3868a7a53560ac2b@users.noreply.github.com","name":"Phillip Holmes"},"message":"Route53 fix - forcing zone_in, record_in to lower case\n\nFixed the .tolower to .lower() for correct syntax (copied change from older notes).","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/7402827950b873d35ff7d25174e57d71bff5c598"},{"sha":"37d99031693cb9672d4337bd8c2f1935531ced94","author":{"email":"291d3e10d996b9590a718565f00bfbbd0ad02bef@ephaone.org","name":"Michael Scherer"},"message":"Add hostname to generated user SSH key comment\n\nThe default is not very useful to sort between different\nkeys and user. Adding the hostname in the comment permit to later\nsort them if you start to reuse the key and set them in different\nservers. See https://github.com/ansible/ansible/pull/7420\nfor the rational.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/37d99031693cb9672d4337bd8c2f1935531ced94"},{"sha":"82af0743820fa901423132cd2afa8ee1358315ce","author":{"email":"a578cfb8ec30df6d990d142df1db2e8c1703e52f@gmail.com","name":"kustodian"},"message":"Set selinux state to 'permissive' for state=disabled","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/82af0743820fa901423132cd2afa8ee1358315ce"},{"sha":"ff36edbb1103274a787f3158bcb7b1d0555a2722","author":{"email":"9e30857935f3f93181dd1d4a7ae9ae4aa4c7a0d9@gmail.com","name":"billwanjohi"},"message":"user: add expired state\n\nported from\nhttps://github.com/ansible/ansible/pull/6303\n\nIt's very useful and routine to disable a *nix user.\nI implemented expired instead of locked because this prevents any use of\nthe account, safer than just preventing password-based authentication.\n\nI have tests [1], but since none of the suite came along with the core\nmodules, I'm unsure how to submit them.\n\n[1] https://github.com/billwanjohi/ansible/blob/add_locked_state/test/integration/roles/test_user/tasks/main.yml","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/ff36edbb1103274a787f3158bcb7b1d0555a2722"},{"sha":"204a0dc1313f7b64ce8e279cfd5f31f7b22a3658","author":{"email":"4f3407de78bccc8cc160ee4d278d5efe7162e6b5@bx.psu.edu","name":"Nate Coraor"},"message":"Fix a few bugs and misbehavior in the hg module:\n1. Don't pull when `dest` is already at the desired changeset.\n2. Don't change the working copy when cleaning or pulling and a revision\n   was specified.\n3. Change the default for the `revision` param to match the behavior of\n   hg.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/204a0dc1313f7b64ce8e279cfd5f31f7b22a3658"},{"sha":"6157a6552fa236c92a938e70f975c62cc87ac780","author":{"email":"9355b8a682187462bec475f58ff00e802f2680e7@gmail.com","name":"Brendan Jurd"},"message":"Add word boundary in apache2_module regexp\n\nAdd a word boundary \\b to the regexp for checking the output of a2{en,dis}mod,\nto avoid a false positive for a module that ends with the same text as the\nmodule we're working on.\n\nFor example, the previous regexp r'.*spam already enabled' would also match\nagainst 'eggs_spam already enabled'.\n\nAlso, get rid of the redundant '.*' from the end of the regexp.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/6157a6552fa236c92a938e70f975c62cc87ac780"},{"sha":"1b973907658374d146f7a4126a6b9979171db326","author":{"email":"3692bfa45759a67d83aedf0045f6cb635a966abf@rwigo.com","name":"Jonathan Armani"},"message":"Add enable / disable of services for OpenBSD if rcctl is present","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/1b973907658374d146f7a4126a6b9979171db326"},{"sha":"c46e030100dbde9f8a8d71263a3a9a11b45880df","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Make \"enabled\" code aware of --check mode.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/c46e030100dbde9f8a8d71263a3a9a11b45880df"},{"sha":"1a8cdb5e3e68b710cec1c0b2d7d94f48d7f1586b","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Check rc and print error message if any.\n\nIt is probably good to use stdout before printing a generic error\nmessage as well.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/1a8cdb5e3e68b710cec1c0b2d7d94f48d7f1586b"},{"sha":"c6dd88c1d162e1a16749577e5d4df9693583389e","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Fail if \"rcctl status\" writes to stderr.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/c6dd88c1d162e1a16749577e5d4df9693583389e"},{"sha":"5f37624eb4debca831a3b44a39b2b3b96f1872aa","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Tweak error checking for \"enabled\" code.\n\nBased on input from @jarmani:\n* A return value of 2 now means a service does not exist. Instead of\n  trying to handle the different meanings of rc after running \"status\",\n  just look at stderr to know if something failed.\n* Skip looking at stdout to make the code cleaner. Any errors should\n  turn up on stderr.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/5f37624eb4debca831a3b44a39b2b3b96f1872aa"},{"sha":"924cf20cf8ad955af8a187cbdbe508852eac0d3c","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Depend more on rcctl if it is present.\n\n* Make the module support enable/disable of special services like pf via rcctl.\n  Idea and method from @jarmani.\n* Make the module handle when the user supplied 'arguments' variable does not\n  match the current flags in rc.conf.local.\n* Update description now that the code tries to use rcctl for everything if it\n  is available.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/924cf20cf8ad955af8a187cbdbe508852eac0d3c"},{"sha":"e463400412d30b0c26c474af625f799241a7c4cf","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Simplify self.arguments logic.\n\nStrange logic pointed out by @jarmani, thanks!","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/e463400412d30b0c26c474af625f799241a7c4cf"},{"sha":"8d9f6053d3c4f8c15bf2d1fdff79efe3f6637255","author":{"email":"53ad72de00776e2588cecf3bed73e82eaae6f23f@odecee.com.au","name":"Ricky Cook"},"message":"Simplify command module option parsing","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/8d9f6053d3c4f8c15bf2d1fdff79efe3f6637255"},{"sha":"25789b3f474cb2a6c2d5692b49cde7e47d926180","author":{"email":"53ad72de00776e2588cecf3bed73e82eaae6f23f@odecee.com.au","name":"Ricky Cook"},"message":"Add comment to command options dict","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/25789b3f474cb2a6c2d5692b49cde7e47d926180"},{"sha":"b195b5a6bb65acfbfddc61885df1fe9d721c34a3","author":{"email":"53ad72de00776e2588cecf3bed73e82eaae6f23f@odecee.com.au","name":"Ricky Cook"},"message":"Get warn option same as other args","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/b195b5a6bb65acfbfddc61885df1fe9d721c34a3"},{"sha":"6db328c79a8c1f406fdab4e901732ecc9682ced3","author":{"email":"53ad72de00776e2588cecf3bed73e82eaae6f23f@odecee.com.au","name":"Ricky Cook"},"message":"Fix regex string format","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/6db328c79a8c1f406fdab4e901732ecc9682ced3"},{"sha":"f9d9c1b6d7d8326142ec1c4e6db46fb4e9e808ab","author":{"email":"748b59d878fadb8a84297cd9da256de0c0414f0b@gmail.com","name":"Patrik Lundin"},"message":"Multiple fixes for OpenBSD rcctl handling.\n\n* Use the newly added 'default' argument to know if the default flags are set\n  or not.\n* Handle that 'status' may either return flags or YES/NO.\n* Centralize flag handling logic.\n* Set action variable after check if we need to keep going.\n\nBig thanks to @ajacoutot for implementing the rcctl 'default' argument.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/f9d9c1b6d7d8326142ec1c4e6db46fb4e9e808ab"},{"sha":"c804beb1fb02361f4fd561c714a71a41534aebbf","author":{"email":"51e575aea6b1cdf9d5285c4f6480fdecf7d64933@asystant.net","name":"Jonathan Armani"},"message":"Merge pull request #2 from eest/openbsd_rcctl\n\nMultiple fixes for OpenBSD rcctl handling.","distinct":false,"url":"https://api.github.com/repos/danderson189/ansible-modules-core/commits/c804beb1fb02361f4fd561c714a71a41534aebbf"}]}