Skip to content

Conversation

@tacerus
Copy link
Member

@tacerus tacerus commented Jan 16, 2026

Vagrant is no longer shipped in Factory and new formulas already were not integrated into the setup anymore but rather ship test suites which can easily be invoked directly (though some logic could be abstracted to a new library).

Vagrant is no longer shipped in Factory and new formulas already were
not integrated into the setup anymore but rather ship test suites which
can easily be invoked directly (though some logic could be abstracted to
a new library).

Signed-off-by: Georg Pfuetzenreuter <[email protected]>
@tacerus tacerus added the Tests label Jan 16, 2026
@tacerus tacerus marked this pull request as ready for review January 16, 2026 21:53
@cboltz
Copy link
Member

cboltz commented Jan 19, 2026

Just wondering - would the lima package (which was announced on the factory ML yesterday) work as a replacement for Vagrant to run these tests?

If the answer is "no", I'm fine with deleting the unused/now useless tests.

@tacerus
Copy link
Member Author

tacerus commented Jan 19, 2026

Interesting, will need to look into it - but it would in any case require writing a new framework around it, at least a quick search does not reveal any existing Pytest/Testinfra integration for it, and I think the files deleted here would not be useful with any new integration (they do not contain any tests, just tooling to execute the tests which are still inside the individual formula directories). Most test suites (like apache_httpd and 389ds which I already wrote independently) run fine against a regular Podman container, merely some run script / Makefile might be useful to gather the needed commands. The Vagrant layer was intended to support tests which need a cluster with nested virtualization, but I never finished #32 to make use of that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants