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

test: execute dmesg task as privileged user #4094

Merged
merged 1 commit into from
May 29, 2024

Conversation

mcattamoredhat
Copy link
Contributor

This pull request includes:

We have seen edge-commit-f39 and edge-installer-f39 failing in CI due to "dmesg: read kernel buffer failed: Operation not permitted"

TASK [check dmesg output] ******************************************************
fatal: [192.168.100.50]: FAILED! => {"changed": true, "cmd": ["dmesg"], "delta": "0:00:00.004286", "end": "2024-04-19 15:07:24.268256", "msg": "non-zero return code", "rc": 1, "start": "2024-04-19 15:07:24.263970", "stderr": "dmesg: read kernel buffer failed: Operation not permitted", "stderr_lines": ["dmesg: read kernel buffer failed: Operation not permitted"], "stdout": "", "stdout_lines": []}

This PR suggests to execute dmesg playbook task as privileged user.

  • adequate testing for the new functionality or fixed issue
  • adequate documentation informing people about the change such as
    • submit a PR for the READMEs listed here
    • submit a PR for the osbuild.org website repository if this PR changed any behavior not covered by the automatically updated READMEs

Copy link

This PR is stale because it has been open 30 days with no activity. Remove "Stale" label or comment or this will be closed in 7 days.

Copy link
Contributor

@bcl bcl left a comment

Choose a reason for hiding this comment

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

Looks good to me

@mvo5 mvo5 merged commit cab72ff into osbuild:main May 29, 2024
49 of 63 checks passed
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.

4 participants