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

Catch no copr owner #648

Conversation

@lachmanfrantisek
Copy link
Member

lachmanfrantisek commented Jan 6, 2020

  • Raise an exception when there is no copr project owner.
  • Related to the #647
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Jan 6, 2020

Build succeeded.

Copy link
Contributor

TomasTomecek left a comment

code LGTM

but not test updates? :(

@lachmanfrantisek

This comment has been minimized.

Copy link
Member Author

lachmanfrantisek commented Jan 6, 2020

but not test updates? :(

I'll take a look and try to add some.

@lachmanfrantisek

This comment has been minimized.

Copy link
Member Author

lachmanfrantisek commented Jan 6, 2020

There are no tests for copr-build at all...;-(

edit: meaning the local copr build subcommand, not the packit-service copr build.

@lachmanfrantisek lachmanfrantisek force-pushed the lachmanfrantisek:catch-no-copr-owner branch from 1ee1c5c to 141a74f Jan 6, 2020
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Jan 6, 2020

Build succeeded.

@packit-as-a-service

This comment has been minimized.

Copy link

packit-as-a-service bot commented Jan 6, 2020

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-648
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@lachmanfrantisek lachmanfrantisek force-pushed the lachmanfrantisek:catch-no-copr-owner branch from 141a74f to 953b2bc Jan 7, 2020
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Jan 7, 2020

Build succeeded.

Signed-off-by: Frantisek Lachman <flachman@redhat.com>
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek lachmanfrantisek force-pushed the lachmanfrantisek:catch-no-copr-owner branch from 953b2bc to 3e5029b Jan 7, 2020
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Jan 7, 2020

Build failed.

@lachmanfrantisek

This comment has been minimized.

Copy link
Member Author

lachmanfrantisek commented Jan 7, 2020

recheck

@lachmanfrantisek

This comment has been minimized.

Copy link
Member Author

lachmanfrantisek commented Jan 7, 2020

@TomasTomecek tests added

@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Jan 7, 2020

Build succeeded.

Copy link
Contributor

TomasTomecek left a comment

bellísimo!

@TomasTomecek TomasTomecek added the mergeit label Jan 7, 2020
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Jan 7, 2020

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit ea0726e into packit-service:master Jan 7, 2020
10 of 11 checks passed
10 of 11 checks passed
packit/rpm-build-fedora-rawhide-x86_64 RPM build has started...
Details
LGTM analysis: Python No new or fixed alerts
Details
ci/dockercloud Your tests passed in Docker Cloud
Details
local/check check status: success
Details
local/gate gate status: success
Details
packit/rpm-build-fedora-30-x86_64 RPMs were built successfully.
Details
packit/rpm-build-fedora-31-x86_64 RPMs were built successfully.
Details
packit/srpm-build SRPM was built successfully.
Details
packit/testing-farm-fedora-30-x86_64 All tests passed
Details
packit/testing-farm-fedora-31-x86_64 All tests passed
Details
packit/testing-farm-fedora-rawhide-x86_64 All tests passed
Details
@lachmanfrantisek lachmanfrantisek deleted the lachmanfrantisek:catch-no-copr-owner branch Jan 7, 2020
@lachmanfrantisek lachmanfrantisek mentioned this pull request Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.