system-tests icon indicating copy to clipboard operation
system-tests copied to clipboard

Add CI for Nix

Open lloeki opened this issue 1 year ago • 4 comments

Motivation

Add CI to check for any regression

Changes

Add CI for Nix

Test in CI, the workflow needs access allowed to these actions:

  • DeterminateSystems/nix-installer-action@main
  • DeterminateSystems/magic-nix-cache-action@main

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • [ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
  • [ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
  • [ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
  • [ ] A docker base image is modified?
    • [ ] the relevant build-XXX-image label is present
  • [ ] A scenario is added (or removed)?

lloeki avatar Oct 18 '24 13:10 lloeki

Sure.

Would it be okay to also run this on a scheduled basis?

That would give us some timely-ish "it broke around that time" alerting should something change in system tests that require a change to the Nix side.

lloeki avatar Oct 24 '24 09:10 lloeki

Sure!

cbeauchesne avatar Oct 24 '24 09:10 cbeauchesne

Hi @lloeki,

Is this PR still needed ?

cbeauchesne avatar Feb 03 '25 14:02 cbeauchesne

Hi @lloeki ,

Is this Pr still a project you plan to work on ?

cbeauchesne avatar Oct 21 '25 22:10 cbeauchesne