Reimplement experimental load-balancing tests in scripttest #35480
Conversation
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
|
/test |
014fa44 to
2e8198e
Compare
joamaki
commented
Oct 22, 2024
joamaki
commented
Oct 22, 2024
|
/test |
2e8198e to
378f242
Compare
This comment was marked as resolved.
This comment was marked as resolved.
378f242 to
b1024a5
Compare
This comment was marked as resolved.
This comment was marked as resolved.
|
/test |
joamaki
commented
Oct 23, 2024
|
/cc @DamianSawicki |
Jussi already mentioned this to me on Slack, but thanks for remembering to let me know @ovidiutirla! I'll try to have a look. |
nebril
approved these changes
Oct 24, 2024
b1024a5 to
f655dbe
Compare
f655dbe to
70554d7
Compare
The API resource definitions used by test/controlplane for detecting capabilities are usable in other contexts. Move them to the testutils package. Signed-off-by: Jussi Maki <jussi@isovalent.com>
70554d7 to
7c83fc7
Compare
Add the "k8s" script command for interacting with the client-go fake clientset. Example usage in txtar: k8s add foo.yaml bar.yaml k8s update foo2.yaml k8s delete foo2.yaml bar.yaml -- foo.yaml -- apiVersion: v1 ... Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
As things are maturing, switch most of the logging from Info to Debug. Remove the unused logger from Writer. Signed-off-by: Jussi Maki <jussi@isovalent.com>
Since we don't need to retry Resource[T] events in the reflector, mark the events as handled right away. This way the reflector can be stopped at any point with unhandled objects in the buffer without causing the event finalizer to panic due to uncalled Done(). Signed-off-by: Jussi Maki <jussi@isovalent.com>
Add scripttest runner Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Signed-off-by: Jussi Maki <jussi@isovalent.com>
Test for checking that unexpected BPF map contents are pruned when tables are empty. Signed-off-by: Jussi Maki <jussi@isovalent.com>
Now that tests have been converted to scripttest drop the old test suite. Signed-off-by: Jussi Maki <jussi@isovalent.com>
7c83fc7 to
bc7ab64
Compare
|
/test |
tklauser
approved these changes
Oct 29, 2024
dylandreimerink
approved these changes
Oct 29, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds a "k8s" script command and reimplements the k8s integration test-suite in
pkg/loadbalancer/experimentalas script tests.StateDB is bumped to v0.3.2 to fix the column ordering bug.