{"action":"created","comment":{"url":"https://api.github.com/repos/openSUSE/salt/pulls/comments/243281033","pull_request_review_id":187008014,"id":243281033,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDI0MzI4MTAzMw==","diff_hunk":"@@ -54,8 +54,11 @@ def get_osarch():\n             stderr=subprocess.PIPE).communicate()[0]\n     else:\n         ret = ''.join(list(filter(None, platform.uname()[-2:]))[-1:])\n-\n-    return salt.utils.stringutils.to_str(ret).strip() or 'unknown'\n+    ret = salt.utils.stringutils.to_str(ret).strip() or 'unknown'\n+    ARCH_FIXES_MAPPING = {\n+        \"powerpc64le\": \"ppc64le\"\n+    }\n+    return ARCH_FIXES_MAPPING.get(ret, ret)","path":"salt/utils/pkg/rpm.py","position":10,"original_position":10,"commit_id":"7429243b7f1b88846a9fe1d7878f9f31cc3b7d43","original_commit_id":"7429243b7f1b88846a9fe1d7878f9f31cc3b7d43","user":{"login":"kkaempf","id":37742,"node_id":"MDQ6VXNlcjM3NzQy","avatar_url":"https://avatars2.githubusercontent.com/u/37742?v=4","gravatar_id":"","url":"https://api.github.com/users/kkaempf","html_url":"https://github.com/kkaempf","followers_url":"https://api.github.com/users/kkaempf/followers","following_url":"https://api.github.com/users/kkaempf/following{/other_user}","gists_url":"https://api.github.com/users/kkaempf/gists{/gist_id}","starred_url":"https://api.github.com/users/kkaempf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kkaempf/subscriptions","organizations_url":"https://api.github.com/users/kkaempf/orgs","repos_url":"https://api.github.com/users/kkaempf/repos","events_url":"https://api.github.com/users/kkaempf/events{/privacy}","received_events_url":"https://api.github.com/users/kkaempf/received_events","type":"User","site_admin":false},"body":"What does the `rpm` maintainer say about this ? Did anyone ask ?\r\n```\r\npower-novalink:~ #  rpm --eval '%{_arch}'\r\nppc64le\r\npower-novalink:~ # rpm --eval '%{_host_cpu}'\r\npowerpc64le\r\n```","created_at":"2018-12-20T14:01:06Z","updated_at":"2018-12-20T14:01:07Z","html_url":"https://github.com/openSUSE/salt/pull/116#discussion_r243281033","pull_request_url":"https://api.github.com/repos/openSUSE/salt/pulls/116","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/openSUSE/salt/pulls/comments/243281033"},"html":{"href":"https://github.com/openSUSE/salt/pull/116#discussion_r243281033"},"pull_request":{"href":"https://api.github.com/repos/openSUSE/salt/pulls/116"}},"in_reply_to_id":241378964},"pull_request":{"url":"https://api.github.com/repos/openSUSE/salt/pulls/116","id":238354562,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM4MzU0NTYy","html_url":"https://github.com/openSUSE/salt/pull/116","diff_url":"https://github.com/openSUSE/salt/pull/116.diff","patch_url":"https://github.com/openSUSE/salt/pull/116.patch","issue_url":"https://api.github.com/repos/openSUSE/salt/issues/116","number":116,"state":"open","locked":false,"title":"[WIP] Return the expected powerpc os arch (bsc#1117995)","user":{"login":"dincamihai","id":1615643,"node_id":"MDQ6VXNlcjE2MTU2NDM=","avatar_url":"https://avatars3.githubusercontent.com/u/1615643?v=4","gravatar_id":"","url":"https://api.github.com/users/dincamihai","html_url":"https://github.com/dincamihai","followers_url":"https://api.github.com/users/dincamihai/followers","following_url":"https://api.github.com/users/dincamihai/following{/other_user}","gists_url":"https://api.github.com/users/dincamihai/gists{/gist_id}","starred_url":"https://api.github.com/users/dincamihai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dincamihai/subscriptions","organizations_url":"https://api.github.com/users/dincamihai/orgs","repos_url":"https://api.github.com/users/dincamihai/repos","events_url":"https://api.github.com/users/dincamihai/events{/privacy}","received_events_url":"https://api.github.com/users/dincamihai/received_events","type":"User","site_admin":false},"body":"## I open this PR in order to keep track of the PTF-quick-fix and to discuss the proper solution for [bsc#1117995](https://bugzilla.suse.com/show_bug.cgi?id=1117995)\r\n\r\n\r\n### What does this PR do?\r\nFixes https://bugzilla.suse.com/show_bug.cgi?id=1117995\r\n\r\n`grains.get osarch` returns now powerpc64le which comes from `rpm --eval \"%{_host_cpu}\"`, before is was the same as `cpuarch` grain (ppc64le)\r\n\r\n\r\n\r\n### What issues does this PR fix or reference?\r\n\r\nThis bug was triggered by the following changes:\r\n  - https://github.com/SUSE/salt-board/issues/119\r\n  - https://github.com/saltstack/salt/commit/746d2be3902fd311b34cd32259cfdeac2508d800 https://bugzilla.suse.com/show_bug.cgi?id=1114197\r\n\r\n### Previous Behavior\r\nRemove this section if not relevant\r\n\r\n### New Behavior\r\nRemove this section if not relevant\r\n\r\n### Tests written?\r\n\r\nYes/No\r\n\r\n### Commits signed with GPG?\r\n\r\nYes/No\r\n\r\nPlease review [Salt's Contributing Guide](https://docs.saltstack.com/en/latest/topics/development/contributing.html) for best practices.\r\n\r\nSee GitHub's [page on GPG signing](https://help.github.com/articles/signing-commits-using-gpg/) for more information about signing commits with GPG.\r\n","created_at":"2018-12-13T11:25:39Z","updated_at":"2018-12-20T14:01:07Z","closed_at":null,"merged_at":null,"merge_commit_sha":"d6f22e02edfa7bda7c8a4d86cd3dd97d260f8e04","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/openSUSE/salt/pulls/116/commits","review_comments_url":"https://api.github.com/repos/openSUSE/salt/pulls/116/comments","review_comment_url":"https://api.github.com/repos/openSUSE/salt/pulls/comments{/number}","comments_url":"https://api.github.com/repos/openSUSE/salt/issues/116/comments","statuses_url":"https://api.github.com/repos/openSUSE/salt/statuses/7429243b7f1b88846a9fe1d7878f9f31cc3b7d43","head":{"label":"openSUSE:ptf-bsc-1117995","ref":"ptf-bsc-1117995","sha":"7429243b7f1b88846a9fe1d7878f9f31cc3b7d43","user":{"login":"openSUSE","id":623819,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyMzgxOQ==","avatar_url":"https://avatars2.githubusercontent.com/u/623819?v=4","gravatar_id":"","url":"https://api.github.com/users/openSUSE","html_url":"https://github.com/openSUSE","followers_url":"https://api.github.com/users/openSUSE/followers","following_url":"https://api.github.com/users/openSUSE/following{/other_user}","gists_url":"https://api.github.com/users/openSUSE/gists{/gist_id}","starred_url":"https://api.github.com/users/openSUSE/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openSUSE/subscriptions","organizations_url":"https://api.github.com/users/openSUSE/orgs","repos_url":"https://api.github.com/users/openSUSE/repos","events_url":"https://api.github.com/users/openSUSE/events{/privacy}","received_events_url":"https://api.github.com/users/openSUSE/received_events","type":"Organization","site_admin":false},"repo":{"id":43946944,"node_id":"MDEwOlJlcG9zaXRvcnk0Mzk0Njk0NA==","name":"salt","full_name":"openSUSE/salt","private":false,"owner":{"login":"openSUSE","id":623819,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyMzgxOQ==","avatar_url":"https://avatars2.githubusercontent.com/u/623819?v=4","gravatar_id":"","url":"https://api.github.com/users/openSUSE","html_url":"https://github.com/openSUSE","followers_url":"https://api.github.com/users/openSUSE/followers","following_url":"https://api.github.com/users/openSUSE/following{/other_user}","gists_url":"https://api.github.com/users/openSUSE/gists{/gist_id}","starred_url":"https://api.github.com/users/openSUSE/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openSUSE/subscriptions","organizations_url":"https://api.github.com/users/openSUSE/orgs","repos_url":"https://api.github.com/users/openSUSE/repos","events_url":"https://api.github.com/users/openSUSE/events{/privacy}","received_events_url":"https://api.github.com/users/openSUSE/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/openSUSE/salt","description":"SUSE patches and backports for SaltStack","fork":false,"url":"https://api.github.com/repos/openSUSE/salt","forks_url":"https://api.github.com/repos/openSUSE/salt/forks","keys_url":"https://api.github.com/repos/openSUSE/salt/keys{/key_id}","collaborators_url":"https://api.github.com/repos/openSUSE/salt/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/openSUSE/salt/teams","hooks_url":"https://api.github.com/repos/openSUSE/salt/hooks","issue_events_url":"https://api.github.com/repos/openSUSE/salt/issues/events{/number}","events_url":"https://api.github.com/repos/openSUSE/salt/events","assignees_url":"https://api.github.com/repos/openSUSE/salt/assignees{/user}","branches_url":"https://api.github.com/repos/openSUSE/salt/branches{/branch}","tags_url":"https://api.github.com/repos/openSUSE/salt/tags","blobs_url":"https://api.github.com/repos/openSUSE/salt/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/openSUSE/salt/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/openSUSE/salt/git/refs{/sha}","trees_url":"https://api.github.com/repos/openSUSE/salt/git/trees{/sha}","statuses_url":"https://api.github.com/repos/openSUSE/salt/statuses/{sha}","languages_url":"https://api.github.com/repos/openSUSE/salt/languages","stargazers_url":"https://api.github.com/repos/openSUSE/salt/stargazers","contributors_url":"https://api.github.com/repos/openSUSE/salt/contributors","subscribers_url":"https://api.github.com/repos/openSUSE/salt/subscribers","subscription_url":"https://api.github.com/repos/openSUSE/salt/subscription","commits_url":"https://api.github.com/repos/openSUSE/salt/commits{/sha}","git_commits_url":"https://api.github.com/repos/openSUSE/salt/git/commits{/sha}","comments_url":"https://api.github.com/repos/openSUSE/salt/comments{/number}","issue_comment_url":"https://api.github.com/repos/openSUSE/salt/issues/comments{/number}","contents_url":"https://api.github.com/repos/openSUSE/salt/contents/{+path}","compare_url":"https://api.github.com/repos/openSUSE/salt/compare/{base}...{head}","merges_url":"https://api.github.com/repos/openSUSE/salt/merges","archive_url":"https://api.github.com/repos/openSUSE/salt/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/openSUSE/salt/downloads","issues_url":"https://api.github.com/repos/openSUSE/salt/issues{/number}","pulls_url":"https://api.github.com/repos/openSUSE/salt/pulls{/number}","milestones_url":"https://api.github.com/repos/openSUSE/salt/milestones{/number}","notifications_url":"https://api.github.com/repos/openSUSE/salt/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/openSUSE/salt/labels{/name}","releases_url":"https://api.github.com/repos/openSUSE/salt/releases{/id}","deployments_url":"https://api.github.com/repos/openSUSE/salt/deployments","created_at":"2015-10-09T09:47:37Z","updated_at":"2018-12-18T12:47:55Z","pushed_at":"2018-12-20T11:18:10Z","git_url":"git://github.com/openSUSE/salt.git","ssh_url":"git@github.com:openSUSE/salt.git","clone_url":"https://github.com/openSUSE/salt.git","svn_url":"https://github.com/openSUSE/salt","homepage":null,"size":247128,"stargazers_count":9,"watchers_count":9,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":26,"mirror_url":null,"archived":false,"open_issues_count":3,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":26,"open_issues":3,"watchers":9,"default_branch":"openSUSE-2018.3.0"}},"base":{"label":"openSUSE:openSUSE-2018.3.0","ref":"openSUSE-2018.3.0","sha":"2e7358326d91c4bb97275f82c06ad35d40cac2fa","user":{"login":"openSUSE","id":623819,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyMzgxOQ==","avatar_url":"https://avatars2.githubusercontent.com/u/623819?v=4","gravatar_id":"","url":"https://api.github.com/users/openSUSE","html_url":"https://github.com/openSUSE","followers_url":"https://api.github.com/users/openSUSE/followers","following_url":"https://api.github.com/users/openSUSE/following{/other_user}","gists_url":"https://api.github.com/users/openSUSE/gists{/gist_id}","starred_url":"https://api.github.com/users/openSUSE/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openSUSE/subscriptions","organizations_url":"https://api.github.com/users/openSUSE/orgs","repos_url":"https://api.github.com/users/openSUSE/repos","events_url":"https://api.github.com/users/openSUSE/events{/privacy}","received_events_url":"https://api.github.com/users/openSUSE/received_events","type":"Organization","site_admin":false},"repo":{"id":43946944,"node_id":"MDEwOlJlcG9zaXRvcnk0Mzk0Njk0NA==","name":"salt","full_name":"openSUSE/salt","private":false,"owner":{"login":"openSUSE","id":623819,"node_id":"MDEyOk9yZ2FuaXphdGlvbjYyMzgxOQ==","avatar_url":"https://avatars2.githubusercontent.com/u/623819?v=4","gravatar_id":"","url":"https://api.github.com/users/openSUSE","html_url":"https://github.com/openSUSE","followers_url":"https://api.github.com/users/openSUSE/followers","following_url":"https://api.github.com/users/openSUSE/following{/other_user}","gists_url":"https://api.github.com/users/openSUSE/gists{/gist_id}","starred_url":"https://api.github.com/users/openSUSE/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openSUSE/subscriptions","organizations_url":"https://api.github.com/users/openSUSE/orgs","repos_url":"https://api.github.com/users/openSUSE/repos","events_url":"https://api.github.com/users/openSUSE/events{/privacy}","received_events_url":"https://api.github.com/users/openSUSE/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/openSUSE/salt","description":"SUSE patches and backports for SaltStack","fork":false,"url":"https://api.github.com/repos/openSUSE/salt","forks_url":"https://api.github.com/repos/openSUSE/salt/forks","keys_url":"https://api.github.com/repos/openSUSE/salt/keys{/key_id}","collaborators_url":"https://api.github.com/repos/openSUSE/salt/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/openSUSE/salt/teams","hooks_url":"https://api.github.com/repos/openSUSE/salt/hooks","issue_events_url":"https://api.github.com/repos/openSUSE/salt/issues/events{/number}","events_url":"https://api.github.com/repos/openSUSE/salt/events","assignees_url":"https://api.github.com/repos/openSUSE/salt/assignees{/user}","branches_url":"https://api.github.com/repos/openSUSE/salt/branches{/branch}","tags_url":"https://api.github.com/repos/openSUSE/salt/tags","blobs_url":"https://api.github.com/repos/openSUSE/salt/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/openSUSE/salt/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/openSUSE/salt/git/refs{/sha}","trees_url":"https://api.github.com/repos/openSUSE/salt/git/trees{/sha}","statuses_url":"https://api.github.com/repos/openSUSE/salt/statuses/{sha}","languages_url":"https://api.github.com/repos/openSUSE/salt/languages","stargazers_url":"https://api.github.com/repos/openSUSE/salt/stargazers","contributors_url":"https://api.github.com/repos/openSUSE/salt/contributors","subscribers_url":"https://api.github.com/repos/openSUSE/salt/subscribers","subscription_url":"https://api.github.com/repos/openSUSE/salt/subscription","commits_url":"https://api.github.com/repos/openSUSE/salt/commits{/sha}","git_commits_url":"https://api.github.com/repos/openSUSE/salt/git/commits{/sha}","comments_url":"https://api.github.com/repos/openSUSE/salt/comments{/number}","issue_comment_url":"https://api.github.com/repos/openSUSE/salt/issues/comments{/number}","contents_url":"https://api.github.com/repos/openSUSE/salt/contents/{+path}","compare_url":"https://api.github.com/repos/openSUSE/salt/compare/{base}...{head}","merges_url":"https://api.github.com/repos/openSUSE/salt/merges","archive_url":"https://api.github.com/repos/openSUSE/salt/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/openSUSE/salt/downloads","issues_url":"https://api.github.com/repos/openSUSE/salt/issues{/number}","pulls_url":"https://api.github.com/repos/openSUSE/salt/pulls{/number}","milestones_url":"https://api.github.com/repos/openSUSE/salt/milestones{/number}","notifications_url":"https://api.github.com/repos/openSUSE/salt/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/openSUSE/salt/labels{/name}","releases_url":"https://api.github.com/repos/openSUSE/salt/releases{/id}","deployments_url":"https://api.github.com/repos/openSUSE/salt/deployments","created_at":"2015-10-09T09:47:37Z","updated_at":"2018-12-18T12:47:55Z","pushed_at":"2018-12-20T11:18:10Z","git_url":"git://github.com/openSUSE/salt.git","ssh_url":"git@github.com:openSUSE/salt.git","clone_url":"https://github.com/openSUSE/salt.git","svn_url":"https://github.com/openSUSE/salt","homepage":null,"size":247128,"stargazers_count":9,"watchers_count":9,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":26,"mirror_url":null,"archived":false,"open_issues_count":3,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":26,"open_issues":3,"watchers":9,"default_branch":"openSUSE-2018.3.0"}},"_links":{"self":{"href":"https://api.github.com/repos/openSUSE/salt/pulls/116"},"html":{"href":"https://github.com/openSUSE/salt/pull/116"},"issue":{"href":"https://api.github.com/repos/openSUSE/salt/issues/116"},"comments":{"href":"https://api.github.com/repos/openSUSE/salt/issues/116/comments"},"review_comments":{"href":"https://api.github.com/repos/openSUSE/salt/pulls/116/comments"},"review_comment":{"href":"https://api.github.com/repos/openSUSE/salt/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/openSUSE/salt/pulls/116/commits"},"statuses":{"href":"https://api.github.com/repos/openSUSE/salt/statuses/7429243b7f1b88846a9fe1d7878f9f31cc3b7d43"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"kkaempf"}}