Skip to content

test: data injection flake#2361

Merged
Noxsios merged 13 commits intomainfrom
fix-data-injection-test
Mar 18, 2024
Merged

test: data injection flake#2361
Noxsios merged 13 commits intomainfrom
fix-data-injection-test

Conversation

@lucasrod16
Copy link
Copy Markdown
Contributor

@lucasrod16 lucasrod16 commented Mar 6, 2024

Description

fixes data injection test flake

Related Issue

Fixes #2360

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 6, 2024

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit a7d0762
🔍 Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/65f87c5b85e5f400088e4bbf

@lucasrod16
Copy link
Copy Markdown
Contributor Author

Lucas Rodriguez added 4 commits March 6, 2024 12:10
…g pod

Using the --selector=app=kiwix-serve flag to fetch logs sometimes results in
the query trying to fetch logs from a pod that is terminating, but still
matches the selector.

This commit also changes k8s.GetPods() to  allow callers to use more customizable
query options.
@lucasrod16 lucasrod16 marked this pull request as ready for review March 6, 2024 21:37
Comment thread src/test/e2e/23_data_injection_test.go Outdated
@Noxsios Noxsios changed the title fix: data injection test flake test: data injection flake Mar 18, 2024
@Noxsios Noxsios merged commit 30ffc14 into main Mar 18, 2024
@Noxsios Noxsios deleted the fix-data-injection-test branch March 18, 2024 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: data injection test

4 participants