This is all code that was ready for 2.7 but one of the patches caused a 
regression.

This is almost the exact same patch series but with one patch reduced to have
no side affects (other than being more resilient to missing data).

Specifically, changing the way the Facts rendered into json caused REST 
transport
problems, so I don't change how it's rendered here - I just make the json 
parsing
able to handle nil values and less likely to produce them.

Otherwise, the code is exactly the same other than one test that had to be 
changed
because of the fact that Facts no longer render automatically with json.

The long-term goal of the json support in the indirector request is that they
should be able to pass over an mcollective bus or something similar, rather than
always having to go over http.

lib/puppet/indirector/request.rb     |   51 +++++++++++++++++-
lib/puppet/node.rb                   |   23 ++++++++
lib/puppet/node/facts.rb             |   15 +++--
spec/unit/indirector/request_spec.rb |   95 ++++++++++++++++++++++++++++++++++
spec/unit/node/facts_spec.rb         |   14 +++++
spec/unit/node_spec.rb               |   63 ++++++++++++++++++++++
6 files changed, 253 insertions(+), 8 deletions(-)

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to