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

Add test for building on an air-gapped system #1760

Merged
merged 5 commits into from
Sep 24, 2024

Conversation

Seb-MCaw
Copy link
Contributor

We are currently interested in the possibility of using Alire on an air-gapped system. The idea would be to fetch a crate and its dependencies online, then transfer to an offline system/network before building.

alr get and alr build seem to already implement the necessary functionality, so this PR simply adds a test to verify the process works as expected.

@Seb-MCaw Seb-MCaw marked this pull request as ready for review September 18, 2024 09:20
Copy link
Member

@mosteo mosteo left a comment

Choose a reason for hiding this comment

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

The test is essentially OK but please fix these minor comments.

@Fabien-Chouteau, I guess you need to review this one too.

testsuite/tests/workflows/air-gapping/test.py Outdated Show resolved Hide resolved
testsuite/tests/workflows/air-gapping/test.py Outdated Show resolved Hide resolved
testsuite/tests/workflows/air-gapping/test.py Outdated Show resolved Hide resolved
@Fabien-Chouteau
Copy link
Member

Hi Alejandro,

@Fabien-Chouteau, I guess you need to review this one too.

You don't need my reviews for @Seb-MCaw 's contributions :)

@Seb-MCaw Seb-MCaw requested a review from mosteo September 23, 2024 10:10
@mosteo
Copy link
Member

mosteo commented Sep 23, 2024

You don't need my reviews for @Seb-MCaw 's contributions :)

OK, noted.

@mosteo mosteo merged commit 9742e81 into alire-project:master Sep 24, 2024
26 checks passed
@mosteo
Copy link
Member

mosteo commented Sep 24, 2024

Merged, thanks!

@mosteo mosteo mentioned this pull request Oct 1, 2024
mosteo pushed a commit to mosteo/alire that referenced this pull request Oct 8, 2024
* Add test for offline build

* Move constraints to YAML file

* Remove redundant build check

* Change crates from git repos to .tgz archives
@Seb-MCaw Seb-MCaw deleted the feat/Air-gap-test branch November 22, 2024 11:57
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.

3 participants