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

add action to run against asan #212

Merged
merged 29 commits into from
Oct 25, 2023
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
2e2012b
initial work on getting asan support
ezra-varady Oct 18, 2023
fd85dcd
touch ups
ezra-varady Oct 18, 2023
7a43d86
docker stuff working
ezra-varady Oct 19, 2023
dabbcb7
fixup test so it doesnt fail under asan (failure was due to spurious …
ezra-varady Oct 22, 2023
4ca6f3d
remove branch from action
ezra-varady Oct 22, 2023
0e22078
update gitignore trap docker kill
ezra-varady Oct 22, 2023
ab21350
cache the value instead of a pointer to it, this fixes a potential us…
ezra-varady Oct 22, 2023
c214f07
clang-format
ezra-varady Oct 23, 2023
c871059
fix paths for github
ezra-varady Oct 23, 2023
88d343f
postgres creates its own user
ezra-varady Oct 23, 2023
d23b597
forgot to drop requirement for removed action
ezra-varady Oct 23, 2023
31270c8
chown requires sudo
ezra-varady Oct 23, 2023
9e116ad
accidentally deleted one too many lines
ezra-varady Oct 23, 2023
87733b0
switching to PR, should help iron out the alst few bugs at least
ezra-varady Oct 23, 2023
dbd33e0
mark more things sudo
ezra-varady Oct 23, 2023
1d049b6
remove files commited by accident
ezra-varady Oct 23, 2023
17d99a2
chmod not chown
ezra-varady Oct 23, 2023
ca82060
if I'm not root can I just run postgres?
ezra-varady Oct 23, 2023
245c1af
missed quotes
ezra-varady Oct 23, 2023
15065de
unclear whats failing, possibly permissions issues in tmp?
ezra-varady Oct 23, 2023
57ea329
coredump not coredumps
ezra-varady Oct 23, 2023
9a777f5
add step to cache build if it worked so that its not contingent on te…
ezra-varady Oct 23, 2023
ed0161c
add additional versions, run weekly
ezra-varady Oct 23, 2023
d4956b3
flush cache on retriever area reset
ezra-varady Oct 23, 2023
e64ccab
add support for ubsan in docker
ezra-varady Oct 24, 2023
9c7ff77
add some notes about using the sanitizer container, update workflow t…
ezra-varady Oct 25, 2023
aad535a
log postgres output separately
ezra-varady Oct 25, 2023
e7eb240
pull down the correct ref
ezra-varady Oct 25, 2023
6f80253
rebase onto main, lower ef_search minimum value
ezra-varady Oct 25, 2023
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
Prev Previous commit
Next Next commit
log postgres output separately
ezra-varady committed Oct 25, 2023
commit aad535a62a3747f73aa021687e18328d77f86f7b
10 changes: 5 additions & 5 deletions .github/workflows/sanitizer-build-and-test.yaml
Original file line number Diff line number Diff line change
@@ -204,14 +204,14 @@ jobs:
name: Regression diff ${{ matrix.os }} ${{ env.name }} ${{ matrix.pg }}
path: |
regression.log
postgres.log
- name: Save PostgreSQL log
if: always()
- name: Save postgres log
if: always() && steps.collectlogs.outputs.regression_diff == 'true'
uses: actions/upload-artifact@v3
with:
name: PostgreSQL log ${{ matrix.os }} ${{ env.name }} ${{ matrix.pg }}
path: /var/log/postgresql/*.log
name: Regression diff ${{ matrix.os }} ${{ env.name }} ${{ matrix.pg }}
path: |
postgres.log
- name: Stack trace
if: always() && steps.collectlogs.outputs.coredumps == 'true'