Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-02 14:48:21 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Convert to using permissions to check for access to cirrus variables in benchmark script

Browse source
This commit is contained in:
Tim Wojtulewicz 2020-06-19 15:17:33 -07:00
parent 94ba6303f3
commit aa323a1841
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
ci/benchmark.sh
View file

@ -5,9 +5,9 @@ ZEEK_BENCHMARK_ENDPOINT="/zeek"
# Setting this causes any command failures to immediately cause the script to fail. # Setting this causes any command failures to immediately cause the script to fail.
set -e set -e
# Don't do this for any branch that isn't from the main zeek repo. # Don't do this unless the user has access to the encrypted variables. This will
# TODO: is it possible to do this from cirrus.yml instead of here? # basically exclude any PR that doesn't come from the main zeek repo.
if [ "${CIRRUS_REPO_OWNER}" != "zeek" ]; then if [ "${CIRRUS_USER_PERMISSION}" != "admin" -a "${CIRRUS_USER_PERMISSION}" != "write" ]; then
echo "Benchmarks are skipped for repositories outside of the main Zeek project" echo "Benchmarks are skipped for repositories outside of the main Zeek project"
exit 0 exit 0
fi fi