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

[system test] [doc] bridge package #10225

Merged
merged 20 commits into from
Sep 5, 2024
Merged

Conversation

see-quick
Copy link
Member

@see-quick see-quick commented Jun 13, 2024

Type of change

  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

This PR adds documentation for the Bridge package in our tests. I would like to split this into more minor PRs for easier reviews, rather than opening one big PR for all Suites within STs.

Checklist

  • Make sure all tests pass
  • Update documentation

@see-quick see-quick added this to the 0.42.0 milestone Jun 13, 2024
@see-quick see-quick self-assigned this Jun 13, 2024
@see-quick see-quick requested a review from a team June 13, 2024 13:33
systemtest/pom.xml Outdated Show resolved Hide resolved
systemtest/pom.xml Show resolved Hide resolved
systemtest/pom.xml Outdated Show resolved Hide resolved
@see-quick see-quick marked this pull request as draft June 28, 2024 08:04
@scholzj scholzj modified the milestones: 0.42.0, 0.43.0 Jul 1, 2024
@see-quick see-quick marked this pull request as ready for review July 25, 2024 11:12
development-docs/systemtests/labels/annotation.md Outdated Show resolved Hide resolved
development-docs/systemtests/labels/annotation.md Outdated Show resolved Hide resolved
development-docs/systemtests/labels/configuration.md Outdated Show resolved Hide resolved
development-docs/systemtests/labels/label.md Outdated Show resolved Hide resolved
development-docs/systemtests/labels/plaintext.md Outdated Show resolved Hide resolved
@scholzj scholzj modified the milestones: 0.43.0, 0.44.0 Aug 19, 2024
@see-quick see-quick marked this pull request as draft August 22, 2024 08:13
@see-quick see-quick marked this pull request as ready for review September 4, 2024 08:17
Copy link
Member

@Frawless Frawless left a comment

Choose a reason for hiding this comment

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

That's exactly how I thought it will looks like. LGTM.

Signed-off-by: see-quick <[email protected]>

# Conflicts:
#	systemtest/src/test/java/io/strimzi/systemtest/kafka/KafkaNodePoolST.java

# Conflicts:
#	systemtest/pom.xml
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
see-quick and others added 16 commits September 4, 2024 13:35
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>

# Conflicts:
#	development-docs/systemtests/labels/configuration.md
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Signed-off-by: see-quick <[email protected]>
Copy link
Member

@scholzj scholzj left a comment

Choose a reason for hiding this comment

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

Nice work. Thanks.

@see-quick see-quick added the ready for merge Label for PRs which are ready for merge label Sep 5, 2024
@scholzj scholzj merged commit 09680cd into strimzi:main Sep 5, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge Label for PRs which are ready for merge System tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants