Skip to content

Commit

Permalink
Merge pull request moby#18309 from WeiZhang555/time
Browse files Browse the repository at this point in the history
Consolidate time format for API
  • Loading branch information
thaJeztah committed Dec 2, 2015
2 parents 61773e5 + 9daca12 commit 6deec02
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 7 deletions.
11 changes: 11 additions & 0 deletions api/client/version.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import (
"encoding/json"
"runtime"
"text/template"
"time"

"github.com/docker/docker/api"
"github.com/docker/docker/api/types"
Expand Down Expand Up @@ -72,6 +73,16 @@ func (cli *DockerCli) CmdVersion(args ...string) (err error) {
}

defer func() {
// first we need to make BuildTime more human friendly
t, errTime := time.Parse(time.RFC3339Nano, vd.Client.BuildTime)
if errTime == nil {
vd.Client.BuildTime = t.Format(time.ANSIC)
}
t, errTime = time.Parse(time.RFC3339Nano, vd.Server.BuildTime)
if errTime == nil {
vd.Server.BuildTime = t.Format(time.ANSIC)
}

if err2 := tmpl.Execute(cli.out, vd); err2 != nil && err == nil {
err = err2
}
Expand Down
2 changes: 2 additions & 0 deletions docs/reference/api/docker_remote_api.md
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,8 @@ This section lists each version from latest to oldest. Each listing includes a
* `GET /info` Now returns `Architecture` and `OSType` fields, providing information
about the host architecture and operating system type that the daemon runs on.
* `GET /networks/(name)` now returns a `Name` field for each container attached to the network.
* `GET /version` now returns the `BuildTime` field in RFC3339Nano format to make it
consistent with other date/time values returned by the API.

### v1.21 API changes

Expand Down
13 changes: 7 additions & 6 deletions docs/reference/api/docker_remote_api_v1.22.md
Original file line number Diff line number Diff line change
Expand Up @@ -1994,14 +1994,15 @@ Show the docker version information
Content-Type: application/json

{
"Version": "1.5.0",
"Version": "1.10.0-dev",
"Os": "linux",
"KernelVersion": "3.18.5-tinycore64",
"GoVersion": "go1.4.1",
"GitCommit": "a8a31ef",
"KernelVersion": "3.19.0-23-generic",
"GoVersion": "go1.4.2",
"GitCommit": "e75da4b",
"Arch": "amd64",
"ApiVersion": "1.20",
"Experimental": false
"ApiVersion": "1.22",
"BuildTime": "2015-12-01T07:09:13.444803460+00:00",
"Experimental": true
}

Status Codes:
Expand Down
2 changes: 1 addition & 1 deletion hack/make.sh
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ if command -v git &> /dev/null && git rev-parse &> /dev/null; then
if [ -n "$(git status --porcelain --untracked-files=no)" ]; then
GITCOMMIT="$GITCOMMIT-dirty"
fi
BUILDTIME=$(date -u)
BUILDTIME=$(date --rfc-3339 ns | sed -e 's/ /T/')
elif [ "$DOCKER_GITCOMMIT" ]; then
GITCOMMIT="$DOCKER_GITCOMMIT"
else
Expand Down

0 comments on commit 6deec02

Please sign in to comment.