Merge branch 'feature/abitrary-python-imports' of https://github.com/stackabletech/superset-operator into feature/abitrary-python-imports

Author: Maleware

CHANGELOG.md

@@ -11,7 +11,6 @@
 - Reduce CRD size from `472KB` to `45KB` by accepting arbitrary YAML input instead of the underlying schema for the following fields ([#528]):
   - `podOverrides`
   - `affinity`
-- Bumping stackable-operator from 0.73.0 to 0.74.0 ([#530])
 
 [#528]: https://github.com/stackabletech/superset-operator/pull/528
 [#530]: https://github.com/stackabletech/superset-operator/pull/530

rust/operator-binary/src/superset_controller.rs

@@ -533,10 +533,11 @@ fn build_rolegroup_config_map(
 
     let mut config_file = Vec::new();
 
+    // By removing the keys from `config_properties`, we avoid pasting the Python code into a Python variable as well
+    // (which would be bad)
     if let Some(header) = config_properties.remove(CONFIG_OVERRIDE_FILE_HEADER_KEY) {
         writeln!(config_file, "{}", header).context(WriteToConfigFileStringSnafu)?;
     }
-    // removing key from `config_properties` to avoid key value match. Append it later.
     let temp_file_footer = config_properties.remove(CONFIG_OVERRIDE_FILE_FOOTER_KEY);
 
     flask_app_config_writer::write::<SupersetConfigOptions, _, _>(