-
Notifications
You must be signed in to change notification settings - Fork 429
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
Copy ingest-pipeline to ingest_pipeline #153
Conversation
For now, both exist. As soon as Kibana is updated, the old one can be removed. This PR is only in draft yet, as it also requires the registry to be updated to elastic/package-registry#581 first.
PR updated as elastic/package-registry#581 was merged. |
I think we need to inform @jonathan-buttner about such changes. |
@mtojek Thanks for the reminder. Yes @jonathan-buttner you also need to update your repo. |
|
||
func main() { | ||
// Find all directories with ingest-pipeline | ||
ingestPipelineDirs, _ := filepath.Glob("./*/dataset/*/elasticsearch/ingest-pi*") |
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.
@ruflin just noticed this file: I don't think it should be versioned. Is that on purpose?
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.
@mtojek No, I forgot to remove it. I also need to take it out on the storage. On the storage layer I thought to keep it in for when I remove the old directories and then directly remove it. We could do the same here or remove it directly.
For now, both exist. As soon as Kibana is updated, the old one can be removed. This PR is only in draft yet, as it also requires the registry to be updated to elastic/package-registry#581 first.
For now, both exist. As soon as Kibana is updated, the old one can be removed.