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

DM-40691: Add watch support for namespaces #205

Merged
merged 1 commit into from
Sep 12, 2023
Merged

DM-40691: Add watch support for namespaces #205

merged 1 commit into from
Sep 12, 2023

Conversation

rra
Copy link
Member

@rra rra commented Sep 12, 2023

Add watch, field selector, and label selector support to list_namespace in the Kubernetes mock.

read_namespace and list_namespace in the Kubernetes mock now only return namespace objects that have been explicitly created, not implicit namespaces created by creating another object without making a namespace first. This more closely matches the behavior of Kubernetes while still making it easy to use the mock in a test environment simulating a pre-existing namespace.

Fix the documentation for label_selector in list_* functions to reflect that it is applied to the regular list return, not just the watch.

Add watch, field selector, and label selector support to
list_namespace in the Kubernetes mock.

read_namespace and list_namespace in the Kubernetes mock now
only return namespace objects that have been explicitly created,
not implicit namespaces created by creating another object without
making a namespace first. This more closely matches the behavior
of Kubernetes while still making it easy to use the mock in a test
environment simulating a pre-existing namespace.

Fix the documentation for label_selector in list_* functions to
reflect that it is applied to the regular list return, not just the
watch.
@rra rra merged commit d683157 into main Sep 12, 2023
5 checks passed
@rra rra deleted the tickets/DM-40691 branch September 12, 2023 22: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.

1 participant