Skip to content

Enable gcplogs driver on windows#37717

Merged
AkihiroSuda merged 1 commit intomoby:masterfrom
taylanisikdemir:gcplogs-windows
Aug 28, 2018
Merged

Enable gcplogs driver on windows#37717
AkihiroSuda merged 1 commit intomoby:masterfrom
taylanisikdemir:gcplogs-windows

Conversation

@taylanisikdemir
Copy link
Copy Markdown

Signed-off-by: Taylan Isikdemir taylani@google.com

- What I did
Enabled gcplogs (stackdriver) log driver on windows.

- How to verify it

  1. Install dockerd on a Windows VM which includes the change in this PR
  2. Run a container with gcplogs driver. See gcplogs documentation for configuration details.
    e.g. docker run --log-driver=gcplogs microsoft/windowsservercore cmd /C echo "hello"
  3. Verify that container logs show up in your target stackdriver

Signed-off-by: Taylan Isikdemir <taylani@google.com>
Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM as long as CI is happy.

@taylanisikdemir
Copy link
Copy Markdown
Author

Docker-PRs build failed because of context timed out during DockerSwarmSuite.TestAPISwarmLeaderElection test.
I see the same test passing in other jenkins builds of this PR.
If this test is known to be flaky can someone re-trigger the build?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants