Bug 1159191

Summary:

rlPhaseEnd truncates phase names containing ':' when reporting results back to Beaker

Product:

[Fedora] Fedora

Reporter:

Karel Srot <ksrot>

Component:

beakerlib

Assignee:

Dalibor Pospíšil <dapospis>

Status:

CLOSED CURRENTRELEASE

QA Contact:

Fedora Extras Quality Assurance <extras-qa>

Severity:

low

Docs Contact:

Priority:

low

Version:

rawhide

CC:

aigao, amarecek, asaha, dapospis, dcallagh, ncoghlan, pmuller, psplicha

Target Milestone:

---

Keywords:

Patch

Target Release:

---

Hardware:

Unspecified

OS:

Unspecified

Whiteboard:

Fixed In Version:

beakerlib-1.10-1

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2014-12-12 14:48:00 UTC

Type:

Bug

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Bug Depends On:

Bug Blocks:

1136360, 1136362

Attachments:

Description	Flags
patch	none

Description Karel Srot 2014-10-31 07:53:49 UTC

Description of problem:

I have a test with test phases:

rlPhaseStartTest "Scenario: foo"
and
rlPhaseStartTest "Scenario: bar"

tcms-results in that case warns about duplicate phase names.

 WARNING  Duplicate phase name Scenario in /CoreOS/...

It seems that it cuts-off all the text starting the ':' character.

Version-Release number of selected component (if applicable):
qa-tools-workstation-3.3-22.x86_64

Comment 1 Karel Srot 2014-10-31 09:03:33 UTC

I just realized that the beaker is truncating phase names, not tcms-results.

Comment 2 Karel Srot 2014-10-31 09:06:09 UTC

Reopening against the Beaker component.

If the test phase contain ':' character the phase name gets truncated.

Comment 3 Dan Callaghan 2014-11-19 03:54:34 UTC

This is actually a bug in beakerlib's handling of phase names. Patch to come.

Comment 4 Dan Callaghan 2014-11-19 03:55:12 UTC

Created attachment 958863 [details]
patch

Comment 5 Dan Callaghan 2014-11-20 07:41:20 UTC

*** Bug 1146918 has been marked as a duplicate of this bug. ***

Comment 6 Dalibor Pospíšil 2014-11-20 12:24:51 UTC

Thanks for patch. I will include it in the next release.

Comment 7 Dalibor Pospíšil 2014-11-20 16:50:03 UTC

fixed by https://git.fedorahosted.org/cgit/beakerlib.git/commit/?id=2ac7f25fd77775949216d129178b4b3204b734d4