-
Notifications
You must be signed in to change notification settings - Fork 0
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
Remove index template from backend #111
Conversation
Hey @CohenIdo, please reference to the related PR on |
@@ -5,7 +5,7 @@ | |||
* 2.0. | |||
*/ | |||
|
|||
export const CSP_KUBEBEAT_INDEX_PATTERN = 'logs-k8s_cis*'; | |||
export const CSP_KUBEBEAT_INDEX_PATTERN = 'logs-k8s_cis.result*'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this change necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nope. just thought that is more accurarte. want me to revert it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets put this on hold for now until we merge our code to upstream Kibana, we need to have an easy setup, and since we don't have an integration working yet, lets keep it in Kibana server for now
a2c9478
to
c4fd592
Compare
@CohenIdo Please rebase this to solve conflicts and we'll merge this in |
c4fd592
to
2cc3e27
Compare
*/ | ||
import type { MappingTypeMapping } from '@elastic/elasticsearch/lib/api/types'; | ||
|
||
export const mapping: MappingTypeMapping = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI the current findings needs this to do search/sort @kfirpeled, i think it will either not work or may throw now
No description provided.