Skip to content

Conversation

@samfreund
Copy link
Member

@samfreund samfreund commented Dec 16, 2025

Description

Runs the smoketest on each image after we build it in CI. This helps ensure that we check every image that we build has the requisite JNI libraries, and that they can be loaded. This is needed cause we've been breaking that, and our current smoketests don't actually test all our libraries since some are gated behind platform checks.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2025.3.2
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

@samfreund samfreund force-pushed the smokiest-test branch 3 times, most recently from 88320fd to d033115 Compare December 16, 2025 07:31
@samfreund

This comment was marked as outdated.

@samfreund samfreund force-pushed the smokiest-test branch 2 times, most recently from 8e4c929 to 629ff45 Compare December 16, 2025 20:29
@github-actions github-actions bot added the backend Things relating to photon-core and photon-server label Dec 16, 2025
@samfreund samfreund force-pushed the smokiest-test branch 4 times, most recently from b61ac48 to f2aeb57 Compare December 16, 2025 22:00
@samfreund samfreund marked this pull request as ready for review December 16, 2025 22:00
@samfreund samfreund requested a review from a team as a code owner December 16, 2025 22:00
@samfreund samfreund force-pushed the smokiest-test branch 3 times, most recently from f238c19 to 87e6f68 Compare December 16, 2025 23:57
@samfreund samfreund force-pushed the smokiest-test branch 2 times, most recently from c5202c9 to 9855c26 Compare December 17, 2025 07:30
Copy link
Member

@Gold856 Gold856 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much nicer JNI loading

@samfreund samfreund merged commit 50f2285 into PhotonVision:main Dec 17, 2025
85 of 91 checks passed
@samfreund samfreund deleted the smokiest-test branch December 17, 2025 16:16
samfreund added a commit to samfreund/photonvision that referenced this pull request Dec 18, 2025
Runs the smoketest on each image after we build it in CI. This helps
ensure that we check every image that we build has the requisite JNI
libraries, and that they can be loaded. This is needed cause we've been
breaking that, and our current smoketests don't actually test all our
libraries since some are gated behind platform checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Things relating to photon-core and photon-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants