Skip to content

filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256#48026

Merged
mauri870 merged 7 commits intoelastic:mainfrom
mauri870:kafka-sasl-plain-test
Dec 29, 2025
Merged

filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256#48026
mauri870 merged 7 commits intoelastic:mainfrom
mauri870:kafka-sasl-plain-test

Conversation

@mauri870
Copy link
Copy Markdown
Member

@mauri870 mauri870 commented Dec 10, 2025 •
edited
Loading

Proposed commit message

Our SASL integration tests for the Kafka input currently only cover SCRAM-SHA-512. Add tests for PLAIN and SCRAM-SHA-256 as well.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

How to test this PR locally

(cd filebeat && mage docker:composeBuild && mage docker:composeUp)
go test -tags integration -v -run ^TestSASLAuthentication$ ./filebeat/input/kafka -v -count=1
./script/stresstest.sh --tags integration ./filebeat/input/kafka ^TestSASLAuthentication$ -p 1
(cd filebeat && mage docker:composeDown)

@mauri870 mauri870 self-assigned this Dec 10, 2025
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Dec 10, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 10, 2025

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mauri870 mauri870 changed the title filebeat: add kafka input testcase for SASL PLAIN and SCRAM-SHA-256 filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 Dec 10, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Dec 10, 2025

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @mauri870? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@mauri870 mauri870 marked this pull request as ready for review December 10, 2025 16:18
@mauri870 mauri870 requested a review from a team as a code owner December 10, 2025 16:18
@mauri870 mauri870 requested review from AndersonQ and faec December 10, 2025 16:18
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Dec 10, 2025

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@mauri870 mauri870 added the backport-active-all Automated backport with mergify to all the active branches label Dec 10, 2025
@mauri870 mauri870 force-pushed the kafka-sasl-plain-test branch from b10e21a to 75c7841 Compare December 10, 2025 16:19
@mauri870 mauri870 merged commit 36d04b5 into elastic:main Dec 29, 2025
197 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 29, 2025

@Mergifyio backport 8.19 9.1 9.2 9.3

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Dec 29, 2025 •
edited
Loading

backport 8.19 9.1 9.2 9.3

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Dec 29, 2025
@mauri870 @mergify
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
d4a0979 
)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)
mergify bot pushed a commit that referenced this pull request Dec 29, 2025
@mauri870 @mergify
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
f6f29c9 
)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)
mergify bot pushed a commit that referenced this pull request Dec 29, 2025
@mauri870 @mergify
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
071d131 
)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)

# Conflicts:
#	filebeat/input/kafka/kafka_integration_test.go
mergify bot pushed a commit that referenced this pull request Dec 29, 2025
@mauri870 @mergify
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
0fb89e0 
)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)
This was referenced Dec 29, 2025
Merged
Merged
Merged
Merged
mauri870 added a commit that referenced this pull request Dec 29, 2025
@mergify @mauri870
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
791774b 
) (#48266)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
mauri870 added a commit that referenced this pull request Dec 30, 2025
@mergify @mauri870
[8.19](backport #48026) filebeat: add kafka input tests for SASL PLAI…
1c53f3d 
…N and SCRAM-SHA-256 (#48264)

* filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)

# Conflicts:
#	filebeat/input/kafka/kafka_integration_test.go

* fix conflicts

---------

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
mauri870 added a commit that referenced this pull request Dec 30, 2025
@mergify @mauri870
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
3d4a974 
) (#48263)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
mauri870 added a commit that referenced this pull request Dec 30, 2025
@mergify @mauri870
filebeat: add kafka input tests for SASL PLAIN and SCRAM-SHA-256 (#48026
95b86b2 
) (#48265)

* filebeat: add testcase for SASL PLAIN and SCRAM-SHA-256 in kafka input

* fix linter errors

(cherry picked from commit 36d04b5)

Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
@rdner rdner mentioned this pull request Feb 10, 2026
Merged
This was referenced Feb 26, 2026
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndersonQ AndersonQ AndersonQ approved these changes

@faec faec Awaiting requested review from faec faec is a code owner automatically assigned from elastic/elastic-agent-data-plane

Assignees

@mauri870 mauri870

Labels

backport-active-all Automated backport with mergify to all the active branches cleanup skip-changelog Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mauri870 @elasticmachine @AndersonQ