Skip to content

Disable service_enabled templated test for service_bluetooth_disabled#12211

Merged
Mab879 merged 1 commit into
ComplianceAsCode:masterfrom
mildas:exclude_bluetooth_test
Jul 23, 2024
Merged

Disable service_enabled templated test for service_bluetooth_disabled#12211
Mab879 merged 1 commit into
ComplianceAsCode:masterfrom
mildas:exclude_bluetooth_test

Conversation

@mildas
Copy link
Copy Markdown
Contributor

@mildas mildas commented Jul 23, 2024

Description:

Disable test scenario that tries to start bluetooth service and then expect it to fail.

Rationale:

The service is dependent on loaded btusb kernel module. However, this module is not loaded by default and templated test scenario doesn't load it either.

We had similar problem in the past #11949 and now it started happening also on RHEL 8.8 and 9.2. I tried it manually on those systems and to properly start the service, I had to modprobe btusb (https://wiki.archlinux.org/title/Bluetooth#Installation). It will be better to disable the test scenario completely rather than creating Contest waiver.

Review Hints:

Try Automatus for service_bluetooth_disabled and see what test scenarios are not run.

@mildas mildas added the Test Suite Update in Test Suite. label Jul 23, 2024
@github-actions
Copy link
Copy Markdown

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

@github-actions
Copy link
Copy Markdown

🤖 A k8s content image for this PR is available at:
ghcr.io/complianceascode/k8scontent:12211
This image was built from commit: 2234dcc

Click here to see how to deploy it

If you alread have Compliance Operator deployed:
utils/build_ds_container.py -i ghcr.io/complianceascode/k8scontent:12211

Otherwise deploy the content and operator together by checking out ComplianceAsCode/compliance-operator and:
CONTENT_IMAGE=ghcr.io/complianceascode/k8scontent:12211 make deploy-local

@Mab879 Mab879 self-assigned this Jul 23, 2024
@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit 2234dcc and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 59.4% (0.0% change).

View more on Code Climate.

@Mab879 Mab879 added this to the 0.1.74 milestone Jul 23, 2024
@Mab879 Mab879 merged commit f82c486 into ComplianceAsCode:master Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Test Suite Update in Test Suite.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants