bmarini / knife-inspect

Inspect your chef repo as is compares to what is on your chef server
MIT License
55 stars 13 forks source link

Use parallel lib to speed up rest api operations #27

Closed kamaradclimber closed 10 years ago

kamaradclimber commented 10 years ago

On my setup we go from 50sec for knife inpect to 10 seconds.

coveralls commented 10 years ago

Coverage Status

Coverage decreased (-0.31%) when pulling da2862696d8143d2617d353c74599468e0741a7b on kamaradclimber:parallel into 84b850714e65c1ff241beedbdd723958ff1b5b26 on bmarini:master.

gregkare commented 10 years ago

Hi, thanks a lot for the pull-request! I've added some comments to da28626, can you rebase to remove the metadata.json support?

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.25%) when pulling 35eb4d357ab6dfb10b5535d08fa2698c649e7b37 on kamaradclimber:parallel into 84b850714e65c1ff241beedbdd723958ff1b5b26 on bmarini:master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.25%) when pulling 35eb4d357ab6dfb10b5535d08fa2698c649e7b37 on kamaradclimber:parallel into 84b850714e65c1ff241beedbdd723958ff1b5b26 on bmarini:master.

gregkare commented 10 years ago

Looks good to me, I'll do a 0.10.0 release including this and the metadata.json support.

Thanks!