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

Run Vagrant in CI #269

Merged
merged 8 commits into from
Sep 13, 2024
Merged

Run Vagrant in CI #269

merged 8 commits into from
Sep 13, 2024

Conversation

lae
Copy link
Owner

@lae lae commented Sep 13, 2024

This just runs vagrant with the development Vagrantfile, mostly just to provide a sanity check/guideline for PRs, and a start to resolve #166. It should be replaced later to re-use existing test playbooks.

Copy link
Contributor

amplify-security bot commented Sep 13, 2024

✨ Amplify has finished checking this pull request

👍 Everything looks good! No issues detected in 📄 2 files and ❇️ 65 lines of code.

Security Pipeline

Tool Configured Result
Semgrep

Last updated by commit bfb18e3 at 2024-09-13 18:04:11 UTC.

Note

To ignore a finding, append @amplify-ignore in a comment to the end of the vulnerable line like // @amplify-ignore or # @amplify-ignore. For more details, visit Amplify Security.

@lae lae force-pushed the feature/gha branch 3 times, most recently from fbe76af to a992fa5 Compare September 13, 2024 15:43
@lae lae force-pushed the feature/gha branch 2 times, most recently from 54a486b to dded4e6 Compare September 13, 2024 16:11
@lae
Copy link
Owner Author

lae commented Sep 13, 2024

waiting 15 minutes when you know the run is gonna fail sure is painful.

@lae
Copy link
Owner Author

lae commented Sep 13, 2024

This error is impossible for me to debug. Maybe Ceph doesn't like the virtual block devices anymore...?

https://github.com/lae/ansible-role-proxmox/actions/runs/10852828353/job/30119748820#step:7:15657

at this point this CI job is meant as a diagnostic and not a required test, so just ignore failures until the LXC-based CI workflow is ported over
@lae lae merged commit 2bdad73 into develop Sep 13, 2024
2 checks passed
@lae lae deleted the feature/gha branch September 13, 2024 18:03
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.

Move tests off Travis CI to another CI deployment
1 participant