Skip to content

Add fuzzing to CI #444

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

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Add fuzzing to CI #444

merged 1 commit into from
Oct 9, 2023

Conversation

SuperQ
Copy link
Contributor

@SuperQ SuperQ commented Sep 5, 2023

Add a 60s fuzzing test to the CI test pipeline.

@TomSellers
Copy link
Contributor

Note that the following warning is expected on GOARCH 386 as there currently isn't any support for coverage on that platform.
warning: the test binary was not built with coverage instrumentation, so fuzzing will run without coverage guidance and may be inefficient

Add a 60s fuzzing test to the CI test pipeline.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ SuperQ requested review from TomSellers and TimRots and removed request for TomSellers October 8, 2023 20:10
@SuperQ
Copy link
Contributor Author

SuperQ commented Oct 8, 2023

Moved fuzz to a dedicated pipeline

Copy link
Member

@TimRots TimRots left a comment

Choose a reason for hiding this comment

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

Nice

@SuperQ SuperQ merged commit d306ed5 into master Oct 9, 2023
@SuperQ SuperQ deleted the superq/fuzz_ci branch October 9, 2023 06:42
codeboten referenced this pull request in open-telemetry/opentelemetry-collector-contrib Nov 24, 2023
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/gosnmp/gosnmp](https://togithub.com/gosnmp/gosnmp) |
require | minor | `v1.36.1` -> `v1.37.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>gosnmp/gosnmp (github.com/gosnmp/gosnmp)</summary>

### [`v1.37.0`](https://togithub.com/gosnmp/gosnmp/releases/tag/v1.37.0)

[Compare
Source](https://togithub.com/gosnmp/gosnmp/compare/v1.36.1...v1.37.0)

#### What's Changed

- Make InitPacket and InitSecurityKeys public by
[@&#8203;upsampled](https://togithub.com/upsampled) in
[https://github.com/gosnmp/gosnmp/pull/447](https://togithub.com/gosnmp/gosnmp/pull/447)
- Refactor TrapListener's Close Method by
[@&#8203;TimRots](https://togithub.com/TimRots) in
[https://github.com/gosnmp/gosnmp/pull/449](https://togithub.com/gosnmp/gosnmp/pull/449)
- Allow RequestID to be shrunk if possible by
[@&#8203;upsampled](https://togithub.com/upsampled) in
[https://github.com/gosnmp/gosnmp/pull/451](https://togithub.com/gosnmp/gosnmp/pull/451)
- Add net-snmp validation testing by
[@&#8203;upsampled](https://togithub.com/upsampled) in
[https://github.com/gosnmp/gosnmp/pull/452](https://togithub.com/gosnmp/gosnmp/pull/452)
- Add fuzzing to CI by [@&#8203;SuperQ](https://togithub.com/SuperQ) in
[https://github.com/gosnmp/gosnmp/pull/444](https://togithub.com/gosnmp/gosnmp/pull/444)
- Allow global password cache to be turned off by
[@&#8203;upsampled](https://togithub.com/upsampled) in
[https://github.com/gosnmp/gosnmp/pull/454](https://togithub.com/gosnmp/gosnmp/pull/454)
- build(deps): bump github.com/google/go-cmp from 0.5.9 to 0.6.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/gosnmp/gosnmp/pull/455](https://togithub.com/gosnmp/gosnmp/pull/455)

#### New Contributors

- [@&#8203;upsampled](https://togithub.com/upsampled) made their first
contribution in
[https://github.com/gosnmp/gosnmp/pull/447](https://togithub.com/gosnmp/gosnmp/pull/447)

**Full Changelog**:
gosnmp/gosnmp@v1.36.1...v1.37.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any
time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
vma pushed a commit to sipsolutions/gosnmp that referenced this pull request May 6, 2025
Add a 60s fuzzing test to the CI test pipeline.

Signed-off-by: SuperQ <superq@gmail.com>
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.

3 participants