ansible / mazer

Experimental Ansible Galaxy Content Manager
GNU General Public License v3.0
113 stars 18 forks source link

Remove any remaining unused traditional role related code, incl direct inst of trad roles #202

Closed alikins closed 5 years ago

alikins commented 5 years ago

Bug Report

There is a fair amount of code in mazer that is for loading traditional roles, parsing role data, and installing traditional roles.

Some of that is needed for 'migrate_role', but a lot of it will not be used long term. Some of the support for directly installing traditional style roles (as in 'mazer install geerlingguy.ntp' which is separate from code for supporting installing collections that may include collection style roles).

There is a lot of code related to parsing of trad role names and uri's on the cli, and lots of code for parsing meta/main.yml and requirements.yml that can likely be removed.

Code that is still needed for the purposes of 'migrate_role' can be isolated/extracted (for ex, to ansible_galaxy/role/).

The existing code is often intertwined with collection support code which ends up making the code more complicated and harder to debug, maintain, replace, and extend in the future.

MAZER VERSION
name = mazer
version = 0.3.0
config_file = /home/adrian/.ansible/mazer.yml
uname = Linux, newswoop, 4.18.16-100.fc27.x86_64, #1 SMP Sun Oct 21 09:33:00 UTC 2018, x86_64
executable_location = /home/adrian/venvs/galaxy-cli-py3-2/bin/mazer
python_version = 3.6.6 (default, Jul 19 2018, 16:29:00) [GCC 7.3.1 20180303 (Red Hat 7.3.1-5)]
python_executable = /home/adrian/venvs/galaxy-cli-py3-2/bin/python3.6