)]}'
{"manifests/db/sync.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"203acc183aae0160dc68cc35614c8c516424b5ea","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*exec_path*]"},{"line_number":16,"context_line":"#   (optional) The path  to use for finding the magnum-db-manage binary."},{"line_number":17,"context_line":"#   Defaults to /usr/bin"},{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"class magnum::db::sync("},{"line_number":20,"context_line":"  $user         \u003d \u0027magnum\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf3e73b7_b62d10aa","line":17,"updated":"2017-08-04 22:31:33.000000000","message":"I\u0027m torn on this because techincally we only support packaged versions of these things.  Additionally if you\u0027re running out of a venv you would be able to work around the path by putting something like the following in your composition layer where you\u0027re including this class:\n\nExec\u003c| title \u003d\u003d \u0027magnum-db-sync\u0027 |\u003e {\n  path \u003d \u0027/my/cool/venv\u0027\n}","commit_id":"f739aa363b7d536c8f561a1a1fbe81a09719de56"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"292a9741489fd24399df660a6d648f97259973e5","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*exec_path*]"},{"line_number":16,"context_line":"#   (optional) The path  to use for finding the magnum-db-manage binary."},{"line_number":17,"context_line":"#   Defaults to /usr/bin"},{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"class magnum::db::sync("},{"line_number":20,"context_line":"  $user         \u003d \u0027magnum\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf3e73b7_d609b431","line":17,"updated":"2017-08-04 22:32:15.000000000","message":"sorry that should have read path \u003d\u003e \u0027/my/cool/venv\u0027 but you get the point","commit_id":"f739aa363b7d536c8f561a1a1fbe81a09719de56"},{"author":{"_account_id":12634,"name":"Risto Laurikainen","email":"risto.laurikainen@csc.fi","username":"rlaurika"},"change_message_id":"40f5b339b6482919eff8b39e8d980b9413ea9af7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*exec_path*]"},{"line_number":16,"context_line":"#   (optional) The path  to use for finding the magnum-db-manage binary."},{"line_number":17,"context_line":"#   Defaults to /usr/bin"},{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"class magnum::db::sync("},{"line_number":20,"context_line":"  $user         \u003d \u0027magnum\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf3e73b7_b783b856","line":17,"in_reply_to":"bf3e73b7_b62d10aa","updated":"2017-08-07 05:53:53.000000000","message":"I\u0027m currently running Magnum out of a virtualenv and this was the only patch I needed to make the module work out of the box without workarounds. Everything else is already there, like the ability to not install packages and not manage or start services.\n\nI\u0027ve also had mixed success in the past with overriding parameters like that. I\u0027ve found it to be buggy in older versions of Puppet, which I\u0027m somewhat stuck with in my environment.","commit_id":"f739aa363b7d536c8f561a1a1fbe81a09719de56"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"7a4f703e50294d32ee35b58a88c0ea6aef7ec302","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*exec_path*]"},{"line_number":16,"context_line":"#   (optional) The path  to use for finding the magnum-db-manage binary."},{"line_number":17,"context_line":"#   Defaults to /usr/bin"},{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"class magnum::db::sync("},{"line_number":20,"context_line":"  $user         \u003d \u0027magnum\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf3e73b7_20ff7602","line":17,"in_reply_to":"bf3e73b7_b783b856","updated":"2017-08-07 14:38:00.000000000","message":"Yea I get that, it\u0027s just one of those things that we\u0027ve consistently not necessarily supported because the official stance has always been package focused.  I guess this is ok for now.  We probably should make this be an array [\u0027/bin\u0027, \u0027/usr/bin\u0027] but that\u0027s just a nit.","commit_id":"f739aa363b7d536c8f561a1a1fbe81a09719de56"},{"author":{"_account_id":12634,"name":"Risto Laurikainen","email":"risto.laurikainen@csc.fi","username":"rlaurika"},"change_message_id":"40f5b339b6482919eff8b39e8d980b9413ea9af7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*exec_path*]"},{"line_number":16,"context_line":"#   (optional) The path  to use for finding the magnum-db-manage binary."},{"line_number":17,"context_line":"#   Defaults to /usr/bin"},{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"class magnum::db::sync("},{"line_number":20,"context_line":"  $user         \u003d \u0027magnum\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf3e73b7_57824c51","line":17,"in_reply_to":"bf3e73b7_d609b431","updated":"2017-08-07 05:53:53.000000000","message":"Done","commit_id":"f739aa363b7d536c8f561a1a1fbe81a09719de56"}]}