Skip to content
This repository was archived by the owner on May 6, 2020. It is now read-only.

fix(scheduler): show more information when there is a HTTP error in Kubernetes#1041

Merged
helgi merged 1 commit intodeis:masterfrom
helgi:more_exception_info
Sep 10, 2016
Merged

fix(scheduler): show more information when there is a HTTP error in Kubernetes#1041
helgi merged 1 commit intodeis:masterfrom
helgi:more_exception_info

Conversation

@helgi
Copy link
Copy Markdown
Contributor

@helgi helgi commented Sep 7, 2016

With the response below the user would get the very unhelpful 422 Invalid - This extends that

{
    'code': 422,
    'reason': 'Invalid',
    'apiVersion': 'v1',
    'metadata': {},
    'kind': 'Status',
    'details': {
        'kind': 'Service',
        'causes': [
            {'field': 'spec.clusterIP', 'reason': 'FieldValueInvalid', 'message': "invalid value '', Details: range is full"}
        ],
        'name': 'test-760092409'
    },
    'message': 'Service "test-760092409" is invalid: spec.clusterIP: invalid value \'\', Details: range is full',
    'status': 'Failure'
}

@helgi helgi added this to the v2.6 milestone Sep 7, 2016
@helgi helgi self-assigned this Sep 7, 2016
@helgi helgi force-pushed the more_exception_info branch 2 times, most recently from 41530bc to 21259bd Compare September 7, 2016 18:17
@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 7, 2016

Current coverage is 86.87% (diff: 100%)

No coverage report found for master at 3e1f9d6.

Powered by Codecov. Last update 3e1f9d6...19bd7f5

@helgi helgi force-pushed the more_exception_info branch 2 times, most recently from d2e2c8f to 19bd7f5 Compare September 8, 2016 18:32
@helgi helgi force-pushed the more_exception_info branch from 19bd7f5 to 8f03fb3 Compare September 9, 2016 22:13
@kmala kmala added the LGTM1 label Sep 10, 2016
@helgi helgi merged commit 29d51d9 into deis:master Sep 10, 2016
@helgi helgi deleted the more_exception_info branch September 10, 2016 22:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants