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.

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

Mark GCE as passing conformance test. #10995

Merged
merged 1 commit into from
Jul 13, 2015

Conversation

erictune
Copy link
Member

@erictune erictune commented Jul 9, 2015

No description provided.

@k8s-bot
Copy link

k8s-bot commented Jul 9, 2015

GCE e2e build/test failed for commit 30fe02d.

@erictune
Copy link
Member Author

erictune commented Jul 9, 2015

jenkins was a build timeout.

-------------------- | ------------ | ------ | ---------- | ------------------------------------------------------------------------------ | ----------- | ---------------------------- | -----
GKE | | | GCE | [docs](https://cloud.google.com/container-engine) | | Commercial | Uses K8s version 0.15.0
GKE | | | GCE | [docs](https://cloud.google.com/container-engine) | [✓](1) | Commercial | Uses K8s version 0.15.0
Copy link
Member

Choose a reason for hiding this comment

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

Why indirect through the footnote? Why not just link directly?

Copy link
Member Author

Choose a reason for hiding this comment

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

  1. so that the column just reads as either a checkmark or not. More visual impact
  2. to limit column width so it fits on screen, both rendered, and in text mode.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, I guess only reason 2b is valid: so that the text version of the md has reasonable line length.

@lavalamp
Copy link
Member

LGTM

@lavalamp lavalamp added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 13, 2015
@erictune
Copy link
Member Author

@thockin docs only change: okay to merge?

thockin added a commit that referenced this pull request Jul 13, 2015
Mark GCE as passing conformance test.
@thockin thockin merged commit f5cc7cd into kubernetes:master Jul 13, 2015
@erictune erictune deleted the conform_check branch July 28, 2015 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants