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

[WIP] Add Jazzer configuration to run in an Action #664

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

Conversation

sahansera
Copy link

@sahansera sahansera commented Dec 7, 2021

Description

Based on the discussion on #510 I thought it might be a good idea to receive some early feedback on some of the questions I had per comment here. In summary, what this does is:

  • Add a new workflow to trigger the Jazzer testing. Currently, I target the previously built version to keep this as a separate job.
  • Invoke jazzer.sh which is a Bash script I wrote to run Jazzer against a given set of classes and methods. Currently, the Class and method names are hardcoded under the jobs.build.runs-on.strategy.matrix.include for this proof-of-concept. This can be sourced differently (eg: YAML or a code scan)
  • In each Jazzer run, a Docker container gets spun up, runs autofuzzing, and dumps the file(s) to a mounted volume on the current Action container.
  • These collected files are published to the workflow pipeline. An example can be found here under the Artifacts section.

Happy to know any feedback on this.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #510

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@sahansera sahansera requested review from a team as code owners December 7, 2021 13:47
@ghost
Copy link

ghost commented Dec 7, 2021

CLA assistant check
All CLA requirements met.

@sahansera sahansera force-pushed the jazzer-integration-510 branch 2 times, most recently from bc4a648 to 84000ca Compare December 8, 2021 12:54
sahansera and others added 3 commits December 9, 2021 00:15
Bumps [actions/setup-java](https://github.com/actions/setup-java) from 2.3.1 to 2.4.0.
- [Release notes](https://github.com/actions/setup-java/releases)
- [Commits](actions/setup-java@v2.3.1...v2.4.0)

---
updated-dependencies:
- dependency-name: actions/setup-java
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Sahan Serasinghe <[email protected]>
Signed-off-by: Sahan Serasinghe <[email protected]>
- class: DaprClientBuilder
package: io.dapr.client
methods: 'withObjectSerializer withStateSerializer'
version: 1.3.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can the version be picked up automatically from the pom.xml file?

@@ -0,0 +1,76 @@
#!/bin/bash
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move to scripts folder, please.

@cicoyle
Copy link
Contributor

cicoyle commented Dec 21, 2023

gentle ping - @sahansera

@cicoyle
Copy link
Contributor

cicoyle commented Feb 19, 2024

gentle ping @sahansera

Copy link

codecov bot commented Feb 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (14cc3f8) 77.60% compared to head (9cb7dc7) 77.60%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #664   +/-   ##
=========================================
  Coverage     77.60%   77.60%           
  Complexity     1570     1570           
=========================================
  Files           144      144           
  Lines          4765     4765           
  Branches        554      554           
=========================================
  Hits           3698     3698           
  Misses          781      781           
  Partials        286      286           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cicoyle
Copy link
Contributor

cicoyle commented Jun 19, 2024

gentle ping @sahansera

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate with Google’s OSS Fuzz service
3 participants