Skip to content

Move to nproc and Speed Up Ansible Lint#14328

Open
Mab879 wants to merge 1 commit intoComplianceAsCode:masterfrom
Mab879:ci_speed_up_2026_01_23
Open

Move to nproc and Speed Up Ansible Lint#14328
Mab879 wants to merge 1 commit intoComplianceAsCode:masterfrom
Mab879:ci_speed_up_2026_01_23

Conversation

@Mab879
Copy link
Member

@Mab879 Mab879 commented Jan 23, 2026

Description:

  • -j2 to -j$(nproc)
  • Parallelize ansible lint

Rationale:

Trying to reduce time in CI.

Review Hints:

Review the CI logs.

@Mab879 Mab879 added this to the 0.1.80 milestone Jan 23, 2026
@Mab879 Mab879 changed the title Move to nproc and Speed Up Move to nproc and Speed Up Ansible Lint Jan 23, 2026
@github-actions
Copy link

ATEX Test Results

Test artifacts have been submitted to Testing Farm.

Results: View Test Results
Workflow Run: View Workflow Details

This comment was automatically generated by the ATEX workflow.

@vojtapolasek vojtapolasek self-assigned this Jan 26, 2026
Copy link
Collaborator

@vojtapolasek vojtapolasek left a comment

Choose a reason for hiding this comment

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

Thank you for the PR.
Claude warned me about few errors which seem relevant. What do you think?

@vojtapolasek
Copy link
Collaborator

@Mab879 I see there are some Ansible lint warnings produced... is this something what was not waived in the past or are these new findings?

@Mab879
Copy link
Member Author

Mab879 commented Feb 2, 2026

@Mab879 I see there are some Ansible lint warnings produced... is this something what was not waived in the past or are these new findings?

The findings were here before, xargs just exposes them. I might be able to rewrite parallelization to better account for this. Or I can tweak the Ansible config to make it pass. Thoughts?

@Mab879 Mab879 force-pushed the ci_speed_up_2026_01_23 branch from 3b38087 to c4b5263 Compare February 9, 2026 21:41
@Mab879 Mab879 requested a review from vojtapolasek February 9, 2026 21:42
@openshift-ci
Copy link

openshift-ci bot commented Feb 10, 2026

@Mab879: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-openshift-node-compliance c4b5263 link true /test e2e-aws-openshift-node-compliance

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@vojtapolasek
Copy link
Collaborator

/packit retest-failed

1 similar comment
@Mab879
Copy link
Member Author

Mab879 commented Feb 10, 2026

/packit retest-failed

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.

3 participants