search

influxdata / influxdata-operator

A k8s operator for InfluxDB
MIT License
76 stars 32 forks source link

influx-operator image? #3

Open kashline opened 5 years ago

kashline commented 5 years ago

I'm trying to find the correct image for the operator deployment but I can't find an official 'influx-operator' image and my pods are crashlooping trying to find an executable when using the influxdb official image.

Should I be using this image for the operator? https://hub.docker.com/r/dustinbarnes/influxdb-operator

pydevops commented 5 years ago

@kashline At the moment, you need to build & push your docker image hosted in a docker registry. Please check https://github.com/influxdata/influxdata-operator#developer-notes. You need operator-sdk v 0.0.7 installed as well. Please take a look at the Makefile's deploy and build target.

dv01d commented 5 years ago

Looks like this now only works for operator-sdk v0.1.x. Throws an error 1 on all other versions of operator-sdk I was able to to test.

pydevops commented 5 years ago

@dv01d I am curious which versions you have tried. The following version is what works for me. 

operator-sdk --version
operator-sdk version v0.3.0+git

Here is the release date for operator-sdk that I pulled from the git repo. 2019-02-08 11:00:34 -0800 (tag: v0.5.0) 2019-02-07 15:44:20 -0500 (tag: v0.4.1) 2019-01-18 12:00:08 -0800 (tag: v0.4.0) 2018-12-12 14:33:09 -0800 (tag: v0.3.0) 2018-12-10 14:27:06 -0800 (tag: v0.2.1, origin/v0.2.x) 2018-11-30 14:32:13 -0800 (tag: v0.2.0) 2018-11-05 11:35:19 -0800 (tag: v0.1.1, origin/v0.1.x) 2018-10-31 14:12:00 -0700 (tag: v0.1.0) 2018-10-15 14:06:52 -0700 (tag: v0.0.7, origin/v0.0.7)

dv01d commented 5 years ago

confirmed it also seems to work on v0.3.x, which was the only version I didn't try. I tried 0.0.7, v0.2.x, v0.4.x, and v0.5.x branches without much luck, so figured I'd save some folks the time and trouble and at least have it here. Need to run a make dep prior to building as well.

pydevops commented 5 years ago

Thanks for confirming. I think we should update the README.md on developer instructions for better experience. The other possible change is not to check in vendor folder as well.