joshcooper
commented
1 year ago The problem is catalog-diff requests a catalog in pson format so puppetserver cannot serialize the catalog in a way that preserves rich data, see
To request a "rich data" catalog you'll need to create the loaders on the client side and specify the rich data catalog mime type. If you're using the catalog v3 endpoint, then you can just use the builtin http client:
require 'puppet'
Puppet.initialize_settings
Puppet::Util::Log.newdestination(:console)
Puppet[:log_level] = :info
env = Puppet::Node::Environment.remote("production")
Puppet.push_context({
current_environment: env,
loaders: Puppet::Pops::Loaders.new(env, true),
rich_data: true
})
# Probably want to load previously saved facts using `Puppet::Node::Facts.from_data_hash` instead
facts = Puppet::Node::Facts.indirection.find(Puppet[:certname])
client = Puppet.runtime[:http]
session = client.create_session
service = session.route_to(:puppet)
response, catalog = service.post_catalog(Puppet[:certname], facts: facts, environment: env.name)
pp catalog.to_data_hashFor example, I see the Regexp data type:
"resources"=>
[{"type"=>"Stage", "title"=>"main", "tags"=>["stage"], "exported"=>false, "kind"=>"compilable_type"},
{"type"=>"Class", "title"=>"Settings", "tags"=>["class", "settings"], "exported"=>false, "kind"=>"unknown"},
{"type"=>"Class", "title"=>"Main", "tags"=>["class"], "exported"=>false, "kind"=>"unknown", "parameters"=>{"name"=>"main"}},
{"type"=>"Node", "title"=>"default", "tags"=>["node", "default", "class"], "exported"=>false, "kind"=>"unknown"},
{"type"=>"Class", "title"=>"Abc", "tags"=>["class", "abc", "node", "default"], "exported"=>false, "kind"=>"unknown"},
{"type"=>"Class",
"title"=>"Abc::Bridges",
"tags"=>["class", "abc::bridges", "abc", "bridges", "node", "default"],
"exported"=>false,
"kind"=>"unknown",
"parameters"=>{"bridgenames"=>{"__ptype"=>"Regexp", "__pvalue"=>"^.*$"}}},If you need to call the v4 catalog endpoint, then you'll need to pass the correct Accept header, which can be resolved like:
format = Puppet::Network::FormatHandler.format_for(:rich_data_json)
headers['Accept'] = format.mimeAnd pass that to https://github.com/voxpupuli/puppet-catalog_diff/blob/952b5d468676936036f9823499f5543413450b56/lib/puppet/catalog-diff/compilecatalog.rb#L103
vchepkov
With puppetlabs/mysql module switching to Deferred function, I noticed that catalog_diff fails to evaluate the catalog. In puppetserver.log I observe the following error:
puppet 7.26.0 puppetserver 2021.7.5.17