Return value of charm_pull is not checked in fetch_source

[johnsca]

models/other.py#L316

If charm pull fails, that is ignored and it tries to use the directory that no longer exists, resulting in a very confusing / misleading stack trace.