Build rust rpm's #2443
Build rust rpm's #2443
Conversation
The sccahe installation from crates is not functioning properly on centos 7. Instead, pull the repo down and build sccache in release mode with no default features. This fixes the issue. Additionally, add a "--clean" flag that will rebuild all of the rpms. This is useful if only a specific rpm is being built but others also need to be built to ensure that db migrations are baked into each service. The operation will still be fairly quick since sccache will be used. Signed-off-by: johnsonw <wjohnson@whamcloud.com>
| cd /integrated-manager-for-lustre \ | ||
| && [ "$1" = "--clean" ] && cargo clean || echo "Attempting to build rpm's without cleaning." \ |
There was a problem hiding this comment.
Does it clean sscache as well? This seems a bit unexpected, since cargo doesn't know about sscache (or does it?).
There was a problem hiding this comment.
In my case, I made a change to one of the migrations and changed a small piece of code in one of the services. When building the rpm's, cargo just saw that one service was changed, so when it built the rpms, it used the cache data for the other services. This ultimately led to the other services having the previous SQL migrations baked into them while the service I made a change to baked in the new migration script. So when I installed the rpms, one service had a different migration baked into it, which created a race condition to run the SQL migrations and ultimately caused the services to crash. What this does is clean out cargo to ensure the other services are also built. Sccache will still be used to speed up compile time.
What is the issue? |
The sccahe installation from crates is not functioning properly on
centos 7. Instead, pull the repo down and build sccache in release mode
with no default features. This fixes the issue. Additionally, add a
"--clean" flag that will rebuild all of the rpms. This is useful if only
a specific rpm is being built but others also need to be built to ensure
that db migrations are baked into each service. The operation will still
be fairly quick since sccache will be used.
Signed-off-by: johnsonw wjohnson@whamcloud.com
This change is