{"action":"closed","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/39692","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/39692/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/39692/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/39692/events","html_url":"https://github.com/saltstack/salt/issues/39692","id":210575788,"number":39692,"title":"tuned module and state are broken on 7.3 families. ","user":{"login":"djsly","id":4981802,"avatar_url":"https://avatars1.githubusercontent.com/u/4981802?v=3","gravatar_id":"","url":"https://api.github.com/users/djsly","html_url":"https://github.com/djsly","followers_url":"https://api.github.com/users/djsly/followers","following_url":"https://api.github.com/users/djsly/following{/other_user}","gists_url":"https://api.github.com/users/djsly/gists{/gist_id}","starred_url":"https://api.github.com/users/djsly/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djsly/subscriptions","organizations_url":"https://api.github.com/users/djsly/orgs","repos_url":"https://api.github.com/users/djsly/repos","events_url":"https://api.github.com/users/djsly/events{/privacy}","received_events_url":"https://api.github.com/users/djsly/received_events","type":"User","site_admin":false},"labels":[{"id":41387723,"url":"https://api.github.com/repos/saltstack/salt/labels/Bug","name":"Bug","color":"e11d21","default":false},{"id":41387237,"url":"https://api.github.com/repos/saltstack/salt/labels/Execution%20Module","name":"Execution Module","color":"006b75","default":false},{"id":41388135,"url":"https://api.github.com/repos/saltstack/salt/labels/High%20Severity","name":"High Severity","color":"ff9143","default":false},{"id":194766302,"url":"https://api.github.com/repos/saltstack/salt/labels/P3","name":"P3","color":"0a3d77","default":false},{"id":194768064,"url":"https://api.github.com/repos/saltstack/salt/labels/Platform","name":"Platform","color":"fef2c0","default":false},{"id":279790703,"url":"https://api.github.com/repos/saltstack/salt/labels/TEAM%20Platform","name":"TEAM Platform","color":"A9A9A9","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/saltstack/salt/milestones/13","html_url":"https://github.com/saltstack/salt/milestone/13","labels_url":"https://api.github.com/repos/saltstack/salt/milestones/13/labels","id":9265,"number":13,"title":"Approved","description":"All issues that are ready to be worked on, both bugs and features.","creator":{"login":"thatch45","id":507599,"avatar_url":"https://avatars3.githubusercontent.com/u/507599?v=3","gravatar_id":"","url":"https://api.github.com/users/thatch45","html_url":"https://github.com/thatch45","followers_url":"https://api.github.com/users/thatch45/followers","following_url":"https://api.github.com/users/thatch45/following{/other_user}","gists_url":"https://api.github.com/users/thatch45/gists{/gist_id}","starred_url":"https://api.github.com/users/thatch45/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thatch45/subscriptions","organizations_url":"https://api.github.com/users/thatch45/orgs","repos_url":"https://api.github.com/users/thatch45/repos","events_url":"https://api.github.com/users/thatch45/events{/privacy}","received_events_url":"https://api.github.com/users/thatch45/received_events","type":"User","site_admin":false},"open_issues":3440,"closed_issues":4338,"state":"open","created_at":"2011-05-14T04:00:56Z","updated_at":"2017-02-28T22:15:18Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2017-02-27T19:06:06Z","updated_at":"2017-02-28T22:15:17Z","closed_at":"2017-02-28T22:15:17Z","body":"### Description of Issue/Question\r\n\r\ntuned module broken on centOS 7.3\r\n\r\nthe tuned.list module is broken now that `tuned-adm` returns a description inline with the tune profile.\r\n\r\n````\r\nsalt-call cmd.run \"tuned-adm list\"\r\nlocal:\r\n    Available profiles:\r\n    - balanced                    - General non-specialized tuned profile\r\n    - desktop                     - Optmize for the desktop use-case\r\n    - latency-performance         - Optimize for deterministic performance at the cost of increased power consumption\r\n    - network-latency             - Optimize for deterministic performance at the cost of increased power consumption, focused on low latency network performance\r\n    - network-throughput          - Optimize for streaming network throughput.  Generally only necessary on older CPUs or 40G+ networks.\r\n    - powersave                   - Optimize for low power consumption\r\n    - throughput-performance      - Broadly applicable tuning that provides excellent performance across a variety of common server workloads.  This is the default profile for RHEL7.\r\n    - virtual-guest               - Optimize for running inside a virtual guest.\r\n    - virtual-host                - Optimize for running KVM guests\r\n    Current active profile: balanced\r\n````\r\n\r\nCode \r\n\r\n````\r\nvalid_profiles = __salt__['tuned.list']()\r\n\r\n    # check valid profiles, and return error if profile name is not valid\r\n    if profile not in valid_profiles:\r\n        raise salt.exceptions.SaltInvocationError('Invalid Profile Name')\r\n````\r\n\r\nhttps://github.com/saltstack/salt/blob/3b81c37a531c48ff54a2332c4ea9c0d108ce0e74/salt/modules/tuned.py#L36-L51\r\n````\r\ndef list_():\r\n    '''\r\n    List the profiles available\r\n    CLI Example:\r\n    .. code-block:: bash\r\n        salt '*' tuned.list\r\n    '''\r\n\r\n    result = __salt__['cmd.run']('tuned-adm list').splitlines()\r\n    result.pop(0)\r\n    result.pop()\r\n    result = [i.lstrip('- ') for i in result]\r\n    return result\r\n````\r\n\r\n### Versions Report\r\n````\r\nsalt-call --version\r\nsalt-call 2016.11.1 (Carbon)\r\n````"}}
{"actor":{"display_login":"cachedout"}}