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

[Cloud Security] fix cloud defend error logs #171278

Merged

Conversation

CohenIdo
Copy link
Contributor

solves https://github.com/elastic/security-team/issues/7870

Summary

The Cloud Defend metering data task uses an explicit index name, unlike other metering tasks in cloud security products that utilize an index pattern. This divergence causes the Cloud Defend metering task to throw an error when the index does not exist. This pull request addresses and resolves this issue.

@CohenIdo CohenIdo added release_note:skip Skip the PR/issue when compiling release notes 8.12 candidate labels Nov 15, 2023
@CohenIdo CohenIdo marked this pull request as ready for review November 15, 2023 11:13
@CohenIdo CohenIdo requested review from a team as code owners November 15, 2023 11:13
@CohenIdo CohenIdo marked this pull request as draft November 15, 2023 11:14
@CohenIdo CohenIdo marked this pull request as ready for review November 15, 2023 11:17
@CohenIdo CohenIdo added the backport:skip This commit does not require backporting label Nov 15, 2023
@CohenIdo
Copy link
Contributor Author

@elasticmachine merge upstream

@CohenIdo CohenIdo force-pushed the update-cloud-defend-serverless-metering branch from 76fd4d4 to 2b8037d Compare November 15, 2023 11:36
@CohenIdo
Copy link
Contributor Author

@elasticmachine merge upstream

@CohenIdo
Copy link
Contributor Author

@elasticmachine merge upstream

@CohenIdo
Copy link
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Security Solution Cypress Tests #9 / Changing alert status Closing alerts can close an alert can close an alert

Metrics [docs]

✅ unchanged

History

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

@CohenIdo
Copy link
Contributor Author

pinging @elastic/security-solution

@CohenIdo CohenIdo merged commit 8f91b39 into elastic:main Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.12 candidate backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants