{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/26029","repository_url":"https://api.github.com/repos/ansible/ansible","labels_url":"https://api.github.com/repos/ansible/ansible/issues/26029/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/26029/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/26029/events","html_url":"https://github.com/ansible/ansible/pull/26029","id":238008834,"number":26029,"title":"New module: access Cisco ACI (network/aci/aci_rest)","user":{"login":"dagwieers","id":388198,"avatar_url":"https://avatars4.githubusercontent.com/u/388198?v=4","gravatar_id":"","url":"https://api.github.com/users/dagwieers","html_url":"https://github.com/dagwieers","followers_url":"https://api.github.com/users/dagwieers/followers","following_url":"https://api.github.com/users/dagwieers/following{/other_user}","gists_url":"https://api.github.com/users/dagwieers/gists{/gist_id}","starred_url":"https://api.github.com/users/dagwieers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dagwieers/subscriptions","organizations_url":"https://api.github.com/users/dagwieers/orgs","repos_url":"https://api.github.com/users/dagwieers/repos","events_url":"https://api.github.com/users/dagwieers/events{/privacy}","received_events_url":"https://api.github.com/users/dagwieers/received_events","type":"User","site_admin":false},"labels":[{"id":562292166,"url":"https://api.github.com/repos/ansible/ansible/labels/affects_2.4","name":"affects_2.4","color":"ededed","default":false},{"id":61436281,"url":"https://api.github.com/repos/ansible/ansible/labels/needs_revision","name":"needs_revision","color":"eb6450","default":false},{"id":339653091,"url":"https://api.github.com/repos/ansible/ansible/labels/networking","name":"networking","color":"c7def8","default":false},{"id":153577825,"url":"https://api.github.com/repos/ansible/ansible/labels/new_plugin","name":"new_plugin","color":"bfe5bf","default":false},{"id":636327968,"url":"https://api.github.com/repos/ansible/ansible/labels/support:community","name":"support:community","color":"ededed","default":false},{"id":636352796,"url":"https://api.github.com/repos/ansible/ansible/labels/support:core","name":"support:core","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2017-06-23T00:24:59Z","updated_at":"2017-07-16T10:32:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/26029","html_url":"https://github.com/ansible/ansible/pull/26029","diff_url":"https://github.com/ansible/ansible/pull/26029.diff","patch_url":"https://github.com/ansible/ansible/pull/26029.patch"},"body":"##### SUMMARY\r\nThis PR includes:\r\n- Check-mode support\r\n- Cosmetic changes to documentation\r\n- Examples in YAML format\r\n- Removal of incorrect requirements (for this module)\r\n- Do not log passwords\r\n- Implement native fetch_url instead of requests\r\n- Use standard hostname, username and password parameters\r\n- Add alias src for parameter config_file\r\n- Add mutual exclusive content option for inline data (and show some inline examples)\r\n- Add timeout parameter\r\n- Add validate_certs parameter\r\n- Handling ACI result output (identical for JSON as XML input)\r\n- Parse/expose ACI error output to user\r\n\r\n##### ISSUE TYPE\r\n<!--- Pick one below and delete the rest: -->\r\n - New Module Pull Request\r\n\r\n##### COMPONENT NAME\r\n<!--- Name of the module/plugin/module/task -->\r\naci_rest\r\n\r\n##### ANSIBLE VERSION\r\n<!--- Paste verbatim output from “ansible --version” between quotes below -->\r\nv2.4"},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/315600193","html_url":"https://github.com/ansible/ansible/pull/26029#issuecomment-315600193","issue_url":"https://api.github.com/repos/ansible/ansible/issues/26029","id":315600193,"user":{"login":"ansibot","id":6585283,"avatar_url":"https://avatars4.githubusercontent.com/u/6585283?v=4","gravatar_id":"","url":"https://api.github.com/users/ansibot","html_url":"https://github.com/ansibot","followers_url":"https://api.github.com/users/ansibot/followers","following_url":"https://api.github.com/users/ansibot/following{/other_user}","gists_url":"https://api.github.com/users/ansibot/gists{/gist_id}","starred_url":"https://api.github.com/users/ansibot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ansibot/subscriptions","organizations_url":"https://api.github.com/users/ansibot/orgs","repos_url":"https://api.github.com/users/ansibot/repos","events_url":"https://api.github.com/users/ansibot/events{/privacy}","received_events_url":"https://api.github.com/users/ansibot/received_events","type":"User","site_admin":false},"created_at":"2017-07-16T10:32:41Z","updated_at":"2017-07-16T10:32:41Z","body":"The test `ansible-test sanity --test pep8` failed with the following errors:\n```\ntest/units/modules/network/aci/test_aci_rest.py:5:1: E302 expected 2 blank lines, found 1\ntest/units/modules/network/aci/test_aci_rest.py:8:21: E225 missing whitespace around operator\ntest/units/modules/network/aci/test_aci_rest.py:24:161: E501 line too long (168 > 160 characters)\ntest/units/modules/network/aci/test_aci_rest.py:36:21: E225 missing whitespace around operator\ntest/units/modules/network/aci/test_aci_rest.py:44:161: E501 line too long (230 > 160 characters)\ntest/units/modules/network/aci/test_aci_rest.py:86:22: E231 missing whitespace after ','\ntest/units/modules/network/aci/test_aci_rest.py:89:34: E126 continuation line over-indented for hanging indent\ntest/units/modules/network/aci/test_aci_rest.py:90:33: E131 continuation line unaligned for hanging indent\ntest/units/modules/network/aci/test_aci_rest.py:96:30: E126 continuation line over-indented for hanging indent\ntest/units/modules/network/aci/test_aci_rest.py:97:32: E121 continuation line under-indented for hanging indent\ntest/units/modules/network/aci/test_aci_rest.py:100:29: E122 continuation line missing indentation or outdented\ntest/units/modules/network/aci/test_aci_rest.py:116:161: E501 line too long (1286 > 160 characters)\ntest/units/modules/network/aci/test_aci_rest.py:120:161: E501 line too long (625 > 160 characters)\ntest/units/modules/network/aci/test_aci_rest.py:138:21: E225 missing whitespace around operator\ntest/units/modules/network/aci/test_aci_rest.py:154:161: E501 line too long (168 > 160 characters)\ntest/units/modules/network/aci/test_aci_rest.py:163:21: E225 missing whitespace around operator\ntest/units/modules/network/aci/test_aci_rest.py:168:21: E225 missing whitespace around operator\ntest/units/modules/network/aci/test_aci_rest.py:191:21: E225 missing whitespace around operator\n```\n<!-- job_id: 596b3f3f41e4070700ff4401 -->\n<!-- job_id: 596b3f3fa5f0dd0800fd2b79 -->\n<!-- job_id: /testresults/ansible-test-sanity-pep8.json -->\n\n[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)\n<!--- boilerplate: shippable_test_result --->"}}
{"actor":{"display_login":"ansibot"}}