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

Packit status - show latest Copr builds #579

Merged
merged 3 commits into from Dec 4, 2019

Conversation

@lbarcziova
Copy link
Contributor

lbarcziova commented Oct 24, 2019

Fixes #445

@packit-as-a-service

This comment has been minimized.

Copy link

packit-as-a-service bot commented Oct 24, 2019

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-579
  • And now you can install the packages.

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

@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Oct 24, 2019

Build succeeded.

Copy link
Member

jpopelka left a comment

Your code is very neat/readable.

packit/status.py Outdated Show resolved Hide resolved
packit/status.py Outdated Show resolved Hide resolved
packit/status.py Outdated Show resolved Hide resolved
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Oct 29, 2019

Build succeeded.

@lbarcziova lbarcziova force-pushed the lbarcziova:status_show_copr_build branch from bdbee10 to be475e2 Oct 31, 2019
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Oct 31, 2019

Build succeeded.


projects = [
project.name
for project in reversed(client.project_proxy.get_list(ownername="packit"))

This comment has been minimized.

Copy link
@TomasTomecek

TomasTomecek Oct 31, 2019

Contributor

in future, we could use packit service api here :D

project.name
for project in reversed(client.project_proxy.get_list(ownername="packit"))
if project.name.startswith(
f"{self.up.local_project.namespace}-{self.up.local_project.repo_name}-"

This comment has been minimized.

Copy link
@TomasTomecek

TomasTomecek Oct 31, 2019

Contributor

sadly this doesn't work for me because my origin is my fork, not the upstream repo; hence it tries to look for TomasTomecek-packit instead of packit-service-packit

I'm gonna open an issue to solve this: #586

This comment has been minimized.

Copy link
@lbarcziova

lbarcziova Nov 13, 2019

Author Contributor

Should I wait for solving of #586 or changing of the condition to self.up.local_project.repo_name in project.name will be enough?

This comment has been minimized.

Copy link
@TomasTomecek

TomasTomecek Nov 13, 2019

Contributor

I would leave the complete solution to #586 so that you are not blocked.

Please add some tests and we're good to merge.

Copy link
Contributor

TomasTomecek left a comment

no tests :(

@lbarcziova lbarcziova force-pushed the lbarcziova:status_show_copr_build branch from be475e2 to 9c1be35 Nov 13, 2019
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Nov 13, 2019

Build failed.

@TomasTomecek

This comment has been minimized.

Copy link
Contributor

TomasTomecek commented Nov 13, 2019

recheck

@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Nov 13, 2019

Build failed.

@TristanCacqueray

This comment has been minimized.

Copy link

TristanCacqueray commented Nov 13, 2019

recheck

@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Nov 13, 2019

Build succeeded.

@jpopelka jpopelka added the mergeit label Nov 29, 2019
@jpopelka jpopelka force-pushed the lbarcziova:status_show_copr_build branch from 9c1be35 to d1db861 Nov 29, 2019
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Nov 29, 2019

Build failed.

@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Nov 29, 2019

Build failed.

@jpopelka

This comment has been minimized.

Copy link
Member

jpopelka commented Nov 29, 2019

@lbarcziova I've rebased in order to merge, but there's some integration tests failing now. Can you re-check?

@lachmanfrantisek

This comment has been minimized.

Copy link
Member

lachmanfrantisek commented Dec 2, 2019

@lbarcziova I've rebased in order to merge, but there's some integration tests failing now. Can you re-check?

This is probably a problem with OGR caused by this:
packit-service/ogr#276

(I am wondering why the reverse-dep tests passed.)

@lbarcziova lbarcziova force-pushed the lbarcziova:status_show_copr_build branch from d1db861 to 656a8fa Dec 3, 2019
@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Dec 3, 2019

Build failed.

@TomasTomecek TomasTomecek removed the mergeit label Dec 3, 2019
@TomasTomecek

This comment has been minimized.

Copy link
Contributor

TomasTomecek commented Dec 3, 2019

Franta is waiting for packit-service/ogr#287 to be merged, then we can merge this one.

@lachmanfrantisek

This comment has been minimized.

Copy link
Member

lachmanfrantisek commented Dec 3, 2019

recheck

@softwarefactory-project-zuul

This comment has been minimized.

Copy link
Contributor

softwarefactory-project-zuul bot commented Dec 3, 2019

Build succeeded.

@TomasTomecek TomasTomecek added the mergeit label Dec 4, 2019
@jpopelka jpopelka merged commit c34de44 into packit-service:master Dec 4, 2019
9 checks passed
9 checks passed
ci/dockercloud Your tests passed in Docker Cloud
Details
local/check check 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/rpm-build-fedora-rawhide-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
@TomasTomecek

This comment has been minimized.

Copy link
Contributor

TomasTomecek commented Dec 4, 2019

Party time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.