Skip to content

🌱Add scheduled github worklow job for Fuzzing#3122

Merged
metal3-io-bot merged 1 commit intometal3-io:mainfrom
Nordix:add-fuzz-workflow
Apr 10, 2026
Merged

🌱Add scheduled github worklow job for Fuzzing#3122
metal3-io-bot merged 1 commit intometal3-io:mainfrom
Nordix:add-fuzz-workflow

Conversation

@smoshiur1237
Copy link
Copy Markdown
Member

@smoshiur1237 smoshiur1237 commented Mar 26, 2026

Adding scheduled workflow for Fuzz test

@smoshiur1237
Copy link
Copy Markdown
Member Author

smoshiur1237 commented Mar 26, 2026

/hold

@metal3-io-bot metal3-io-bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 26, 2026
@smoshiur1237
Copy link
Copy Markdown
Member Author

smoshiur1237 commented Mar 26, 2026

/retest

@peppi-lotta
Copy link
Copy Markdown
Member

peppi-lotta commented Apr 8, 2026

/lgtm

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 8, 2026
@metal3-io-bot metal3-io-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 8, 2026
smoshiur1237
smoshiur1237 commented Apr 8, 2026
View reviewed changes
Copy link
Copy Markdown
Member Author

@smoshiur1237 smoshiur1237 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

/cc @tuminoid @peppi-lotta

@metal3-io-bot metal3-io-bot requested a review from tuminoid April 8, 2026 11:08
@metal3-io-bot
Copy link
Copy Markdown
Contributor

metal3-io-bot commented Apr 8, 2026

@smoshiur1237: GitHub didn't allow me to request PR reviews from the following users: adam.

Note that only metal3-io members and repo collaborators can review this PR, and authors cannot review their own PRs.

Details

In response to this:

/cc @tuminoid @adam @peppi-lotta

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@smoshiur1237
Copy link
Copy Markdown
Member Author

smoshiur1237 commented Apr 8, 2026

/retest

@smoshiur1237
Copy link
Copy Markdown
Member Author

smoshiur1237 commented Apr 8, 2026

/unhold

@metal3-io-bot metal3-io-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 8, 2026
tuminoid
tuminoid reviewed Apr 8, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@tuminoid tuminoid left a comment

Choose a reason for hiding this comment

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

Also in Makefile, we need to change L139 to:

go test  -fuzz=$$fuzz_test -fuzztime='$(FUZZ_TIME)' || exit 1; \

to avoid injection when called via workflow.

Comment thread .github/workflows/scheduled-fuzz.yml Outdated
@smoshiur1237
Add scheduled github worklow job for Fuzzing
484990f 
Signed-off-by: smoshiur1237 <moshiur.rahman@est.tech>
@smoshiur1237 smoshiur1237 requested a review from tuminoid April 9, 2026 08:34
@smoshiur1237
Copy link
Copy Markdown
Member Author

smoshiur1237 commented Apr 9, 2026

/retest

1 similar comment
@smoshiur1237
Copy link
Copy Markdown
Member Author

smoshiur1237 commented Apr 9, 2026

/retest

tuminoid
tuminoid approved these changes Apr 9, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@tuminoid tuminoid left a comment

Choose a reason for hiding this comment

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

Tested locally, it works. No user configurable things that can inject stuff.

/lgtm

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 9, 2026
@tuminoid
Copy link
Copy Markdown
Member

tuminoid commented Apr 9, 2026

/hold

Needs project-infra PR to go first.

@metal3-io-bot metal3-io-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 9, 2026
smoshiur1237
smoshiur1237 commented Apr 10, 2026
View reviewed changes
Copy link
Copy Markdown
Member Author

@smoshiur1237 smoshiur1237 left a comment

Choose a reason for hiding this comment

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

/unhold

@metal3-io-bot metal3-io-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 10, 2026
Rozzii
Rozzii reviewed Apr 10, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@Rozzii Rozzii left a comment

Choose a reason for hiding this comment

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

/approve

@metal3-io-bot
Copy link
Copy Markdown
Contributor

metal3-io-bot commented Apr 10, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Rozzii, tuminoid

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 10, 2026
@metal3-io-bot metal3-io-bot merged commit 88dd929 into metal3-io:main Apr 10, 2026
40 of 43 checks passed
@metal3-io-bot metal3-io-bot deleted the add-fuzz-workflow branch April 10, 2026 06:31
@metal3-io-bot metal3-io-bot added this to the BMO - v0.13 milestone Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rozzii Rozzii Rozzii left review comments

@tuminoid tuminoid tuminoid approved these changes

@adilGhaffarDev adilGhaffarDev Awaiting requested review from adilGhaffarDev

@iurygregory iurygregory Awaiting requested review from iurygregory

@peppi-lotta peppi-lotta Awaiting requested review from peppi-lotta

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

BMO - v0.13

Development

Successfully merging this pull request may close these issues.

5 participants

@smoshiur1237 @peppi-lotta @metal3-io-bot @tuminoid @Rozzii