Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Beats CM now handle the new response format from Kibana. #11377

Merged
merged 2 commits into from
Mar 28, 2019

Conversation

ph
Copy link
Contributor

@ph ph commented Mar 21, 2019

Format is defined in elastic/kibana#27408
Main objective is to easier bubbling or errors and messages.

export interface ReturnType {
  error?: {
    message: string;
    code?: number;
  };
  success: boolean;
}

INTEGRATION_TESTS = os.environ.get('INTEGRATION_TESTS', False)
# Disable because waiting artifacts from https://github.com/elastic/kibana/pull/31660
# INTEGRATION_TESTS = os.environ.get('INTEGRATION_TESTS', False)
INTEGRATION_TESTS = False
Copy link
Contributor Author

@ph ph Mar 21, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

chicken-egg, we need snapshots of Kibana and ES to have the integration suite working. So I am disabling it.

Format is defined in elastic/kibana#27408
Main objective is to easier bubbling or errors and messages.

```
export interface ReturnType {
  error?: {
    message: string;
    code?: number;
  };
  success: boolean;
}

```
@ph ph force-pushed the cm/better-error-resposne-from-kibana branch from ccc8f67 to a1784cb Compare March 21, 2019 19:04
Error ErrorResponse `json:"error,omitempty"`
}

// ErrorResponse contains human reable and machine readable information when an error happen.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace with ErrorResponse contains human readable and machine readable information when an error happens.

@ph
Copy link
Contributor Author

ph commented Mar 27, 2019

rebased+typo, waiting on green to merge.

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

Successfully merging this pull request may close these issues.

2 participants