Code review comment for lp://staging/~pwlars/ubuntu-ci-services-itself/imagebuild-keystone-auth-fix

Revision history for this message
Vincent Ladeuil (vila) wrote :

Yeah for robustness !

Nit (no need to fix that for now):

18 + raise ImageStoreException(
19 + 'No valid glance endpoints found for the specified region. '
20 + 'Check that the cloud credentials specified are valid')

Expanding 'endpoints' and 'region' could help diagnosis.

We know there is no glance endpoints, but having the existing endpoints (even empty) could give us some hints.

review: Approve

« Back to merge proposal