generated from kyma-project/template-repository
-
Notifications
You must be signed in to change notification settings - Fork 31
test: refactor selfmonitor tests with shared helpers and HTTP 400/429 fault injection #3132
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
Open
jeffreylimnardy
wants to merge
88
commits into
kyma-project:main
Choose a base branch
from
jeffreylimnardy:virtual-service-return-500
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
661676e
set virtual service to return non retryable error (500)
jeffreylimnardy 6ba6f51
lower fault injection since now retries are not happening
jeffreylimnardy 48af641
Merge branch 'main' into virtual-service-return-500
jeffreylimnardy a3f3526
increase fault injection again to 85
jeffreylimnardy 3b3bc2c
Merge branch 'main' into virtual-service-return-500
jeffreylimnardy 00d9f54
try debugging with tmate
jeffreylimnardy e4d871e
detach
jeffreylimnardy d233392
Merge branch 'main' into virtual-service-return-500
jeffreylimnardy 3d35faa
lower fault injection again to make tests flaky
jeffreylimnardy 95b43b0
Merge branch 'main' into virtual-service-return-500
jeffreylimnardy 5079112
remove tmate
jeffreylimnardy 8093303
debug self monitor on failure
jeffreylimnardy 34ea29e
dont use canceled context
jeffreylimnardy 2c9bc3f
move
jeffreylimnardy 60f3576
Merge branch 'main' into virtual-service-return-500
k15r b55805a
improve tests for istio
k15r 48174a8
in progress
k15r 9432dd3
simplify the selfmonitor tests
k15r 096f207
test
k15r 8576297
hostpath cleanup
k15r 2653a56
add cleanup
k15r 6a5ebb4
ds cleanup
k15r f4e7a6b
another try
k15r b33622e
Merge branch 'main' into virtual-service-return-500
k15r 0090ecc
move fluentbit cleanup to setuptest
k15r 1d59311
create virtual services before generators and backends
k15r 4870ada
increase timeout as 5min rate window can cause more than 10 mins for the
k15r 4cebea2
implement code review comments
k15r 3b8a121
Address review: buffer alert ops note, VS chain, drop unused delay fault
k15r 26e5943
chore: extract fluent-bit buffer alert changes to separate PR
k15r 6bb4138
Merge branch 'main' into virtual-service-return-500
k15r e7839b6
Address review: rename backendScaledToZero, add default panics, clari…
k15r 3587481
Address review: add default panic to switch, clarify comments, fix cl…
k15r 8b5e77d
feat: replace Istio fault injection with lightweight mock-backend
k15r 31f5090
chore: remove accidentally committed mock-backend binary
k15r 955cf36
use mock-backend
k15r 34329dc
Merge remote-tracking branch 'upstream/main' into virtual-service-ret…
k15r 562a9dc
add support for delays in mock-backend
k15r e47bad0
Address review: improve comments on fault injection semantics and sta…
k15r a05dbf3
Merge branch 'main' into virtual-service-return-500
k15r f557691
use the correct mock-backend
k15r 7834f15
make generate
k15r dbc239f
Address review: replace busybox:1.36 with mirrored alpine, normalize …
k15r 8825f04
add make targets
k15r ab4b4b1
Address review: add useIstio path for metric-agent, fix backend.go co…
k15r 589bc81
fix: clarify Istio sourceLabels semantics in VirtualService builder
k15r 1bfe8e1
Merge branch 'main' into virtual-service-return-500
k15r b94d959
feat(selfmonitor): implement runtime FaultEnabler pattern for self-mo…
k15r 5ac9e96
test(selfmonitor): log when faults are enabled via VirtualService
k15r 307df69
chore: use :dev tag for mock-backend image to prevent always-pull wit…
k15r 64e85da
chore: use :main tag for mock-backend image to match other image conv…
k15r 40a368b
test(selfmonitor): remove redundant healthy check in assertFlowDegrad…
k15r 4419646
test(selfmonitor): drop FlowHealthy(true) from transition sequence — …
k15r e687a05
test(selfmonitor): rename flowHealthyThenDegraded to degradedReasons
k15r 42b2e5d
Merge branch 'main' into virtual-service-return-500
k15r a6db751
Merge upstream/main into virtual-service-return-500
k15r 43eb07f
chore: rename mock-backend to fault-backend in testkit and self-monit…
k15r 5b44b44
increase rate
k15r 0b6205c
skip healthy baseline for outage tests where alert fires from boot
k15r 950f92b
make alertConditionDescription component-aware for Fluent Bit
k15r b9f966b
use gRPC status fault for metric-agent Istio VS to fix missing send_f…
k15r 49ed72e
decrease the rate while querying the Selfmonitor tests
k15r 581f574
Merge branch 'main' into virtual-service-return-500
k15r 72b15c7
wait for FlowHealthy=True before enabling faults in backpressure tests
k15r bbac34f
add missing SomeTelemetryDataDropped descriptions to alertConditionDe…
k15r c4b97b4
wait for Prometheus rate metrics to be non-zero before enabling faults
k15r 6039d89
log rate baseline query results in assertSelfMonitorRateNonZero
k15r f12c5e5
configure selfmonitor tests to use 1 gateway replica
k15r ff966b3
Merge branch 'main' into virtual-service-return-500
k15r ab30a79
fix lint in helpers.go
k15r c7cc184
log selfmonitor scrape targets in eventually debug output
k15r ec08629
improve rate baseline log messages to distinguish no-data from query …
k15r cf572f8
log selfmonitor targets in assertSelfMonitorRateNonZero eventually block
k15r a4cbf07
add separator line before rate baseline check output
k15r 2d52176
increase rate baseline timeout to 5 minutes
k15r c601246
set kubernetes SD refresh_interval to 1 minute in selfmonitor scrape …
k15r 2d85a50
revert refresh_interval: not a valid field in kubernetes_sd_configs
k15r b77beda
increase k3d server memory to 4g
k15r e3ffd8b
revert k3d server memory: no limit is the default (adding one would c…
k15r 6262d46
fix selfmonitor target discovery: use endpointslice role for Promethe…
k15r d8391f5
log dropped targets in selfmonitor target output
k15r 2409312
revert endpointslice back to endpoints role in selfmonitor scrape config
k15r 336f61d
add workload status logging to selfmonitor debug output
k15r ecbf6d1
log k8s endpoints in selfmonitor rate baseline loop
k15r 35ce474
bump selfmonitor resource limits
k15r 0de0f57
fix selfmonitor scrape: use endpointslice role with correct service l…
k15r 7a5317b
grant manager endpointslices RBAC to allow self-monitor role creation
k15r 5238480
add endpointslices RBAC to self-monitor role
k15r File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we establish documentation guidelines simply as a claude skill?