Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature request: Allow deadline config show to be parsable #585

Open
Ahuge opened this issue Feb 4, 2025 · 1 comment · May be fixed by #586
Open

Feature request: Allow deadline config show to be parsable #585

Ahuge opened this issue Feb 4, 2025 · 1 comment · May be fixed by #586
Labels
enhancement New feature or request

Comments

@Ahuge
Copy link
Contributor

Ahuge commented Feb 4, 2025

Describe the problem

I need the ability to parse the output of deadline config show to display the current settings to the user.

Currently I can build a custom parser for the output of deadline config show but that has already changed once (343bf7b) and it feels like a reasonable feature request to have the output parsable in a format such as JSON.

Proposed Solution

I propose adding flag to the deadline config show command called --output which accepts text and json to control the output style for that command.

By default it will stay at text which will maintain the same formatting.

Example Use Cases

If you are writing a tool that needs to describe the current settings to the user. Especially if the tool does not have a python scripting environment.
@Ahuge Ahuge added enhancement New feature or request needs triage A new report that needs a first look labels Feb 4, 2025
@Ahuge Ahuge linked a pull request Feb 4, 2025 that will close this issue
Open
2 tasks
@epmog
Copy link
Contributor

epmog commented Feb 5, 2025

Thanks for filing an issue and PR! Yea absolutely, I've commented on the PR but we'll want to match other CLI commands with the same same options ("verbose" instead of "text").

Anything that needs to be parse-able by machinery should be able to leverage something like --output json to ensure other log lines don't interfere with the parsing and make it more robust.

@epmog epmog removed the needs triage A new report that needs a first look label Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Add an --output flag to config show Ahuge/deadline-cloud
2 participants
@Ahuge @epmog