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

add flux version --instance #1810

Closed
garlick opened this issue Nov 8, 2018 · 1 comment
Closed

add flux version --instance #1810

garlick opened this issue Nov 8, 2018 · 1 comment
Assignees
@garlick

Comments

@garlick
Copy link
Member

garlick commented Nov 8, 2018

This is slightly silly, but should flux version be given an option to connect to the broker and ask it for a version?

We were talking about this sort of thing over in #1809 and it just seemed like, in addition to the challenges of keeping protocols interoperable, it's just going to be fun figuring out what's going on in some cases, and tools might be helpful.
@grondo
Copy link
Contributor

grondo commented Nov 8, 2018

That is a good thought! The version of the broker should at least be set in an attribute (?). If FLUX_URI is set, maybe flux version could query that attribute and report it, even by default.

garlick added a commit to garlick/flux-core that referenced this issue Nov 8, 2018
@garlick
cmd/flux-version: add --remote option
727049b 
If --remote option is given, connect to broker and
get 'version' attribute

Fixes flux-framework#1810
@garlick garlick self-assigned this Nov 8, 2018
garlick added a commit to garlick/flux-core that referenced this issue Nov 8, 2018
@garlick
cmd/flux-version: add --remote option
53ce99c 
If --remote option is given, connect to broker and
get 'version' attribute

Fixes flux-framework#1810
@garlick garlick mentioned this issue Nov 8, 2018
Merged
garlick added a commit to garlick/flux-core that referenced this issue Nov 8, 2018
@garlick
cmd/flux-version: show cmd, lib, and broker version
e7416c1 
Fixes flux-framework#1810
@grondo grondo closed this as completed in 15f9574 Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@garlick garlick

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@garlick @grondo