Skip to content
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

Use /usr/sbin/sestatus instead of /usr/bin/sestatus #2720

Merged
merged 1 commit into from
Mar 1, 2024

Conversation

guoguojenna
Copy link
Contributor

On a RHEL-8.9 compose, there's no /usr/bin/sestatus file, only /usr/sbin/sestatus,

which made the test preparation failed.

Command 'rpm -q --whatprovides /usr/bin/dmesg /usr/bin/sestatus /usr/sbin/ausearch /usr/bin/flock || dnf install -y /usr/bin/dmesg /usr/bin/sestatus /usr/sbin/ausearch /usr/bin/flock' returned 1,

so changing /usr/bin/sestatus to /usr/sbin/sestatus.

$ rpm -ql policycoreutils-2.9-25.el8.x86_64 | grep bin/sestatus
/usr/sbin/sestatus

On a RHEL-9.4 compose:
$ rpm -ql policycoreutils-3.6-1.el9.x86_64 | grep bin/sestatus
/usr/bin/sestatus
/usr/sbin/sestatus

Pull Request Checklist

  • implement the feature
  • write the documentation
  • extend the test coverage
  • update the specification
  • adjust plugin docstring
  • modify the json schema
  • mention the version
  • include a release note

Fix "no such file /usr/bin/sestatus" issue on RHEL-8
policycoreutils-2.9-25.el8
@happz happz added command | tests tmt tests command area | check Test check implementation labels Mar 1, 2024
@happz happz added this to the 1.32 milestone Mar 1, 2024
@happz happz added the ci | full test Pull request is ready for the full test execution label Mar 1, 2024
@happz
Copy link
Collaborator

happz commented Mar 1, 2024

/packit build

@happz happz merged commit 278b18c into teemtee:main Mar 1, 2024
21 of 22 checks passed
@happz
Copy link
Collaborator

happz commented Mar 1, 2024

Oh, I didn't realize all required checks are already done. The plan was to wait for provision check to finish before merging this PR :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area | check Test check implementation ci | full test Pull request is ready for the full test execution command | tests tmt tests command
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants