aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJordan Bayles <jophba@chromium.org>2021-08-01 10:42:29 -0700
committerOpenscreen LUCI CQ <openscreen-scoped@luci-project-accounts.iam.gserviceaccount.com>2021-08-02 19:44:43 +0000
commitb9be95eef079e1c04520a05a3bd057bec59b77a3 (patch)
tree8c37cb1440c8376975f498a703c9ed7bec8575d9
parentc6fdd48ace63566701b8d8315741d402162fe425 (diff)
downloadopenscreen-b9be95eef079e1c04520a05a3bd057bec59b77a3.tar.gz
Add informational cast standalone bot
This PR adds an experimental (i.e. non-blocking) bot called linux64_cast_debug, that builds the standalone cast binaries in order to run the standalone_e2e.py test script in a later patch. This does NOT add the dependency libs for cast standalone, which will need future work with the infrastructure team. Finally, this patch adds some improved logging to the cast standalone script. Change-Id: I02448420c44a0e9184f260fa2ce02710c90693b5 Reviewed-on: https://chromium-review.googlesource.com/c/openscreen/+/3063815 Commit-Queue: Jordan Bayles <jophba@chromium.org> Reviewed-by: Ryan Keane <rwkeane@google.com>
-rwxr-xr-xcast/standalone_e2e.py18
-rw-r--r--docs/continuous_build.md1
-rw-r--r--infra/config/global/commit-queue.cfg4
-rw-r--r--infra/config/global/cr-buildbucket.cfg29
-rw-r--r--infra/config/global/luci-milo.cfg12
-rw-r--r--infra/config/global/luci-scheduler.cfg11
6 files changed, 68 insertions, 7 deletions
diff --git a/cast/standalone_e2e.py b/cast/standalone_e2e.py
index b27e3c65..c64ab5e7 100755
--- a/cast/standalone_e2e.py
+++ b/cast/standalone_e2e.py
@@ -177,13 +177,17 @@ class StandaloneCastTest(unittest.TestCase):
return
logging.debug('Credentials out of date, generating new ones...')
- subprocess.check_output(
- [
- cls.build_paths.cast_receiver,
- '-g', # Generate certificate and private key.
- '-v' # Enable verbose logging.
- ],
- stderr=subprocess.STDOUT)
+ try:
+ subprocess.check_output(
+ [
+ cls.build_paths.cast_receiver,
+ '-g', # Generate certificate and private key.
+ '-v' # Enable verbose logging.
+ ],
+ stderr=subprocess.STDOUT)
+ except subprocess.CalledProcessError as e:
+ print('Generation failed with output: ', e.output.decode())
+ raise
def launch_receiver(self):
"""Launches the receiver process with discovery disabled."""
diff --git a/docs/continuous_build.md b/docs/continuous_build.md
index 1c38a130..53c68e21 100644
--- a/docs/continuous_build.md
+++ b/docs/continuous_build.md
@@ -12,6 +12,7 @@ Current builders include:
| linux64_gcc_debug | x86-64 | Ubuntu Linux 18.04 | gcc-7 | debug | |
| linux64_tsan | x86-64 | Ubuntu Linux 18.04 | clang | release | TSAN enabled |
| linux64_coverage_debug | x86-64 | Ubuntu Linux 18.04 | clang | debug | used for code coverage |
+| linux64_cast_e2e | x86-64 | Ubuntu Linux 18.04 | clang | debug | Builds cast standalone |
| mac_debug | x86-64 | Mac OS X/Xcode | clang | debug | |
| chromium_linux64_debug | x86-64 | Ubuntu Linux 18.04 | clang | debug | built with chromium |
| chromium_mac_debug | x86-64 | Mac OS X 10.15 | clang | debug | built with chromium |
diff --git a/infra/config/global/commit-queue.cfg b/infra/config/global/commit-queue.cfg
index 22a5b113..4f19b682 100644
--- a/infra/config/global/commit-queue.cfg
+++ b/infra/config/global/commit-queue.cfg
@@ -53,6 +53,10 @@ config_groups {
name: "openscreen/try/linux64_coverage_debug"
experiment_percentage: 100
}
+ builders {
+ name: "openscreen/try/linux64_cast_e2e"
+ experiment_percentage: 100
+ }
retry_config {
single_quota: 1
global_quota: 2
diff --git a/infra/config/global/cr-buildbucket.cfg b/infra/config/global/cr-buildbucket.cfg
index 02bd50f7..5ff8b9ad 100644
--- a/infra/config/global/cr-buildbucket.cfg
+++ b/infra/config/global/cr-buildbucket.cfg
@@ -77,6 +77,17 @@ builder_mixins {
}
builder_mixins {
+ name: "cast_standalone"
+ recipe {
+ properties_j: "have_ffmpeg:true"
+ properties_j: "have_libsdl2:true"
+ properties_j: "have_libopus:true"
+ properties_j: "have_libvpx:true"
+ properties_j: "cast_allow_developer_certificate:true"
+ }
+}
+
+builder_mixins {
name: "linux"
dimensions: "os:Ubuntu-18.04"
}
@@ -275,6 +286,15 @@ buckets {
mixins: "ci"
mixins: "goma_rbe_ats"
}
+
+ builders {
+ name: "linux64_cast_e2e"
+ mixins: "linux"
+ mixins: "debug"
+ mixins: "x64"
+ mixins: "cast_standalone"
+ mixins: "goma_rbe_ats"
+ }
}
}
@@ -386,5 +406,14 @@ buckets: {
mixins: "code_coverage"
mixins: "goma_rbe_ats"
}
+
+ builders {
+ name: "linux64_cast_e2e"
+ mixins: "linux"
+ mixins: "debug"
+ mixins: "x64"
+ mixins: "cast_standalone"
+ mixins: "goma_rbe_ats"
+ }
}
}
diff --git a/infra/config/global/luci-milo.cfg b/infra/config/global/luci-milo.cfg
index 70df9ce0..c0445bb6 100644
--- a/infra/config/global/luci-milo.cfg
+++ b/infra/config/global/luci-milo.cfg
@@ -54,6 +54,12 @@ consoles {
category: "linux|x64"
short_name: "coverage"
}
+
+ builders {
+ name: "buildbucket/luci.openscreen.ci/linux64_cast_e2e"
+ category: "linux|x64"
+ short_name: "cast"
+ }
}
consoles {
@@ -110,4 +116,10 @@ consoles {
category: "linux|x64"
short_name: "coverage"
}
+
+ builders {
+ name: "buildbucket/luci.openscreen.ci/linux64_cast_e2e"
+ category: "linux|x64"
+ short_name: "cast"
+ }
}
diff --git a/infra/config/global/luci-scheduler.cfg b/infra/config/global/luci-scheduler.cfg
index 13612af8..0ec867b9 100644
--- a/infra/config/global/luci-scheduler.cfg
+++ b/infra/config/global/luci-scheduler.cfg
@@ -28,6 +28,7 @@ trigger {
triggers: "linux_arm64_debug"
triggers: "mac_debug"
triggers: "linux64_coverage_debug"
+ triggers: "linux64_cast_e2e"
}
trigger {
@@ -121,3 +122,13 @@ job {
builder: "linux64_coverage_debug"
}
}
+
+job {
+ id: "linux64_cast_e2e"
+ acl_sets: "default"
+ buildbucket: {
+ server: "cr-buildbucket.appspot.com"
+ bucket: "luci.openscreen.ci"
+ builder: "linux64_cast_e2e"
+ }
+}