internal/worker: remove DB experiment logic
Change-Id: Ic7771aa0de242141119489154bd29e2e1e9fd8c5
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/260097
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
Reviewed-by: Julie Qiu <julie@golang.org>
diff --git a/internal/worker/server.go b/internal/worker/server.go
index a57a387..7c5e484 100644
--- a/internal/worker/server.go
+++ b/internal/worker/server.go
@@ -188,9 +188,6 @@
// manual: clear-cache clears the redis cache.
handle("/clear-cache", rmw(s.errorHandler(s.clearCache)))
- // manual: update-experiment updates a given experiment.
- handle("/update-experiment", rmw(s.errorHandler(s.updateExperiment)))
-
// manual: delete the specified module version.
handle("/delete/", http.StripPrefix("/delete", rmw(s.errorHandler(s.handleDelete))))
@@ -494,22 +491,6 @@
return nil
}
-func (s *Server) updateExperiment(w http.ResponseWriter, r *http.Request) error {
- name := r.FormValue("name")
- description := r.FormValue("description")
- rollout, err := strconv.Atoi(r.FormValue("rollout"))
- if err != nil || rollout < 0 || rollout > 100 {
- return &serverError{http.StatusBadRequest, fmt.Errorf("rollout is invalid: %q", rollout)}
- }
-
- if err := s.db.UpdateExperiment(r.Context(), &internal.Experiment{Name: name, Description: description, Rollout: uint(rollout)}); err != nil {
- return &serverError{http.StatusInternalServerError, err}
- }
-
- fmt.Fprintf(w, "Updated %q experiment rollout to %d percent", name, rollout)
- return nil
-}
-
func (s *Server) handleHealthCheck(w http.ResponseWriter, r *http.Request) {
if err := s.db.Underlying().Ping(); err != nil {
http.Error(w, fmt.Sprintf("DB ping failed: %v", err), http.StatusInternalServerError)
