Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

+ansible #469

Merged
merged 7 commits into from
Mar 8, 2023
Merged

+ansible #469

merged 7 commits into from
Mar 8, 2023

Conversation

ddobrinskiy
Copy link
Contributor

@ddobrinskiy ddobrinskiy commented Mar 7, 2023

Resolves #300

Stuff we need to do before merging:

@what-the-diff
Copy link

what-the-diff bot commented Mar 7, 2023

  • A new project was added to the projects directory.
  • The hosts file contains a single host, localhost with ansible_connection=local
  • package yml has been created for this project and it is using github tags as version source (versions:github)
  • It also defines dependencies on python 3+ and pipenv from pypa/pipenv repo which will be installed in virtual environment during build phase of tea-build process
  • Build script creates venv enviroment under {{prefix}}/venv path where all executables are copied into bin folder inside that venv dir so they can be used by other packages without conflicts between different versions of same executable name like 'ansible' or 'ansible-playbook'. This way we have multiple instances of these binaries available at once but only one instance per each binary name is visible outside the virtual environments because those names are registered in TEA registry database when building them via tea-build command line tool

@ddobrinskiy ddobrinskiy marked this pull request as draft March 7, 2023 15:56
@ddobrinskiy ddobrinskiy changed the title [DRAFT] +ansible +ansible Mar 7, 2023
@ddobrinskiy ddobrinskiy marked this pull request as ready for review March 8, 2023 18:36
@mxcl
Copy link
Member

mxcl commented Mar 8, 2023

Possibly symlinks would work. I dunno.

tests prove it works! I say LFG

@mxcl mxcl merged commit 966e702 into pkgxdev:main Mar 8, 2023
@ddobrinskiy ddobrinskiy deleted the ansible.com branch March 9, 2023 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

+ansible
2 participants