-
Notifications
You must be signed in to change notification settings - Fork 458
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
fix: port service mapping #4132
Conversation
pkg/k8s/K8sCommonService.go
Outdated
@@ -496,15 +524,25 @@ func (impl K8sCommonServiceImpl) PortNumberExtraction(resp []BatchResourceRespon | |||
impl.logger.Warnw("value not found in resourceTreeVal, unable to extract port no") | |||
continue | |||
} | |||
serviceNameRes, ok := value[k8sCommonBean.K8sClusterResourceNameKey] |
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.
why are we checking K8sClusterResourceNameKey for service value?
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.
changed K8sClusterResourceNameKey to K8sClusterResourceMetadataNameKey
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Description
These changes correctly maps port list with service names in case of helm deployed applications
Fixes #4025
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:
Does this PR introduce a user-facing change?