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

kubevirt / containerized-data-importer / #5923
50%

Build:
DEFAULT BRANCH: main
Ran 29 Mar 2026 10:47PM UTC
Jobs 1
Files 157
Run time 1min
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

29 Mar 2026 10:34PM UTC coverage: 49.418% (+0.1%) from 49.301%
#5923

push

travis-ci

web-flow
Fix snapshot clone watcher to re-trigger reconciliation when snapshot becomes ready (#4061)

* Fix snapshot clone permanently stuck when snapshot is not immediately ready

when cloning from a VolumeSnapshot that is not yet ReadyToUse, the clone populator stops reconciling and never retries. Since no watch re-triggers reconciliation when the snapshot becomes ready, the clone stays stuck in CloneFromSnapshotSourceInProgress permanently

This affects CSI drivers that report RestoreSize before ReadyToUse, creating a window where a clone can be initiated against a snapshot that isn't fully usable yet

requeueing with a short delay when the snapshot is not ready so the controller retries until ReadyToUse becomes true

Signed-off-by: Noam Assouline <nassouli@redhat.com>
Assisted-by: Claude (Cursor)

* Fix clone source watcher for cross-namespace clones

getPopulationSourceNamespace only checked DataSourceRef.Namespace to determine the source namespace, but for cross-namespace clones CDI leaves that field nil and stores the source namespace in the AnnDataSourceNamespace annotation instead.
This caused the PVC to be indexed under its own namespace rather than the source namespace, preventing the clone source watcher from finding it.

Check AnnDataSourceNamespace first before falling back to DataSourceRef.Namespace.

Signed-off-by: Noam Assouline <nassouli@redhat.com>
Assisted-by: Claude (Cursor)

---------

Signed-off-by: Noam Assouline <nassouli@redhat.com>

16 of 26 new or added lines in 4 files covered. (61.54%)

14782 of 29912 relevant lines covered (49.42%)

0.55 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
56.86
0.0% pkg/controller/populators/clone-populator.go
3
54.81
1.23% pkg/controller/populators/forklift-populator.go
6
68.24
6.94% pkg/controller/populators/populator-base.go
Jobs
ID Job ID Ran Files Coverage
1 #5923.1 29 Mar 2026 10:47PM UTC 157
49.42
Source Files on build #5923
  • Tree
  • List 157
  • Changed 6
  • Source Changed 0
  • Coverage Changed 6
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • 6f47c6a2 on github
  • Prev Build on main
  • Next Build on main
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

© 2026 Coveralls, Inc