Skip to content

third_party: update mavlink#2709

Merged
julianoes merged 3 commits intomainfrom
pr-update-mavlink
Nov 20, 2025
Merged

third_party: update mavlink#2709
julianoes merged 3 commits intomainfrom
pr-update-mavlink

Conversation

@julianoes
Copy link
Copy Markdown
Collaborator

This is required for figure eight messages for PX4 MAVSDK tests.

@julianoes
Copy link
Copy Markdown
Collaborator Author

@JonasVautherin any idea what is happening here in CI? It seems to run out of space.

This is required for figure eight messages for PX4 MAVSDK tests.
@JonasVautherin
Copy link
Copy Markdown
Collaborator

JonasVautherin commented Nov 20, 2025

I can't access the logs, it says "Failed to generate URL to download logs." 🤔.

EDIT: actually I could download a log archive, but there is no complaint in what's there... it just appears to hang. Since it's marked as "failed" by github, I would expect github to tell us something...

@julianoes
Copy link
Copy Markdown
Collaborator Author

I think it runs out of space, but it's a mystery. It's literally due to the mavlink change. I'm going to try to replicate it locally.

@julianoes
Copy link
Copy Markdown
Collaborator Author

Oh, it runs out of space, so I added the action to save some space but I added it to arm64 instead of android-arm64!

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

@julianoes julianoes merged commit 5282579 into main Nov 20, 2025
53 of 54 checks passed
@julianoes julianoes deleted the pr-update-mavlink branch November 20, 2025 23:00
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.

2 participants