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

patch: fix failing webpa-common tests #1055

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

denopink
Copy link
Contributor

@denopink denopink commented Oct 21, 2024

  • fix failing webpa-common tests

@denopink denopink added the bug label Oct 21, 2024
@denopink denopink self-assigned this Oct 21, 2024
Copy link

guardrails bot commented Oct 21, 2024

⚠️ We detected 1 security issue in this pull request:

Hard-Coded Secrets (1)
Severity Details Docs
Medium Title: Github Key
uses: xmidt-org/shared-go/.github/workflows/ci.yml@3d552804b9a20638508bf359cdd2e9b46bc06879 # v4.4.24
📚

More info on how to fix Hard-Coded Secrets in General.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

@denopink denopink force-pushed the denopink/patch/ci-and-device_drain_tests branch from 137a120 to 7903139 Compare October 21, 2024 17:24
- `TestDrainer` tests weren't well made, they led to a race-like condition where a slow running machine will fail some draining tests
- fix metric registry_test.go tests (add default debugger logger)
@denopink denopink force-pushed the denopink/patch/ci-and-device_drain_tests branch from 7903139 to fc0d191 Compare October 21, 2024 17:43
@denopink denopink marked this pull request as draft October 22, 2024 18:06
@denopink denopink changed the title patch: update ci and fix TestDrainers tests patch: fix failing webpa-common tests Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant