Skip to content

Show memory leak summary when valgrind check fails#570

Open
sshedi wants to merge 1 commit intovmware:devfrom
sshedi:valgrind-improvements
Open

Show memory leak summary when valgrind check fails#570
sshedi wants to merge 1 commit intovmware:devfrom
sshedi:valgrind-improvements

Conversation

@sshedi
Copy link
Copy Markdown
Contributor

@sshedi sshedi commented Apr 7, 2026

No description provided.

@sshedi sshedi changed the title Show memeory leak summary when valgrind check fails Show memory leak summary when valgrind check fails Apr 7, 2026
@sshedi
Copy link
Copy Markdown
Contributor Author

sshedi commented Apr 7, 2026

error: unshare with flags x40020000 failed: Operation not permitted
error: Plugin unshare: hook scriplet_fork_post failed
warning: %post(git-2.43.7-2.ph5.x86_64) scriptlet failed, exit status 126
package git-2.43.7-2.ph5.x86_64: script warning in %postin
Error(1525) : rpm transaction failed
Error: Process completed with exit code 245.

There is a fix in progress for this.

@oliverkurth
Copy link
Copy Markdown
Contributor

Thank you for this. But how does this fix the valgrind issue? I understand we'll see the error output, but why is it not failing any more?

@sshedi sshedi force-pushed the valgrind-improvements branch from 579fdbf to ba811cd Compare April 7, 2026 19:39
@sshedi
Copy link
Copy Markdown
Contributor Author

sshedi commented Apr 7, 2026

Thank you for this. But how does this fix the valgrind issue? I understand we'll see the error output, but why is it not failing any more?

Fixed it. The error was not captured because I removed exit on first error.

@oliverkurth
Copy link
Copy Markdown
Contributor

error: unshare with flags x40020000 failed: Operation not permitted
error: Plugin unshare: hook scriplet_fork_post failed
warning: %post(git-2.43.7-2.ph5.x86_64) scriptlet failed, exit status 126
package git-2.43.7-2.ph5.x86_64: script warning in %postin
Error(1525) : rpm transaction failed
Error: Process completed with exit code 245.

There is a fix in progress for this.

I am trying to fix a similar issue here: vmware/photon-os-installer#41

Signed-off-by: Shreenidhi Shedi <shreenidhi.shedi@broadcom.com>
@sshedi sshedi force-pushed the valgrind-improvements branch from ba811cd to 363e8a1 Compare April 8, 2026 15:38
@sshedi
Copy link
Copy Markdown
Contributor Author

sshedi commented Apr 8, 2026

error: unshare with flags x40020000 failed: Operation not permitted
error: Plugin unshare: hook scriplet_fork_post failed
warning: %post(git-2.43.7-2.ph5.x86_64) scriptlet failed, exit status 126
package git-2.43.7-2.ph5.x86_64: script warning in %postin
Error(1525) : rpm transaction failed
Error: Process completed with exit code 245.

There is a fix in progress for this.

I am trying to fix a similar issue here: vmware/photon-os-installer#41

This is also the right thing to do, at some point we will have to align with rpm upstream and consider scriptlet errors as fatal. rpm-6.0.1-3 is published, please rebase #569

@sshedi
Copy link
Copy Markdown
Contributor Author

sshedi commented Apr 8, 2026

The intention behind this PR is, if we are force merging a PR ignoring memory leaks; we can have a look at the leak summary and decide if it indeed from tdnf or from third party library.

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.

2 participants