Replies: 2 comments 3 replies
-
FYI @cwasicki @tiyash-basu-frequenz @matthias-wende-frequenz |
Beta Was this translation helpful? Give feedback.
0 replies
-
My vote goes to only test on tag pushes, then we make a pre-release before each release to make sure everything works in arm64 and we are good to go. It seems like a waste of CPU cycles to test in arm64 all the time. We could also trigger this on a schedule, nightly or weekly depending on how active a project is, so we make sure some breakage doesn't stay for too long unnoticed and it is only uncovered before a release. |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Testing in arm64 is slow and expensive for now, as we do it by emulating arm64. This is supposed to change in the future when GitHub rolls out native arm64 runners, but we need to see what to do in the mean time.
For the SDK and channels repos, we are testing in the merge queue and pushes, which makes the slowness less problematic, as we don't suffer it while iterating over a PR, but still means quite a few extra CI minutes are consumed.
An alternative would be to just test on tag pushes, which should ensure releases work properly in arm64. In the experience with the SDK, I think we never saw anything failing in arm64 and not in amd64, so it seems like the likelihood of this happening is pretty low, making it harder to justify running the CI on arm64 all the time.
We can also have different approaches for different types of project, for example libraries being more critical could keep the merge queue testing, and actors could be tested only on tag pushes.
To support this, we could have a new setting in the cookiecutter template, like a choice:
arm64: ["disabled", "tags", "merge_queue"]
to be able to easily configure different triggers for different repos, and even have a different default for different repo types.References:
Beta Was this translation helpful? Give feedback.
All reactions