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

kubevirt / containerized-data-importer / #5510 / 1
49%
main: 49%

Build:
DEFAULT BRANCH: main
Ran 30 Jul 2025 08:38AM UTC
Files 155
Run time 4s
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

30 Jul 2025 08:25AM UTC coverage: 59.346% (+0.01%) from 59.336%
#5510.1

push

travis-ci

web-flow
fix: add DataSource reference handling to authorize utils (#3831)

* fix: add DataSource reference to authorize_utils

A previous PR[1] introduced the concept of DataSource references as a way
for a DataSource to reference another DataSource.

A DV that has its sourceRef set to a DataSource reference that resides
in a different namespace would fail as authorize utils did not include
the necessary changes to the clone source handler to accomodate
DataSource references.

This commit fixes that and introduces a functional test for it.

[1] https://github.com/kubevirt/containerized-data-importer/pull/3760

Signed-off-by: Adi Aloni <aaloni@redhat.com>

* datasource-controller: prohibit cross-namespace references

Previously a DataSource could reference a DataSource in a namespace
different from its own. This is unwanted behavior and with this commit
we now error when we create such DataSources.

Signed-off-by: Adi Aloni <aaloni@redhat.com>

---------

Signed-off-by: Adi Aloni <aaloni@redhat.com>

17173 of 28937 relevant lines covered (59.35%)

0.66 hits per line

Source Files on job #5510.1
  • Tree
  • List 155
  • Changed 4
  • Source Changed 0
  • Coverage Changed 4
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 5510
  • d64a975c on github
  • Prev Job for on main (##5507.1)
  • Next Job for on main (##5513.1)
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