Qiskit / qiskit / 11583319838
90%
main: 89%

LAST BUILD BRANCH: dag-concatenate
DEFAULT BRANCH: main
Ran
Jobs 1
Files 855
Run time 2min
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
coverage: 89.754% (-0.01%) from 89.765%
11583319838

Pull #13380

github

mergify[bot] 
Move QPY tests to GitHub Actions and increase inter-symengine tests (#13273)

This commit has two major goals:

- fix the caching of the QPY files for both the `main` and `stable/*`
  branches

- increase the number of compatibility tests between the different
  symengine versions that might be involved in the generation and
  loading of the QPY files.

Achieving both of these goals also means that it is sensible to move the
job to GitHub Actions at the same time, since it will put more pressure
on the Azure machine concurrency we use.

Caching
-------

The previous QPY tests attempted to cache the generated files for each
historical version of Qiskit, but this was unreliable.  The cache never
seemed to hit on backport branches, which was a huge slowdown in the
critical path to getting releases out.  The cache restore keys were also
a bit lax, meaning that we might accidentally have invalidated files in
the cache by changing what we wanted to test, but the restore keys
wouldn't have changed.

The cache files would fail to restore as a side-effect of ed79d42
(gh-11526); QPY was moved to be on the tail end of the lint run, rather
than in a test run.  This meant that it was no longer run as part of the
push event when updating `main` or one of the `stable/*` branches.  In
Azure (and GitHub Actions), the "cache" action accesses a _scoped_
cache, not a universal one for the repository [^1][^2].  Approximately,
base branches each have their own scope, and PR events open a new scope
that is a child of the target branch, the default branch, and the source
branch, if appropriate.  A cache task can read from any of its parent
scopes, but write events go to the most local scope.  This means that we
haven't been writing to long-standing caches for some time now.  PRs
would typically miss the cache on the first attempt, hit their
cache for updates, then miss again once entering the merge queue.

The fix for this is to run the QPY job on branch-update events as wel... (continued)
Pull Request #13380: Move QPY tests to GitHub Actions and increase inter-symengine tests (backport #13273)

66891 of 74527 relevant lines covered (89.75%)

216540.14 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
94.02
 -0.24% crates/qasm2/src/expr.rs
4
92.23
 -0.25% crates/qasm2/src/lex.rs
6
97.62
 -0.46% crates/qasm2/src/parse.rs
Jobs
ID Job ID Ran Files Coverage
1 11583319838.1 855
89.75
 GitHub Action Run
Source Files on build 11583319838