Skip to content

Comments

feat: run itests using ARM runners#388

Open
michaeldmitry wants to merge 4 commits intomainfrom
OPENG-3328
Open

feat: run itests using ARM runners#388
michaeldmitry wants to merge 4 commits intomainfrom
OPENG-3328

Conversation

@michaeldmitry
Copy link
Contributor

@michaeldmitry michaeldmitry commented Sep 8, 2025

@michaeldmitry michaeldmitry requested a review from a team as a code owner September 8, 2025 10:09
@michaeldmitry
Copy link
Contributor Author

This might break come itests if they don't pass a "arch": "arm64" constraint when deploying the charm. Should we make enabling this matrix conditional (e.g., behind a flag)?

@lucabello
Copy link
Contributor

When we discussed this previously, we mentioned that we don't gain much coverage from running tests on multiple architectures, so we decided against it. If you want to rediscuss it, can you create a café topic for this?

@simskij
Copy link
Member

simskij commented Dec 11, 2025

When we discussed this previously, we mentioned that we don't gain much coverage from running tests on multiple architectures, so we decided against it. If you want to rediscuss it, can you create a café topic for this?

@lucabello We also said that for machine collectors, we will still need to do it for arm as well at the very least. That is also why we do it for grafana-agent and (I think?) otel-collector. In other words, this is still needed :)

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.

run itests using ARM runners

3 participants