Skip to content

Fix Kafka tests to not use dockerhub#7094

Merged
eerhardt merged 1 commit intomicrosoft:mainfrom
eerhardt:FixKafkaDockerTests
Jan 14, 2025
Merged

Fix Kafka tests to not use dockerhub#7094
eerhardt merged 1 commit intomicrosoft:mainfrom
eerhardt:FixKafkaDockerTests

Conversation

@eerhardt
Copy link
Copy Markdown
Member

Dockerhub has a rate limit so we can't use it in CI.

Dockerhub has a rate limit so we can't use it in CI.
@eerhardt eerhardt changed the title Fix Kafka to not use dockerhub Fix Kafka tests to not use dockerhub Jan 14, 2025
@eerhardt eerhardt requested a review from joperezr January 14, 2025 00:27
@eerhardt eerhardt enabled auto-merge (squash) January 14, 2025 00:27
: host.Services.GetRequiredService<IProducer<string, string>>())
{
for (int i = 0; i < 100; i++)
for (int i = 0; i < 5; i++)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The tests were taking 45 seconds each on my machine. I thought making this less would take the time away, but the time appears to be waiting for the first "Consume" to work.


  Standard Output: 
1/14/2025 12:24:21 AM: produced message 0
1/14/2025 12:24:21 AM: produced message 1
1/14/2025 12:24:21 AM: produced message 2
1/14/2025 12:24:21 AM: produced message 3
1/14/2025 12:24:21 AM: produced message 4
1/14/2025 12:24:21 AM: before flush
1/14/2025 12:24:22 AM: after flush
1/14/2025 12:24:22 AM: after Subscribe
1/14/2025 12:25:05 AM: consumed message 0
1/14/2025 12:25:05 AM: consumed message 1
1/14/2025 12:25:05 AM: consumed message 2
1/14/2025 12:25:05 AM: consumed message 3
1/14/2025 12:25:05 AM: consumed message 4

@eerhardt
Copy link
Copy Markdown
Member Author

Tests passed on Linux Helix runs. Merging to unblock CI.

@eerhardt eerhardt disabled auto-merge January 14, 2025 01:09
@eerhardt eerhardt merged commit 2110899 into microsoft:main Jan 14, 2025
@eerhardt eerhardt deleted the FixKafkaDockerTests branch January 14, 2025 01:09
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 13, 2025
@github-actions github-actions Bot added the area-integrations Issues pertaining to Aspire Integrations packages label Mar 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-integrations Issues pertaining to Aspire Integrations packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants