SHA-4096 commented on code in PR #286:
URL: https://github.com/apache/dubbo-kubernetes/pull/286#discussion_r1609491201


##########
pkg/admin/model/instance.go:
##########
@@ -73,3 +78,105 @@ type State struct {
        Tip   string `json:"tip"`
        Value string `json:"value"`
 }
+
+type InstanceDetailResp struct {
+       RpcPort         []string `json:"rpcPort"`
+       Ip              []string `json:"ip"`
+       AppName         []string `json:"appName"`
+       WorkloadName    []string `json:"workloadName"`
+       Labels          []string `json:"labels"`
+       CreateTime      []string `json:"createTime"`
+       ReadyTime       []string `json:"readyTime"`
+       RegisterTime    []string `json:"registerTime"`
+       RegisterCluster []string `json:"registerCluster"`
+       DeployCluster   []string `json:"deployCluster"`
+       Node            []string `json:"node"`
+       Image           []string `json:"image"`
+       Probes          struct {
+               StartupProbe struct {
+                       Type string `json:"type"`
+                       Port int    `json:"port"`
+                       Open bool   `json:"open"`
+               } `json:"startupProbe"`
+               ReadinessProbe struct {
+                       Type string `json:"type"`
+                       Port int    `json:"port"`
+                       Open bool   `json:"open"`
+               } `json:"readinessProbe"`
+               LivenessProbe struct {
+                       Type string `json:"type"`
+                       Port int    `json:"port"`
+                       Open bool   `json:"open"`
+               } `json:"livenessProbe"`
+       } `json:"probes"`
+}
+
+func (r *InstanceDetailResp) FromInstanceDetail(id *InstanceDetail) 
*InstanceDetailResp {
+       r.AppName = id.AppName.Values()
+       r.RpcPort = id.RpcPort.Values()
+       r.Ip = id.Ip.Values()
+       r.WorkloadName = id.WorkloadName.Values()
+       r.Labels = id.Labels.Values()
+       r.CreateTime = id.CreateTime.Values()
+       r.ReadyTime = id.ReadyTime.Values()
+       r.RegisterTime = id.RegisterTime.Values()
+       r.RegisterCluster = id.RegisterClusters.Values()
+       r.DeployCluster = id.DeployCluster.Values()
+       r.Node = id.Node.Values()
+       r.Image = id.Image.Values()
+       return r
+}
+
+type InstanceDetail struct {
+       RpcPort          Set
+       Ip               Set
+       AppName          Set
+       WorkloadName     Set
+       Labels           Set
+       CreateTime       Set
+       ReadyTime        Set
+       RegisterTime     Set
+       RegisterClusters Set
+       DeployCluster    Set
+       Node             Set
+       Image            Set
+}

Review Comment:
   
#https://github.com/apache/dubbo-kubernetes/blob/f493d76f1a41b5ac5b30bfb0a51155b2dfe5c274/pkg/admin/model/application.go#L60
   I referenced the ApplicationDetail Struct to write InstanceDetail, but it 
seems that it's indeed not a good practice.
   
   This interface would return something like this for now:
   ```json
   "data": [
           {
               "rpcPort": [
                   "20000"
               ],
               "ip": [
                   "172.23.237.86:20000"
               ],
           }
   ]
   ```
   
   I'll have it changed soon.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org
For additional commands, e-mail: notifications-h...@dubbo.apache.org

Reply via email to