Codecov Report

Merging#1337 (01d5b50) intomain (ad8d9dd) willincrease coverage by0.56%.
The diff coverage is100.00%.

@@            Coverage Diff             @@##             main    #1337      +/-   ##==========================================+ Coverage   66.44%   67.01%   +0.56%==========================================  Files         284      288       +4       Lines       18584    18857     +273       Branches      235      241       +6     ==========================================+ Hits        12349    12637     +288+ Misses       4967     4932      -35- Partials     1268     1288      +20

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatead8d9dd...01d5b50. Read thecomment docs.

@kylecarbs I think the Terraform provider has to be updated supportarm_7 as an arch as well? Is it as simple as changing the validation rule?

https://github.com/coder/terraform-provider-coder/blob/4531753ebf98fbbd912eec8503c45dde6313a459/internal/provider/provider.go#L162-L167

Oh, and the agent scripts:

I also noticed that the binary isn't being uploaded to[coder-url]/bin/coder-arm_7. In fact, none of the arm builds are. Any ideas?

Edit: I think it has to do with the coder slim build only building amd64 binaries. I'll test in the morning

coder/terraform-provider-coder#18

still need to a bit of work to expose the agent start script

Testing with my raspberry pi now :)

Edit: Running into an issue parsing templates

Edit 2: got this working!

So, just a few doubts - but I will test this.

1. Does this have compatible Docker images?
2. Will this work on arm64 now that we got scripts in the build?
3. -Most Important- how much time does it take to build Coder freshly?

So, just a few doubts - but I will test this.

1. Does this have compatible Docker images?
2. Will this work on arm64 now that we got scripts in the build?
3. -Most Important- how much time does it take to build Coder freshly?

All great questions.

1. You need to specify armv7 compatible images in the template. I'll write a guide on how to do this. Sadlyarmv7:ubuntu won't work because it doesn't containcurl
2. Yes
3. Not long (~3 minutes), but these changes mean a build is provided (.deb, .apk, or binary) for each release

Thanks for the answers, I will start my build now.

You need to specify armv7 compatible images in the template. I'll write a guide on how to do this.

I know how to modify the main.tf if that's what you mean. What I will have to do(i hope it's right) is go to Docker Hub and find an ARM64(my pi is 64-bit ARM) image. Then, I should add that image name to its respective section in the main.tf file. Then I should create a workspace. Am I right on this?
P.S. my pi is arm64, so (don't wanna sound rude, it happens with me) I think it could be better if you refer to this as arm. Just arm if it's fine with you.

I know how to modify the main.tf if that's what you mean. What I will have to do(i hope it's right) is go to Docker Hub and find an ARM64(my pi is 64-bit ARM) image. Then, I should add that image name to its respective section in the main.tf file. > Then I should create a workspace. Am I right on this?

Correct. That image needs to havecurl in it. You also need to change the coder_agentarch to use arm64.

P.S. my pi is arm64, so (don't wanna sound rude, it happens with me) I think it could be better if you refer to this as arm. > Just arm if it's fine with you.

Yeah, I initially submitted this to add support for armv7 but it grew in scope when I realized the arm64 scripts weren't being added

Umm.... We got a situation here. I am encountering this:
run parse: recv parse source: manager closed: EOF

Any solutions? My Pi's Neofetch if it helps:

     _,met$$$$$gg.          pi@geniuspi     ,g$$$$$$$$$$$$$$$P.       -----------   ,g$$P"     """Y$$.".        OS: Debian GNU/Linux 11 (bullseye) aarch64  ,$$P'              `$$$.     Host: Raspberry Pi 4 Model B Rev 1.4 ',$$P       ,ggs.     `$$b:   Kernel: 5.15.32-v8+ `d$$'     ,$P"'   .    $$$    Uptime: 15 mins  $$P      d$'     ,    $$P    Packages: 2074 (dpkg)  $$:      $$.   -    ,d$$'    Shell: bash 5.1.4  $$;      Y$b._   _,d$P'      Resolution: 1920x1080  Y$$.    `.`"Y$$$$P"'         DE: Plasma 5.20.5  `$$b      "-.__              WM: KWin   `Y$$                        WM Theme: ChromeOS-light    `Y$$.                      Theme: Breeze [Plasma], Breeze [GTK2/3]      `$$b.                    Icons: Tela-circle [Plasma], Tela-circle [GTK2/3]        `Y$$b.                 Terminal: konsole           `"Y$b._             CPU: BCM2835 (4) @ 1.800GHz               `"""            Memory: 762MiB / 7812MiB

Umm.... We got a situation here. I am encountering this:
run parse: recv parse source: manager closed: EOF

Any solutions? My Pi's Neofetch if it helps:

Apologies! This is a known issue and a bug in the upstream Terraform library. Installing Terraform on your Pi will fix this as a workaround.

I know this is merged and all that, but@bpmct did it work properly for you? I inserted custom images that use Ubuntu ARM, two from Canonical's kinda proxies ARMv8 provider(that's arm64, if I am not wrong) and one from a person or org named osrf. All of them failed, so are there any working images that use ARM64?

@im-coder-lg can we discuss this in your issue or in Discord?