Skip to content

Conversation

@Honny1
Copy link
Member

@Honny1 Honny1 commented Apr 6, 2023

The main purpose of this PR is to create a separate EPEL 8 release job. Additionally, this PR contains test synchronization.

dest: plans/
- src: .fmf/
dest: .fmf/
- src: tests/
Copy link
Contributor

Choose a reason for hiding this comment

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

We haven't had to do that before. What's changed?

Copy link
Member Author

Choose a reason for hiding this comment

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

Why didn't we have to do that? I think it was missing by mistake. Downstream CI tried to execute the tmt plan but could not find the test.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Does not look as it was not found: https://artifacts.dev.testing-farm.io/78e9d3cd-7980-472a-920e-1e7b4d6454ae/

PackIt finds tests because it uses a GitHub repository. For downstream is used downstream repo:
https://artifacts.dev.testing-farm.io/e0b683ea-6977-4e6f-b034-e6daf66ac1bd/

Copy link
Contributor

Choose a reason for hiding this comment

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

Got it.

Copy link
Member Author

Choose a reason for hiding this comment

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

I check how PackIt uses tmt etc. PackIt discovers tests from upstream. I will change it in another PR.

@evgenyz evgenyz merged commit 7383bfc into OpenSCAP:main Apr 11, 2023
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