{"shas":[["c5448b79e690d72536f4d98b636f450d37de627a","00d761b23995d0adadf5b7f5eea7dc44b892ddb7@gmail.com","(#7624) Auditing should not be enabled by default for purged resources.\n\n- Auditing creates logging messages as of 2.6.5 so it should not\n  be enabled by default.\n\n- This patch removes the :audit => :all settting from resources\n  created via self.instances (which is used for purging).\n\nPlease note that we believe this change to be safe, and *should* not result in\nuser-visible behavioural differences when you use the `instances` method on a\ntype, but we can't give you a perfect assurance of that.\n\nIf you do have code that depends on the current behaviour, and it misbehaves\nafter this patch, please let us know so we can weep ^W find another solution\nthat works for everyone.\n\nReviewed-By: Daniel Pittman <daniel@puppetlabs.com>\nReviewed-By: Nigel Kersten <nigel@puppetlabs.com>","Dan Bode"],["31bf55c2f9c090c40e48f38c5f64e4373a402261","9956a36b5e6e3cca7ae8066f76f964d5f08c7b1e@gmail.com","(#7632) Make secret_agent application compatible with secret_agent face\n\nRunning `puppet secret_agent` was failing with an error in validate_args (in\ninterface/action.rb), because the application was trying to pass the certname as\nan argument to the synchronize action. Also, it was trying to submit the report\na second time. Reviewing the code with Nick0, we found that the application\nwasn't inheriting from FaceBase and was duplicating a lot of work, and were able\nto resolve the issue by basically deleting the whole thing.\n\nThis patch makes secret_agent behave like the other Faces apps, and makes\nsynchronize the default action of the secret_agent face. We left the `run_mode\n:agent` line in the application because of bug #7802.\n\nPaired-with: Nick Lewis <nick@puppetlabs.com>","nfagerlund"],["834191c81e6e390d9c147bddba286f1660780aa1","9956a36b5e6e3cca7ae8066f76f964d5f08c7b1e@gmail.com","Merge branch 'ticket/2.7rc/7632-secret_agent_application' into 2.7rc","nfagerlund"],["c8df02703ac29b00140648640f85ec044a9cfd6e","1fa2ef4755a9226cb9a0a4840bd89b158ac71391@puppetlabs.com","(#7193) Fix path issues with acceptance tests that call old shell tests\n\nSome rearranging of the acceptance test directories caused some ruby\ntests that referenced old shell tests to lose their relative paths.\n\nPaired-with: Dominic Maraglia <dominic@puppetlabs.com>","Matt Robinson"],["d4e6c268aecabdbf36ece126f960ce2384576fcc","6b3799be4300e44489a08090123f3842e6419da5@puppetlabs.com","(#7624) Manually fetch all properties in instances.\n\nWhen we removed the `:audit => :all` flag on creation of an instance from a\nPuppet type, we stopped fetching all the properties.  This had flow-on\nconsequences that were visible from the outside; while some places did their\nown work to ensure that properties were fetched, others didn't.\n\nWe now open-code the loop that creates and fetches those properties, to ensure\nthat we have the same data without going through the :audit machinery.\n\nThis resolves the excessive logging, and also eliminates the behavioural\nchange that we introduced in the previous commit.\n\nReviewed-By: Daniel Pittman <daniel@puppetlabs.com>","Luke Kaines"],["d0d1e2ef64c4a6eecf3465dff82b62cf2cc79afb","3d0f3b9ddcacec30c4008c5e030e6c13a478cb4f@puppetlabs.com","Merge branch 'bug/2.7rc/7624-resource-purging-causes-audit-messages' into 2.7rc","Daniel Pittman"],["4922409a2bdf3ad6a094762c9e2e60371ed16a1b","1fa2ef4755a9226cb9a0a4840bd89b158ac71391@puppetlabs.com","Merge branch '2.7rc' into 2.7.x\n\n* 2.7rc:\n  (#7624) Manually fetch all properties in instances.\n  (#7193) Fix path issues with acceptance tests that call old shell tests\n  (#7632) Make secret_agent application compatible with secret_agent face\n  (#7624) Auditing should not be enabled by default for purged resources.","Matt Robinson"]],"ref":"refs/heads/2.7.x","size":7,"head":"4922409a2bdf3ad6a094762c9e2e60371ed16a1b","push_id":33258988}