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

Enable packit CI and create CI plan #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Koncpa
Copy link
Contributor

@Koncpa Koncpa commented Mar 7, 2024

No description provided.

@Koncpa Koncpa requested a review from sopos March 7, 2024 11:44
@Koncpa Koncpa self-assigned this Mar 7, 2024
@Koncpa Koncpa force-pushed the pk_enable_packit branch 2 times, most recently from 3761a31 to 88b9271 Compare March 7, 2024 11:50
@sopos
Copy link
Contributor

sopos commented Mar 8, 2024

What's the point of execution all the tests for every single change?
Shouldn't we limit the test execution to the changed tests only?
Running the full set would be good to the component updates, not the test updates.

@Koncpa
Copy link
Contributor Author

Koncpa commented Mar 18, 2024

I think that would be beneficial for us to run all tests, because It could help us to catch issues with not proper cleanup of each test. With new tooling we will often run tests as set of test and not as single host task so there is importance of proper setup and cleanup of enviroment..Also in future we could use this general CI plan in other repositories as imported plan..

@Koncpa Koncpa force-pushed the pk_enable_packit branch from 88b9271 to 6a756da Compare March 19, 2024 13:55
@Koncpa Koncpa force-pushed the pk_enable_packit branch from b610375 to 772f587 Compare May 7, 2024 12:17
@Koncpa
Copy link
Contributor Author

Koncpa commented May 7, 2024

/packit test

@Koncpa Koncpa force-pushed the pk_enable_packit branch 10 times, most recently from 3bf6a08 to 0dddf1f Compare May 14, 2024 08:52
@Koncpa Koncpa force-pushed the pk_enable_packit branch 9 times, most recently from 3178f28 to a5f86d2 Compare February 4, 2025 10:45
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.

2 participants