Karl Skewes

Results 50 comments of Karl Skewes

Because the bug/behaviour change lies elsewhere shall we close this issue? Or is it useful to anyone staying open? I'm easy either way.

An alternative to a `ServiceMonitor` is a [`PodMonitor`](https://github.com/coreos/prometheus-operator/blob/master/Documentation/design.md#podmonitor). We tend to go with PodMonitor's by default now as not all apps take inbound traffic, thus simpler than adding a service...

If I could find where the `core-dns.yaml` or similar lives (on GitHub presumably?) I would raise a PR to add the metrics port to try and progress this. We ended...

Agree. The `iptables` mechanism is a pretty big ask for those using EKS (a managed service). Encoding the `iptables` steps into our Terraform is possible but with Calico and other...

@alexbrand @pires Hey guys, have you had any time to progress this? The original pull request with subsequent conversation about overriding configmap with env vars for client/data/master role variances looks...

For me I fixed the error by adding `pull-requests: write` permission to the job. ``` jobs: pulumi: name: Pulumi Preview permissions: contents: read pull-requests: write # this line required ```...

@spinnakerbot remove-label to-be-closed

Solving this requires someone to complete PR's to each of the services. I don't think I'll get to it sorry.

We have just hit this on 1.17.6 with a `NetworkPolicy` list: ``` --- apiVersion: networking.k8s.io/v1 kind: NetworkPolicyList items: - apiVersion: networking.k8s.io/v1 kind: NetworkPolicy metadata: labels: ``` We also have some...

It looks like Spinnaker treats `metadata` as a required field in Kubernetes v2 here: https://github.com/spinnaker/clouddriver/blob/master/clouddriver-kubernetes-v2/src/main/java/com/netflix/spinnaker/clouddriver/kubernetes/v2/description/manifest/KubernetesManifest.java#L102 Kubernetes spec says for `Lists`: > Every list or simple kind SHOULD have the following...