Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(278)

Unified Diff: milo/appengine/buildbot/html.go

Issue 2944983003: [milo] {buildbucket,buildbot,swarming,logdog} -> backends/*. (Closed)
Patch Set: fix the tests Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « milo/appengine/buildbot/grpc_test.go ('k') | milo/appengine/buildbot/html_data.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: milo/appengine/buildbot/html.go
diff --git a/milo/appengine/buildbot/html.go b/milo/appengine/buildbot/html.go
deleted file mode 100644
index c5f63f7b06f82793d3206a787887d5984cca285e..0000000000000000000000000000000000000000
--- a/milo/appengine/buildbot/html.go
+++ /dev/null
@@ -1,100 +0,0 @@
-// Copyright 2015 The LUCI Authors. All rights reserved.
-// Use of this source code is governed under the Apache License, Version 2.0
-// that can be found in the LICENSE file.
-
-package buildbot
-
-import (
- "fmt"
- "net/http"
- "strconv"
-
- "github.com/luci/luci-go/milo/appengine/common"
- "github.com/luci/luci-go/server/router"
- "github.com/luci/luci-go/server/templates"
-)
-
-// BuildHandler Renders the buildbot build page.
-func BuildHandler(c *router.Context) {
- master := c.Params.ByName("master")
- if master == "" {
- common.ErrorPage(c, http.StatusBadRequest, "No master specified")
- return
- }
- builder := c.Params.ByName("builder")
- if builder == "" {
- common.ErrorPage(c, http.StatusBadRequest, "No builder specified")
- return
- }
- buildNum := c.Params.ByName("build")
- if buildNum == "" {
- common.ErrorPage(c, http.StatusBadRequest, "No build number")
- return
- }
- num, err := strconv.Atoi(buildNum)
- if err != nil {
- common.ErrorPage(c, http.StatusBadRequest,
- fmt.Sprintf("%s does not look like a number", buildNum))
- return
- }
-
- result, err := Build(c.Context, master, builder, num)
- if err != nil {
- var code int
- switch err {
- case errBuildNotFound:
- code = http.StatusNotFound
- case errNotAuth:
- code = http.StatusUnauthorized
- default:
- code = http.StatusInternalServerError
- }
- common.ErrorPage(c, code, err.Error())
- return
- }
-
- templates.MustRender(c.Context, c.Writer, "pages/build.html", templates.Args{
- "Build": result,
- })
-}
-
-// BuilderHandler renders the buildbot builder page.
-// Note: The builder html template contains self links to "?limit=123", which could
-// potentially override any other request parameters set.
-func BuilderHandler(c *router.Context) {
- master := c.Params.ByName("master")
- if master == "" {
- common.ErrorPage(c, http.StatusBadRequest, "No master specified")
- return
- }
- builder := c.Params.ByName("builder")
- if builder == "" {
- common.ErrorPage(c, http.StatusBadRequest, "No builder specified")
- return
- }
- limit, err := common.GetLimit(c.Request)
- if err != nil {
- common.ErrorPage(c, http.StatusBadRequest, err.Error())
- return
- }
- if limit < 0 {
- limit = 25
- }
- cursor := c.Request.FormValue("cursor")
-
- result, err := builderImpl(c.Context, master, builder, limit, cursor)
- _, notFound := err.(errBuilderNotFound)
- switch {
- case err == nil:
- templates.MustRender(c.Context, c.Writer, "pages/builder.html", templates.Args{
- "Builder": result,
- })
- case err == errNotAuth:
- common.ErrorPage(c, http.StatusUnauthorized, err.Error())
- case notFound:
- common.ErrorPage(c, http.StatusNotFound, err.Error())
- default: // err != nil
- common.ErrorPage(c, http.StatusInternalServerError, err.Error())
- }
- return
-}
« no previous file with comments | « milo/appengine/buildbot/grpc_test.go ('k') | milo/appengine/buildbot/html_data.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698