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

qvm-ls --raw-data: also remove the []{}=> chars #2444

Closed
rootkovska opened this Issue Nov 20, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@rootkovska
Member

rootkovska commented Nov 20, 2016

Just make it easier to write bash scripts, avoid piping through additional tr -d :)

@jpouellet

This comment has been minimized.

Show comment
Hide comment
@jpouellet

jpouellet Nov 22, 2016

Contributor

I am currently relying on this in one of my scripts, and using --raw-list where plain names are desired. I'm more than happy to change my parsing as I think it would actually simplify things anyway, but one question I even had at the time was what Qubes interfaces should be considered stable? This would be nice to have an actual documented policy on.

As for the specifics of this case, I can easily change it to check ip back != n/a instead of {}, and type = Tpl instead of [].

That leaves only =>. What do you suggest is the best way to expose that? Additional field hidden from default view?

Contributor

jpouellet commented Nov 22, 2016

I am currently relying on this in one of my scripts, and using --raw-list where plain names are desired. I'm more than happy to change my parsing as I think it would actually simplify things anyway, but one question I even had at the time was what Qubes interfaces should be considered stable? This would be nice to have an actual documented policy on.

As for the specifics of this case, I can easily change it to check ip back != n/a instead of {}, and type = Tpl instead of [].

That leaves only =>. What do you suggest is the best way to expose that? Additional field hidden from default view?

@jpouellet

This comment has been minimized.

Show comment
Hide comment
@jpouellet

jpouellet Nov 22, 2016

Contributor

FWIW, core3 looks like the stable interface I actually want.

Contributor

jpouellet commented Nov 22, 2016

FWIW, core3 looks like the stable interface I actually want.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Nov 22, 2016

Member

That leaves only =>. What do you suggest is the best way to expose that? Additional field hidden from default view?

See qubes-prefs.

FWIW, core3 looks like the stable interface I actually want.

The ugliness of the current interface (and the code behind it) is one of the reason for core3...

@rootkovska does --raw-list solve your case? Otherwise I think we may introduce new field like raw_name to not break compatibility.

Member

marmarek commented Nov 22, 2016

That leaves only =>. What do you suggest is the best way to expose that? Additional field hidden from default view?

See qubes-prefs.

FWIW, core3 looks like the stable interface I actually want.

The ugliness of the current interface (and the code behind it) is one of the reason for core3...

@rootkovska does --raw-list solve your case? Otherwise I think we may introduce new field like raw_name to not break compatibility.

@jpouellet

This comment has been minimized.

Show comment
Hide comment
@jpouellet

jpouellet Nov 27, 2016

Contributor

There is also <dispN>

Contributor

jpouellet commented Nov 27, 2016

There is also <dispN>

jpouellet added a commit to jpouellet/qubes-core-admin that referenced this issue Nov 27, 2016

Add raw_name field for qvm-ls
Useful to avoid needing to `... | tr -d '<>{}[]='` just to get clean
VM names in scripts.

Fixes QubesOS/qubes-issues#2444

@jpouellet jpouellet referenced this issue in QubesOS/qubes-core-admin Nov 27, 2016

Merged

Add raw_name field for qvm-ls #78

marmarek added a commit to QubesOS/qubes-core-admin that referenced this issue Mar 14, 2017

Add name-raw field for qvm-ls
Useful to avoid needing to `... | tr -d '<>{}[]='` just to get clean
VM names in scripts.

Fixes QubesOS/qubes-issues#2444

(cherry picked from commit 80450da)

@qubesos-bot qubesos-bot referenced this issue in QubesOS/updates-status Mar 16, 2017

Closed

core-admin v3.2.13 (r3.2) #13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment