Change in bazel[master]: Bump the size of //src/test/shell/bazel:execroot_test

0 views
Skip to first unread message

Damien Martin-guillerez (Gerrit)

unread,
Jul 25, 2016, 8:39:45 AM7/25/16
to bazel-de...@googlegroups.com
Damien Martin-guillerez has uploaded a new change for review.

https://bazel-review.googlesource.com/4156

Change subject: Bump the size of //src/test/shell/bazel:execroot_test
......................................................................

Bump the size of //src/test/shell/bazel:execroot_test

It was timeout flaky on ci.bazel.io

Fixes #1559.

Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
---
M src/test/shell/bazel/BUILD
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/src/test/shell/bazel/BUILD b/src/test/shell/bazel/BUILD
index 5ed4d00..d0aec83 100644
--- a/src/test/shell/bazel/BUILD
+++ b/src/test/shell/bazel/BUILD
@@ -357,7 +357,7 @@

sh_test(
name = "execroot_test",
- size = "small",
+ size = "medium",
srcs = ["execroot_test.sh"],
data = [":test-deps"],
)

--
To view, visit https://bazel-review.googlesource.com/4156
To unsubscribe, visit https://bazel-review.googlesource.com/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
Gerrit-PatchSet: 1
Gerrit-Project: bazel
Gerrit-Branch: master
Gerrit-Owner: Damien Martin-guillerez <dmar...@google.com>

Yun Peng (Gerrit)

unread,
Jul 25, 2016, 8:40:49 AM7/25/16
to Damien Martin-guillerez
Yun Peng has posted comments on this change.

Change subject: Bump the size of //src/test/shell/bazel:execroot_test
......................................................................


Patch Set 1: Code-Review+2
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
Gerrit-PatchSet: 1
Gerrit-Project: bazel
Gerrit-Branch: master
Gerrit-Owner: Damien Martin-guillerez <dmar...@google.com>
Gerrit-Reviewer: Yun Peng <pcl...@google.com>
Gerrit-HasComments: No

Damien Martin-guillerez (Gerrit)

unread,
Jul 25, 2016, 10:28:23 AM7/25/16
to Yun Peng
Hello Yun Peng,

I'd like you to reexamine a change. Please visit

https://bazel-review.googlesource.com/4156

to look at the new patch set (#2).

Change subject: Bump the size of //src/test/shell/bazel:execroot_test
......................................................................

Bump the size of //src/test/shell/bazel:execroot_test

It was timeout flaky on ci.bazel.io

Fixes #1559.

--
Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
Reviewed-on: https://bazel-review.googlesource.com/#/c/4156
MOS_MIGRATED_REVID=128347912
---
M src/test/shell/bazel/BUILD
1 file changed, 1 insertion(+), 1 deletion(-)


Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
Gerrit-PatchSet: 2
Gerrit-Project: bazel
Gerrit-Branch: master
Gerrit-Owner: Damien Martin-guillerez <dmar...@google.com>
Gerrit-Reviewer: Yun Peng <pcl...@google.com>

Damien Martin-guillerez (Gerrit)

unread,
Jul 25, 2016, 10:28:26 AM7/25/16
to Yun Peng
Damien Martin-guillerez has submitted this change and it was merged.

Change subject: Bump the size of //src/test/shell/bazel:execroot_test
......................................................................


Bump the size of //src/test/shell/bazel:execroot_test

It was timeout flaky on ci.bazel.io

Fixes #1559.

--
Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
Reviewed-on: https://bazel-review.googlesource.com/#/c/4156
MOS_MIGRATED_REVID=128347912
---
M src/test/shell/bazel/BUILD
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/src/test/shell/bazel/BUILD b/src/test/shell/bazel/BUILD
index 5ed4d00..d0aec83 100644
--- a/src/test/shell/bazel/BUILD
+++ b/src/test/shell/bazel/BUILD
@@ -357,7 +357,7 @@

sh_test(
name = "execroot_test",
- size = "small",
+ size = "medium",
srcs = ["execroot_test.sh"],
data = [":test-deps"],
)

--
To view, visit https://bazel-review.googlesource.com/4156
To unsubscribe, visit https://bazel-review.googlesource.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie116fd70cf167ac152782f766b9ae541cc041ab4
Gerrit-PatchSet: 2
Gerrit-Project: bazel
Gerrit-Branch: master
Gerrit-Owner: Damien Martin-guillerez <dmar...@google.com>
Gerrit-Reviewer: Damien Martin-guillerez <dmar...@google.com>
Gerrit-Reviewer: Yun Peng <pcl...@google.com>
Reply all
Reply to author
Forward
0 new messages