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

feat(workflows): Add multi-os stable tests #839

Merged

Conversation

craciunoiuc
Copy link
Member

Prerequisite checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Ran make fmt on your commit series before opening this PR;
  • Updated relevant documentation.

Description of changes

This adds testing when pushing to stable on all OSes available through GitHub on Linux and Mac.
Of course we should catch this before pushing to stable, but it will take A Lot of time ti run all those so it's better if we only do it ~once a week.

I only copied the existing tests and added some duplication + matrixes.

@craciunoiuc craciunoiuc added kind/enhancement New feature or request area/.github/workflows Issue or PR related to GitHub Actions workflows labels Sep 27, 2023
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/thorough-e2e-tests-stable branch 2 times, most recently from cb1f18c to 0c9fcf3 Compare September 27, 2023 11:29
@antoineco
Copy link
Contributor

Can we temporarily enable this workflow for this PR, just to at least ensure that it completes? Then we can revert back to only running on stable.

@craciunoiuc
Copy link
Member Author

sure, we can remove it after it completes

@craciunoiuc craciunoiuc force-pushed the craciunoiuc/thorough-e2e-tests-stable branch from 0c9fcf3 to 5545251 Compare September 27, 2023 11:35
@craciunoiuc
Copy link
Member Author

Added, let's see

@craciunoiuc craciunoiuc force-pushed the craciunoiuc/thorough-e2e-tests-stable branch from 5545251 to e31b409 Compare September 27, 2023 11:57
Signed-off-by: Cezar Craciunoiu <cezar.craciunoiu@unikraft.io>
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/thorough-e2e-tests-stable branch from e31b409 to 4ea47da Compare September 27, 2023 12:23
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/thorough-e2e-tests-stable branch from 4ea47da to 16d6eb4 Compare September 27, 2023 12:37
Signed-off-by: Cezar Craciunoiu <cezar.craciunoiu@unikraft.io>
Signed-off-by: Cezar Craciunoiu <cezar.craciunoiu@unikraft.io>
@craciunoiuc craciunoiuc force-pushed the craciunoiuc/thorough-e2e-tests-stable branch from 16d6eb4 to 75118f1 Compare September 27, 2023 12:47
@antoineco antoineco merged commit da029f9 into unikraft:staging Sep 27, 2023
5 checks passed
@craciunoiuc craciunoiuc deleted the craciunoiuc/thorough-e2e-tests-stable branch October 17, 2023 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/.github/workflows Issue or PR related to GitHub Actions workflows kind/enhancement New feature or request
Projects
Status: 🚀 Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants