Skip to content

Conversation

@ehelms
Copy link
Member

@ehelms ehelms commented May 1, 2025

This is using Vagrant to provide a way to do external database testing. There is a command dedicated to setting up an external database in the development CLI. I am starting with a separate job focused on just testing the external database.

@ehelms ehelms force-pushed the test-external-database branch from ef3d1e3 to 766b80f Compare May 7, 2025 21:06
@ehelms
Copy link
Member Author

ehelms commented May 7, 2025

I updated this for now to be based off the other prototype (#143) that includes development CLI, that allows building in this deployment, e.g.

./forge vms start --vms database
./forge remote-database

Where the last command will configure a database host to be external for development and testing. This starts to feel like what Forklift could also be, or Forklift could evolve into this repo or a mix of the two.

@ehelms ehelms force-pushed the test-external-database branch from 766b80f to a86546d Compare May 8, 2025 18:48
@ehelms ehelms force-pushed the test-external-database branch from a86546d to 184a7c7 Compare November 3, 2025 20:38
@ehelms ehelms marked this pull request as ready for review November 3, 2025 20:40
@ehelms ehelms force-pushed the test-external-database branch 11 times, most recently from 0f102a9 to 57acf22 Compare November 7, 2025 19:27
@ehelms
Copy link
Member Author

ehelms commented Nov 7, 2025

This is passing now! This includes a commit that re-works how the test command generates the ssh-config to base it off the Ansible inventory instead of Vagrant. This addresses the error that Vagrant throws if not all boxes are provisioned when requesting ssh-config and makes the test command able to work with more environments.

@ehelms ehelms force-pushed the test-external-database branch 2 times, most recently from accd6f5 to 8b1dc76 Compare November 13, 2025 15:24
@ehelms
Copy link
Member Author

ehelms commented Nov 13, 2025

Added documentation

@ehelms ehelms force-pushed the test-external-database branch 2 times, most recently from 3d2d8b1 to 155e0d0 Compare November 14, 2025 22:18
@ehelms ehelms force-pushed the test-external-database branch 2 times, most recently from 0635182 to 6e21655 Compare November 25, 2025 16:27
@ehelms
Copy link
Member Author

ehelms commented Nov 25, 2025

This includes #326 since the database checks fail for foreman-proxy setup (which doesn't require a database).

@ehelms
Copy link
Member Author

ehelms commented Dec 2, 2025

I split out one of the commits as it would be useful beyond this -- #346

@evgeni evgeni force-pushed the test-external-database branch from 9c73b92 to 0793e5e Compare December 3, 2025 06:40
@evgeni evgeni force-pushed the test-external-database branch 2 times, most recently from f85528a to 0ff351e Compare December 3, 2025 07:42
Signed-off-by: Eric D. Helms <ericdhelms@gmail.com>
@evgeni evgeni force-pushed the test-external-database branch from 0ff351e to ca4a53f Compare December 3, 2025 08:18
@evgeni evgeni merged commit 41edc80 into theforeman:master Dec 3, 2025
9 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.

2 participants