)]}'
{"manifests/compute.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"dd195451431082760d649c6a8d8abf527bf90525","unresolved":false,"context_lines":[{"line_number":134,"context_line":") {"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  include ::nova::params"},{"line_number":137,"context_line":"  include ::stdlib"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  nova_config {"},{"line_number":140,"context_line":"    \u0027DEFAULT/reserved_host_memory_mb\u0027:           value \u003d\u003e $reserved_host_memory;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"3ab451fb_24a5bd03","line":137,"updated":"2015-04-17 13:25:21.000000000","message":"not sure you have to include stdlib here, we already use ensure_resource at some places without doing it.","commit_id":"e8717146bef6b4962fbe83da73a65a5e23665dbf"},{"author":{"_account_id":7468,"name":"Sergey Vasilenko","email":"svasilenko@mirantis.com","username":"xenolog"},"change_message_id":"947500100470de84bb351a21f3bcc1b93bcdd6ae","unresolved":false,"context_lines":[{"line_number":134,"context_line":") {"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  include ::nova::params"},{"line_number":137,"context_line":"  include ::stdlib"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  nova_config {"},{"line_number":140,"context_line":"    \u0027DEFAULT/reserved_host_memory_mb\u0027:           value \u003d\u003e $reserved_host_memory;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"3ab451fb_e477d5d2","line":137,"in_reply_to":"3ab451fb_24a5bd03","updated":"2015-04-17 13:46:33.000000000","message":"is it happens without error, because library included on early stage? may be.","commit_id":"e8717146bef6b4962fbe83da73a65a5e23665dbf"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"3b0aa0a97cf2b615f03919227d38bbc577acc73a","unresolved":false,"context_lines":[{"line_number":134,"context_line":") {"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  include ::nova::params"},{"line_number":137,"context_line":"  include ::stdlib"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  nova_config {"},{"line_number":140,"context_line":"    \u0027DEFAULT/reserved_host_memory_mb\u0027:           value \u003d\u003e $reserved_host_memory;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"dac4157f_d5444675","line":137,"in_reply_to":"3ab451fb_e477d5d2","updated":"2015-04-21 14:54:20.000000000","message":"Anyway, it would be more safe to include it, rather than not, right?","commit_id":"e8717146bef6b4962fbe83da73a65a5e23665dbf"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"b0b11b23eb8e2eaedfccbe690146ba1556797a47","unresolved":false,"context_lines":[{"line_number":134,"context_line":") {"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  include ::nova::params"},{"line_number":137,"context_line":"  include ::stdlib"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  nova_config {"},{"line_number":140,"context_line":"    \u0027DEFAULT/reserved_host_memory_mb\u0027:           value \u003d\u003e $reserved_host_memory;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"3afb71cf_b72bdc93","line":137,"in_reply_to":"7ad32947_bd2460af","updated":"2015-06-09 01:01:35.000000000","message":"Agreed with Emilien. The stdlib class is not relevant to the ensure_package function https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/manifests/init.pp","commit_id":"e8717146bef6b4962fbe83da73a65a5e23665dbf"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"ee1c733d28cd014b544028931a982ea8657fd513","unresolved":false,"context_lines":[{"line_number":134,"context_line":") {"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  include ::nova::params"},{"line_number":137,"context_line":"  include ::stdlib"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  nova_config {"},{"line_number":140,"context_line":"    \u0027DEFAULT/reserved_host_memory_mb\u0027:           value \u003d\u003e $reserved_host_memory;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"7ad32947_bd2460af","line":137,"in_reply_to":"dac4157f_d5444675","updated":"2015-04-27 16:05:07.000000000","message":"I agree it works anyway, but for code consistency with other modules, I would suggest we don\u0027t include it. There is no reason to do so.","commit_id":"e8717146bef6b4962fbe83da73a65a5e23665dbf"}]}