
<html>


<head>


<meta http-equiv="Content-Type" content="text/html; charset=utf-8">


</head>


<body>


<div dir="ltr">Hi all,


<div><br>


</div>


<div>So we seem to be adopting a pattern of using UPDATE_HORIZON_CONFIG in the enabled files to add a legacy/angular toggle to the settings. I don't like this, because in settings.py the enabled files are processed *after* local_settings.py imports, meaning


 the angular panel will always be enabled, and would require a local/enabled file change to disable it.</div>


<div><br>


</div>


<div>My suggestion would be:</div>


<div><br>


</div>


<div>- Remove current UPDATE_HORIZON_CONFIG change in the swift panel and images panel patch</div>


<div>- Add equivalents ('angular') to the settings.py HORIZON_CONFIG dict, and then the 'legacy' version to the test settings.</div>


<div><br>


</div>


<div>I think that should run UTs as expected, and allow the legacy/angular panel to be toggled via local_settings.</div>


<div><br>


</div>


<div>Was there a reason we chose to use UPDATE_HORIZON_CONFIG, rather than just updating the dict in settings.py? I couldn't recall a reason, and the original patch ( <a href="https://review.openstack.org/#/c/293168/">https://review.openstack.org/#/c/293168/</a>


 ) doesn't seem to indicate why.</div>


<div><br>


</div>


<div>Rob</div>


</div>


</body>


</html>