P 419 deploy automation #2418
P 419 deploy automation #2418
Conversation
| SGX_SDK=/opt/intel/sgxsdk | ||
| SGX_ENCLAVE_SIGNER=$SGX_SDK/bin/x64/sgx_sign | ||
|
|
||
| VERSION_PATTERN="^p[0-9]+\.[0-9]+\.[0-9]+-[0-9]+-w[0-9]+\.[0-9]+\.[0-9]+-[0-9]+$" |
There was a problem hiding this comment.
it seems that this pattern supports only versions with patch version like p0.9.17-9172-w0.0.1-102 , should we also support something like p0.9.18-w0.0.2 in case we have major release?
There was a problem hiding this comment.
I would say we will have p0.9.18-9180-w0.0.2-100.
| if sudo -l -U $USER 2>/dev/null | grep -q 'may run the following'; then | ||
| source "$SGX_SDK/environment" | ||
| else | ||
| echo "$USER doesn't have sudo permission" | ||
| exit 1 | ||
| fi |
There was a problem hiding this comment.
We also can use this command to determine whether the current user is sudo
if [ $(id -u) -eq 0 ]; then
echo "sudo"
else
echo "Doesn't have sudo permission"
fi
There was a problem hiding this comment.
That won't work though
root has uid 0, but non-zero uid doesn't mean it can't sudo. In fact, the code will always land in else branch unless executed with root.
| sudo mkdir -p "$BASEDIR" | ||
| sudo chown -R $USER:$GROUPS "$BASEDIR" | ||
| for d in "$LOG_BACKUP_BASEDIR" "$WORKER_BACKUP_BASEDIR" "$RELAYCHAIN_ALICE_BASEDIR" "$RELAYCHAIN_BOB_BASEDIR" \ | ||
| "$PARACHAIN_ALICE_BASEDIR" "$WORKER_BASEDIR"; do | ||
| mkdir -p "$d" | ||
| done |
There was a problem hiding this comment.
Another version:
folders=("folder1" "folder2" "folder3" "folder4")
for folder in "${folders[@]}"
do
if [ ! -d "$folder" ]; then
mkdir "$folder"
else
echo "exists"
fi
done| fi | ||
| } | ||
|
|
||
| function restart_services { | ||
| sudo systemctl daemon-reload | ||
| restart_parachain_services | ||
| restart_worker_services | ||
| echo "Done" |
There was a problem hiding this comment.
echo "The service has been successfully restarted."
| for ((i = 0; i < $WORKER_COUNT; i++)); do | ||
| worker_dir="$WORKER_BASEDIR/w$i" | ||
| mkdir -p "$worker_dir" | ||
| for f in 'key.txt' 'spid.txt' 'enclave.signed.so' 'litentry-worker'; do |
There was a problem hiding this comment.
Same as above, if there are more than two, put them into an array for iteration.
| @@ -30,7 +30,7 @@ DISCORD_AUTH_TOKEN= | |||
| ACHAINABLE_AUTH_KEY= | |||
| ONEBLOCK_NOTION_KEY= | |||
| NODEREAL_API_KEY= | |||
| GENIIDATA_API_KEY=142cf1b0-1ca7-11ee-bb5e-9d74c2e854ac | |||
| GENIIDATA_API_KEY= | |||
There was a problem hiding this comment.
Will it affect any of our tests?
I remember @kziemianek mentioned this is the public key
There was a problem hiding this comment.
it won't affect any tests. The key is always overwritten.
| @@ -156,10 +157,16 @@ def offset_port(offset): | |||
|
|
|||
|
|
|||
| def setup_environment(offset, config, parachain_dir): | |||
| if not os.path.isfile("./local-setup/.env"): | |||
| shutil.copy("./local-setup/.env.dev", "./local-setup/.env") | |||
|
|
|||
There was a problem hiding this comment.
Is this needed?
I see the following line is
load_dotenv("./local-setup/.env.dev")
Old description is deprecated.
Auto deploy script moved into private repo.
Here is only some minor change/fix.