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 Test #2

Closed
wants to merge 1 commit into from
Closed

Packit Test #2

wants to merge 1 commit into from

Conversation

LorbusChris
Copy link
Owner

No description provided.

@LorbusChris
Copy link
Owner Author

/packit copr-build

@TomasTomecek
Copy link

TomasTomecek commented Sep 11, 2019

tried locally, and packit srpm worked just fine and even mock --rebuild $SRPM started the build process.

It feels like that packit-service is not hooked in this repo - is it installed?

@LorbusChris
Copy link
Owner Author

/packit copr-build

@LorbusChris
Copy link
Owner Author

It is installed. Just removed and added it again, but still doesn't seem to work. Maybe it's because this is a fork?

@TomasTomecek
Copy link

okay, packit service is getting the requests but this is the problem:

git.exc.GitCommandError: Cmd('git') failed due to: exit code(128)                                                                                                            
  cmdline: git clone --tags -v https://github.com/LorbusChris/origin.git /sandcastle                                                                                         
  stderr: 'Cloning into '/sandcastle'...                                                                                                                                     
POST git-upload-pack (gzip 5477 to 2788 bytes)                                                                                                                               
fatal: write error: No space left on device 
fatal: index-pack failed

@TomasTomecek
Copy link

within a few weeks, we are planning to migrate the packit service to a new cluster where storage shouldn't be a problem - would you be okay with waiting or should we fix our storage now?

@LorbusChris
Copy link
Owner Author

This is a blocker for OKD on FCOS. Not immediate priority, but the sooner the better :)

Some context:
I only need to run packit once on this fork in order to verify that it works, but I'll need to it set up for the main openshift/origin and oc repos asap, go get continuous builds of them (we can probably go with just COPR PR and master builds for a start (I hope), but in the longer run what we really need is a packit-service trigger for koji builds of master).
If it works here as expected, I'll work on getting packit added to the openshift org.
I am unsure how much resource consumption CI builds from those two repos will generate, btw.

@cgwalters
Copy link

This is a blocker for OKD on FCOS. Not immediate priority, but the sooner the better :)

Is it? We can build RPMs anywhere...among them, api.ci for example.

@LorbusChris
Copy link
Owner Author

Are you suggesting to add a job on OpenShift's prow?
We could do that.

Wondering though, would all the build info be publicly visible?

@cgwalters
Copy link

Are you suggesting to add a job on OpenShift's prow?

Yep.

Wondering though, would all the build info be publicly visible?

Yes.

@LorbusChris
Copy link
Owner Author

I'll look into Prow for PR CI builds.

I'm not sure if we can have per commit/timed builds of master there as well (that's what's done in ART for RHELCOS RPMs AIUI).

I would still like master builds in Koji via Packit-Service, but if that isn't possible at the moment due to resource constraints, I'm sure we can find a way to promote a CI build from Prow for use in FCOS composes somehow.

@TomasTomecek
Copy link

I would still like master builds in Koji via Packit-Service

We don't support that right now.

@LorbusChris
Copy link
Owner Author

Right, I believe the issue for this is packit/packit-service#47

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