Skip to content

Conversation

@ganga1980
Copy link
Contributor

@ganga1980 ganga1980 commented Jan 7, 2026

This pull request makes a minor fix to the service records filtering logic in the in_kube_podinventory plugin. The change ensures that the correct key is used when checking whether to exclude a Kubernetes service record.

  • Updated the call to KubernetesApiClient.isExcludeResourceItem to use the Namespace key instead of namespace when filtering service records in in_kube_podinventory.rb.

Fix has been verified. DCR with include and "kube-system" namespace .

image

@ganga1980 ganga1980 requested a review from a team as a code owner January 7, 2026 16:39
@ganga1980
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@zanejohnson-azure
Copy link
Contributor

This pull request makes a minor fix to the service records filtering logic in the in_kube_podinventory plugin. The change ensures that the correct key is used when checking whether to exclude a Kubernetes service record.

  • Updated the call to KubernetesApiClient.isExcludeResourceItem to use the Namespace key instead of namespace when filtering service records in in_kube_podinventory.rb.

Fix has been verified.

how was it verified? please share results if you have.

@ganga1980
Copy link
Contributor Author

This pull request makes a minor fix to the service records filtering logic in the in_kube_podinventory plugin. The change ensures that the correct key is used when checking whether to exclude a Kubernetes service record.

  • Updated the call to KubernetesApiClient.isExcludeResourceItem to use the Namespace key instead of namespace when filtering service records in in_kube_podinventory.rb.

Fix has been verified.

how was it verified? please share results if you have.

updated the screenshot before and after fix.

@ganga1980 ganga1980 closed this Jan 7, 2026
@ganga1980 ganga1980 reopened this Jan 7, 2026
@ganga1980
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ganga1980 ganga1980 enabled auto-merge (squash) January 7, 2026 17:27
@ganga1980 ganga1980 merged commit 5c0bca0 into ci_prod Jan 7, 2026
26 checks passed
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.

4 participants