Skip to content
This repository has been archived by the owner on Nov 16, 2020. It is now read-only.

Version API and CLI #500

Merged
merged 1 commit into from
Jun 8, 2018
Merged

Version API and CLI #500

merged 1 commit into from
Jun 8, 2018

Conversation

imikushin
Copy link
Contributor

Add /v1/version API endpoint and use it in dispatch version CLI to return server side version info.

Now you can do this:

➜  dispatch git:(version) dispatch version
Client: v0.1.16-5-gf535bc07
Server: v0.1.16-5-gf535bc07
➜  dispatch git:(version)
➜  dispatch git:(version) dispatch version --json
{
  "client": {
    "version": "v0.1.16-5-gf535bc07",
    "commit": "f535bc07c3f84ce2ad9f296c1eaca9aab45aa853",
    "buildDate": "2018-06-06T17:23:30Z",
    "goVersion": "go1.10.2",
    "compiler": "gc",
    "platform": "darwin/amd64"
  },
  "server": {
    "version": "v0.1.16-5-gf535bc07",
    "commit": "f535bc07c3f84ce2ad9f296c1eaca9aab45aa853",
    "buildDate": "2018-06-06T17:23:30Z",
    "goVersion": "go1.10.2",
    "compiler": "gc",
    "platform": "linux/amd64"
  }
}

berndtj
berndtj previously approved these changes Jun 7, 2018
Copy link
Contributor

@berndtj berndtj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me!

kars7e
kars7e previously approved these changes Jun 7, 2018
neosab
neosab previously approved these changes Jun 7, 2018
Copy link
Contributor

@neosab neosab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@imikushin imikushin dismissed stale reviews from neosab, kars7e, and berndtj via 480f21a June 7, 2018 22:50
@imikushin imikushin force-pushed the version branch 2 times, most recently from 480f21a to cd0f5a5 Compare June 7, 2018 22:51
@imikushin imikushin merged commit 93a1770 into vmware-archive:master Jun 8, 2018
@imikushin imikushin mentioned this pull request Jun 8, 2018
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants