{"action":"created","issue":{"url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/issues/3","labels_url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/issues/3/comments","events_url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/issues/3/events","html_url":"https://github.com/ccin2p3/puppet-etc_services/issues/3","id":58560207,"number":3,"title":"Type signature","user":{"login":"riton","id":553730,"avatar_url":"https://avatars.githubusercontent.com/u/553730?v=3","gravatar_id":"","url":"https://api.github.com/users/riton","html_url":"https://github.com/riton","followers_url":"https://api.github.com/users/riton/followers","following_url":"https://api.github.com/users/riton/following{/other_user}","gists_url":"https://api.github.com/users/riton/gists{/gist_id}","starred_url":"https://api.github.com/users/riton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/riton/subscriptions","organizations_url":"https://api.github.com/users/riton/orgs","repos_url":"https://api.github.com/users/riton/repos","events_url":"https://api.github.com/users/riton/events{/privacy}","received_events_url":"https://api.github.com/users/riton/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"riton","id":553730,"avatar_url":"https://avatars.githubusercontent.com/u/553730?v=3","gravatar_id":"","url":"https://api.github.com/users/riton","html_url":"https://github.com/riton","followers_url":"https://api.github.com/users/riton/followers","following_url":"https://api.github.com/users/riton/following{/other_user}","gists_url":"https://api.github.com/users/riton/gists{/gist_id}","starred_url":"https://api.github.com/users/riton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/riton/subscriptions","organizations_url":"https://api.github.com/users/riton/orgs","repos_url":"https://api.github.com/users/riton/repos","events_url":"https://api.github.com/users/riton/events{/privacy}","received_events_url":"https://api.github.com/users/riton/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-02-23T09:10:47Z","updated_at":"2015-02-23T09:24:18Z","closed_at":null,"body":"Thanks for releasing this module on Puppet Forge. I'm using it for my own company, but I've made some changes to how the type is specified. Instead of:\r\n\r\n```\r\n    etc_services { 'kerberos/udp':\r\n      port    => '88',\r\n      aliases => [ 'kerberos5', 'krb5', 'kerberos-sec' ],\r\n      comment => 'Kerberos v5'\r\n    }\r\n```\r\n\r\nI use:\r\n\r\n```\r\n    etc_services { 'kerberos':\r\n      ports    => ['tcp/88',\r\n      aliases => [ 'kerberos5', 'krb5', 'kerberos-sec' ],\r\n      comment => 'Kerberos v5'\r\n    }\r\n```\r\n\r\nThis more closely matches the /etc/services syntax."},"comment":{"url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/issues/comments/75510914","html_url":"https://github.com/ccin2p3/puppet-etc_services/issues/3#issuecomment-75510914","issue_url":"https://api.github.com/repos/ccin2p3/puppet-etc_services/issues/3","id":75510914,"user":{"login":"alexjurkiewicz","id":379509,"avatar_url":"https://avatars.githubusercontent.com/u/379509?v=3","gravatar_id":"","url":"https://api.github.com/users/alexjurkiewicz","html_url":"https://github.com/alexjurkiewicz","followers_url":"https://api.github.com/users/alexjurkiewicz/followers","following_url":"https://api.github.com/users/alexjurkiewicz/following{/other_user}","gists_url":"https://api.github.com/users/alexjurkiewicz/gists{/gist_id}","starred_url":"https://api.github.com/users/alexjurkiewicz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexjurkiewicz/subscriptions","organizations_url":"https://api.github.com/users/alexjurkiewicz/orgs","repos_url":"https://api.github.com/users/alexjurkiewicz/repos","events_url":"https://api.github.com/users/alexjurkiewicz/events{/privacy}","received_events_url":"https://api.github.com/users/alexjurkiewicz/received_events","type":"User","site_admin":false},"created_at":"2015-02-23T09:24:18Z","updated_at":"2015-02-23T09:24:18Z","body":"Actually, there's a problem. You can't specify tcp & udp with the same $name. You could either keep the current design or have an ugly solution (since I think it's rare to have this situation):\r\n\r\n    etc_services { 'kerberos udp':\r\n        title => 'kerberos',\r\n        port => 'udp/88',\r\n     }\r\n    etc_services { 'kerberos tcp':\r\n        title => 'kerberos',\r\n        port => 'tcp/88',\r\n     }\r\n    etc_services { 'http':\r\n        port => 'tcp/80',\r\n     }\r\n\r\nPersonally, I don't need tcp & udp entries, so I'm happy to have an ugly solution for that case. But I'll track this module whatever you decide."}}