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

kubevirt / containerized-data-importer / #4653
49%

Build:
DEFAULT BRANCH: main
Ran 23 May 2024 10:37AM UTC
Jobs 1
Files 153
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

23 May 2024 10:13AM UTC coverage: 58.47% (-0.005%) from 58.475%
#4653

push

travis-ci

web-flow
Add watch for DataImportCron-labeled PVCs deletion (#3285)

When the DataImportCron last import DV is manually deleted, the
controller reconciles, but due to k8s default background cascading
deletion, the PVC may still temporarily exist, so the controller will
not re-create the DV even after the PVC is deleted, unless it reconciles
due to other watched CR like DataSource. In the scenario of CNV-39688,
since we move from pvc source format to snapshot, the DataSource won’t
update until a snapshot is created, which will never happen. To solve it
we add a watch for deletion of DataImportCron-labeled PVCs. The change
was tested locally, but since it required two storage classes it’s
currently out of scope for the existing CI lanes, so it requires tier-2
test.

Signed-off-by: Arnon Gilboa <agilboa@redhat.com>

0 of 9 new or added lines in 1 file covered. (0.0%)

16005 of 27373 relevant lines covered (58.47%)

0.65 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
9
65.05
-0.53% pkg/controller/dataimportcron-controller.go
Jobs
ID Job ID Ran Files Coverage
1 #4653.1 23 May 2024 10:36AM UTC 153
58.47
Source Files on build #4653
  • Tree
  • List 153
  • Changed 2
  • Source Changed 0
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • 4342b3ca 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