-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[kbn/es-archiver] move to a package #72318
[kbn/es-archiver] move to a package #72318
Conversation
42422fe
to
6826eec
Compare
* under the License. | ||
*/ | ||
|
||
export * from '../../../../src/legacy/utils/streams'; |
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.
This is temporary until I can move esArchiver off of node streams.
Pinging @elastic/kibana-operations (Team:Operations) |
@elasticmachine merge upstream |
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
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.
Code LGTM. I successfully verified save
, load
and --help
flag.
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.
LGTM
Co-authored-by: spalger <spalger@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> # Conflicts: # .github/CODEOWNERS # src/dev/code_coverage/ingest_coverage/team_assignment/ingestion_pipeline_painless.json
Co-authored-by: spalger <spalger@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> # Conflicts: # .github/CODEOWNERS # src/dev/code_coverage/ingest_coverage/team_assignment/ingestion_pipeline_painless.json
Co-authored-by: spalger <spalger@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> # Conflicts: # .github/CODEOWNERS # src/dev/code_coverage/ingest_coverage/team_assignment/ingestion_pipeline_painless.json
As a part of moving the es-archiver to a more modern place I'd like to move the es_archiver logic to a package that is isolated from the rest of Kibana.
This PR uses the new
RunWithCommands
helper from@kbn/dev-utils
added in #72311