-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rocotostat/rocotocheck xml output #21
Comments
Adding a -x flag to rocotostat and rocotocheck commands to request output in XML format is a good idea. |
Lets discuss the XML structure to make sure it is extensible without modifying existing behavior in case new things need to be added to it later. Otherwise, downstream tools that use it will break when we change it. |
Chris, I'll send you some ideas shortly. The goal is to make an XML form of the tables that rocotostat dumps. As a last resort, in the future, if we need to make backward-incompatible changes to the XML, we could specify a version. rocotostat ... --xml-version 1.3 # get Rocoto 1.3's XML output style |
The main use case is for an automated tool to either display, or act on, the status of a workflow. The expected structure is the same as the workflow document. However, this will only be parsed by software, not humans, so the syntax should be expressive rather than compact. I suggest we put in every piece of information that is readily available, but not likely to change significantly from version to version:
|
That looks like a reasonable proposal to start from. |
I'm also considering having it be a new program. The rocotostat is complex enough as it is. I was thinking of something like "rocotodump." If I were coming into the system from outside as a new developer, it would sound like a program to dump the state in a non-human-readable form. |
Rocoto state viewing commands do not provide an option to generate their output in XML format so that it is easily parsed by downstream tools.
The text was updated successfully, but these errors were encountered: