Skip to content
This repository has been archived by the owner on Jun 11, 2020. It is now read-only.

Delete repo fail #1176

Closed
JackDandy opened this issue Nov 25, 2018 · 22 comments
Closed

Delete repo fail #1176

JackDandy opened this issue Nov 25, 2018 · 22 comments
Assignees

Comments

@JackDandy
Copy link

JackDandy commented Nov 25, 2018

Two repos's were added inside build.snapcraft.io from a GitHub account that links an organizational release repo.

I want to delete one of the two repo's from the snap, but this is failing.

@bartaz
Copy link
Contributor

bartaz commented Nov 26, 2018

@JackDandy Can you give us more details about the error you are receiving?

Can you make a screenshot of browser JS console when you try to delete the repo? What's the error there?

Thanks!

@JackDandy
Copy link
Author

JackDandy commented Nov 26, 2018

@bartaz ... Here's an anim gif of the failure ... https://vgy.me/35g81N.gif

Note: I changed the registered name on the repo in order for it to intentionally fail a build because that is the testing/dev repo that I would now like to delete.

@bartaz
Copy link
Contributor

bartaz commented Nov 27, 2018

@JackDandy Thanks for a video, that's helpful. Unfortunately I cannot reproduce the issue myself, and the timeout error that shows up for you doesn't tell much.

Is this the repo you want removed? https://github.com/JackDandy/SickGear
I can try do to it manually.

@JackDandy
Copy link
Author

Yes please.

Also, I would like SickGear to be the base snap url when it is made public, and for SickGear/SickGear repo to be used. I would also like that repo to auto build, but given SickGear is an org repo with no direct signin, then how do I set that up in GH settings for that org repo?

Thanks

@JackDandy
Copy link
Author

If this is going to take another 24 hrs, or if you have any other questions, then can we speed things up by chatting in IRC/#snappy ?

@bartaz
Copy link
Contributor

bartaz commented Nov 27, 2018

@JackDandy Hi, sorry - I have some issues accessing our prod env to delete the repo. I'll try to sort it out tomorrow.

@bartaz
Copy link
Contributor

bartaz commented Nov 28, 2018

Sorry it takes so long. It seems that your repo is one of a bunch affected by a Launchpad bug (https://bugs.launchpad.net/launchpad/+bug/1803163)

It means that we will likely wont be able to delete it (either via build.snapcraft or manually in Launchpad) until it's fixed. I'll give it a try as it may success eventually, but for now I'm marking this issue blocked until LP bug is fixed.

@JackDandy
Copy link
Author

JackDandy commented Nov 28, 2018

ugh, indefinite period of waiting nightmare.

thanks for the feedback.

@bartaz
Copy link
Contributor

bartaz commented Nov 28, 2018

@JackDandy But, issues around deleting this repo, shouldn't affect building the other one.

I'm not really sure what you meant here:

Also, I would like SickGear to be the base snap url when it is made public, and for SickGear/SickGear repo to be used. I would also like that repo to auto build, but given SickGear is an org repo with no direct signin, then how do I set that up in GH settings for that org repo?

As far as I see you have SickGear/SickGear repo set up in BSI. By default repos added are built on every push to the repo, so no additional configuration should be needed.

@JackDandy
Copy link
Author

JackDandy commented Nov 28, 2018

I merged on SickGear organisation account yesterday, and had to do a manual build in snap webui.

If I make the snap public, are the failed ones also public ? I want just snap/sickgear (SickGear) that is built from the org account (i.e. SickGear/SickGear @ github) to be public in the store.

[edit]: ok i just made it public, seems ok :-) I dont think the build will be auto but have to wait until next time i do that process, now that it is "live".

@bartaz
Copy link
Contributor

bartaz commented Nov 28, 2018

If a build fails this revision of a snap is not published to the store (because it didn't build properly), so only successful builds are published in the store snap (edge channel).

Once they are published you can decide to promote them to any other channel (beta/candidate/stable).

@bartaz
Copy link
Contributor

bartaz commented Nov 28, 2018

@JackDandy OK. Not building on commit is a different issue - not sure why this may happening. My first guess is that maybe if you moved the repo the webhook that is set on GitHub is still old (refers to old repo location or has a secret generated for old repo). If that's the case you may need to remove the webhook manually, remove repo from build.snapcraft ,and add it again (to make sure webhook is recreated).

If you notice that builds don't trigger on push or merge feel free to open a new issue here.

@JackDandy
Copy link
Author

JackDandy commented Nov 28, 2018

Sure, thanks.

q: how do I set "verified" status, e.g. like "Plex" has ? Your systems have pulled the org repo, therefore, the account pulled the linked official repo.

@bartaz
Copy link
Contributor

bartaz commented Nov 28, 2018

@JackDandy As far as I understand currently only a selected group of publishers got a Verified status, I don't know if there is any process to apply for it, but it's better to ask on forum: https://forum.snapcraft.io/t/how-to-become-verified-publisher/7442

@JackDandy
Copy link
Author

It's been over a couple months and no shift on this from LP.

So, thinking from the side of Canonical, can you make it in the UI so that we can stop a repo from even being sent to LP.

A pause button alongside the delete -- so that we don't have to wait so long for builds to complete that we don't care for (which is why the delete was attempted)

@brlin-tw
Copy link

brlin-tw commented Mar 13, 2019

I also encountered a similar problem, the delete request returned with 504 Gateway timeout.

This particular snap has been transferred to the upstream, I've removed the webhook at my repository at the GitHub end but buid.snapcraft.io still build the snap due to "dependency changed" build trigger and keep publishing invalid builds to the edge channel (which is differentiable due to the different versioning style)

@brlin-tw
Copy link

The snap in particular: https://github.com/Lin-Buo-Ren/gallery-dl-snap

Is there anything you can do from the backend until the bug is resolved?

@brlin-tw
Copy link

brlin-tw commented Mar 21, 2019

One possible workaround is to change the registered snap name to make it mismatch with the one specified in the Snapcraft recipe.

Screenshot

A build is still triggered somehow but probably the built snap would not be published to the store due to name mismatch.

Yep it works:

@JackDandy
Copy link
Author

@Lin-Buo-Ren that's how i do my workaround too

@bartaz
Copy link
Contributor

bartaz commented Apr 3, 2019

Also reported in #1211

@bartaz bartaz closed this as completed Apr 3, 2019
@bartaz bartaz reopened this Apr 3, 2019
@kerolloz
Copy link

@Lukewh
image

@solazio
Copy link
Contributor

solazio commented Jun 3, 2020

This is fixed in the Builds tab in snapcraft.io

@solazio solazio closed this as completed Jun 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants