{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/45541","repository_url":"https://api.github.com/repos/ansible/ansible","labels_url":"https://api.github.com/repos/ansible/ansible/issues/45541/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/45541/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/45541/events","html_url":"https://github.com/ansible/ansible/pull/45541","id":359330730,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE0ODQ2OTAz","number":45541,"title":"[WIP] New module to control launchd services on target macOS hosts.","user":{"login":"martinm82","id":616250,"node_id":"MDQ6VXNlcjYxNjI1MA==","avatar_url":"https://avatars2.githubusercontent.com/u/616250?v=4","gravatar_id":"","url":"https://api.github.com/users/martinm82","html_url":"https://github.com/martinm82","followers_url":"https://api.github.com/users/martinm82/followers","following_url":"https://api.github.com/users/martinm82/following{/other_user}","gists_url":"https://api.github.com/users/martinm82/gists{/gist_id}","starred_url":"https://api.github.com/users/martinm82/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinm82/subscriptions","organizations_url":"https://api.github.com/users/martinm82/orgs","repos_url":"https://api.github.com/users/martinm82/repos","events_url":"https://api.github.com/users/martinm82/events{/privacy}","received_events_url":"https://api.github.com/users/martinm82/received_events","type":"User","site_admin":false},"labels":[{"id":529643857,"node_id":"MDU6TGFiZWw1Mjk2NDM4NTc=","url":"https://api.github.com/repos/ansible/ansible/labels/WIP","name":"WIP","color":"fef2c0","default":false},{"id":1043397454,"node_id":"MDU6TGFiZWwxMDQzMzk3NDU0","url":"https://api.github.com/repos/ansible/ansible/labels/affects_2.8","name":"affects_2.8","color":"ededed","default":false},{"id":496359346,"node_id":"MDU6TGFiZWw0OTYzNTkzNDY=","url":"https://api.github.com/repos/ansible/ansible/labels/module","name":"module","color":"1d76db","default":false},{"id":723150340,"node_id":"MDU6TGFiZWw3MjMxNTAzNDA=","url":"https://api.github.com/repos/ansible/ansible/labels/new_contributor","name":"new_contributor","color":"c2e0c6","default":false},{"id":473225334,"node_id":"MDU6TGFiZWw0NzMyMjUzMzQ=","url":"https://api.github.com/repos/ansible/ansible/labels/new_module","name":"new_module","color":"c2e0c6","default":false},{"id":153577825,"node_id":"MDU6TGFiZWwxNTM1Nzc4MjU=","url":"https://api.github.com/repos/ansible/ansible/labels/new_plugin","name":"new_plugin","color":"bfe5bf","default":false},{"id":636327968,"node_id":"MDU6TGFiZWw2MzYzMjc5Njg=","url":"https://api.github.com/repos/ansible/ansible/labels/support:community","name":"support:community","color":"ededed","default":false},{"id":636352796,"node_id":"MDU6TGFiZWw2MzYzNTI3OTY=","url":"https://api.github.com/repos/ansible/ansible/labels/support:core","name":"support:core","color":"ededed","default":false},{"id":468273802,"node_id":"MDU6TGFiZWw0NjgyNzM4MDI=","url":"https://api.github.com/repos/ansible/ansible/labels/test","name":"test","color":"0e8a16","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-09-12T06:18:36Z","updated_at":"2018-09-15T18:28:24Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/45541","html_url":"https://github.com/ansible/ansible/pull/45541","diff_url":"https://github.com/ansible/ansible/pull/45541.diff","patch_url":"https://github.com/ansible/ansible/pull/45541.patch"},"body":"##### SUMMARY\r\nThis PR is about introducing a new core module that can be used to control launchd services on macOS targets.\r\n\r\n<!--- HINT: Include \"Fixes #nnn\" if you are fixing an existing issue -->\r\n\r\n##### ISSUE TYPE\r\n- New Module Pull Request\r\n\r\n##### COMPONENT NAME\r\nlaunchd\r\n\r\n##### ANSIBLE VERSION\r\nTested with\r\n```\r\nansible 2.4.2.0\r\n```\r\n\r\n##### ADDITIONAL INFORMATION\r\nI am still missing the testing part. That is why this PR is still marked as WIP but I wanted to see already some test results in shippable and gather some input from the community."},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/421612336","html_url":"https://github.com/ansible/ansible/pull/45541#issuecomment-421612336","issue_url":"https://api.github.com/repos/ansible/ansible/issues/45541","id":421612336,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMTYxMjMzNg==","user":{"login":"martinm82","id":616250,"node_id":"MDQ6VXNlcjYxNjI1MA==","avatar_url":"https://avatars2.githubusercontent.com/u/616250?v=4","gravatar_id":"","url":"https://api.github.com/users/martinm82","html_url":"https://github.com/martinm82","followers_url":"https://api.github.com/users/martinm82/followers","following_url":"https://api.github.com/users/martinm82/following{/other_user}","gists_url":"https://api.github.com/users/martinm82/gists{/gist_id}","starred_url":"https://api.github.com/users/martinm82/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinm82/subscriptions","organizations_url":"https://api.github.com/users/martinm82/orgs","repos_url":"https://api.github.com/users/martinm82/repos","events_url":"https://api.github.com/users/martinm82/events{/privacy}","received_events_url":"https://api.github.com/users/martinm82/received_events","type":"User","site_admin":false},"created_at":"2018-09-15T18:28:24Z","updated_at":"2018-09-15T18:28:24Z","author_association":"NONE","body":"@mattclay, do you have somewhere an example on how to write an integration test for macOS? I see a lot under the tests/integration folder but how do I ensure that the test will be executed only on macOS?\r\n\r\nShall I add just some conditions (aka ansible_family == 'Darwin') to my tasks? Or is there another way of controlling this?\r\n\r\nMaybe you have some readme for writing integration tests. This one (https://docs.ansible.com/ansible/2.6/dev_guide/testing_integration.html) is not really helpful.\r\n\r\nThanks a lot"}}
{"actor":{"display_login":"martinm82"}}