Skip to content

rename "pipeline_id" param to "pipeline"#15618

Merged
talevy merged 1 commit intoelastic:feature/ingestfrom
talevy:rename_pipeline_param
Dec 22, 2015
Merged

rename "pipeline_id" param to "pipeline"#15618
talevy merged 1 commit intoelastic:feature/ingestfrom
talevy:rename_pipeline_param

Conversation

@talevy
Copy link
Copy Markdown
Contributor

@talevy talevy commented Dec 22, 2015

No description provided.

@talevy talevy added :Distributed/Ingest Node Execution or management of Ingest Pipelines review labels Dec 22, 2015
@martijnvg
Copy link
Copy Markdown
Member

The rename also needs to be done in the ingest-with-mustache qa module. OTT LGTM

@talevy talevy force-pushed the rename_pipeline_param branch from d72f672 to 44d64c8 Compare December 22, 2015 20:30
@talevy
Copy link
Copy Markdown
Contributor Author

talevy commented Dec 22, 2015

updated the qa/ingest-with-metadata rest test with the change

@martijnvg
Copy link
Copy Markdown
Member

👍

talevy added a commit that referenced this pull request Dec 22, 2015
rename "pipeline_id" param to "pipeline"
@talevy talevy merged commit fe94593 into elastic:feature/ingest Dec 22, 2015
@talevy talevy deleted the rename_pipeline_param branch December 22, 2015 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Ingest Node Execution or management of Ingest Pipelines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants