setup icon indicating copy to clipboard operation
setup copied to clipboard

docker-compose.yml: Update to v9.0.0-beta

Open jpmckinney opened this issue 3 years ago • 4 comments

As described here: https://github.com/getredash/redash/blob/master/CHANGELOG.md#upgrading

jpmckinney avatar Apr 14 '21 18:04 jpmckinney

Thanks for this @jpmckinney, I've applied this in the alternative fork over here, in case that helps. :smile:

justinclift avatar Jul 24 '21 11:07 justinclift

I think this PR is outdated because it releases the v10+ version now.

peter279k avatar May 06 '22 06:05 peter279k

The current setup.sh does not seem to produce the intended result because it replaces the version string. I thought a different approach was needed.

ref: https://github.com/getredash/setup/blob/e611a6af966124eb974d1d953f64a10c5e9767e6/setup.sh#L69

example:

$ TZ=UTC date
Wed Aug 23 04:17:07 AM UTC 2023
$ curl -s "https://version.redash.io/api/releases?channel=stable" | json_pp | grep "docker_image" | head -n 1 | awk 'BEGIN{FS=":"}{print $3}' | awk 'BEGIN{FS="\""}{print $1}'
8.0.0.b32245

An idea, To avoid misunderstandings. Maybe make this part a placeholder with something that doesn't exist (other than version)?

ref: https://github.com/getredash/setup/blob/e611a6af966124eb974d1d953f64a10c5e9767e6/data/docker-compose.yml#L3

The other, as is also done here Do we get rid of the Version string replacement?

ref: https://github.com/getnewdash/setup/blob/f46decba85e20408e43c311348ce2ce715297379/setup.sh#L58

snickerjp avatar Aug 23 '23 04:08 snickerjp

Hmmmm, the goal is to install the most recent "production" quality release. There are several ways we could achieve that.

Personally, I don't have a strong preference for a "best" solution. So, maybe pick whichever way you're most happy with and we can just use that?

If some much better idea appears later on, we can just change to that then. :smile:

justinclift avatar Aug 23 '23 04:08 justinclift

@jpmckinney Thanks for putting together this PR. :smile:

Looking over it, the commit that only changes the docker image version isn't needed any more (as mentioned by @peter279k).

The other commit though (removes celery queue, adds worker container) is still good though.

So that commit has been merged now (https://github.com/getredash/setup/commit/92ca3da2dd2e710c40f308db0f0f78a0145f3eaf).

justinclift avatar Mar 21 '24 11:03 justinclift