Skip to content

ci: improve the state of some flaky tests - #5899

Merged
lengau merged 6 commits into
mainfrom
work/improve-flaky-tests/SNAPCRAFT-4919
Dec 19, 2025
Merged

ci: improve the state of some flaky tests#5899
lengau merged 6 commits into
mainfrom
work/improve-flaky-tests/SNAPCRAFT-4919

Conversation

@bepri

@bepri bepri commented Dec 4, 2025

Copy link
Copy Markdown
Member

Improves/fixes a number of flaky tests. I recommend reviewing per-commit.

The integration store tests are currently broken and seem to need store-side fixes, so they will be left out of this PR. The remote build tests are failing for similar reasons. The snapd failures are upstream build errors, which I have alerted the snapd team of.


  • I've followed the contribution guidelines.
  • I've signed the CLA.
  • I've successfully run make lint && make test.
  • I've added or updated any relevant documentation.
  • I've updated the relevant release notes.

@bepri bepri self-assigned this Dec 4, 2025
@bepri bepri added the PR: Run Manual Spread This PR should run the manual test suite. label Dec 4, 2025
@bepri bepri changed the title Work/improve flaky tests/snapcraft 4919 ci: improve the state of some flaky tests Dec 15, 2025
@bepri
bepri force-pushed the work/improve-flaky-tests/SNAPCRAFT-4919 branch from 87064f7 to bd10d45 Compare December 15, 2025 14:34
@bepri
bepri marked this pull request as ready for review December 15, 2025 15:44
@bepri
bepri requested review from a team, lengau and smethnani and removed request for a team December 15, 2025 15:44

@smethnani smethnani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks reasonable to me. Thanks @bepri!

Comment thread spread.yaml
@lengau
lengau merged commit 9db1272 into main Dec 19, 2025
26 of 29 checks passed
@lengau
lengau deleted the work/improve-flaky-tests/SNAPCRAFT-4919 branch December 19, 2025 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Run Manual Spread This PR should run the manual test suite.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants