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

release-engineering / pushsource / 366 / 3
100%
master: 100%

Build:
Build:
LAST BUILD BRANCH: use-https-for-errata
DEFAULT BRANCH: master
Ran 04 Jun 2021 12:47AM UTC
Files 35
Run time 3s
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

04 Jun 2021 12:44AM UTC coverage: 100.0%. Remained the same
TOX_ENV=cov-travis DEPLOY=1

push

travis-ci

rohanpm
koji: use NSVCA as module push item name

Testing this code with Pub revealed a behavior difference: in Pub
code, we load modulemd files and use their NSVCA as the push item
name. In this library we were not doing that, and instead using
the modulemd filename as push item name.

Although it's a bit annoying that we have to parse the modulemd
files before we can return push items, overall the Pub behavior
seems better as the NSVCA is a much better identifier for what's
being pushed than a generic filename like modulemd.<arch>.txt.
So let's do the same here.

As a compromise, we still use the filename as a fallback when the
file isn't available. This is done since loading the file needs
koji's NFS volume available, which is true in production but won't
be the case in many development and testing scenarios.

1051 of 1051 relevant lines covered (100.0%)

1.0 hits per line

Source Files on job 366.3 (TOX_ENV=cov-travis DEPLOY=1)
  • Tree
  • List 0
  • Changed 2
  • Source Changed 2
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 305
  • Travis Job 366.3
  • 292dbe51 on github
  • Prev Job for TOX_ENV=cov-travis DEPLOY=1 on rohan/set-module-nsvca (#355.3)
  • Next Job for TOX_ENV=cov-travis DEPLOY=1 on rohan/set-module-nsvca (#368.3)
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