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

upload will fail if a file with the same hash is already up #9479

Merged
merged 2 commits into from
Sep 10, 2018

Conversation

gabreal
Copy link
Contributor

@gabreal gabreal commented Sep 5, 2018

still about this issue: https://gitlab.parity.io/parity/parity-ethereum/-/jobs/96821

which happens if a snap with the same checksum is already uploaded. allow_failure here will at least the pipeline proceed to the next stage.

(the before_script statements with *determine_version are duplicate)

@parity-cla-bot
Copy link

It looks like @gabreal signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

.gitlab-ci.yml Outdated
@@ -222,33 +222,28 @@ publish-linux-snap-amd64: &publish_snap
BUILD_ARCH: amd64
dependencies:
- package-linux-snap-amd64
allow_failure: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is this possible, we are building nightly only once per day. This should ideally not fail. Can't we fix this on snap side?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The upload only fails for uploads of exactly the same snap for the same version/architecture. So if there are no changes in the code nightly builds will still produce a snap which will be upload and hash check on snapcraft refuses it then. I found no way of changing this behaviour.

IMHO the easiest way to fix this would be to have a version like the current timestamp (e.g. 20180905) or git commit like before: https://imgur.com/a/bgpoG7T

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version is wrong anyways, this is not 2.1.0. This need to be fixed in the script that figures out the version. Adding a commit hash and date would help here if nightly.

@5chdn 5chdn added A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. M1-ci 🙉 Continuous integration. labels Sep 5, 2018
@5chdn 5chdn added this to the 2.1 milestone Sep 5, 2018
Copy link
Contributor

@5chdn 5chdn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@5chdn 5chdn added A8-looksgood 🦄 Pull request is reviewed well. B9-blocker 🚧 This pull request blocks the next release from happening. Use only in extreme cases. and removed A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. labels Sep 8, 2018
Copy link
Contributor

@General-Beck General-Beck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM

@General-Beck General-Beck merged commit eb0986c into master Sep 10, 2018
@gabreal gabreal deleted the allow-snap-publishing-fail branch September 10, 2018 15:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. B9-blocker 🚧 This pull request blocks the next release from happening. Use only in extreme cases. M1-ci 🙉 Continuous integration.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants