Skip to content

Conversation

taisho6339
Copy link
Contributor

@taisho6339 taisho6339 commented Aug 26, 2025

Summary

I've implemented OpenSearch integration tests for verifying OpenSearch compatibility.

Vector configuration

How did you test this PR?

vdev int test elasticsearch

Change Type

  • Bug fix
  • New feature
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

closes: #16761

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • cargo fmt --all
      • cargo clippy --workspace --all-targets -- -D warnings
      • cargo nextest run --workspace (alternatively, you can run cargo test --all)
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run cargo vdev build licenses to regenerate the license inventory and commit the changes (if any). More details here.

@github-actions github-actions bot added the domain: ci Anything related to Vector's CI environment label Aug 26, 2025
@taisho6339 taisho6339 marked this pull request as ready for review August 26, 2025 10:39
@taisho6339 taisho6339 requested a review from a team as a code owner August 26, 2025 10:39
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Aug 26, 2025
Copy link
Member

@pront pront left a comment

Choose a reason for hiding this comment

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

Nice!

@pront pront enabled auto-merge August 26, 2025 15:58
@pront pront added this pull request to the merge queue Aug 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 26, 2025
@thomasqueirozb
Copy link
Contributor

Some integration tests failed in the merge queue unfortunately 😕

@thomasqueirozb thomasqueirozb added sink: elasticsearch Anything `elasticsearch` sink related meta: awaiting author Pull requests that are awaiting their author. labels Aug 29, 2025
@taisho6339 taisho6339 marked this pull request as draft August 31, 2025 03:27
@github-actions github-actions bot removed the meta: awaiting author Pull requests that are awaiting their author. label Aug 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: ci Anything related to Vector's CI environment no-changelog Changes in this PR do not need user-facing explanations in the release changelog sink: elasticsearch Anything `elasticsearch` sink related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document and integration test OpenSearch support
3 participants