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

Add Enrich Origin #48098

Merged

Conversation

jbaiera
Copy link
Member

@jbaiera jbaiera commented Oct 16, 2019

This PR adds an origin for the Enrich feature, and modifies the background maintenance task to use the origin when executing client operations. Without this fix, the maintenance task fails to execute when security is enabled.

@jbaiera jbaiera added >non-issue :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP labels Oct 16, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Ingest)

Copy link
Contributor

@hub-cap hub-cap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, this was my mistake. LGTM. I knew i was going to mess the origin (or lack thereof) stuff up.

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jbaiera
Copy link
Member Author

jbaiera commented Oct 16, 2019

@elasticmachine update branch

@jbaiera
Copy link
Member Author

jbaiera commented Oct 16, 2019

@elasticmachine run elasticsearch-ci/packaging-sample-matrix

@jbaiera
Copy link
Member Author

jbaiera commented Oct 17, 2019

@elasticmachine update branch

@jbaiera
Copy link
Member Author

jbaiera commented Oct 19, 2019

@elasticmachine update branch

@jbaiera jbaiera merged commit ef5755b into elastic:master Oct 21, 2019
@jbaiera jbaiera deleted the enrich-fix-maintenance-task-origin-error branch October 21, 2019 15:11
jbaiera added a commit to jbaiera/elasticsearch that referenced this pull request Oct 21, 2019
This PR adds an origin for the Enrich feature, and modifies the background 
maintenance task to use the origin when executing client operations. 
Without this fix, the maintenance task fails to execute when security is 
enabled.
jbaiera added a commit to jbaiera/elasticsearch that referenced this pull request Oct 21, 2019
This PR adds an origin for the Enrich feature, and modifies the background 
maintenance task to use the origin when executing client operations. 
Without this fix, the maintenance task fails to execute when security is 
enabled.
jbaiera added a commit that referenced this pull request Oct 21, 2019
This PR adds an origin for the Enrich feature, and modifies the background 
maintenance task to use the origin when executing client operations. 
Without this fix, the maintenance task fails to execute when security is 
enabled.
jbaiera added a commit that referenced this pull request Oct 21, 2019
This PR adds an origin for the Enrich feature, and modifies the background 
maintenance task to use the origin when executing client operations. 
Without this fix, the maintenance task fails to execute when security is 
enabled.
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Oct 22, 2019
* elastic/master:
  [Docs] Fix opType options in IndexRequest API example. (elastic#48290)
  Simplify Shard Snapshot Upload Code (elastic#48155)
  Mute ClassificationIT tests (elastic#48338)
  Reenable azure repository tests and remove some randomization in http servers  (elastic#48283)
  Use an env var for the classpath of jar hell task (elastic#48240)
  Refactor FIPS BootstrapChecks to simple checks (elastic#47499)
  Add "format" to "range" queries resulted from optimizing a logical AND (elastic#48073)
  [DOCS][Transform] document limitation regarding rolling upgrade with 7.2, 7.3 (elastic#48118)
  Fail with a better error when if there are no ingest nodes (elastic#48272)
  Fix executing enrich policies stats (elastic#48132)
  Use MultiFileTransfer in CCR remote recovery (elastic#44514)
  Make BytesReference an interface (elastic#48171)
  Also validate source index at put enrich policy time. (elastic#48254)
  Add 'javadoc' task to lifecycle check tasks (elastic#48214)
  Remove option to enable direct buffer pooling (elastic#47956)
  [DOCS] Add 'Selecting gateway and seed nodes' section to CCS docs (elastic#48297)
  Add Enrich Origin (elastic#48098)
  fix incorrect comparison (elastic#48208)
matriv added a commit to matriv/elasticsearch that referenced this pull request Oct 28, 2019
Seems that the issue has been fixed with: elastic#48098

Closes: elastic#48117
matriv added a commit that referenced this pull request Oct 28, 2019
Seems that the issue has been fixed with: #48098

Closes: #48117
matriv added a commit that referenced this pull request Oct 28, 2019
Seems that the issue has been fixed with: #48098

Closes: #48117
(cherry picked from commit 4703623)
matriv added a commit that referenced this pull request Oct 28, 2019
Seems that the issue has been fixed with: #48098

Closes: #48117
(cherry picked from commit 4703623)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >non-issue v7.5.0 v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants