Add support for arm64, armV7, armV8

[utegental]

What would you like to be added:
There is no arm support. It would be good to see support this for those architectures.

Why is this needed:
Getting "standard_init_linux.go:219: exec user process caused: exec format error" for pods scheduled on arm nodes.
Usually it's easy to do - just making image multi-architecture =)

[marquiz]

I agree. There have been attempts in the past e.g. #203 and #327 at least. Now that we have K8s test-infra (gcb as the image builder) as the CI pipeline this might be doable

[Diaoul]

I would be interested in this as well. Can you elaborate on what kind of work is required to make this happen? From the previous PRs, it doesn't seem that a great deal of code is required.

[xunholy]

One is currently being maintained in parity with upstream docker.io/raspbernetes/node-feature-discovery which is built for armv7 & arm64

[Diaoul]

Great! Maybe this can be merged upstream?

[zvonkok]

@marquiz We can easily do this with github actions

jobs:
  build_job:
    runs-on: ubuntu-latest
    name: Build on ${{ matrix.arch }}

    strategy:
      matrix:
        include:
          - arch: armv7
            distro: ubuntu20.04
          - arch: aarch64
            distro: ubuntu20.04
          - arch: s390x
            distro: ubuntu20.04
          - arch: ppc64le
            distro: ubuntu20.04
    steps:

[zvonkok]

The only question is does gcr.io support manifest lists?

[marquiz]

@marquiz We can easily do this with github actions

Yeah, sure. But we use prow for building images

[anthr76]

Any updates on this?

[anta5010]

@marquiz Are there any plans to progress with this issue?
It looks like the main problem is not in updating the Makefile. This change is quite simple if buildx can be used. But, before creating a PR the CI pipelines need to be updated to support buildx. Do you know if it's possible and what help is required?

[marquiz]

@marquiz Are there any plans to progress with this issue?
It looks like the main problem is not in updating the Makefile. This change is quite simple if buildx can be used. But, before creating a PR the CI pipelines need to be updated to support buildx. Do you know if it's possible and what help is required?

Hmm, kubernetes test-infra might actually support docker buildx nowadays so we could do this. What needs to be done is

• update Makefile
• (probably) update scripts/test-infra/* to build images on all architectures
• update docs (docs/get-started/deployment-and-usage.md and docs/advanced/developer-guide.md quickly come to mind)

Patches are welcome 😄

[ArangoGutierrez]

Best example I could find is https://github.com/kubernetes-sigs/service-catalog/blob/master/Makefile#L343
but I am not big fan of cross compilation tho, so the question is, do we want to do it like that?

[ArangoGutierrez]

https://kubernetes.slack.com/archives/C01A82FKSQK/p1628088075000500

[marquiz]

I am not big fan of cross compilation tho, so the question is, do we want to do it like that?

With docker buildx you shouldn't need that. Just something like docker buildx build --platform linux/arm64 . (haven't experimented this myself, though). Thus, I think we should be ok with relatively simple modifications to the Makefile