profile
viewpoint
David Benque dbenque Datadog France Golang, C++, Python Distributed systems, microservice, PaaS Kubernetes, Openshift, Docker, Prometheus

AmadeusITGroup/Redis-Operator 130

Redis Operator creates/configures/manages Redis clusters atop Kubernetes

AmadeusITGroup/kubervisor 36

The Kubervisor allow you to control which pods should receive traffic or not based on anomaly detection.It is a new kind of health check system.

dbenque/delveAppengine 9

Helper to automatically aatch Delve debugger to the latest instance of an Appengine module

clamoriniere/k8snssetup 7

Dummy tool to configure a multi tenants Kubernetes cluster

DataDog/draino 1

Automatically cordon and drain Kubernetes nodes based on node conditions

dbenque/GAE-Image-Server 1

Use GAE Blobstore to host your images. Upload image and profiles (Quality/Size). Serve Image for a given profile (resize on the fly).

dbenque/api 0

The canonical location of the Kubernetes API definition.

dbenque/appengine_apps 0

Appengine applications definitions

dbenque/autoscaler 0

Autoscaling components for Kubernetes

issue commentplanetlabs/draino

No actions being taken by draino for the condition Ready=Unknown

@dliao-tyro ok, but in order to have the drain completed you had to restart the kubelet, which will make the node responsive again (transition from unknown (aka notReady) to ready state). I was expecting the solution to work on top of a node that remains notReady. IMO it can due to the way the eviction API works, waiting for kubelet status to confirm pod eviction.

prabhatnagpal

comment created time in 4 days

Pull request review commentDataDog/kubernetes

use 'nf_conntrack' instead of 'nf_conntrack_ipv4' for linux kernel >= 4.19

 func CanUseIPVSProxier(handle KernelHandler, ipsetver IPSetVersioner) (bool, err 	} 	wantModules := sets.NewString() 	loadModules := sets.NewString()+	linuxKernelHandler := NewLinuxKernelHandler()+	_, ipvsModules, _ := utilipvs.GetKernelVersionAndIPVSMods(linuxKernelHandler.executor)

ok

EricMountain

comment created time in 2 months

Pull request review commentDataDog/kubernetes

use 'nf_conntrack' instead of 'nf_conntrack_ipv4' for linux kernel >= 4.19

 func CanUseIPVSProxier(handle KernelHandler, ipsetver IPSetVersioner) (bool, err 	} 	wantModules := sets.NewString() 	loadModules := sets.NewString()+	linuxKernelHandler := NewLinuxKernelHandler()+	_, ipvsModules, _ := utilipvs.GetKernelVersionAndIPVSMods(linuxKernelHandler.executor)

Check err ?

EricMountain

comment created time in 2 months

issue closedplanetlabs/draino

Consider Cluster-Autoscaler `safe-to-evict` annotation by default

Draino should not evict pods that are marked with "cluster-autoscaler.kubernetes.io/safe-to-evict": "false"

https://github.com/kubernetes/autoscaler/blob/master/cluster-autoscaler/FAQ.md#what-types-of-pods-can-prevent-ca-from-removing-a-node

I can do the PR if there is an agreement on that.

closed time in 3 months

dbenque

pull request commentplanetlabs/draino

Allow multiple labels to be AND or OR together

Such transformation (dnf/cnf) would make it completely obscure while with a simple OR there is a natural way of expressing group of labels that define the group of nodes. I am afraid the syntax proposed here https://github.com/planetlabs/draino/pull/75#issuecomment-616872086 would make it impossible to use for me and my users.

cpoole

comment created time in 3 months

more