Bug 1150754

Summary: nodes.conf is blank by default, but the config loader does not have defaults for at least one setting
Product: [Retired] Pulp Reporter: Randy Barlow <rbarlow>
Component: nodesAssignee: Jeff Ortel <jortel>
Status: CLOSED UPSTREAM QA Contact: pulp-qe-list
Severity: medium Docs Contact:
Priority: high    
Version: MasterCC: mhrivnak
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-02-28 22:39:07 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:

Description Randy Barlow 2014-10-08 21:09:20 UTC
Description of problem:
I noticed today that I could not run pulp-dev.py -I because my nodes.conf has the ca_path setting commented out.

Version-Release number of selected component (if applicable):

How reproducible:
Every time.

Steps to Reproduce:
1. Comment out all of nodes.conf.
2. Run sudo ./pulp-dev -I

Actual results:
Traceback (most recent call last):
  File "<string>", line 2, in <module>
  File "/home/rbarlow/devel/pulp/nodes/common/pulp_node/resources.py", line 43, in node_configuration
  File "/home/rbarlow/devel/pulp/common/pulp/common/config.py", line 296, in validate
    return v.validate(self)
  File "/home/rbarlow/devel/pulp/common/pulp/common/config.py", line 360, in validate
  File "/home/rbarlow/devel/pulp/common/pulp/common/config.py", line 476, in validate
    self.validproperty(section, property)
  File "/home/rbarlow/devel/pulp/common/pulp/common/config.py", line 494, in validproperty
    raise pe
pulp.common.config.PropertyNotFound: Required property "main.ca_path", not found

Expected results:
The ca_path should be allowed to be undefined in nodes.conf.

Comment 1 Jeff Ortel 2014-10-13 20:19:59 UTC
So far, I cannot reproduce on master or 2.5-dev.  Randy, can you attach your nodes.conf?

Comment 2 Brian Bouterse 2015-02-28 22:39:07 UTC
Moved to https://pulp.plan.io/issues/573