[autoroller] Do not unconditionally bypass upload validator.
The original commit was a temporary workaround; by not passing
uploadvalidator~skip we would have Gerrit uploadvalidator plugin to get
a chance to validate the change and makes it easier to test our changes.
Change-Id: I0d80c93c2bf3823dea8f89ca6be7ec2e3f54b70e
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/567725
Reviewed-by: Ravi Mistry <rmistry@google.com>
Auto-Submit: Xin Li <delphij@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
diff --git a/autoroll/go/repo_manager/android_repo_manager.go b/autoroll/go/repo_manager/android_repo_manager.go
index c09c844..f2d57fb 100644
--- a/autoroll/go/repo_manager/android_repo_manager.go
+++ b/autoroll/go/repo_manager/android_repo_manager.go
@@ -505,7 +505,7 @@
}
// Upload the CL to Gerrit.
- uploadArgs := []string{"upload", "--no-verify", "--push-option=uploadvalidator~skip"}
+ uploadArgs := []string{"upload", "--no-verify"}
if rollEmails != nil && len(rollEmails) > 0 {
uploadArgs = append(uploadArgs, fmt.Sprintf("--re=%s", strings.Join(rollEmails, ",")))
}
