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

Feature/no more vm #456

Merged
merged 7 commits into from
Dec 16, 2024
Merged

Feature/no more vm #456

merged 7 commits into from
Dec 16, 2024

Conversation

quartje
Copy link
Contributor

@quartje quartje commented Oct 9, 2024

This repo has changed a lot in the last year, since we went from a CentOS7 VM based install to a Docker based approach on Debian 12. A lot of the documentation, VM setup and tests have become obsolete since then.

This PR also simplifies the provision script. The only argument needed is the environment you want to use.

Two elements are currently missing, which were present before:

  1. Installation of the roles was tested using molecule. It was used to install the roles and check if they were still working
  2. There are scripts and skeletons available to start from scratch. These have not been used or tested in quite a while, installation will fail if you use the current documentation.

The wiki should be updated as well after merging this PR.

There is no more Vagrant support, and all applications run from Docker.
The docs have been updated to reflect this
- Remove Vagrant VM stuff
- Remove remove secrets_file location and use a default value
- Remove the user argument for the provision script
- enviromment_dir is no longer needed. The ansible magic var
  inventory_dir is reintroduced. You need to add localhost to the
inventory for this to work
@quartje quartje force-pushed the feature/no_more_vm branch from 17ac40c to c19587f Compare October 9, 2024 09:42
@quartje quartje requested a review from baszoetekouw October 9, 2024 09:43
Copy link
Member

@baszoetekouw baszoetekouw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lijkt me prima @quartje
Kun jij nog even de conflicts fixen, dan merge ik hem

@quartje quartje merged commit 97e5749 into main Dec 16, 2024
3 checks passed
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.

None yet

2 participants