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

Re-enable copy_to into elastic/security #667

Merged

Conversation

salvatore-campagna
Copy link
Contributor

@salvatore-campagna salvatore-campagna commented Sep 17, 2024

copy_to is used to copy from kubernetes.event.message to message.
Now it is supported in Elasticsearch 8.15 and we can benchmark the security
track including it. We also remove a parameter which was used to run a modified
workflow, which was using kubernetes.event.message instead of message.

This needs backporting to 8.15

@salvatore-campagna salvatore-campagna self-assigned this Sep 17, 2024
@salvatore-campagna
Copy link
Contributor Author

@gareth-ellis if this is green I would like to use it to enable elastic/security in nightlies. I will need to update the other PR in the elasticsearch-benchmarks repo to add the appropriate configs.

@salvatore-campagna salvatore-campagna merged commit 0203b04 into elastic:master Sep 18, 2024
13 checks passed
salvatore-campagna added a commit to salvatore-campagna/rally-tracks that referenced this pull request Sep 18, 2024
`copy_to` is used to copy from `kubernetes.event.message` to `message`.
Now it is supported in Elasticsearch 8.15 and we can benchmark the security
track including it. We also remove a parameter which was used to run a modified
workflow, which was using `kubernetes.event.message` instead of `message`.
salvatore-campagna added a commit that referenced this pull request Sep 18, 2024
`copy_to` is used to copy from `kubernetes.event.message` to `message`.
Now it is supported in Elasticsearch 8.15 and we can benchmark the security
track including it. We also remove a parameter which was used to run a modified
workflow, which was using `kubernetes.event.message` instead of `message`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants