)]}'
{"Gemfile":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"5e07265327a6dc19792fe7bfc914b14adaab46d4","unresolved":false,"context_lines":[{"line_number":9,"context_line":"gem \u0027foodcritic\u0027, \u0027~\u003e 3.0.3\u0027"},{"line_number":10,"context_line":"gem \u0027strainer\u0027"},{"line_number":11,"context_line":"gem \u0027rubocop\u0027, \u0027~\u003e 0.18.1\u0027"},{"line_number":12,"context_line":"gem \u0027chef-vault\u0027"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"9aa7fdbe_3e801e8b","line":12,"updated":"2014-10-29 19:29:46.000000000","message":"this way won\u0027t work for chef-client runs,  need  to use the chef-gem resource to optionally install this as needed.","commit_id":"8ab93394c4622c5dc802a56bd42540ed274d3ed7"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"248ed26f826a2764915b2d13468bd2ffae56a900","unresolved":false,"context_lines":[{"line_number":10,"context_line":"gem \u0027strainer\u0027"},{"line_number":11,"context_line":"gem \u0027rubocop\u0027, \u0027~\u003e 0.18.1\u0027"},{"line_number":12,"context_line":"gem \u0027rake\u0027, \u0027~\u003e 10.0\u0027"},{"line_number":13,"context_line":"gem \u0027chef-vault\u0027, \u0027~\u003e 2.3.0\u0027"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"7a8c0949_543bbdd0","line":13,"updated":"2014-11-06 11:31:33.000000000","message":"More correct to say   ~\u003e 2.3    here?","commit_id":"22d97f479d4e1c5fd5433e09fa2b70745e58698b"}],"libraries/passwords.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"5a9c99cdf799290dfce22b0ff1cf5a4030b9587a","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    begin"},{"line_number":82,"context_line":"      require \u0027chef-vault\u0027"},{"line_number":83,"context_line":"    rescue LoadError"},{"line_number":84,"context_line":"      Chef::Log.warn(\"Missing gem \u0027chef-vault\u0027, use recipe[chef-vault] to install it first.\")"},{"line_number":85,"context_line":"    end"},{"line_number":86,"context_line":"    ::Chef::Log.info \"Loading vault secret #{index} from #{bag_name}\""},{"line_number":87,"context_line":"    ::ChefVault::Item.load(bag_name, index)[index]"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"fa98f980_57be5401","line":84,"updated":"2014-09-19 19:08:10.000000000","message":"new cookbook/recipe for this? So we might need to doc that they need to add in that cookbook, add that recipe to run list and setup the chef-vault before this will work first time thru?\n\nWe it be so bad to just to do that for them in another common recipe we conditionally add to the os-base run list?","commit_id":"8ab93394c4622c5dc802a56bd42540ed274d3ed7"},{"author":{"_account_id":9978,"name":"Pierre Rambaud","email":"pierre.rambaud@numergy.com","username":"GoT"},"change_message_id":"c9fa736d3cc81ad0d7920a12c490de4e78a2a4b3","unresolved":false,"context_lines":[{"line_number":77,"context_line":"    ::Chef::DataBagItem.load(bag_name, index)[index]"},{"line_number":78,"context_line":"  end"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  def vault_secret(bag_name, index)"},{"line_number":81,"context_line":"    begin"},{"line_number":82,"context_line":"      require \u0027chef-vault\u0027"},{"line_number":83,"context_line":"    rescue LoadError"}],"source_content_type":"text/x-ruby","patch_set":4,"id":"7a8c0949_2a7248e5","line":80,"updated":"2014-11-06 13:36:21.000000000","message":"I think you couldn\u0027t use ::Chefvault::Item if chef-vault isn\u0027t loaded.\nThis function should look like:\n\ndef vault_secret(bag_name, index)\n  require \u0027chef-vault\u0027\n  ::Chef::Log.info \"Loading vault secret #{index} from #{bag_name}\"\t86\n  ::ChefVault::Item.load(bag_name, index)[index]\nrescue\n  Chef::Log.warn(\u0027Missing......\u0027)\n  nil\nend","commit_id":"19179ff2f04e0da7591c7ace3fcea17eeaf906fa"},{"author":{"_account_id":9978,"name":"Pierre Rambaud","email":"pierre.rambaud@numergy.com","username":"GoT"},"change_message_id":"c54c3acf70085001f77591f4077c59f3bd6d28ec","unresolved":false,"context_lines":[{"line_number":77,"context_line":"    ::Chef::DataBagItem.load(bag_name, index)[index]"},{"line_number":78,"context_line":"  end"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  def vault_secret(bag_name, index)"},{"line_number":81,"context_line":"    begin"},{"line_number":82,"context_line":"      require \u0027chef-vault\u0027"},{"line_number":83,"context_line":"    rescue LoadError"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"7a8c0949_a11c96c2","line":80,"updated":"2014-11-06 18:16:31.000000000","message":"I think you couldn\u0027t use ::Chefvault::Item if chef-vault isn\u0027t loaded. \nThis function should look like:\n\ndef vault_secret(bag_name, index)\n  require \u0027chef-vault\u0027\n  ::Chef::Log.info \"Loading vault secret #{index} from #{bag_name}\" 86\n  ::ChefVault::Item.load(bag_name, index)[index]\nrescue\n  ::Chef::Log.warn(\u0027Missing......\u0027)\n  nil\nend","commit_id":"794cc3d461131a5453d2477db5580cebb408fa28"},{"author":{"_account_id":11915,"name":"Jan Klare","email":"klare@osism.tech","username":"jklare"},"change_message_id":"da4d13dfa9b58ad3d5bd9cc2f2bdb9b9112f9917","unresolved":false,"context_lines":[{"line_number":77,"context_line":"    ::Chef::DataBagItem.load(bag_name, index)[index]"},{"line_number":78,"context_line":"  end"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"  def vault_secret(bag_name, index)"},{"line_number":81,"context_line":"    begin"},{"line_number":82,"context_line":"      require \u0027chef-vault\u0027"},{"line_number":83,"context_line":"    rescue LoadError"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"7a8c0949_81787af3","line":80,"in_reply_to":"7a8c0949_a11c96c2","updated":"2014-11-10 14:22:10.000000000","message":"I think its better to just rescue LoadErrors and not everything since the warning is just for that special error and the chef-run is meant to fail at that specific poing if you cant load chef-vault and not later when you are trying to use it. So you will get the warning out of the rescue and the instant fail when trying to load the item.","commit_id":"794cc3d461131a5453d2477db5580cebb408fa28"}],"recipes/default.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"248ed26f826a2764915b2d13468bd2ffae56a900","unresolved":false,"context_lines":[{"line_number":106,"context_line":""},{"line_number":107,"context_line":"if node[\u0027openstack\u0027][\u0027databag_type\u0027] \u003d\u003d \u0027vault\u0027"},{"line_number":108,"context_line":"  chef_gem \u0027chef-vault\u0027 do"},{"line_number":109,"context_line":"    version \u00272.3.0\u0027"},{"line_number":110,"context_line":"  end"},{"line_number":111,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"7a8c0949_b1731314","line":109,"updated":"2014-11-06 11:31:33.000000000","message":"Should this version be an attribute?\n\nAnd should this be ~\u003e 2.3    like the gem file?    Not sure on this as I have not done much with gem in recipes before.","commit_id":"22d97f479d4e1c5fd5433e09fa2b70745e58698b"}]}