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

feat(redpanda): Add option to enable topic auto-creation #1360

Merged

Conversation

ladislavmacoun
Copy link
Contributor

What does this PR do?

Adds an option to enable topic auto-creation

Why is it important?

The option to enable automatic topic creation is particularly useful for testing environments, allowing users to produce data into topics without the need to create them beforehand.

@ladislavmacoun ladislavmacoun requested a review from a team as a code owner July 12, 2023 20:16
@netlify
Copy link

netlify bot commented Jul 12, 2023

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 621b475
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/64ca6ba017801d00074702bd
😎 Deploy Preview https://deploy-preview-1360--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ladislavmacoun ladislavmacoun force-pushed the feat/allow-auto-topic-creation branch 4 times, most recently from 963f3e9 to fb331cb Compare July 14, 2023 12:00
@eddumelendez
Copy link
Member

Thanks for your contribution! So far, I haven't been able to find any reference about .bootstrap.yml. can you share why it is needed in both places? I only found the redpanda.yml config but when enabled only there nothing happens. I would like to understand more about it.

@ladislavmacoun
Copy link
Contributor Author

Thanks for your contribution! So far, I haven't been able to find any reference about .bootstrap.yml. can you share why it is needed in both places? I only found the redpanda.yml config but when enabled only there nothing happens. I would like to understand more about it.

The bootstrapping mechanism is mentioned in the production deployment documentation, there is a reference in the code for bootstrap.yaml configuration.

My understanding of this is that the bootstrap.yaml is responsible for the configuration of the entire cluster, and it is only assessed at the time of the initial start-up. On the other hand, adjustments can be made to the redpanda.yaml config as needed.

The option to enable automatic topic creation is particularly useful for testing
environments, allowing users to produce data into topics without the need to
create them beforehand.

Signed-off-by: Ladislav Macoun <ladislavmacoun@gmail.com>
Signed-off-by: Ladislav Macoun <ladislavmacoun@gmail.com>
@sonarcloud
Copy link

sonarcloud bot commented Aug 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Collaborator

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

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

This PR LGTM! Although I'd like to postpone the final approval to @eddumelendez, who had some other concerns.

Thanks for your time and patience!

@mdelapenya mdelapenya self-assigned this Aug 2, 2023
@mdelapenya mdelapenya added the enhancement New feature or request label Aug 2, 2023
@mdelapenya mdelapenya merged commit 14d6569 into testcontainers:main Aug 2, 2023
59 checks passed
@mdelapenya
Copy link
Collaborator

Great work! 💪

@ladislavmacoun ladislavmacoun deleted the feat/allow-auto-topic-creation branch August 2, 2023 16:45
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Aug 3, 2023
* main:
  chore(deps): bump github.com/hashicorp/consul/api in /examples/consul (testcontainers#1381)
  chore(deps): bump github.com/twmb/franz-go in /modules/redpanda (testcontainers#1387)
  chore(deps): bump github.com/nats-io/nats.go in /examples/nats (testcontainers#1389)
  feat(redpanda): Add option to enable topic auto-creation (testcontainers#1360)
  chore(deps): bump golang.org/x/text from 0.9.0 to 0.11.0 in /modulegen (testcontainers#1388)
  chore(deps): bump google.golang.org/api from 0.125.0 to 0.134.0 in /examples (testcontainers#1412)
  chore(deps): bump golang.org/x/mod and github.com/aws/aws-sdk-go-v2/config in /modules/localstack (testcontainers#1410)
  test: fix TestGetDockerConfig (testcontainers#1413)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants