Skip to content

Install kayobe-config requirements.txt #55

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

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Install kayobe-config requirements.txt #55

merged 1 commit into from
Jun 21, 2024

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Jun 21, 2024

CIS benchmark hardening, and HashiCorp Vault playbooks require additional python dependencies.

CIS benchmark hardening, and HashiCorp Vault playbooks require
additional python dependencies.
@jovial jovial requested a review from a team as a code owner June 21, 2024 14:48
@jovial jovial merged commit 4c1dffd into main Jun 21, 2024
@jovial jovial deleted the bugfix/requirements branch June 21, 2024 15:27
jovial added a commit that referenced this pull request Jun 26, 2024
A regression was introduced in:

#55

which meant that the Kayobe docker image failed to build. This is
because requirements.txt was updated to point a local path that did not
exist inside the docker image during the build. I've changed the logic
to update the requirements.txt to use the ansible variables that
configure the kayobe checkout.
markgoddard added a commit that referenced this pull request Jul 24, 2024
Yoga & earlier deployments have been broken since the following related
changes:

#55
#59
#60

The above changes switched to installing Kayobe using SKC's
requirements.txt, avoiding the need for a local copy of the repository.
This breaks Yoga & earlier releases which require symlinks in
etc/kayobe/ansible/ to correctly resolve to a Kayobe source repo.

This change reverts to cloning a Kayobe repository when using Yoga &
earlier releases.
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.

2 participants