Skip to content

Conversation

@lsm5
Copy link
Member

@lsm5 lsm5 commented Dec 26, 2024

No description provided.

@packit-as-a-service
Copy link

Failed to load packit config file:

Please correct data and retry.

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

2 similar comments
@packit-as-a-service
Copy link

Failed to load packit config file:

Please correct data and retry.

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@packit-as-a-service
Copy link

Failed to load packit config file:

Please correct data and retry.

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@packit-as-a-service
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs[4].packages': 'Undefined package(s) referenced: containers-common-fedora.', 'jobs[5].packages': 'Undefined package(s) referenced: containers-common-centos.'})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

2 similar comments
@packit-as-a-service
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs[4].packages': 'Undefined package(s) referenced: containers-common-fedora.', 'jobs[5].packages': 'Undefined package(s) referenced: containers-common-centos.'})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@packit-as-a-service
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs[4].packages': 'Undefined package(s) referenced: containers-common-fedora.', 'jobs[5].packages': 'Undefined package(s) referenced: containers-common-centos.'})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@lsm5 lsm5 changed the title Tmt TMT: Initial Enablement Dec 26, 2024
@lsm5
Copy link
Member Author

lsm5 commented Dec 26, 2024

We can speed up tmt tests by only running the podman tests relevant to conmon. Let me know which ones and I can adjust the test config.

@lsm5
Copy link
Member Author

lsm5 commented Feb 20, 2025

@saschagrunert @haircommander With packit rpm build jobs in place, do we need the cirrus Packaging for Fedora N[-1] jobs? If no, I guess we could also delete contrib/ and .rpmbuild dirs along with those 2 jobs.

@lsm5 lsm5 force-pushed the tmt branch 2 times, most recently from 39d9f91 to 082ff01 Compare February 20, 2025 13:45
@lsm5 lsm5 marked this pull request as ready for review February 20, 2025 14:54
@lsm5
Copy link
Member Author

lsm5 commented Feb 20, 2025

Marking ready for review but maybe best to wait until containers/podman#24369 is merged. I'll then update the TMT plan to fetch tests from podman main branch instead of my own fork.

@saschagrunert
Copy link
Member

@saschagrunert @haircommander With packit rpm build jobs in place, do we need the cirrus Packaging for Fedora N[-1] jobs? If no, I guess we could also delete contrib/ and .rpmbuild dirs along with those 2 jobs.

I'd say no, but I'm not exactly sure how we should handle the CI here.

@lsm5
Copy link
Member Author

lsm5 commented Feb 24, 2025

@saschagrunert @haircommander With packit rpm build jobs in place, do we need the cirrus Packaging for Fedora N[-1] jobs? If no, I guess we could also delete contrib/ and .rpmbuild dirs along with those 2 jobs.

I'd say no, but I'm not exactly sure how we should handle the CI here.

Doesn't look like any Cirrus job depends on fedora_packaging. So, I think it should be safe to remove. I can leave that for a followup though.

The current PR should be good to go. PTAL. I'll leave CentOS and aarch64 for a followup as well.

@lsm5
Copy link
Member Author

lsm5 commented Feb 24, 2025

Ah, there's still the fact that I'm fetching tests from my podman fork. I'll leave it to you if you feel comfy merging with that. I can keep my fork active and change the URL here once the podman PR is merged. Your call.

Removing EPEL (RHEL) jobs from here, as we have removed them from our
other projects as well. RHEL copr envrionments are often out of date,
leading to failed builds and tests.

Testing on CentOS Stream should suffice to guarantee that whatever
enters RHEL has passed upstreawm tests.

This commit also separates out ELN jobs so that fedora copr targets can
be conveniently reused in TMT tests that will be added in future
commits.

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
@lsm5
Copy link
Member Author

lsm5 commented Jun 30, 2025

@saschagrunert @haircommander How much does conmon care about nix package builds? Do we still need to keep the static build jobs?

@saschagrunert
Copy link
Member

@saschagrunert @haircommander How much does conmon care about nix package builds? Do we still need to keep the static build jobs?

Yeah I would prefer keeping them because we still ship conmon this way.

@lsm5
Copy link
Member Author

lsm5 commented Jun 30, 2025

@saschagrunert @haircommander How much does conmon care about nix package builds? Do we still need to keep the static build jobs?

Yeah I would prefer keeping them because we still ship conmon this way.

ack, I'll get to those in another PR. Once the existing TMT tests pass here, I'll remove the Packaging for Fedora.. Cirrus jobs from here.

Run podman local rootful and rootless tests on every PR.

Packit will handle syncing of the TMT tests and gating config
from upstream to downstream Fedora with every upstream release.

TODO: enable CentOS Stream test jobs.

Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
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