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 of chroots of Copr Project owned by Packit rises an error #1283

Closed
lachmanfrantisek opened this issue Nov 15, 2021 · 8 comments
Closed
Labels
area/copr Related to the integration with copr.fedorainfracloud.org/ kind/bug Something isn't working. triaged This issue was already processed by the team.

Comments

@lachmanfrantisek
Copy link
Member

lachmanfrantisek commented Nov 15, 2021

If someone creates the PR with a set of chroots and change it later on, there is a permission problem.

See this example: RedHat-SP-Security/keylime-tests#13 (comment)

@lachmanfrantisek lachmanfrantisek added kind/bug Something isn't working. area/copr Related to the integration with copr.fedorainfracloud.org/ labels Nov 15, 2021
@mfocko
Copy link
Member

mfocko commented Nov 15, 2021

It's not, the issue you cannot update targets while there is a running build (for that target you want to remove? ← might not be true)

@lachmanfrantisek
Copy link
Member Author

@mfocko what can we do here?

@mfocko
Copy link
Member

mfocko commented Nov 15, 2021

• Retrigger or suggest to retrigger once the build is finished. Or, can we cancel the build?

that's probably related to an even older issue #5

anyways, canceling build sounds like the most ideal solution, but we can only fix it on our side, if user specifies Copr repository, they can build from multiple repos / branches there

@lachmanfrantisek
Copy link
Member Author

So, let's start with getting a better comment. The situation is not so common and asking for some human interaction looks ok to me in this case.

@lachmanfrantisek lachmanfrantisek added the triaged This issue was already processed by the team. label Nov 18, 2021
@stale
Copy link

stale bot commented Mar 2, 2022

This issue has been marked as stale because it hasn't seen any
activity for the last 60 days.

Stale issues are closed after 14 days, unless the label is removed
by a maintainer or someone comments on it.

This is done in order to ensure that open issues are still relevant.

Thank you for your contribution! 🦄 🚀 🤖

(Note: issues labeled with pinned or EPIC are
never marked as stale.)

@stale stale bot added the stale Is the issue still valid? label Mar 2, 2022
@TomasTomecek
Copy link
Member

We now produce a better error message: #1397

Since this is a limitation of Copr, I'm not sure if there is anything we can do our side. We could possibly cancel the build or work with Copr to implement this on their side.

@stale stale bot removed the stale Is the issue still valid? label Mar 28, 2022
@stale
Copy link

stale bot commented Jun 12, 2022

This issue has been marked as stale because it hasn't seen any
activity for the last 60 days.

Stale issues are closed after 14 days, unless the label is removed
by a maintainer or someone comments on it.

This is done in order to ensure that open issues are still relevant.

Thank you for your contribution! 🦄 🚀 🤖

(Note: issues labeled with pinned or EPIC are
never marked as stale.)

@stale stale bot added the stale Is the issue still valid? label Jun 12, 2022
@TomasTomecek
Copy link
Member

Is this still a problem?

@stale stale bot removed the stale Is the issue still valid? label Jun 27, 2022
@csomh csomh closed this as not planned Won't fix, can't repro, duplicate, stale Sep 7, 2022
Repository owner moved this from backlog to done in Packit Kanban Board Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/copr Related to the integration with copr.fedorainfracloud.org/ kind/bug Something isn't working. triaged This issue was already processed by the team.
Projects
Archived in project
Development

No branches or pull requests

4 participants