-
Notifications
You must be signed in to change notification settings - Fork 14.6k
ci: introduce px4-dev container with multi arch support #24022
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
Conversation
24ede0c
to
93b6f72
Compare
6a3204f
to
0fe5fe9
Compare
thanks for the fix @alexcekay I pushed a commit to the libdronecan lib for the stringop-overflow error, however the remaining failures are not present anymore, hopefully it means they got fixed elsewhere! |
22512aa
to
94a4697
Compare
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: |
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: |
It appears that this issue is not a toolchain change. It is presenting itself in 1.16-beta as well as 1.15.4 as well |
yes, I'm working out how to build both container images in independent native runners so its faster, it just needs an last push, I just haven't had the time, I will however get to it this week |
ef37052
to
b3ee267
Compare
Finally managed to get the containers to work, here's the multi-arch builds published to both registries. Next step is to make pushing to registry conditional to when we push a tag, this way we only have tagged containers going forward. |
3ced9bf
to
271e3da
Compare
The biggest issue with the cross-compiler container for development is the image size of >5gb, which non-surprisingly comes from installing If we dive into the image to inspect its contentsd this is what we find:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few nitpicks.
@mrpollo now that release 1.16 is branched off, can we get this in? Or what's our approach to update the compiler? |
Signed-off-by: Ramon Roche <mrpollo@gmail.com>
271e3da
to
3b89952
Compare
Signed-off-by: Ramon Roche <mrpollo@gmail.com>
Signed-off-by: Ramon Roche <mrpollo@gmail.com>
Let's discuss on the Dev Call @sfuhrer |
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/px4-team-sync-and-community-q-a-may-6-2025/45433/2 |
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/px4-dev-call-may-14-2025-team-sync-and-community-q-a/45516/1 |
Use the px4-dev container to build and publish all targets