Fix Rust deployment

[benjaminwinger]

Fixes #1927.

The rustup install should probably be added to the dockerfile instead of included in the workflow, but I thought it would be easier to test it this way.

I've tested it and it's working: https://github.com/kuzudb/kuzu/actions/runs/5858238195/job/15881791303

I also had to fix the version parsing since 0.0.6.5 is not a valid semantic version (so I made the fourth component be turned into a pre-release version, e.g. 0.0.6-5).

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.01% ⚠️

Comparison is base (f4f1a77) 89.84% compared to head (54df97e) 89.83%.
Report is 2 commits behind head on master.

❗ Current head 54df97e differs from pull request most recent head 757a4b8. Consider uploading reports for the commit 757a4b8 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1932      +/-   ##
==========================================
- Coverage   89.84%   89.83%   -0.01%     
==========================================
  Files         867      867              
  Lines       31557    31557              
==========================================
- Hits        28351    28350       -1     
- Misses       3206     3207       +1     

see 1 file with indirect coverage changes

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

[mewim]

It seems this breaks rust format: https://github.com/kuzudb/kuzu/actions/runs/5858442707/job/15894933233?pr=1931

Let's breifly discuss it tomorrow. Redeploying the Docker containers maybe required to make this work.

[benjaminwinger]

Oh, yes with the minimal profile it won't install rustfmt. Either dropping the --profile minimal, or adding rustup component add rustfmt should fix that.

[mewim]

I think we should consider seperate the building environment from the regular testing environment. Since the builder now mutates the environment by running rustup, we should not run it on the testing container, so we can keep a stable environment for testing.

[mewim]

Or, we can just install rust via rustup in Dockerfile and avoid running it everytime during the build process.

[benjaminwinger]

Yes, I think installing it in the Dockerfile is a better idea.

I take it it's not starting up a fresh container per job/workflow then? I hadn't realised that this would propagate to the other jobs.

[mewim]

We have a fixed set of containers for each environment instead of launching a new one for each job. I will redeploy the containers to see if it fixes the issue.

[mewim]

Verified by https://github.com/kuzudb/kuzu/actions/runs/5868767569/job/15912311132