Remove resource size check. #6572

Merged
merged 1 commit into from Nov 17, 2016

Conversation

Projects
None yet
3 participants
Contributor

natefinch commented Nov 17, 2016

We can't be guaranteed to get content-length from the server, so it's
totally reasonable that we might not know the size. We do get the hash,
which is a better check anyway, so just ignore size.

QA Steps

  1. juju deploy cs:~natefinch/starsay
  2. Ensure it gets deployed fully
    (if the bug is not fixed, it'll never get past the install step)
Remove resource size check.
We can't be guaranteed to get content-length from the server, so it's
totally reasonable that we might not know the size. We do get the hash,
which is a better check anyway, so just ignore size.
Contributor

natefinch commented Nov 17, 2016

$$merge$$

Contributor

jujubot commented Nov 17, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 1c2833b into juju:develop Nov 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment