Skip to content

snap: deploy: fix major channels not being pushed #4222

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 17, 2020
Merged

snap: deploy: fix major channels not being pushed #4222

merged 1 commit into from
Jul 17, 2020

Conversation

casperdcl
Copy link
Contributor

@casperdcl casperdcl commented Jul 17, 2020 •
edited
Loading

  • push to snap channels in one (rather than two) deploy step(s)
  • in the meantime update the current v1 channels to match latest
snapcraft release dvc 626 v1/stable  # updates 1.1.7 to 1.1.10
snapcraft release dvc 641 v1/edge    # updates 1.1.7 to 1.1.10-15-g91f3036c

fixes https://travis-ci.com/github/iterative/dvc/jobs/360297196#L2901-L2902

The store was unable to accept this snap.
  - binary_sha3_384: A file with this exact same content has already been uploaded

@casperdcl casperdcl added bug Did we break something? c1-quick-fix research labels Jul 17, 2020
@casperdcl casperdcl self-assigned this Jul 17, 2020
@casperdcl casperdcl requested a review from efiop July 17, 2020 00:49
@efiop efiop merged commit 4edf368 into iterative:master Jul 17, 2020
@efiop
Copy link
Contributor

efiop commented Jul 17, 2020

Thanks!

@casperdcl
Copy link
Contributor Author

just also manually pushed 1.1.11 since it was released before this PR was merged :)

@weekly-digest weekly-digest bot mentioned this pull request Jul 19, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@efiop efiop Awaiting requested review from efiop

Assignees

@casperdcl casperdcl

Labels
bug Did we break something? research
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@casperdcl @efiop