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.

Sign up for GitHub

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

Jump to bottom

covermerger: syz-bq tool and integration with datastore and bigquery #4804

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

covermerger: syz-bq tool and integration with datastore and bigquery #4804

wants to merge 4 commits into from

Conversation

tarasmadan
Copy link
Collaborator

No description provided.

@codecov Codecov
Copy link

codecov bot commented May 14, 2024 •
edited

Codecov Report

Attention: Patch coverage is 28.17680% with 130 lines in your changes missing coverage. Please review.

Project coverage is 61.0%. Comparing base (121701b) to head (35913e0).

Additional details and impacted files
Files Coverage Δ
pkg/vcs/vcs.go 50.5% <ø> (ø)
pkg/covermerger/lines_matcher.go 89.7% <0.0%> (-10.3%) ⬇️
pkg/vcs/fuchsia.go 0.0% <0.0%> (ø)
pkg/covermerger/file_line_merger.go 63.8% <26.1%> (-36.2%) ⬇️
pkg/vcs/git.go 59.1% <0.0%> (-2.0%) ⬇️
pkg/covermerger/covermerger.go 66.7% <60.8%> (-0.4%) ⬇️
pkg/covermerger/repos.go 0.0% <0.0%> (-65.2%) ⬇️

... and 1 file with indirect coverage changes

@tarasmadan tarasmadan requested review from corkamig and removed request for corkamig May 17, 2024 12:03
@tarasmadan tarasmadan force-pushed the merger-add-functionality branch 6 times, most recently from 1e4c8df to da384f6 Compare June 5, 2024 18:54
@tarasmadan tarasmadan requested review from dvyukov and mvanotti and removed request for mvanotti June 6, 2024 15:43
@tarasmadan tarasmadan marked this pull request as ready for review June 6, 2024 15:44
@tarasmadan
Copy link
Collaborator Author

tarasmadan commented Jun 6, 2024 •
edited

Let me split it and add some desctiption.

@tarasmadan
vendor: add cloud.google.com/go/datastore
03c472f
@tarasmadan
covermerger: add more control options and datastore integration
9c371bd
@tarasmadan
tools/syz-bq.sh: initial code
0fa77da 
This tool:
1. snapshot target coverage data to gcs gzip files.
2. download it to the local disk
3. define target commit
4. call aggregation tool asking it to upload result to the datastore
@tarasmadan
pkg/covermerger: mock fs operations in tests
35913e0
@tarasmadan tarasmadan changed the title [prototype] merger, add functionality covermerger: tool and integration with datastore and bigquery Jun 6, 2024
@tarasmadan tarasmadan changed the title covermerger: tool and integration with datastore and bigquery covermerger: syz-bq tool and integration with datastore and bigquery Jun 6, 2024
@tarasmadan tarasmadan mentioned this pull request Jun 7, 2024
Merged
@tarasmadan tarasmadan requested a review from a-nogikh June 10, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dvyukov dvyukov Awaiting requested review from dvyukov

@a-nogikh a-nogikh Awaiting requested review from a-nogikh

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@tarasmadan