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

test: data injection flake #2361

Merged
merged 13 commits into from Mar 18, 2024
Merged

test: data injection flake #2361

merged 13 commits into from Mar 18, 2024

Conversation

lucasrod16
Copy link
Member

@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

Copy link

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
Member Author

Still seeing the flake. More investigation and experimentation needed

https://github.com/defenseunicorns/zarf/actions/runs/8175786653/job/22353838067?pr=2361#step:8:5656

…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
@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
24 checks passed
@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
Status: Done
Status: Approved
Development

Successfully merging this pull request may close these issues.

fix: data injection test
4 participants