{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/31445","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/31445/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/31445/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/31445/events","html_url":"https://github.com/saltstack/salt/pull/31445","id":135911658,"number":31445,"title":"Only use LONGSIZE in rpm.info if available. Otherwise, use SIZE.","user":{"login":"rallytime","id":6547906,"avatar_url":"https://avatars.githubusercontent.com/u/6547906?v=3","gravatar_id":"","url":"https://api.github.com/users/rallytime","html_url":"https://github.com/rallytime","followers_url":"https://api.github.com/users/rallytime/followers","following_url":"https://api.github.com/users/rallytime/following{/other_user}","gists_url":"https://api.github.com/users/rallytime/gists{/gist_id}","starred_url":"https://api.github.com/users/rallytime/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rallytime/subscriptions","organizations_url":"https://api.github.com/users/rallytime/orgs","repos_url":"https://api.github.com/users/rallytime/repos","events_url":"https://api.github.com/users/rallytime/events{/privacy}","received_events_url":"https://api.github.com/users/rallytime/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/saltstack/salt/labels/Pending%20Changes","name":"Pending Changes","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-24T00:30:46Z","updated_at":"2016-02-24T16:21:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/saltstack/salt/pulls/31445","html_url":"https://github.com/saltstack/salt/pull/31445","diff_url":"https://github.com/saltstack/salt/pull/31445.diff","patch_url":"https://github.com/saltstack/salt/pull/31445.patch"},"body":"Fixes #31366 \r\n\r\nI have tested this manually on CentOS 6 and CentsOS 5, but I want to write an automated test. I started a test in #31439, which I want to add to here once that is merged. At that time I'll rebase and add the test.\r\n\r\nWithout this change, a call to `lowpkg.info` gets an error from the `cmd.run_all` call stating a tag is invalid, which is LONGSIZE. This change fixes error and returns successfully. Also added a more helpful error catch so we don't just return `None`  if there is information in the `stderr`, but we still got a `retcode: 0`.\r\n```\r\n# salt-call --local lowpkg.info git\r\n<snipped>\r\nlocal:\r\n    ----------\r\n    git:\r\n        ----------\r\n        arch:\r\n            x86_64\r\n        build_date:\r\n            2013-11-21T19:57:31Z\r\n        build_date_time_t:\r\n            1385081851\r\n        build_host:\r\n            lisse.hasselt.wieers.com\r\n        description:\r\n            GIT comes in two layers. The bottom layer is merely an extremely fast\r\n            and flexible filesystem-based database designed to store directory trees\r\n            with regard to their history. The top layer is a SCM-like tool which\r\n            enables human beings to work with the database in a manner to a degree\r\n            similar to other SCM tools (like CVS, BitKeeper or Monotone).\r\n        group:\r\n            Development/Tools\r\n        install_date:\r\n            2016-02-23T16:27:28Z\r\n        install_date_time_t:\r\n            1456262848\r\n        license:\r\n            GPL\r\n        packager:\r\n            Dag Wieers <dag@wieers.com>\r\n        release:\r\n            1.el5.rf\r\n        relocations:\r\n            (not relocatable)\r\n        signature:\r\n            DSA/SHA1, Fri Nov 22 01:25:29 2013, Key ID a20e52146b8d79e6\r\n        size:\r\n            17639877\r\n        source_rpm:\r\n            git-1.7.12.4-1.el5.rf.src.rpm\r\n        summary:\r\n            Git core and tools\r\n        url:\r\n            http://git-scm.com/\r\n        vendor:\r\n            Dag Apt Repository, http://dag.wieers.com/apt/\r\n        version:\r\n            1.7.12.4\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/188332068","html_url":"https://github.com/saltstack/salt/pull/31445#issuecomment-188332068","issue_url":"https://api.github.com/repos/saltstack/salt/issues/31445","id":188332068,"user":{"login":"cachedout","id":111616,"avatar_url":"https://avatars.githubusercontent.com/u/111616?v=3","gravatar_id":"","url":"https://api.github.com/users/cachedout","html_url":"https://github.com/cachedout","followers_url":"https://api.github.com/users/cachedout/followers","following_url":"https://api.github.com/users/cachedout/following{/other_user}","gists_url":"https://api.github.com/users/cachedout/gists{/gist_id}","starred_url":"https://api.github.com/users/cachedout/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cachedout/subscriptions","organizations_url":"https://api.github.com/users/cachedout/orgs","repos_url":"https://api.github.com/users/cachedout/repos","events_url":"https://api.github.com/users/cachedout/events{/privacy}","received_events_url":"https://api.github.com/users/cachedout/received_events","type":"User","site_admin":false},"created_at":"2016-02-24T16:21:24Z","updated_at":"2016-02-24T16:21:24Z","body":"@terminalmage Could you review this, please?"}}