{"push_id":1372443521,"size":36,"distinct_size":36,"ref":"refs/heads/master","head":"19072a617e9f8f5f8e003c985440ea2f31edc6ac","before":"f9ec50f0827a879f6b756d1441cc8b3f1c323052","commits":[{"sha":"1053bdd6806ea495799d36b1183d79ae9c747438","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add rspec\n\nYeah... it is time we started adding some tests...","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/1053bdd6806ea495799d36b1183d79ae9c747438"},{"sha":"4b185b9c6b52e4ee91abee2b69bb9a125e298136","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add initial configuration class","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/4b185b9c6b52e4ee91abee2b69bb9a125e298136"},{"sha":"2a34cc2b5fb841266d6b8bf187717c50153950ad","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Use Config for requestor username/password","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/2a34cc2b5fb841266d6b8bf187717c50153950ad"},{"sha":"373d1d0b1ab4ff53fc02d9f39f6e5df392d6f4cf","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Adds middleware configuration","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/373d1d0b1ab4ff53fc02d9f39f6e5df392d6f4cf"},{"sha":"6322144b8522e0678a9175c48bb6560d09f6c2a1","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add stackprof and rails as dev dependencies","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/6322144b8522e0678a9175c48bb6560d09f6c2a1"},{"sha":"47719a9a32e7351c409baf3c0bfa56ae41d7bb00","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add middleware spec\n\nThis commit is here to confirm existing functionality when refactoring\nthe middleware in the future, which will make it easier to test when\nchanging the config.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/47719a9a32e7351c409baf3c0bfa56ae41d7bb00"},{"sha":"e4bc3d7066c4f95be880b15b95ccb2eeadcdf643","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add underscore to active_record and active_support\n\nThis makes sure the camelcase conversion from snakecase to class name\nworks consistently and doesn't require any helpers to convert between\nthe two","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/e4bc3d7066c4f95be880b15b95ccb2eeadcdf643"},{"sha":"06662ac627abe2fc335223eda1e658f9169daa75","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Use config for defining middlewares\n\nThis does a lot of things:\n\n* Use the MiqPerformance.config.middlewares to define which middlewares\n  (from lib/miq_performance/middlewares/) we include in the main\n  middleware when loaded.\n* Define the middlewares on the instance instead of the class\n  (middlewares in Rails seem to be singletons, so this should be fine,\n  and makes it much easier to test (I was considering doing\n  `Object.remove_const` or something to get around this otherwise...\n  bleh...)\n* Removes the `performance_middlewares` class method/variable, which\n  subsequently required removing it from the middleware modules as well\n* Adds tests to confirm that we are only defining what was requested for\n  the middlewares","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/06662ac627abe2fc335223eda1e658f9169daa75"},{"sha":"ef57b37cce9c3b89698264c4676871048135f039","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add configs for host, read_timeout, and ignore_ssl","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/ef57b37cce9c3b89698264c4676871048135f039"},{"sha":"c78db56f5a99b2120f3a32daa276b7d82558d946","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add default_dir configuration\n\nUp until now, the base directory for everything that was saved for\nMiqPerformance was hard coded to \"tmp/miq_performance\", relative to the\ncurrent working directory.  This is still the default, but has been\nupdated to be configurable via the config file.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/c78db56f5a99b2120f3a32daa276b7d82558d946"},{"sha":"70b8fa39196a58dcf1a1bb0e1513263958e61fee","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add requestfile_dir config for requestor\n\nWill eventually be used to allow the user to configure where they will\nfind the Requestfile","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/70b8fa39196a58dcf1a1bb0e1513263958e61fee"},{"sha":"20cb0a6d7f2f9d2ff4ab80debadaf9aff1d2dc90","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Change RequestFileBuilder to RequestfileBuilder\n\nThis class name will better match the outputted file that is created by\nthe builder.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/20cb0a6d7f2f9d2ff4ab80debadaf9aff1d2dc90"},{"sha":"2434077f1e20d21f7ea73f74ef02e35e00cc79aa","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Reset config after all tests\n\nMost tests will be testing different configs and variations, so reseting\nand reloading the config after each test, requiring it to be reloaded\nfor each test, is something that is convenient enough (and not a huge\nperformance hit on the suite overall) to do every time.\n\nNote:  The performance will not be realized in the `after` filter, but in\nthe subsequent test that now has to reload the config.  Again, this is\njust a file read operation, so it should be an acceptable performance\npenalty, if slightly obscure to a new dev.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/2434077f1e20d21f7ea73f74ef02e35e00cc79aa"},{"sha":"1487397019cf262a5be6c3f866020dbd6b90a6f7","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Set `RequestfileBuilder::filepath` via the config","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/1487397019cf262a5be6c3f866020dbd6b90a6f7"},{"sha":"f4ddf85f80d2a87d53a562eb0504fcbc41b8de7a","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Use and .freeze IGNORED_PAYLOAD constant","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/f4ddf85f80d2a87d53a562eb0504fcbc41b8de7a"},{"sha":"dcf1fd2cf4eae9e5dfe38dfee4049cd19cbc0603","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Adds skip_schema_queries to the config\n\nDefaults it to false.\n\nAlso fixes a bug with MiqPerformance::Configuration#[] where if a config\nvalue was falsey, it would return the default value for the config\ninstead of the user configured value.  This was because the code was\ndoing a simple `||`, and switching it to a `.fetch` fixes that bug.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/dcf1fd2cf4eae9e5dfe38dfee4049cd19cbc0603"},{"sha":"3d72deadac586545f60c8e6b8e5718f4c1009ee4","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Fix ~/.miq_performance issue in tests\n\nThere was a non-deterministic spec failure when the specs were run as a\nsuite.  This was caused by certain specs making use of the\nconfiguration, assuming the default, but not stubbing out the\nconfig_file_location and it was processed when running that test,\nmemoized, and then saved for future tests.\n\nAlso, it makes more sense to make sure this is cleared before each test\nthan waiting until a test is finished to have it clean up after itself.\n\nOf note:  `if defined?` is used because not all specs load the\nconfiguration class, so this constant might not be initialized when\nrunning individual tests, or tests in a specific order.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/3d72deadac586545f60c8e6b8e5718f4c1009ee4"},{"sha":"5b17583e61bf361fce6c7028df5c38562316e55a","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Use MiqPerformance::Configuration[] for initialize\n\nAfter setting the config, we want to make use of the `[]` instance\nmethod since it does similar functionality to what was being done here,\nbut also makes use of \"fetch\", which avoids issues with the key existing\nin the config, but being `falsey`.\n\nFor these keys, this might cause issues since they should never be\n`nil`/`false`, but it will provide a more expected behavior to the\nuser/developer when these keys are that instead of jumping to the\nDEFAULTS even though the key exists.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/5b17583e61bf361fce6c7028df5c38562316e55a"},{"sha":"5fb31a46692c1d4e77d6f74b728db8ccec333fe3","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add CLI skeleton","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/5fb31a46692c1d4e77d6f74b728db8ccec333fe3"},{"sha":"1cc1a00240464b2ed813f9e86f8f472be2371ebc","author":{"name":"Nick LaMuro","email":"26402cfb30ef5f9657d054878024167809d01c07@gmail.com"},"message":"Add clean subcommand\n\nUsed for cleaning old run directories the `tmp/miq_performance` (or as\nconfigured) directory.","distinct":true,"url":"https://api.github.com/repos/kbrock/miq_performance/commits/1cc1a00240464b2ed813f9e86f8f472be2371ebc"}]}
{"actor":{"display_login":"kbrock"}}