Skip to content

[kube] support filtering for entire objects in tables#46683

Merged
tigrato merged 2 commits intomasterfrom
tigrato/fix-kube-assertion
Sep 17, 2024
Merged

[kube] support filtering for entire objects in tables#46683
tigrato merged 2 commits intomasterfrom
tigrato/fix-kube-assertion

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Sep 17, 2024

This PR resolves a bug where Teleport incorrectly returned an error when a request attempted to order resources by a field that wasn't included in the *metav1.PartialObjectMetadata objects returned in the table during a kubectl get resource operation.

Changelog: Fixes a bug in Kubernetes access that causes the error expected *metav1.PartialObjectMetadata object when trying to list resources.

Copy link
Copy Markdown
Contributor

@hugoShaka hugoShaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changelog makes sense for us but it's a bit obscure for customers.

WDYT of: Fixes a bug in Kubernetes access that causes the error Unable to list: expected *metav1.PartialObjectMetadata object when trying to list resources.

Comment thread lib/kube/proxy/resource_filters.go Outdated
Comment thread lib/kube/proxy/resource_filters.go
This PR fixes a bug where Teleport incorrectly returned an error when ordering resources by some field because the returned table included the full object instead of the PartialMetadataObject.

This PR extends support for ordering fields by a field that isn't present in default table view.
@tigrato tigrato force-pushed the tigrato/fix-kube-assertion branch from 5f6fece to abc19dc Compare September 17, 2024 17:55
@tigrato tigrato added this pull request to the merge queue Sep 17, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 17, 2024
@tigrato tigrato added this pull request to the merge queue Sep 17, 2024
Merged via the queue into master with commit fe0b3ad Sep 17, 2024
@tigrato tigrato deleted the tigrato/fix-kube-assertion branch September 17, 2024 19:51
@public-teleport-github-review-bot
Copy link
Copy Markdown

@tigrato See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Create PR

smallinsky pushed a commit that referenced this pull request Sep 20, 2024
* [kube] support filtering for entire objects in tables

This PR fixes a bug where Teleport incorrectly returned an error when ordering resources by some field because the returned table included the full object instead of the PartialMetadataObject.

This PR extends support for ordering fields by a field that isn't present in default table view.

* add tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants