Unidata / LDM

The Unidata Local Data Manager (LDM) system includes network client and server programs designed for event-driven data distribution, and is the fundamental component of the Unidata Internet Data Distribution (IDD) system.
http://www.unidata.ucar.edu/software/ldm
Other
43 stars 27 forks source link

Convert runtime files into YAML #68

Open semmerson opened 5 years ago

semmerson commented 5 years ago

Files ldmd.conf, pqact.conf, pqsurf.conf, registry.xml, scour.conf

mustbei commented 3 years ago

As to the registry.xml the new ldmadmin (Python) already parses it seamlessly. Should it be included in the conversion?

semmerson commented 3 years ago

What's crucial is whether or not a programmatic library exists for parsing the particular configuration-file. One exists for XML, so "registry.xml" could be left alone. Parsers for the other configuration-files, however, are hand-coded.

mustbei commented 3 years ago

Indeed, I had once a syntactic error in the ldmd.conf file that the current yy parser could not point me to. Will do for the all .conf files, acknowledging however the impact on their current usage in both the new ldmadmin and the underlying C code (pqact, etc.), as well as their backward incompatibility.

mustbei commented 3 years ago

Oh, and on the user... (the mandatory learning curve and the subsequent ease of use and robustness thereafter)

semmerson commented 3 years ago

This issue is very low priority because 1) the amount of effort is significant; 2) user's are used to the current solution; 3) providing forward and rollback compatibility is very non-trivial (the configuration-files would have to be automatically converted to the new format and changes to the new configuration-files would have to be rolled back into the old configuration-files); and 4) there are, doubtless, more important issues.

So, don't work on it if you were planning too.

This issue was more aspirational (i.e., wishful thinking) on my part.

On Tue, Apr 6, 2021 at 10:59 AM mustbei @.***> wrote:

Indeed, I had once a syntactic error in the ldmd.conf file that the current yy parser could not point me to. Will do for the all .conf files, acknowledging however the impact on their current usage in both the new ldmadmin and the underlying C code (pqact, etc.), as well as their backward incompatibility.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Unidata/LDM/issues/68#issuecomment-814279102, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEVZ7PHKE27RYYWC2LGHEDTHM4YLANCNFSM4GWKUNZA .

mustbei commented 3 years ago

Makes sense. Should it be closed then?

semmerson commented 3 years ago

Leave it open. It's aspirational.

Hope is good.

mustbei commented 3 years ago

Absolutely!