Tim Buckley

Results 52 comments of Tim Buckley

Apologies for the slow response, however I've tentatively fixed this in https://github.com/HewlettPackard/dockerfile-parser-rs/pull/15. It may take a bit to merge and release since all the other reviewers are out for the...

After some investigation, this seems to be some Android bug or some such. When ReceiverService attempt to create a connection to LIFXService via `bindService()`, LIFXService reliably starts without any trouble,...

I've done a pass to apply some of the discussions we've more or less found a consensus on, mainly switching to UUID identifiers and lightly touching on how we'll verify...

Hmm. I can understand why you'd want to just use plain docker-compose since it's so much simpler than a full k8s or the like, the three-node config UX example you...

The idea seems reasonable enough I think, if you're able to come up with a static deployment that meets your needs I don't see any issues merging it. I'm all...

I think `kubernetes_get_host.py` predates nodeName being available in the downward API, but it would be good to switch. Maybe @mhoppal can confirm?

That said, you're right, the cause of the error is probably due to outdated `ipaddress` and `urllib3` packages ([see also](https://github.com/kubernetes-incubator/client-python/blob/master/README.md#hostname-doesnt-match)). Removing `kubernetes_get_host.py` will fix this issue and let the agent...

Output from the latest agent-collector image shows compatible versions: ``` [tim:~]↥ 3s % docker run --rm=true -it monasca/agent-collector:master-20171121-122311 pip list | grep -E '(urllib3|ipaddress)' ipaddress (1.0.18) urllib3 (1.22) ```

Ideally I think metrics should push metrics and logs should push logs, though it may be simpler to implement if we picked one or the other.

side note - sidecar is an optional component, you can disable (or even remove the patch from the image build entirely) without breaking anything, especially in a docker-only environment. Without...