Port https://github.com/rancher/rancher-ai-mcp/pull/99

[bk201]

Port rancher/rancher-ai-mcp#99

[Yu-Jack]

we can combine these into a single function like:

func fetchRancherURL() (string, error) {
    if url := os.Getenv("RANCHER_URL"); url != "" {
        return url, nil
    }

	cfg, err := rest.InClusterConfig()
	if err != nil {
		return "", fmt.Errorf("creating in-cluster config: %w", err)
	}

	dynClient, err := dynamic.NewForConfig(cfg)
	if err != nil {
		return "", fmt.Errorf("creating dynamic client: %w", err)
	}

	gvr := schema.GroupVersionResource{
		Group:    "management.cattle.io",
		Version:  "v3",
		Resource: "settings",
	}

	obj, err := dynClient.Resource(gvr).Get(context.Background(), "internal-server-url", metav1.GetOptions{})
	if err != nil {
		return "", fmt.Errorf("getting internal-server-url setting: %w", err)
	}

	value, _, err := unstructured.NestedString(obj.Object, "value")
	return value, err
}

[bk201]

Thanks, Jack. I'd like to keep it as it is. The function from the writer is meant to only fetch the URL from the internal-server-url setting. So it totally makes sense to separate the overriding of the env variable here.

// fetchRancherURL fetches the Rancher server URL from the internal-server-url Setting
// resource via the Kubernetes API using in-cluster config.
func fetchRancherURL() (string, error) {

[Yu-Jack]

Others LGTM.