{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/32384","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/32384/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/32384/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/32384/events","html_url":"https://github.com/saltstack/salt/issues/32384","id":146249142,"number":32384,"title":"CloudLinux not detected as os_family: RedHat (consequences of #28951)","user":{"login":"afletch","id":1830538,"avatar_url":"https://avatars.githubusercontent.com/u/1830538?v=3","gravatar_id":"","url":"https://api.github.com/users/afletch","html_url":"https://github.com/afletch","followers_url":"https://api.github.com/users/afletch/followers","following_url":"https://api.github.com/users/afletch/following{/other_user}","gists_url":"https://api.github.com/users/afletch/gists{/gist_id}","starred_url":"https://api.github.com/users/afletch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afletch/subscriptions","organizations_url":"https://api.github.com/users/afletch/orgs","repos_url":"https://api.github.com/users/afletch/repos","events_url":"https://api.github.com/users/afletch/events{/privacy}","received_events_url":"https://api.github.com/users/afletch/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/saltstack/salt/labels/Bug","name":"Bug","color":"e11d21"},{"url":"https://api.github.com/repos/saltstack/salt/labels/Grains","name":"Grains","color":"006b75"},{"url":"https://api.github.com/repos/saltstack/salt/labels/High%20Severity","name":"High Severity","color":"ff9143"},{"url":"https://api.github.com/repos/saltstack/salt/labels/P4","name":"P4","color":"031a39"},{"url":"https://api.github.com/repos/saltstack/salt/labels/Platform","name":"Platform","color":"fef2c0"},{"url":"https://api.github.com/repos/saltstack/salt/labels/Regression","name":"Regression","color":"e11d21"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/saltstack/salt/milestones/13","html_url":"https://github.com/saltstack/salt/milestones/Approved","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://avatars.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":3177,"closed_issues":3996,"state":"open","created_at":"2011-05-14T04:00:56Z","updated_at":"2016-09-15T16:18:57Z","due_on":null,"closed_at":null},"comments":10,"created_at":"2016-04-06T09:25:27Z","updated_at":"2016-09-15T16:22:06Z","closed_at":null,"body":"Due to the change in #28951, all CloudLinux versions now have the os_family grain of 'CloudLinux Server'. This is because the `_OS_NAME_MAP` in `grains/core.py:929` was not updated to be consistent with the change.\r\nThe consequences of this, I'm sure you can imagine, are considerable and terminal to state execution.\r\n\r\nCurently:\r\n`'cloudserve': 'CloudLinux',`\r\nThis would need changing to:\r\n`'cloudlinux: 'CloudLinux',`\r\n\r\nThis then cascades to also break the grains `osfinger` and `osmajorrelease` because they rely on `os_family`.\r\nnb. Even setting a manual grain in `/etc/salt/grains` on the minion for `os_family: RedHat` does not fix the broken osfinger/osmajorrelease grains, as the test used to set these depends on the value of `os_family` determined earlier in `grains/core.py` (this is, perhaps, another problem; that manual grains are not respected during determination of other values)\r\n\r\n```\r\n# salt-minion --versions-report\r\nSalt Version:\r\n           Salt: 2015.8.8.2\r\n\r\nDependency Versions:\r\n         Jinja2: 2.2.1\r\n       M2Crypto: 0.20.2\r\n           Mako: Not Installed\r\n         PyYAML: 3.11\r\n          PyZMQ: 14.5.0\r\n         Python: 2.6.6 (r266:84292, Jul 29 2015, 18:30:53)\r\n           RAET: Not Installed\r\n        Tornado: 4.2.1\r\n            ZMQ: 4.0.5\r\n           cffi: Not Installed\r\n       cherrypy: Not Installed\r\n       dateutil: 1.4.1\r\n          gitdb: Not Installed\r\n      gitpython: Not Installed\r\n          ioflo: Not Installed\r\n        libgit2: Not Installed\r\n        libnacl: Not Installed\r\n   msgpack-pure: Not Installed\r\n msgpack-python: 0.4.6\r\n   mysql-python: Not Installed\r\n      pycparser: Not Installed\r\n       pycrypto: 2.6.1\r\n         pygit2: Not Installed\r\n   python-gnupg: Not Installed\r\n          smmap: Not Installed\r\n        timelib: Not Installed\r\n\r\nSystem Versions:\r\n           dist: redhat 6.7 Aleksei Gubarev\r\n        machine: x86_64\r\n        release: 2.6.32-673.8.1.lve1.4.3.el6.x86_64\r\n         system: CloudLinux Server 6.7 Aleksei Gubarev\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/247377422","html_url":"https://github.com/saltstack/salt/issues/32384#issuecomment-247377422","issue_url":"https://api.github.com/repos/saltstack/salt/issues/32384","id":247377422,"user":{"login":"jfindlay","id":702905,"avatar_url":"https://avatars.githubusercontent.com/u/702905?v=3","gravatar_id":"","url":"https://api.github.com/users/jfindlay","html_url":"https://github.com/jfindlay","followers_url":"https://api.github.com/users/jfindlay/followers","following_url":"https://api.github.com/users/jfindlay/following{/other_user}","gists_url":"https://api.github.com/users/jfindlay/gists{/gist_id}","starred_url":"https://api.github.com/users/jfindlay/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jfindlay/subscriptions","organizations_url":"https://api.github.com/users/jfindlay/orgs","repos_url":"https://api.github.com/users/jfindlay/repos","events_url":"https://api.github.com/users/jfindlay/events{/privacy}","received_events_url":"https://api.github.com/users/jfindlay/received_events","type":"User","site_admin":false},"created_at":"2016-09-15T16:22:05Z","updated_at":"2016-09-15T16:22:05Z","body":"@MadsRC, I'm not sure what motivated the change.  You may want to ask someone more knowledgeable like @cachedout or @rallytime."}}
{"actor":{"display_login":"jfindlay"}}