Skip to content

[SKIP SOF-TEST] scripts/fuzz.sh: add a -p "pristine" flag that deletes build-fuzz/ - #8831

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
marc-hb:fuzz-pristine
Feb 7, 2024
Merged

[SKIP SOF-TEST] scripts/fuzz.sh: add a -p "pristine" flag that deletes build-fuzz/#8831
lgirdwood merged 1 commit into
thesofproject:mainfrom
marc-hb:fuzz-pristine

Conversation

@marc-hb

@marc-hb marc-hb commented Feb 2, 2024

Copy link
Copy Markdown
Collaborator

It's slower but safer when changing some parameters.

@marc-hb
scripts/fuzz.sh: add a -p "pristine" flag that deletes build-fuzz/
1849d52 
It's slower but safer when changing some parameters.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb changed the title scripts/fuzz.sh: add a -p "pristine" flag that deletes build-fuzz/ [SKIP SOF-TEST] scripts/fuzz.sh: add a -p "pristine" flag that deletes build-fuzz/ Feb 2, 2024
@marc-hb
marc-hb marked this pull request as ready for review February 2, 2024 22:07
@marc-hb
marc-hb requested a review from andyross February 2, 2024 22:07
@marc-hb

marc-hb commented Feb 2, 2024 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

@lrudyX , @wszypelt the failure in https://sof-ci.01.org/sof-pr-viewer/#/build/PR8831/build13511184 is new, never seen that one before. Some system is down?

14:34:15,148 INFO  - D:\qba2\workspace\44988>robocopy pull-8831-merge_1956549\INSTALL\ \\10.x.x.x\Builds\pull-8831-merge_1956549\ *.ri *.ldc /E
14:34:15,162 INFO  -
14:34:15,162 INFO  - -------------------------------------------------------------------------------
14:34:15,163 INFO  - ROBOCOPY :: Robust File Copy for Windows
14:34:15,163 INFO  - -------------------------------------------------------------------------------
14:34:15,163 INFO  -
14:34:37,178 INFO  - Started : Friday, February 2, 2024 11:34:15 PM
14:34:37,179 INFO  - 2024/02/02 23:34:37 ERROR 53 (0x00000035) Getting File System Type of Destination \\10.x.x.x\Builds\pull-8831-merge_1956549\
14:34:37,179 INFO  - The network path was not found.

In any case this is totally unrelated to this PR.

@marc-hb marc-hb mentioned this pull request Feb 2, 2024
Closed
lgirdwood
lgirdwood approved these changes Feb 5, 2024
View reviewed changes

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@wszypelt @lrudyX good to merge ?

@marc-hb

marc-hb commented Feb 5, 2024

Copy link
Copy Markdown
Collaborator Author

QB can't possibly be using that script.

This being said, I would like a review from @andyross

@wszypelt

wszypelt commented Feb 6, 2024

Copy link
Copy Markdown

@lgirdwood Internal Intel CI Build and tests are ok

@lgirdwood
lgirdwood merged commit e04289d into thesofproject:main Feb 7, 2024
@marc-hb
marc-hb deleted the fuzz-pristine branch February 7, 2024 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lgirdwood lgirdwood lgirdwood approved these changes
@andyross andyross Awaiting requested review from andyross
@kv2019i kv2019i Awaiting requested review from kv2019i kv2019i is a code owner
+1 more reviewer
@cujomalainey cujomalainey cujomalainey approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@marc-hb @wszypelt @cujomalainey @lgirdwood