Simplify telemetry ping

nforgeio · Nov 13, 2023 · 2a3f133 · 2a3f133
1 parent b48e250
commit 2a3f133
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 17 deletions.
diff --git a/Lib/Neon.Kube/ClusterMetadata/ClusterTelemetry.cs b/Lib/Neon.Kube/ClusterMetadata/ClusterTelemetry.cs
@@ -331,12 +331,5 @@ public class ClusterNodeTelemetry
         /// </summary>
         [JsonProperty(PropertyName = "PrivateAddress", Required = Required.Always | Required.AllowNull)]
         public string PrivateAddress { get; set; }
-
-        /// <summary>
-        /// Indicates whether the node is configured and is ready to accept external network traffic
-        /// for the cluster.
-        /// </summary>
-        [JsonProperty(PropertyName = "Ingress", Required = Required.Always | Required.AllowNull)]
-        public bool Ingress { get; set; }
     }
 }
diff --git a/Services/neon-cluster-operator/CronJobs/TelemetryPingJob.cs b/Services/neon-cluster-operator/CronJobs/TelemetryPingJob.cs
@@ -98,16 +98,6 @@ public async Task Execute(IJobExecutionContext context)
                         node.PrivateAddress          = k8sNode.Status.Addresses.FirstOrDefault(address => address.Type == "InternalIP").Address;
                         node.Role                    = k8sNode.Metadata.GetLabel(NodeLabels.LabelRole);
 
-                        // $todo(jefflill):
-                        //
-                        // We need to really figure this out from the node status and Istio config.
-                        // We're currently assuming that only worker nodes support ingress unless
-                        // there are no workers and we're not checking node health either.
-                        //
-                        //      https://github.com/nforgeio/neonKUBE/issues/1816
-
-                        node.Ingress = !hasWorkers || node.Role == NodeRole.Worker;
-
                         if (k8sNode.Status.Capacity.TryGetValue("cpu", out var cores))
                         {
                             node.VCpus = cores.ToInt32();