Skip to content

tests: check blocking concurrent confdb accesses - #17027

Merged
miguelpires merged 1 commit into
canonical:masterfrom
miguelpires:confdb-blocking-test
May 15, 2026
Merged

tests: check blocking concurrent confdb accesses#17027
miguelpires merged 1 commit into
canonical:masterfrom
miguelpires:confdb-blocking-test

Conversation

@miguelpires

@miguelpires miguelpires commented May 6, 2026

Copy link
Copy Markdown
Contributor

Tests that concurrent confdb accesses to the same databag block when appropriate (no concurrent writes, accesses that can't be parallelized are serviced in order to prevent starvation) and that it's possible to supply a timeout.

https://warthogs.atlassian.net/browse/SNAPDENG-35594

@miguelpires miguelpires added the confdb confdb work (previously called registries and before aspects) label May 6, 2026
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown

Fri May 15 08:53:37 UTC 2026
The following results are from: https://github.com/canonical/snapd/actions/runs/25457000467

Failures:

Preparing:

  • openstack:opensuse-16.0-64:
  • openstack:opensuse-15.6-64:

Restoring:

  • openstack:opensuse-16.0-64:
  • openstack:opensuse-15.6-64:

Skipped tests from snapd-testing-skip

If you wish to have any of the below tests run in your PR, in your PR description, add 'unskip:' followed by a copy-and-pasted list (without variants) of the below tests you wish to run (unskip plus test list must be valid yaml)

  • garden:ubuntu-25.10-64:tests/main/apparmor-prompting-support

Tests that concurrent confdb accesses to the same databag block when
appropriate (no concurrent writes, accesses that can't be parallelized
are serviced in order to prevent starvation) and that it's possible to
supply a timeout.

Signed-off-by: Miguel Pires <miguel.pires@canonical.com>
@miguelpires
miguelpires force-pushed the confdb-blocking-test branch from 777b3b3 to 5659226 Compare May 6, 2026 19:39
@codecov

codecov Bot commented May 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.06%. Comparing base (89b23a7) to head (5659226).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #17027      +/-   ##
==========================================
- Coverage   79.07%   79.06%   -0.01%     
==========================================
  Files        1378     1375       -3     
  Lines      191273   191326      +53     
  Branches     2465     2465              
==========================================
+ Hits       151247   151271      +24     
- Misses      30920    30944      +24     
- Partials     9106     9111       +5     
Flag Coverage Δ
unittests 79.06% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andrewphelpsj andrewphelpsj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!


restore: |
snap unset system experimental.confdb
rm first_read last_read || true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
rm first_read last_read || true
rm first-read last-read password=last-password || true

But tbh I don't even know if this is needed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

password=last-password isn't a file, it's the key-value set for one of the writes

@maykathm maykathm 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.

thanks

@miguelpires
miguelpires merged commit a7d863d into canonical:master May 15, 2026
220 of 224 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

confdb confdb work (previously called registries and before aspects)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants