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

Added new env to run multiple policies #889

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

athiruma
Copy link
Collaborator

Type of change

Note: Fill x in []

  • bug
  • enhancement
  • documentation
  • dependencies

Description

POLICIES_LIST: To run multiple policies.

For security reasons, all pull requests need to be approved first before running any automated CI

@athiruma athiruma added documentation Improvements or additions to documentation ok-to-test PR ok to test labels Jan 16, 2025
@athiruma athiruma requested a review from ebattat January 16, 2025 12:01
@athiruma athiruma self-assigned this Jan 16, 2025
@ebattat
Copy link
Collaborator

ebattat commented Jan 16, 2025

We can run multiple policies, why we need it ?

@athiruma
Copy link
Collaborator Author

We can run multiple policies, why we need it ?

Currently, we are not supporting multiple policies in one run. We need to depend on external scripts to run multiple policies. If we switch to pod it is very useful to run multiple policies.

@ebattat
Copy link
Collaborator

ebattat commented Jan 16, 2025

Did u test that the CG CI is keep working with this change ?

@athiruma
Copy link
Collaborator Author

Did u test that the CG CI is keep working with this change ?

yes.

Copy link
Collaborator

@ebattat ebattat left a comment

Choose a reason for hiding this comment

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

/LGTM

@ebattat ebattat merged commit 31112f4 into redhat-performance:main Jan 16, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation ok-to-test PR ok to test
Projects
Development

Successfully merging this pull request may close these issues.

2 participants