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

[WIP] Testing upstream package builds #257

Closed
wants to merge 2 commits into from
Closed

[WIP] Testing upstream package builds #257

wants to merge 2 commits into from

Conversation

gridhead
Copy link
Member

@gridhead gridhead commented May 5, 2022

This tests the included packit.yaml Packit configuration on the packit branch to build packages to the t0xic0der/fedora-messaging COPR namespace on every pull request made or release created for Fedora 35.

This is a work in progress as I was not able to make the tests run.

Verifies working for #253. Does not close it.

@gridhead gridhead added the no-mergify Add this label to pull requests to stop automatic merging label May 5, 2022
@gridhead gridhead self-assigned this May 5, 2022
packit.yaml Outdated Show resolved Hide resolved
packit.yaml Outdated Show resolved Hide resolved
packit.yaml Outdated Show resolved Hide resolved
packit.yaml Outdated Show resolved Hide resolved
setup.py Show resolved Hide resolved
@StephenCoady
Copy link
Contributor

I know its just for testing but just a quick review so we remember the changes needed.

@gridhead gridhead force-pushed the packit branch 2 times, most recently from 393fbdc to 56d1b8a Compare May 6, 2022 04:08
Signed-off-by: Akashdeep Dhar <[email protected]>
@packit-as-a-service
Copy link

Failed to load packit config file:

Cannot parse package config: ValidationError({'jobs': {0: {'branch': ['Unknown field.']}, 1: {'branch': ['Unknown field.']}, 2: {'branch': ['Unknown field.']}, 3: {'branch': ['Unknown field.']}}}).

For more info, please check out the documentation: https://packit.dev/docs/packit-service or contact us - Packit team

Signed-off-by: Akashdeep Dhar <[email protected]>
@gridhead
Copy link
Member Author

gridhead commented May 6, 2022

Apparently the pull request also needs to have the Packit configuration for the CI to kick off. I tried once by targetting a branch which DOES NOT have a Packit configuration to a branch that DOES but it did not work until I added one to the SOURCE BRANCH in a subsequent commit.

@gridhead
Copy link
Member Author

gridhead commented May 6, 2022

/packit test

1 similar comment
@gridhead
Copy link
Member Author

gridhead commented May 6, 2022

/packit test

@StephenCoady StephenCoady deleted the branch fedora-infra:packit May 12, 2022 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-mergify Add this label to pull requests to stop automatic merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants