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

Supply a reason for "available images" being blank #861

Merged
merged 1 commit into from
Jan 24, 2018
Merged

Conversation

squaremo
Copy link
Member

@squaremo squaremo commented Dec 1, 2017

When flux has not managed to fetch the image metadata, it will supply
a null in the "available images" field. But usually, we can figure out
why there's no result -- it could be because we didn't have
credentials for the registry, or perhaps because we just haven't tried
to fetch them yet.

This commit effectively adds a field to the API return value to hold
that reason, and gives a placeholder reason in the case that available
images weren't found.

More work is needed to plumb through the actual reasons from the
registry. But it's probably worth waiting for #851 to be merged before
doing that.

@samb1729
Copy link
Contributor

samb1729 commented Jan 2, 2018

it's probably worth waiting for #851 to be merged

anything else worth waiting for?

When flux has not managed to fetch the image metadata, it will supply
a null in the "available images" field. But usually, we can figure out
why there's no result -- it could be because we didn't have
credentials for the registry, or perhaps because we just haven't tried
to fetch them yet.

This commit effectively adds a field to the API return value to hold
that reason, and gives a placeholder reason in the case that available
images weren't found. More work is needed to plumb through the
_actual_ reasons from the registry.
@squaremo
Copy link
Member Author

squaremo commented Jan 2, 2018

it's probably worth waiting for #851 to be merged

anything else worth waiting for?

Nope; but, after this is merged, we can try to plumb through some more informative errors.

@squaremo squaremo requested a review from samb1729 January 23, 2018 12:50
@squaremo squaremo merged commit 9ce750a into master Jan 24, 2018
@squaremo squaremo deleted the images-error branch January 24, 2018 17:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants