)]}'
{"manifests/init.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"ab54caa5393780f22d895257bd6056d4497f8d4b","unresolved":false,"context_lines":[{"line_number":392,"context_line":""},{"line_number":393,"context_line":"  package { \u0027python-lesscpy\u0027:"},{"line_number":394,"context_line":"    ensure  \u003d\u003e $package_ensure,"},{"line_number":395,"context_line":"  }"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"  exec { \u0027refresh_horizon_django_cache\u0027:"},{"line_number":398,"context_line":"    command     \u003d\u003e \"${::horizon::params::manage_py} collectstatic --noinput --clear \u0026\u0026 ${::horizon::params::manage_py} compress --force\","}],"source_content_type":"text/x-puppet","patch_set":2,"id":"dae33548_87abfdd5","line":395,"updated":"2016-02-16 09:51:22.000000000","message":"agree with Thomas about lesscpy.\n\nBut about executing collectstatic, there is can be situation, when someone want to override current default horizon scheme from puppet - and then we MUST run collectstatic.\n\nSo, for me, we can proceed with next way:\n\n  exec { \u0027refresh_horizon_django_cache\u0027:\n    command     \u003d\u003e \"${::horizon::params::manage_py} collectstatic --noinput --clear \u0026\u0026 ${::horizon::params::manage_py} compress --force\",\n    refreshonly \u003d\u003e true,\n    require     \u003d\u003e Package[\u0027horizon\u0027],\n  }\n\n  if $::os_package_type \u003d\u003d \u0027rpm\u0027 {\n    if $compress_offline {\n      Concat[$::horizon::params::config_file] ~\u003e Exec[\u0027refresh_horizon_django_cache\u0027]\n    }\n  }\n\nIn this case, we will define resource exec[\u0027refresh_horizon_django_cache\u0027], but i will not be executed until we trigger that - option refreshonly \u003d\u003e true provides us such opportunity and we use it for rpm case or some specific situation - so, logic is unchanged.\n\nAnd then, we can trigger it if we need it.","commit_id":"dc25ea1deeb4d306330cccaa3458da1644c89ab1"},{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"fbe8eae804cbe007ab53199b312f614710bf627b","unresolved":false,"context_lines":[{"line_number":392,"context_line":""},{"line_number":393,"context_line":"  package { \u0027python-lesscpy\u0027:"},{"line_number":394,"context_line":"    ensure  \u003d\u003e $package_ensure,"},{"line_number":395,"context_line":"  }"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"  exec { \u0027refresh_horizon_django_cache\u0027:"},{"line_number":398,"context_line":"    command     \u003d\u003e \"${::horizon::params::manage_py} collectstatic --noinput --clear \u0026\u0026 ${::horizon::params::manage_py} compress --force\","}],"source_content_type":"text/x-puppet","patch_set":2,"id":"dae33548_47029570","line":395,"in_reply_to":"dae33548_87abfdd5","updated":"2016-02-16 09:57:20.000000000","message":"it\u0027s a good idea I think. We won\u0027t trigger collecstatic by default, but when we really need to do it for ubuntu/debian packages - we can.\n\nThomas, Emilien, what do you think?","commit_id":"dc25ea1deeb4d306330cccaa3458da1644c89ab1"}]}