fix(docker): improve docker tags to be cleared and avoid conflicts

[mistercrunch]

Improving the docker tag naming scheme to:

• we now suffix most tags with -arm for platofrm = linux-arm64, and leave the amd builds without suffix since they're used already, before this fix we would have conflicts between tags / images. More on this here Issue with the apache/superset:dockerize #26788
• we now point the main dev tag (ie apache/supserset:dev, and apache/supserset:dev-arm) to top of the master branch. This tag only gets generated then.
• on release, this should create the right top-level docker tags like apache/supserset:lean and apache/supserset:py310, ...
• on release, the very top level for the whole repo is the lean/amd image

A question remains

Where should we point top level builds like apache/supserset:lean and apache/supserset:lean-arm or even the top-most level apache/supserset? Seems it should be pointed to the latest official ASF release version/build, unclear if it should be done in CI, or a simple docker tag command as part of the ASF release process, pointing to the image of the right SHA (that tag should be computed and exist, so all we need is to point to the right place)

[request-info]

We would appreciate it if you could provide us with more info about this issue/pr! Please do not leave the title or description empty.

[eschutho]

unclear if it should be done in CI, or a simple docker tag command as part of the ASF release process

@mistercrunch for the ASF release, the docker image creation is still handled through the CI when the sha is marked for release, so there's currently no separate or manual process that's done.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a53c537) 67.29% compared to head (8efb21f) 67.29%.
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #26787   +/-   ##
=======================================
  Coverage   67.29%   67.29%           
=======================================
  Files        1895     1895           
  Lines       74249    74249           
  Branches     8257     8257           
=======================================
  Hits        49968    49968           
  Misses      22206    22206           
  Partials     2075     2075           

Flag	Coverage Δ
mysql	77.98% <ø> (ø)
postgres	78.08% <ø> (ø)
python	78.21% <ø> (ø)
sqlite	77.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sfirke]

Note that when we talk about the dev image, its main difference from lean seems to be that it bundles the headless browser and driver needed for Alerts & Reports to work. I would like eventually to see those bundled with the main images if possible, it confuses people that they have to run an image called "dev" to get that functionality.

I'm only bringing that up here to ask that we retain functionality such that when a version is released tagged like 3.1.0, a corresponding version 3.1.0-dev is also built that contains the tools necessary for Alerts & Reports.

[craig-rueda]

LGTM