• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

xapi-project / xen-api / 13541476888
80%

Build:
DEFAULT BRANCH: master
Ran 26 Feb 2025 10:13AM UTC
Jobs 1
Files 38
Run time 305min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

26 Feb 2025 10:00AM UTC coverage: 78.516%. Remained the same
13541476888

push

github

web-flow
Restructure valid allowed_operations computation (#6253)

In an effort to make the logic clearer, we restructure the part(s) of
`Xapi_pool_helpers` responsible for determining which operations are
"valid" (i.e. become members of `allowed_operations`).

The previous code is somewhat tedious to understand because it is
unconditionally ineffective - the logical delineation of parts of the
code is implicit, as the code computes the valid operation table in
order, but many of the operations will have no effect (as later code to
populate an operation's entry in the validity table do nothing).

This change makes the precedence of blocking and waiting operations more
apparent by explicitly casing on them (in decreasing order of
precedence). The differences are somewhat subtle and explained in the
commit message.

---

BVT+BST (211251) all green except for a few known issues. In a distant
future, the relation between operations could be stated declaratively
and the `allowed_operations` computed incrementally (based on ambient
state changes in the database, etc.).

The code populates a table with a tri-state value (allowed, unknown, and
disallowed). However, absence from the table could signify the same
(that the operation is valid), as every possible value is accounted for
(therefore, `Unknown` is redundant).

3512 of 4473 relevant lines covered (78.52%)

0.79 hits per line

Jobs
ID Job ID Ran Files Coverage
1 python3.11 - 13541476888.1 26 Feb 2025 10:13AM UTC 38
78.52
GitHub Action Run
Source Files on build 13541476888
  • Tree
  • List 38
  • Changed 0
  • Source Changed 0
  • Coverage Changed 0
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #13541476888
  • 46dec4bc on github
  • Prev Build on master (#13541306988)
  • Next Build on master (#13546102248)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc