{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/47716","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/47716/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/47716/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/47716/events","html_url":"https://github.com/saltstack/salt/pull/47716","id":324261846,"node_id":"MDExOlB1bGxSZXF1ZXN0MTg4OTE1NzM0","number":47716,"title":"add docker proxy minion","user":{"login":"gtmanfred","id":732321,"node_id":"MDQ6VXNlcjczMjMyMQ==","avatar_url":"https://avatars0.githubusercontent.com/u/732321?v=4","gravatar_id":"","url":"https://api.github.com/users/gtmanfred","html_url":"https://github.com/gtmanfred","followers_url":"https://api.github.com/users/gtmanfred/followers","following_url":"https://api.github.com/users/gtmanfred/following{/other_user}","gists_url":"https://api.github.com/users/gtmanfred/gists{/gist_id}","starred_url":"https://api.github.com/users/gtmanfred/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gtmanfred/subscriptions","organizations_url":"https://api.github.com/users/gtmanfred/orgs","repos_url":"https://api.github.com/users/gtmanfred/repos","events_url":"https://api.github.com/users/gtmanfred/events{/privacy}","received_events_url":"https://api.github.com/users/gtmanfred/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2018-05-18T03:54:04Z","updated_at":"2018-06-04T19:59:07Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/saltstack/salt/pulls/47716","html_url":"https://github.com/saltstack/salt/pull/47716","diff_url":"https://github.com/saltstack/salt/pull/47716.diff","patch_url":"https://github.com/saltstack/salt/pull/47716.patch"},"body":"### What does this PR do?\r\nThis proxy minion + executor combo allows for using the docker.call module to run salt modules inside of docker containers.\r\n\r\nThis also adds the ability to run a highstate on the docker container.\r\n\r\n### Tests written?\r\n\r\nYes\r\n\r\n### Commits signed with GPG?\r\n\r\nYes"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/394479534","html_url":"https://github.com/saltstack/salt/pull/47716#issuecomment-394479534","issue_url":"https://api.github.com/repos/saltstack/salt/issues/47716","id":394479534,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NDQ3OTUzNA==","user":{"login":"gtmanfred","id":732321,"node_id":"MDQ6VXNlcjczMjMyMQ==","avatar_url":"https://avatars0.githubusercontent.com/u/732321?v=4","gravatar_id":"","url":"https://api.github.com/users/gtmanfred","html_url":"https://github.com/gtmanfred","followers_url":"https://api.github.com/users/gtmanfred/followers","following_url":"https://api.github.com/users/gtmanfred/following{/other_user}","gists_url":"https://api.github.com/users/gtmanfred/gists{/gist_id}","starred_url":"https://api.github.com/users/gtmanfred/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gtmanfred/subscriptions","organizations_url":"https://api.github.com/users/gtmanfred/orgs","repos_url":"https://api.github.com/users/gtmanfred/repos","events_url":"https://api.github.com/users/gtmanfred/events{/privacy}","received_events_url":"https://api.github.com/users/gtmanfred/received_events","type":"User","site_admin":false},"created_at":"2018-06-04T19:59:07Z","updated_at":"2018-06-04T19:59:07Z","author_association":"MEMBER","body":"Ok, this has been failing for a while.\r\n\r\nIt was just not caught, because `self.assertTrue` was seeing the return of test.ping as true, even though it was just returning a string about not being able to import yaml.\r\n\r\nIt looks like it is being caused by the changes that were made to making the thing tarball for salt-ssh, by @isbm \r\n\r\n```\r\n       ('ERROR: Traceback (most recent call last):\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/salt-call\", line 27, in <module>\\n'\r\n        '    salt_call()\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/scripts.py\", line 395, in '\r\n        'salt_call\\n'\r\n        '    import salt.cli.call\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/cli/call.py\", line 5, in '\r\n        '<module>\\n'\r\n        '    import salt.utils.parsers\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/utils/parsers.py\", line 27, in '\r\n        '<module>\\n'\r\n        '    import salt.config as config\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/config/__init__.py\", line 24, '\r\n        'in <module>\\n'\r\n        '    import salt.utils.data\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/utils/data.py\", line 17, in '\r\n        '<module>\\n'\r\n        '    import salt.utils.dictupdate\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/utils/dictupdate.py\", line 15, '\r\n        'in <module>\\n'\r\n        '    from salt.serializers.yamlex import merge_recursive as '\r\n        '_yamlex_merge_recursive\\n'\r\n        '  File \"/tmp/salt.docker.2694e1/pyall/salt/serializers/yamlex.py\", line '\r\n        '118, in <module>\\n'\r\n        '    import yaml\\n'\r\n        'ImportError: No module named yaml')\r\n```\r\n\r\nBo, do you have time to look at this error?  I am not sure what the changes were made for using `pyall` and why it isn't finding the `yaml` import on python3 versions using docker.call.\r\n\r\nThanks,\r\nDaniel"}}
{"actor":{"display_login":"gtmanfred"}}