Skip to content

test: fix missing mocker of r,tree for rocky/redat package builds#6808

Merged
blackboxsw merged 1 commit into
canonical:mainfrom
blackboxsw:test-rh-subscription
Mar 28, 2026
Merged

test: fix missing mocker of r,tree for rocky/redat package builds#6808
blackboxsw merged 1 commit into
canonical:mainfrom
blackboxsw:test-rh-subscription

Conversation

@blackboxsw
Copy link
Copy Markdown
Collaborator

@blackboxsw blackboxsw commented Mar 27, 2026

Found during package builds for recent copr upload.
Inability to pass unittests during package build as non-root user running command:

./tools/run-container --source-package --artifacts=./srpm rockylinux/9

Resulting unittest falilure is due to an ineffective mock.path which should have been mocker.patch.
This doesn't affect Ubuntu because Ubuntu doesn't have directories:

  • /var/cache/yum
  • /var/cache/dnf

Proposed Commit Message

test: fix missing mocker of rmtree for rocky/redat package builds

Additional Context

Prevents upload to copr test repos

Test Steps

./tools/run-container --source-package --artifacts=./srpm rockylinux/9

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

Copy link
Copy Markdown
Member

@holmanb holmanb left a comment

Choose a reason for hiding this comment

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

Please fix the commit message. Otherwise this looks good.

@blackboxsw blackboxsw merged commit a870544 into canonical:main Mar 28, 2026
17 checks passed
blackboxsw added a commit to blackboxsw/cloud-init that referenced this pull request Apr 13, 2026
Fixes leaked rmtree call during package builds for rocky and redhat.
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