{"push_id":880049564,"size":14,"distinct_size":14,"ref":"refs/heads/transactions","head":"5f4a24a767cb9a3f88d376871745473a59ded6aa","before":"a305335ffaf7bc7f251e134f2d121b7f6c6fbc51","commits":[{"sha":"030a6769151de552f877780d914c7360ccf01d52","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"add support for LWRP specs which load multiple recipes\n\nThis will be used by the pacemaker_transaction LWRP spec.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/030a6769151de552f877780d914c7360ccf01d52"},{"sha":"7e37091126fa95421dfd3abaadba4a720ab5305a","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"add support for LWRP specs which load test recipes\n\nThis will be used by the pacemaker_transaction LWRP spec.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/7e37091126fa95421dfd3abaadba4a720ab5305a"},{"sha":"a2c133de9fa7bcb4961c0499bdd389a552ea00c8","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"allow the possibility to test LWRPs via normal recipes\n\nRather than the shared context always artificially constructing a\nresource via test code, split out the construction into a different\ncontext so that the original shared context can be used by other\nprovider specs.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/a2c133de9fa7bcb4961c0499bdd389a552ea00c8"},{"sha":"46f2183408cbc16d85c8f0229e09c6ad308130d1","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"add KEYSTONE_{PRIMITIVE,LOCATION}_NAME constants\n\nThese will be used by the pacemaker_transaction LWRP spec.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/46f2183408cbc16d85c8f0229e09c6ad308130d1"},{"sha":"8dc82641553a97b7f673b6522659c5da17280cc9","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"add missing \"name\" attribute to hawk and xfs cookbooks\n\nThis omission was highlighted by the newer version of chefspec.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/8dc82641553a97b7f673b6522659c5da17280cc9"},{"sha":"5a3f411d13b8e852047217d528fe7c4103ed81d7","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"remove space to keep HoundCI happy","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/5a3f411d13b8e852047217d528fe7c4103ed81d7"},{"sha":"81ba291d4b2b0e339adfac989e5b7540a32cd893","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"fix step_into value\n\nIt seems this was wrong all along, but it didn't matter since we were\nnot running LWRP actions from a real recipe.  However now we want to\ndo that to test the pacemaker_transaction LWRP, so we need to fix it.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/81ba291d4b2b0e339adfac989e5b7540a32cd893"},{"sha":"7d7ebd27082a597f30d1011e6020fd6f0c9c90ee","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"remove voodoo used for instantiating a new Chef resource\n\nWe already have @new_resource.class, so use that instead of doing\nsurgery on class names.  This is cleaner, and anyway the surgery breaks\nwith newer versions of Chef where the provider class is a singleton\nclass with a human-readable string as the class name, rather than\nChef::Provider::Foo.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/7d7ebd27082a597f30d1011e6020fd6f0c9c90ee"},{"sha":"aa011ef6e2e3306b9126d71b6971498af2857997","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"fix \"already initialized constant\" warnings\n\nchef and chefspec are both a bit crap when it comes to loading code,\nand end up loading classes multiple times, so fix the object type\nregistration to avoid defining constants more than once.\n\nFor reference, here are two sample stack traces from the same run\nwhich show different code paths both loading the same\nPacemaker::Constraint::Colocation class:\n\n$cookbook_path/libraries/pacemaker/constraint/colocation.rb:4:in `<class:Colocation>'\n$cookbook_path/libraries/pacemaker/constraint/colocation.rb:3:in `<top (required)>'\n$cookbook_path/spec/libraries/pacemaker/constraint/colocation_spec.rb:3:in `require_relative'\n$cookbook_path/spec/libraries/pacemaker/constraint/colocation_spec.rb:3:in `<top (required)>'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/configuration.rb:1361:in `load'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/configuration.rb:1361:in `block in load_spec_files'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/configuration.rb:1359:in `each'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/configuration.rb:1359:in `load_spec_files'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/runner.rb:102:in `setup'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/runner.rb:88:in `run'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/runner.rb:73:in `run'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/lib/rspec/core/runner.rb:41:in `invoke'\n$cookbook_path/vendor/bundle/gems/rspec-core-3.4.1/exe/rspec:4:in `<top (required)>'\n$cookbook_path/vendor/bundle/bin/rspec:23:in `load'\n$cookbook_path/vendor/bundle/bin/rspec:23:in `<main>'\n\n$cookbook_path/libraries/pacemaker/constraint/colocation.rb:4:in `<class:Colocation>'\n$cookbook_path/libraries/pacemaker/constraint/colocation.rb:3:in `<top (required)>'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:191:in `load'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:191:in `block in load_libraries_from_cookbook'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:188:in `each'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:188:in `load_libraries_from_cookbook'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:99:in `block in compile_libraries'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:98:in `each'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:98:in `compile_libraries'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context/cookbook_compiler.rb:71:in `compile'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/run_context.rb:158:in `load'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/policy_builder/expand_node_object.rb:91:in `setup_run_context'\n~/.rvm/rubies/ruby-2.1.0/lib64/ruby/2.1.0/forwardable.rb:183:in `setup_run_context'\n$cookbook_path/vendor/bundle/gems/chef-12.5.1/lib/chef/client.rb:485:in `setup_run_context'\n~/SUSE/cloud/chef/git/chefspec/lib/chefspec/solo_runner.rb:117:in `converge'","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/aa011ef6e2e3306b9126d71b6971498af2857997"},{"sha":"4c7192fd4690135e1792d1dd9c5201935ccb1371","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"tweaks to improve RSpec context / example names","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/4c7192fd4690135e1792d1dd9c5201935ccb1371"},{"sha":"349c808b9171c51b8759ef56e118fa5b836eda4e","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"ditch unneeded require","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/349c808b9171c51b8759ef56e118fa5b836eda4e"},{"sha":"fd921852247e27ccdb1765e854dba249822177e2","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"tweak Guardfile for more efficient debugging","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/fd921852247e27ccdb1765e854dba249822177e2"},{"sha":"4c826f3853197c3051a4bc5ea47d7e5e7d5c52fc","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"revamp Mixlib::ShellOut mocking\n\nRSpec deprecates #expect_any_instance_of, and the existing mocking code\nwas overly complex (unnecessarily handling succeeding and failing\ncommands differently), and imprecise (not setting expectations for the\nexact commands being run, or defining exactly what STDOUT/STDERR should\nbe).  The new approach also makes for clearer test code.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/4c826f3853197c3051a4bc5ea47d7e5e7d5c52fc"},{"sha":"5f4a24a767cb9a3f88d376871745473a59ded6aa","author":{"email":"a3b1ac66d44425905084924ee6cef90b31d2e761@suse.com","name":"Adam Spiers"},"message":"add pacemaker_transaction LWRP\n\nThis will allow multiple objects to be configured in Pacemaker\nwithin a single transaction.  An example is visible in the\npacemaker_test::keystone_transaction recipe added by this commit.","distinct":true,"url":"https://api.github.com/repos/aspiers/crowbar-ha/commits/5f4a24a767cb9a3f88d376871745473a59ded6aa"}]}