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

Update Packit configuration #77

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

Conversation

packit-public-repos-bot

In case of any questions, please contact Packit team.

This commit fixes the configuration for the forthcoming Packit 1.0.0.
See [our blog post](https://packit.dev/posts/packit_1_0_0_action_required) for more details.
  - Job type `build` has been changed to `copr_build`.
  - Job type `production_build` has been changed to `upstream_koji_build`.
  - Key `upstream_project_name` has been changed to `upstream_package_name`.
  - Key `synced_files` has been changed to `files_to_sync`.

Please review and merge me before January 2025 otherwise packit-service jobs will fail because of an invalid configuration.
upstream_project_url: https://github.com/fedora-iot/rpm-head-signing
downstream_package_name: rpm-head-signing
downstream_package_name: python-rpm-head-signing
Copy link
Member

Choose a reason for hiding this comment

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

That is not correct, the downstream is as it is there.

Copy link
Contributor

@majamassarini majamassarini Dec 11, 2024

Choose a reason for hiding this comment

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

Sorry, I didn't realize the script was changing more things than needed. I will investigate why (seems like the config was taken from an old version). The only needed change is renaming synced_files to files_to_sync (before January 25 - when we will release the new Packit 1.0.0).

The script is not able to force-push new changes, so I kindly ask you to close this PR and fix the config in a new PR. Thank you.

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.

3 participants