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

[Reporting] Download report: add debug logging around sending of 5xx codes #183852

Merged

Conversation

tsullivan
Copy link
Member

Summary

These logs help correlate errors one may see from investigating HTTP 5xx status codes that are sent in response to the Download Report API.

Related to: #183846

@tsullivan tsullivan added release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) labels May 20, 2024
@tsullivan tsullivan marked this pull request as ready for review May 20, 2024 16:49
@tsullivan tsullivan requested a review from a team as a code owner May 20, 2024 16:49
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@rshen91
Copy link
Contributor

rshen91 commented May 20, 2024

 proc [kibana] [2024-05-20T14:21:23.327-06:00][DEBUG][plugins.reporting.download-report] Report job 2e8e9914-cc44-45e2-ba77-3ac0f01328f1 is processing. Sending statusCode: 503 {"service":{"node":{"roles":["background_tasks","ui"]}}}
 proc [kibana] [2024-05-20T14:21:23.883-06:00][DEBUG][plugins.reporting.download-report] Report job 2e8e9914-cc44-45e2-ba77-3ac0f01328f1 is processing. Sending statusCode: 503 {"service":{"node":{"roles":["background_tasks","ui"]}}}
 proc [kibana] [2024-05-20T14:21:24.426-06:00][DEBUG][plugins.reporting.download-report] Report job 2e8e9914-cc44-45e2-ba77-3ac0f01328f1 is processing. Sending statusCode: 503 {"service":{"node":{"roles":["background_tasks","ui"]}}}
 proc [kibana] [2024-05-20T14:21:24.981-06:00][DEBUG][plugins.reporting.download-report] Report job 2e8e9914-cc44-45e2-ba77-3ac0f01328f1 is processing. Sending statusCode: 503 {"service":{"node":{"roles":["background_tasks","ui"]}}}
 proc [kibana] [2024-05-20T14:21:25.518-06:00][DEBUG][plugins.reporting.download-report] Report job 2e8e9914-cc44-45e2-ba77-3ac0f01328f1 is processing. Sending statusCode: 503 {"service":{"node":{"roles":["background_tasks","ui"]}}}
 proc [kibana] [2024-05-20T14:21:26.055-06:00][DEBUG][plugins.reporting.download-report] Report job 2e8e9914-cc44-45e2-ba77-3ac0f01328f1 is processing. Sending statusCode: 503 {"service":{"node":{"roles":["background_tasks","ui"]}}}```
export const reportingDebugLogger = {
    name: 'plugins.reporting',
    level: 'debug',
    appenders: ['console'],
  };

export default createTestConfig({
  serverlessProject: 'es',
  testFiles: [require.resolve('.')],
  junit: {
    reportName: 'Serverless Search API Integration Tests',
  },
  suiteTags: { exclude: ['skipSvlSearch'] },

  // include settings from project controller
  // https://github.com/elastic/project-controller/blob/main/internal/project/esproject/config/elasticsearch.yml
  esServerArgs: [],
  kbnServerArgs: [`--logging.loggers=${JSON.stringify([reportingDebugLogger])}`],
});

@tsullivan tsullivan merged commit 105baf4 into elastic:main May 20, 2024
18 checks passed
@tsullivan tsullivan deleted the reporting/download-api-status-loggers branch May 20, 2024 20:29
@kibanamachine kibanamachine added v8.15.0 backport:skip This commit does not require backporting labels May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants