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

Ensure Ansible installed with Python3.9 #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

brucellino
Copy link

This fixes mmul-it/kpa#5

This PR contains:

  1. Packer template (written to correspond to Dockerfile) -- however ensures pip3 with python3.9 is invoked when installing ansible
  2. pre-commit hooks and sundry

The packer template diverges from the Dockerfile in that it makes greater use of declarative versions -- e.g. the requirements.txt file for the image, etc.

@brucellino
Copy link
Author

@rascasoft can you review this please? It contains updates to the pipeline so that you can build the image with packer (nothing has been removed.)

The pipeline is also triggered on PRs now, instead of only main branch commits -- no pushes occur on that yet to the docker registry.

When this is merged, we can deal with the semantic release bits

@rascasoft
Copy link
Member

@brucellino I've implemented a way of managing releases that should fit in what we have right now, without the need of integrating anything else.
It is all part of the v1.0.0, specifically in the d1e80a7 commit which takes the branch and generate the tags.
I would like to test this infrastructure and see if it fits our need, so lets put this in the fridge for the moment.

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.

Fix Python version to 3.9 inside the kpa container
2 participants