Skip to content

Drop manually specifying copr repo in packit#4620

Open
LecrisUT wants to merge 1 commit into
teemtee:mainfrom
LecrisUT:chore/drop-artifacts
Open

Drop manually specifying copr repo in packit#4620
LecrisUT wants to merge 1 commit into
teemtee:mainfrom
LecrisUT:chore/drop-artifacts

Conversation

@LecrisUT
Copy link
Copy Markdown
Member

Not quite sure why this was used, but the fact that it is only set for rawhide and epel9 is fishy.

Pull Request Checklist

  • implement the feature

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the manual specification of the teemtee/stable Copr repository from the .packit.yaml configuration. This change deletes hardcoded repository URLs for fedora-rawhide and epel-9 test environments. No issues were found in the submitted changes.

@LecrisUT LecrisUT added the ci | full test Pull request is ready for the full test execution label Feb 27, 2026
@LecrisUT LecrisUT force-pushed the chore/drop-artifacts branch 2 times, most recently from fccece2 to a1d2531 Compare March 3, 2026 20:26
@happz happz added this to planning Mar 3, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Mar 3, 2026
@happz happz moved this from backlog to review in planning Mar 3, 2026
@psss psss added this to the 1.74 milestone May 7, 2026
Signed-off-by: Cristian Le <git@lecris.dev>
@LecrisUT LecrisUT force-pushed the chore/drop-artifacts branch from a1d2531 to 7afae50 Compare May 11, 2026 11:53
Comment thread .packit.yaml
id: https://copr.fedorainfracloud.org/coprs/g/teemtee/stable/repo/fedora-rawhide/group_teemtee-stable-fedora-rawhide.repo
- type: repository-file
# yamllint disable-line rule:line-length
id: https://copr.fedorainfracloud.org/coprs/g/teemtee/stable/repo/epel-9/group_teemtee-stable-epel-9.repo
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These repositories were intentionally enabled in #3452 to allow testing with fresh fmf bits as soon as they are released. As discussed today, it would be good to verify that this actually still works, because of concerns about the exact python version required. We could possibly do that if teemtee/fmf#293 is finished and we do a new fmf release.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This packit issue could be useful here. Basically we could just add the appropriate dependency at the copr level and let copr do its magic there.

@psss psss added status | blocked The merging of PR is blocked on some other issue status | discuss Needs more discussion before closing labels May 14, 2026
@psss
Copy link
Copy Markdown
Member

psss commented May 14, 2026

Blocked on the hacking session discussion, moving to the sprint backlog.

@psss psss moved this from review to backlog in planning May 14, 2026
@bajertom bajertom removed the status | discuss Needs more discussion before closing label May 19, 2026
@bajertom
Copy link
Copy Markdown
Contributor

It's blocked on packit/packit#2719

@psss
Copy link
Copy Markdown
Member

psss commented May 19, 2026

Agreed to keep it in the backlog until the packit issue is implemented or rawhide python version breaks the setup. Dropping from the sprint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution status | blocked The merging of PR is blocked on some other issue

Projects

Status: backlog

Development

Successfully merging this pull request may close these issues.

4 participants