Bug 1007118

Summary: Conflicting PressGang and Publican conditions should produce an error/warning
Product: [Community] PressGang CCMS Reporter: Lee Newson <lnewson>
Component: CSProcessorAssignee: Lee Newson <lnewson>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.1CC: mcaspers
Target Milestone: ---   
Target Release: 1.3   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-03 22:16:22 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:

Description Lee Newson 2013-09-12 00:04:41 UTC
Currently if someone includes a Publican condition (one in the publican.cfg file) and in the Content Spec (ie [condtion=test]), then the later will be completely ignored which could result in unexpected build errors. As such if both are specified then an error or warning should be produced saying that only one way will work.

Comment 1 Lee Newson 2013-11-06 00:30:24 UTC
Fixed in 1.3-SNAPSHOT build 201311061019

The validator will now print warnings when a condition is used on a node and in the publican.cfg metadata.

Comment 2 Lee Newson 2013-11-07 04:09:52 UTC
This version is now live on the dev/test server.

Comment 3 Matthew Casperson 2013-11-25 00:17:00 UTC
Confirmed that an warning is produced when there are conflicting conditions.

I'd suggest that the error be changed from:

A condition has been defined in publican.cfg and as such the condition will be ignored.

To

A condition has been defined in publican.cfg, and as such the condition defined against the topic or container will be ignored.

Comment 4 Lee Newson 2013-11-25 01:45:55 UTC
Fixed in 1.3-SNAPSHOT build 201311251126

I've updated the warning message to what Matt mentioned above.

Note: This version is currently live on the development/test server.

Comment 5 Matthew Casperson 2013-11-25 21:16:41 UTC
Confirmed new warning message.