feat: add AMD/ARM 64 images for the docker action.

[dzmitryhil]

Description

Add AMD/ARM 64 images for the docker action.

---

This change is 

[ysv]

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @alinetskyi, @dzmitryhil, @miladz68, and @wojtek-coreum)

---

.github/workflows/build.yml line 46 at r1 (raw file):

          push: true
          build-args: |
            "NEXT_PUBLIC_GRAPHQL_URL=http://localhost:8080/v1/graphql"

but do we really still need all these variables ?

[ysv]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @alinetskyi, @dzmitryhil, @miladz68, and @wojtek-coreum)

---

.github/workflows/build.yml line 46 at r1 (raw file):

Previously, ysv (Yaroslav Savchuk) wrote…

but do we really still need all these variables ?

I see they are for compatibility with znet inside crust
But I suggest to not use this trick but instead set envs inside crust directly
using

infra.Deployment {
	EnvVarsFunc func() []EnvVar
}

[dzmitryhil]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @alinetskyi, @miladz68, @wojtek-coreum, and @ysv)

---

.github/workflows/build.yml line 46 at r1 (raw file):

Previously, ysv (Yaroslav Savchuk) wrote…

I see they are for compatibility with znet inside crust
But I suggest to not use this trick but instead set envs inside crust directly
using

infra.Deployment {
	EnvVarsFunc func() []EnvVar
}

But that's not trick. The values are default values, connected to the localhost, which should be by default.
I can duplicate them in the crust, if you see it's important. But IMO we don't need to do it.

[ysv]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @alinetskyi, @dzmitryhil, @miladz68, and @wojtek-coreum)

---

.github/workflows/build.yml line 46 at r1 (raw file):

Previously, dzmitryhil (Dzmitry Hil) wrote…

But that's not trick. The values are default values, connected to the localhost, which should be by default.
I can duplicate them in the crust, if you see it's important. But IMO we don't need to do it.

In original big dipper repo they don't specify these build-args
https://github.com/forbole/big-dipper-2.0-cosmos/blob/main/.github/workflows/publish.yml#L89

So I suggest we keep it similar. Lets not specify them if they are not needed

Also yes lets put these values in crust pls

[dzmitryhil]

Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @alinetskyi, @miladz68, @wojtek-coreum, and @ysv)

---

.github/workflows/build.yml line 46 at r1 (raw file):

Previously, ysv (Yaroslav Savchuk) wrote…

In original big dipper repo they don't specify these build-args
https://github.com/forbole/big-dipper-2.0-cosmos/blob/main/.github/workflows/publish.yml#L89

So I suggest we keep it similar. Lets not specify them if they are not needed

Also yes lets put these values in crust pls

Removed all apart from "PROJECT_NAME=web-coreum" which is required for the build.

Code quote:

 "PROJECT_NAME=web-coreum"

[ysv]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alinetskyi, @miladz68, and @wojtek-coreum)