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

jnpkrn / clufter / 703
60%
master: 60%

Build:
Build:
LAST BUILD BRANCH: next
DEFAULT BRANCH: master
Ran 08 Nov 2017 10:08PM UTC
Jobs 5
Files 214
Run time 4min
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

pending completion
703

push

travis-ci

jnpkrn
facts: infer_comp+_cmp_ver: handle tail ver. component(s) missing right

Previously, it could happen that components could get some "extras",
accumulated as traversed through the evolutionary line, that in fact
would not reflect the raw data.  Triggering condition in such case
is version comparison with one the operands consisting of less
version components -- a case in point (not present though, yet!):
corosync 2.4 vs 2.4.3, which would incorrectly make the former
awarded "extras" from the latter.

The apparent solution for that to rectify _cmp_ver function so that
it imagines virtual zeroes at the positions missing for version
components parity between the two operands.  That alone, however,
makes the "facts.TestClusterSystem.test_cluster_pcs_flatiron" unit
test fail starting with "6.5" case.  Now, what becomes a trouble
is that doing so breaks the logic a la "requiring version 1.x,
which can clearly be fulfilled with 1.4" as the comparison runs
as "(1, 0) vs. (1, 4)" rather than "1 vs. (1, 4), i.e., 1 vs. 1"
as before (1.x here implied with how corosync=flatiron is translated
under the hood).  Hence, as the next step, introduce new parameter
at _cmp_ver so as to request just a prefix match comparison, and
apply that where it matters, i.e. directly at the leaf "component"
part of the inference mechanism, infer_comp function.

Signed-off-by: Jan Pokorný <jpokorny@redhat.com>

13 of 13 new or added lines in 1 file covered. (100.0%)

3608 of 5983 relevant lines covered (60.3%)

2.98 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
100.0
facts.py
Jobs
ID Job ID Ran Files Coverage
1 703.1 (MAINT_EXTRA=0) 08 Nov 2017 10:12PM UTC 0
59.95
Travis Job 703.1
2 703.2 (MAINT_EXTRA=0) 08 Nov 2017 10:08PM UTC 0
59.65
Travis Job 703.2
5 703.5 (MAINT_EXTRA=0) 08 Nov 2017 10:08PM UTC 0
59.9
Travis Job 703.5
6 703.6 (MAINT_EXTRA=0) 08 Nov 2017 10:09PM UTC 0
59.93
Travis Job 703.6
7 703.7 (MAINT_EXTRA=0) 08 Nov 2017 10:11PM UTC 0
59.61
Travis Job 703.7
Source Files on build 703
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #703
  • 7008574b on github
  • Prev Build on next (#702)
  • Next Build on next (#704)
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