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

Service detail page does not always show load balancer ip #6245

Closed
bashofmann opened this issue Jun 29, 2022 · 3 comments · Fixed by #6246
Closed

Service detail page does not always show load balancer ip #6245

bashofmann opened this issue Jun 29, 2022 · 3 comments · Fixed by #6246
Assignees
Milestone

Comments

@bashofmann
Copy link
Contributor

Detailed Description
The details section of the Service detail page does not always show the external IP or hostname of a LoadBalancer Service.

Currently the external IP is only shown, if it was defined in the Service spec. In most cases the external IP or hostname of a LoadBalancer Service is automatically assigned by the cloud provider though. In that case spec.loadBalancerIP is empty. But you can see the assigned information in status.loadBalancer.ingress.

Context
Displaying the IP or hostname makes it easier for users to copy and paste this information.

@gaktive
Copy link
Member

gaktive commented Feb 3, 2023

This may help with #8087 though I'd have to dig in deeper.

A milestone was left off this work so if it's in To Test, QA can look for 2.7.2.

@slickwarren slickwarren added the team/area2 Hostbusters label Feb 6, 2023
@zube zube bot added the QA/XS label Feb 15, 2023
@thaneunsoo
Copy link

Test Environment:

Rancher version: v2.7-head d559c2f
Rancher cluster type: HA
Docker version: 20.10
Browser: Chrome Build 110


Testing:

Tested this issue with the following steps:

  1. Provision hosted cluster (AKS used)
  2. After cluster becomes active, create loadbalancer service
  3. Leave external IP undefined
  4. Create

Result
External IP was automatically populated and is no longer left blank (addresses redacted)
image.png

@zube zube bot closed this as completed Feb 27, 2023
@zube zube bot removed the [zube]: Done label May 29, 2023
@izaac
Copy link
Contributor

izaac commented Mar 1, 2024

Needs either unit tests and/or E2E. If that's not possible as it seems provisioning with cloud provider is not yet implemented a manual validation on AKS/GKE/EKS should be done. cc @yonasberhe23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants