-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Fixing broken http://kubernetes.io/kubernetes/third_party/swagger-ui/ #1921
Fixing broken http://kubernetes.io/kubernetes/third_party/swagger-ui/ #1921
Conversation
709940c
to
0fe3d1f
Compare
Shoot, I didn't know this was used directly. Should we revert that instead? |
--- | ||
|
||
Kubernetes swagger UI has now been replaced by our generated API reference docs | ||
which can be accessed at http://kubernetes.io/docs/api-reference/README/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this page only have 4 APIs, when we have dozens?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sent #1928 to add missing ones
Emergency merge to stop the badness |
Did this actually fix it?
…On Sun, Dec 11, 2016 at 4:32 PM, Tim Hockin ***@***.***> wrote:
Merged #1921
<#1921>.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1921 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFVgVLNRuwQeO1TYEPIgsLrp4hsPTNIhks5rHJYWgaJpZM4LJ8DI>
.
|
This PR is for 1.5 branch. I can send the same for master branch but I think the plan is to start serving k8s.io from release-1.5 branch this morning. Problem should be fixed when that happens. |
http://kubernetes.io/kubernetes/third_party/swagger-ui/ broke when we removed the gh-pages branch (kubernetes/kubernetes#38309 (comment)).
Adding an index.md file at http://kubernetes.io/kubernetes/third_party/swagger-ui/ to inform users that they should now be using the API reference docs.
http://kubernetes.io/kubernetes/third_party/swagger-ui/ is still one of the top results when searching for "kubernetes swagger" so we cannot abandon it.
cc @thockin who removed the gh-pages branch.
Also cc @devin-donnelly @jaredbhatti
This change is