| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
| |
| |
| | |
This is a much cleaner fix than the xmlrpc version, thankfully. :)
|
|\|
| |
| |
| |
| |
| |
| | |
Conflicts:
lib/puppet/executables/client/certhandler.rb
spec/unit/executables/client/certhandler.rb
|
| |\
| | |
| | |
| | | |
0.24.x
|
| | |
| | |
| | |
| | |
| | |
| | | |
Removed logic to reuse parser and log on server when there is a parsing error.
Now we just make a new parser and if there is an error, raise it up, which will result in errors on the client logs.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The tests were failing when run on a machine with certs on the file system
Stub out failure to read where appropriate
Worked fine at my desk :(
|
|\| |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Also added the fixes to make the certhandler tests pass
even when certs exist; I'll deal with the conflict later.
Conflicts:
CHANGELOG
bin/puppetd
lib/puppet/network/http/handler.rb
lib/puppet/network/http/mongrel/rest.rb
spec/integration/indirector/rest.rb
spec/integration/network/server/mongrel.rb
spec/integration/network/server/webrick.rb
spec/unit/network/http/webrick.rb
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| |/ |
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
I wasn't testing whether the options where still a hash,
and an 'inject' loop wasn't returning correctly, so
the options weren't being set correctly.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
My request authorization code changed the request to set its 'node'
instance variable, rather than using its options, and I apparently
didn't change the Indirection class to do this.
I have *no* idea how these tests were passing before -- they passed for
me yesterday but failed for Andrew, and today they started failing for
me. Frightening.
|
| | |
| | |
| | |
| | | |
FAIL
|
| |/
| |
| |
| |
| |
| | |
Add the class to handle certs, specs and modify puppetd
should fix 1190, 1199, 1200
|
| |\
| | |
| | |
| | | |
git://github.com/lak/puppet into 0.24.x
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
All requests should now have an ipaddress add to them, they
should always be marked authenticated or not, and they should
have the certificate name set as their 'node' if a certificate
is present.
They both use the same methods they use for xmlrpc,
although there's no common code, to facilitate deprecation
of xmlrpc.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This basically consists of the node name (i.e., the certificate name),
the IP, and whether the request is authenticated (which is determined
by whether it provided a valid certificate).
Now the two REST classes (mongrel and webrick) need to correctly
pass this information to the indirection calls they make.
|
| |/
| |
| |
| |
| |
| | |
(i.e., client-side templates). This would really only be used
for composite resources that pass the results of the template
on to generated resources.
|
| |
| |
| |
| |
| | |
Exporting or collecting resources no longer raises an exception
when no storeconfigs is enabled, it just produces a warning.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The Master handler previously provided the support for the :node_name
setting, and that functionality has now been moved into the Node
class. At the same time, the names to search through have been
changed somewhat: Previously, the certificate name and the
hostname were both used for searching, but now, the cert name
is always searched first (unless node_name == facter), but only
the Facter hostname, domain, and fqdn are used otherwise. We no
longer split the cert name, only the hostname/domain/fqdn.
In the general case, this provides no behaviour change, because
people's hostname is the same as their certname. This only
results in a change in behaviour if you specify a certificate
name that is a normal node name, and you want to look that node
up by something other than the full name in the certificate.
Signed-off-by: Luke Kanies <luke@madstop.com>
|
| |
| |
| |
| |
| |
| | |
This idiom is only functional when you have
Rails loaded, which is why it was failing for others
but not me.
|
| |
| |
| |
| |
| | |
I've set the default cadir and vardir to /dev/null, to
hopefully catch this kind of test sooner next time.
|
| |
| |
| |
| |
| | |
I was previously not clearing the cache using the Cacher.invalidate
method, which caused a test to fail.
|
| |
| |
| |
| | |
I just had to remove the '.clear' call.
|
|\|
| |
| |
| |
| |
| |
| | |
Conflicts:
CHANGELOG
lib/puppet/transaction.rb
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
master branch.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
catalogs.
It largely worked previously, except when aliases were set,
which require catalogs. This now converts all parser resources
to trans_objects then to RAL resources, and sets the catalog
in the meantime. This allows aliases to work just
fine when converting directly from parser catalogs to RAL catalogs.
|
| |
| |
| |
| |
| | |
Previously, they failed if you tried to get them to autogenerate
an id, because they assumed that a result would be returned.
|
| |
| |
| |
| |
| |
| | |
This renames the 'facter' confine to 'variable', and it prefers
settings to facts. There shouldn't really be any overlap, so
it shouldn't be a problem.
|
| | |
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
CHANGELOG
spec/integration/defaults.rb
spec/integration/node/catalog.rb
spec/unit/rails.rb
spec/unit/type/mount.rb
|
| | |
|
| |\ |
|
| | | |
|
| | |
| | |
| | |
| | | |
accessor for @@state to make testing a bit cleaner.
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Now each of the test types is a separate subclass
of Confine, so that they can have all of their own
logging and summarizing behaviour.
Also, added a 'feature' type, which can test for
the availability of Puppet features (and log
their absence more usefully).
|
| | |
| | |
| | |
| | |
| | | |
Facter to load facts there if a new enough version of
Facter is used.
|
| | |
| | |
| | |
| | |
| | | |
It mocked :exists? instead of :exist?, and my provider work
changed the method call because :exists? is going away in ruby.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
I created a Confiner module for the Provider class methods,
and then I enhanced the interface between it and the Confine
class to make sure binary paths are searched for fresh each time.
This fixes #1197, which was a result of binary paths being
searched for at startup, rather than at execution.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
This directory only existed as an organizational method,
and the code never matched it, so I'm fixing it.
|