)]}'
{"lib/puppet/provider/cinder_type/openstack.rb":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"95904262c95bf9e83f5611b7e92f5dc8a0637f9f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"require File.join(File.dirname(__FILE__), \u0027..\u0027,\u0027..\u0027,\u0027..\u0027, \u0027puppet/provider/cinder\u0027)"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"Puppet::Type.type(:cinder_type).provide("},{"line_number":4,"context_line":"  :openstack,"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"dab17558_f2d34236","line":1,"updated":"2016-05-16 15:18:52.000000000","message":"this file is already under \u0027puppet/provider\u0027, why we need to step out on 3 dir above and then back here?\n\nwhy not just \u0027..\u0027 and \u0027cinder\u0027 ?\n\nAnd also, i haven\u0027t ever seen this error before. How we can reproduce it?","commit_id":"2065a3a389daa77c37292551afd0774b56f94d8d"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"3d281ad1483456e69d1b87740ecd662fdf04d368","unresolved":false,"context_lines":[{"line_number":1,"context_line":"require File.join(File.dirname(__FILE__), \u0027..\u0027,\u0027..\u0027,\u0027..\u0027, \u0027puppet/provider/cinder\u0027)"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"Puppet::Type.type(:cinder_type).provide("},{"line_number":4,"context_line":"  :openstack,"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"dab17558_92a4764b","line":1,"in_reply_to":"dab17558_d2cc3e37","updated":"2016-05-16 15:25:39.000000000","message":"So, why then some modules work with current path and other don\u0027t?\n\nOr we must to change load path for all providers?\n\nI\u0027d like to see certain answer and solution for this situation, before merge this patch.","commit_id":"2065a3a389daa77c37292551afd0774b56f94d8d"},{"author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"change_message_id":"1aaca55fe180927e53f760804cb0e67ba467fc0e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"require File.join(File.dirname(__FILE__), \u0027..\u0027,\u0027..\u0027,\u0027..\u0027, \u0027puppet/provider/cinder\u0027)"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"Puppet::Type.type(:cinder_type).provide("},{"line_number":4,"context_line":"  :openstack,"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"dab17558_d2cc3e37","line":1,"in_reply_to":"dab17558_f2d34236","updated":"2016-05-16 15:22:59.000000000","message":"I havent tested puppet 3.x but puppet 4.4.2 (puppetserver 2.3.2) I ran into this issue. When I changed the line to the one in the patch, it worked, and when I changed it back it still worked until the puppetserver was bounced.\n\nAnd besides, we already do it this way for some providers :)\n\nhttps://github.com/openstack/puppet-glance/blob/master/lib/puppet/provider/glance_image/openstack.rb","commit_id":"2065a3a389daa77c37292551afd0774b56f94d8d"}]}
