{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/35309","repository_url":"https://api.github.com/repos/ansible/ansible","labels_url":"https://api.github.com/repos/ansible/ansible/issues/35309/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/35309/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/35309/events","html_url":"https://github.com/ansible/ansible/pull/35309","id":291342744,"number":35309,"title":"[WIP] Add `--separate-gvars` option for inventory --list","user":{"login":"ryansb","id":636610,"avatar_url":"https://avatars1.githubusercontent.com/u/636610?v=4","gravatar_id":"","url":"https://api.github.com/users/ryansb","html_url":"https://github.com/ryansb","followers_url":"https://api.github.com/users/ryansb/followers","following_url":"https://api.github.com/users/ryansb/following{/other_user}","gists_url":"https://api.github.com/users/ryansb/gists{/gist_id}","starred_url":"https://api.github.com/users/ryansb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ryansb/subscriptions","organizations_url":"https://api.github.com/users/ryansb/orgs","repos_url":"https://api.github.com/users/ryansb/repos","events_url":"https://api.github.com/users/ryansb/events{/privacy}","received_events_url":"https://api.github.com/users/ryansb/received_events","type":"User","site_admin":false},"labels":[{"id":529643857,"url":"https://api.github.com/repos/ansible/ansible/labels/WIP","name":"WIP","color":"fef2c0","default":false},{"id":530145226,"url":"https://api.github.com/repos/ansible/ansible/labels/ci_verified","name":"ci_verified","color":"b60205","default":false},{"id":48457821,"url":"https://api.github.com/repos/ansible/ansible/labels/feature_pull_request","name":"feature_pull_request","color":"006b75","default":false},{"id":636352796,"url":"https://api.github.com/repos/ansible/ansible/labels/support:core","name":"support:core","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-01-24T20:07:51Z","updated_at":"2018-02-06T04:35:33Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/ansible/ansible/pulls/35309","html_url":"https://github.com/ansible/ansible/pull/35309","diff_url":"https://github.com/ansible/ansible/pull/35309.diff","patch_url":"https://github.com/ansible/ansible/pull/35309.patch"},"body":"This display option doesn't change the interpretation of the variables\r\nor precedence, but displays group variables on the associated group\r\ninstead of flattening and repeating the variable on each host in the\r\ngroup.\r\n\r\n[DON'T MERGE THIS]\r\n\r\n##### SUMMARY\r\n<!--- Describe the change, including rationale and design decisions -->\r\n\r\n<!---\r\nIf you are fixing an existing issue, please include \"Fixes #nnn\" in your\r\ncommit message and your description; but you should still explain what\r\nthe change does.\r\n-->\r\n\r\n##### ISSUE TYPE\r\n<!--- Pick one below and delete the rest: -->\r\n - Feature Pull Request\r\n\r\n##### COMPONENT NAME\r\n<!--- Name of the module, plugin, module or task -->\r\nansible-inventory\r\n\r\n##### ANSIBLE VERSION\r\n<!--- Paste verbatim output from \"ansible --version\" between quotes below -->\r\n```\r\nansible 2.5.0 (efficient-gvars-display 6d367da6c2) last updated 2018/01/24 14:53:13 (GMT -400)\r\n  config file = /home/ryansb/code/scratch/ec2_inv/ansible.cfg\r\n  configured module search path = [u'/home/ryansb/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']\r\n  ansible python module location = /home/ryansb/code/ansible/lib/ansible\r\n  executable location = /home/ryansb/code/ansible/bin/ansible\r\n  python version = 2.7.12 (default, Sep 27 2016, 18:08:33) [GCC 6.2.1 20160916 (Red Hat 6.2.1-2)]\r\n```\r\n\r\n\r\n##### ADDITIONAL INFORMATION\r\n<!---\r\nInclude additional information to help people understand the change here.\r\nFor bugs that don't have a linked bug report, a step-by-step reproduction\r\nof the problem is helpful.\r\n  -->\r\nOne option for #30877\r\n<!--- Paste verbatim command output below, e.g. before and after your change -->\r\n```\r\n\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/363308300","html_url":"https://github.com/ansible/ansible/pull/35309#issuecomment-363308300","issue_url":"https://api.github.com/repos/ansible/ansible/issues/35309","id":363308300,"user":{"login":"bcoca","id":836171,"avatar_url":"https://avatars1.githubusercontent.com/u/836171?v=4","gravatar_id":"","url":"https://api.github.com/users/bcoca","html_url":"https://github.com/bcoca","followers_url":"https://api.github.com/users/bcoca/followers","following_url":"https://api.github.com/users/bcoca/following{/other_user}","gists_url":"https://api.github.com/users/bcoca/gists{/gist_id}","starred_url":"https://api.github.com/users/bcoca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bcoca/subscriptions","organizations_url":"https://api.github.com/users/bcoca/orgs","repos_url":"https://api.github.com/users/bcoca/repos","events_url":"https://api.github.com/users/bcoca/events{/privacy}","received_events_url":"https://api.github.com/users/bcoca/received_events","type":"User","site_admin":false},"created_at":"2018-02-06T04:35:33Z","updated_at":"2018-02-06T04:35:33Z","author_association":"MEMBER","body":"he, this was 'on my list' .... but a few things, there are 2 'group vars', one that is provided by inventory directly and another that is provided by the 'vars plugins', I'm not sure if we should merge those or keep separate.\r\n\r\nAlso to eliminate redundancy you cannot use vars manager to get the 'host vars' in this case, you need to partially reproduce it's logic here, but only the parts for inventory and vars plugins, with the same question for host_vars/ as we had for group_vars/ above.\r\n\r\nbikeshedding: was thinking --optimized or --source-layout as its not just '--group vars'."}}
{"actor":{"display_login":"bcoca"}}