search

canonical / opensearch-dashboards-operator

Opensearch Dashboards Juju Charm
Apache License 2.0
0 stars 1 forks source link

[DESIGN] Introduce YAML parser for dealing with yaml config operations #35

Open Mehdi-Bendriss opened 6 months ago

Mehdi-Bendriss commented 6 months ago

Currently YAML related operations (read / write) is handled as plain text operations. We should use proper yaml parsing when dealing with yaml documents.

github-actions[bot] commented 6 months ago

https://warthogs.atlassian.net/browse/DPE-4308

juditnovak commented 6 months ago

IGNORE, discussed live, see conclusion in the next comment

--

This was a design decision that have been discussed multiple times.

I see no need for YAML operations in the current layout (and I'm not the only one with that view in the Data Platform Team).

Aslo, may I ask why would you label a known design decision would as a bug ...?

Is there a missing functionality that can not be fullfilled with the current method?

In other words: what is the purpose of this open issue? What is the problem we are trying to resolve?

(Part of the reason why I'm asking is because we have a few burning issues around the Dashboards...)

juditnovak commented 6 months ago

Conclusion: we agreed offline, that as long as pyyaml could allow for a simple solution (i.e. not "re-inventing the wheel") it could be a good balance to allow for config management. Taking it up from here