LGTM. @pdion891 would you like to review?

Could you look if it conflict with #1435 ? Because I made a pull request to automatically increment version in docker file when changing release of master via tools/build/setnextversion.sh.

Changes look good to me, but waiting for @pdion891 to tell us more :)

I didn't test #1435 yet, but code looks good. The only difference I see is:

pip install cryptography --force-reinstall

And I used the mysql connector from MySQL.com but would prefer to take it from distribution (as in #1435)

So, should we go forward and merge #1435 + #1769 to master?
I might just need to rebase and update version number of the current commit.

With those 2 PR, we would have master back online on dockerhub so we can get an updater docker image for the simulator!

@resmo could you do a PR on top of #1435 ? or this have to be merged first? so we can get your chances and some of mine.

@pdion891 I would say merge #1435 first

ok, I'll update the PR and merge it I think the PR go all required LGTM. I'll confirmed with John, in case he is in a code freeze period.

@resmo now that PR by @pdion891 is merge, can this be closed? Thanks.

Hi @resmo, I've made some changes on the Docker file so it might not be possible to merge this PR in the current state.

Thanks!

ok, np closing for now