Bug 1727063

Summary: Incorrect HTML on task details page causes unexpected DOM which fails QE automation
Product: Red Hat Satellite Reporter: Mirek Długosz <mzalewsk>
Component: Tasks PluginAssignee: Mirek Długosz <mzalewsk>
Status: CLOSED ERRATA QA Contact: Mirek Długosz <mzalewsk>
Severity: high Docs Contact:
Priority: unspecified    
Version: UnspecifiedCC: aruzicka, inecas, jhutar, pcreech
Target Milestone: 6.6.0Keywords: AutomationBlocker, Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-foreman-tasks-0.15.5.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-10-22 19:52:11 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:
Attachments:
Description Flags
Visual explanation none

Description Mirek Długosz 2019-07-04 10:53:41 UTC
Created attachment 1587350 [details]
Visual explanation

Since foreman-tasks commit a2aa2b7e68d0d6615a8810a4c1f3f77d1ce5a4e5, "Errors" field value on task details page is in <p> instead of <div>. However, <p> is not valid parent of <pre>. This causes browsers to create unexpected DOM tree that Satellite QE automation fails to interpret correctly. As a final result, some of our automation reports failures where they should not.

See attached screenshot for visual explanation.

Comment 2 Bryan Kearney 2019-07-04 14:04:51 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/27234 has been resolved.

Comment 3 Mirek Długosz 2019-07-24 14:35:17 UTC
tests/foreman/ui/test_dashboard.py::test_positive_latest_warning_error_tasks is now passing in automation run and locally.

Version:
Satellite 6.6 snap 11
pulp-server-2.19.1-1.el7sat.noarch
satellite-6.6.0-5.beta.el7sat.noarch
tfm-rubygem-katello-3.12.0.7-1.el7sat.noarch
foreman-1.22.0.9-1.el7sat.noarch
tfm-rubygem-foreman-tasks-0.15.5.2-1.el7sat.noarch
tfm-rubygem-foreman-tasks-core-0.3.2-1.el7sat.noarch

Comment 4 Bryan Kearney 2019-10-22 19:52:11 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2019:3172