Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move signature check and bias fuzzer to gha #9191

Closed
wants to merge 51 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
51 commits
Select commit Hold shift + click to select a range
8439cf6
remove torcharrow and update pyvelox cml
assignUser Mar 21, 2024
dbdd1f0
make build velox usable by pyvelox
assignUser Mar 21, 2024
530177c
get or create function signatures
assignUser Mar 21, 2024
ef77d63
hardcode ref instead of main to workaround broken pr
assignUser Mar 21, 2024
bcd14a3
fix working dir
assignUser Mar 21, 2024
465139a
Add deepdiff
assignUser Mar 21, 2024
88f45b8
fix stash dir on empty stash
assignUser Mar 21, 2024
ee3b675
correctly save signatures
assignUser Mar 21, 2024
01f816d
make python options turn features on only
assignUser Mar 22, 2024
63edae3
move bias fuzzer to scheduled.yml
assignUser Mar 22, 2024
a6bb970
use 8-core
assignUser Mar 22, 2024
aaa61da
add extra cmake flags to debug
assignUser Mar 22, 2024
76b971e
fix typo
assignUser Mar 22, 2024
ac257ba
use EXTRA_CMAKE_FLAGS as intended via envvar
assignUser Mar 22, 2024
6d65226
use 16core for more ram
assignUser Mar 22, 2024
5769a35
run in container
assignUser Mar 22, 2024
79d9313
fix workspace
assignUser Mar 22, 2024
72d1540
install python headers
assignUser Mar 23, 2024
2242e7f
pass correct python executable
assignUser Mar 23, 2024
d910a45
fix python command
assignUser Mar 23, 2024
ac33a23
move fuzzer runs to container as well
assignUser Mar 23, 2024
b3b97d9
add presto bias fuzzer run
assignUser Mar 23, 2024
a3728ef
rebase exchange fuzzer
assignUser Mar 23, 2024
10f1598
try without setting max_map_count
assignUser Mar 23, 2024
31c4ab7
properly set ccache base dir for inside container
assignUser Mar 23, 2024
cdd21bf
adapt prest based agg fuzzer to new workflow setup
assignUser Mar 26, 2024
45bc625
run presto-java agg on ubuntu-latest
assignUser Mar 26, 2024
1bc9dc8
add messaging when velox_build_dir is used
assignUser Mar 27, 2024
e893353
review feedback
assignUser Mar 27, 2024
0ef5d04
add new function to signature.py to export and compare signatures in CI
assignUser Mar 27, 2024
34b8751
fix stash retention days
assignUser Mar 27, 2024
f1dd3a3
don't use argsparse.FileType to better manage file open/close
assignUser Mar 27, 2024
5f3a2af
Removing a function signature for test purposes.
kgpai Mar 28, 2024
d0070d6
revert makefile changes
assignUser Mar 29, 2024
15f859e
fix output names
assignUser Mar 29, 2024
44dca5c
remove ccache_basedir
assignUser Mar 29, 2024
6c2c3e8
trigger ci
assignUser Mar 29, 2024
515e7ee
move errors from step summary to log
assignUser Mar 29, 2024
4e3a08c
fix error file permission
assignUser Mar 29, 2024
081ac53
debug ccache
assignUser Mar 29, 2024
bbec6e5
reduce amount of fuzzer logs produced
assignUser Mar 29, 2024
d16ee3c
typo
assignUser Mar 29, 2024
5bb971c
Revert "Removing a function signature for test purposes."
assignUser Mar 29, 2024
8465268
remove redundant error call out
assignUser Mar 29, 2024
3261b17
remove ccache debug
assignUser Mar 29, 2024
c27b506
remove remaining CCI files
assignUser Mar 29, 2024
ee4db14
improve set_output
assignUser Mar 29, 2024
af26c25
remove python dev install
assignUser Mar 29, 2024
25e734f
Only run bias or unbiased fuzzer not both
assignUser Mar 29, 2024
33a31d9
format
assignUser Mar 29, 2024
a0613c8
Add a noop Circleci build so we can remove the error.
kgpai Apr 1, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
89 changes: 0 additions & 89 deletions .circleci/README.md

This file was deleted.

25 changes: 7 additions & 18 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,28 +17,17 @@ version: 2.1
# This allows us to use CircleCI's dynamic configuration feature
setup: true


# Path-filtering orb is required to continue a pipeline based on
# the path of an updated fileset
orbs:
path-filtering: circleci/[email protected]
jobs:
noop-build:
docker:
- image: cimg/base:2024.02
steps:
- run: circleci-agent step halt

workflows:
version: 2

path-filtering-workflow:
jobs:
- noop-build

- path-filtering/filter:
name: check-sensitive-paths

# Format is: <regex path-to-test> <parameter-to-set> <value-of-pipeline-parameter>
# Regex below will filter out paths with test in them.
mapping: |
velox/expression/((?!.*test).*).* run-longer-expression-fuzzer true
velox/exec/((?!.*test).*).* run-longer-expression-fuzzer true
velox/common/((?!.*test).*).* run-longer-expression-fuzzer true
velox/core/((?!.*test).*).* run-longer-expression-fuzzer true
velox/vector/((?!.*test).*).* run-longer-expression-fuzzer true

config-path: .circleci/dist_compile.yml
Loading
Loading