Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Double "Waiting for network connection" message #1219

Open
DaniilKl opened this issue Jan 29, 2025 · 0 comments
Open

Double "Waiting for network connection" message #1219

DaniilKl opened this issue Jan 29, 2025 · 0 comments
Labels

Comments

@DaniilKl
Copy link
Contributor

Component

Dasharo Tools Suite

Device

other

Dasharo version

Dasharo Tools Suite version

2.2.0

Test case ID

No response

Brief summary

Sometimes network connection could be checked several times in a row.

How reproducible

100%

How to reproduce

Boot DTS and provide credentials. Note that providing credentials not the only place where such situation can occur.

Expected behavior

Network connection check should be properly handled.

Actual behavior

Image

Screenshots

No response

Additional context

This patch might help:

diff --git a/include/dts-functions.sh b/include/dts-functions.sh
index 2f996d790cf4..6b396077bcbf 100644
--- a/include/dts-functions.sh
+++ b/include/dts-functions.sh
@@ -225,10 +225,6 @@ board_config() {
 # set. This is made with a goal to limit global variables declaration to
 # dts-environment.sh and board_config function.
 
-  # We download firmwares via network. At this point, the network connection
-  # must be up already.
-  wait_for_network_connection
-
   echo "Checking if board is Dasharo compatible."
   case "$SYSTEM_VENDOR" in
     "Notebook")
diff --git a/scripts/dasharo-deploy b/scripts/dasharo-deploy
index 29c51a3d254c..27b004e4063b 100644
--- a/scripts/dasharo-deploy
+++ b/scripts/dasharo-deploy
@@ -908,6 +908,9 @@ install_workflow() {
   # Set all global variables needed for installation:
   prepare_env install
 
+  # Check for network connection before downloading binaries:
+  wait_for_network_connection
+
   # Download and verify firmware:
   if [ "$HAVE_EC" == "true" ]; then
     download_ec
@@ -1017,6 +1020,9 @@ update_workflow() {
     fi
   fi
 
+  # Check for network connection before downloading binaries:
+  wait_for_network_connection
+
   if [ "$HAVE_EC" == "true" ]; then
     download_ec
     verify_artifacts ec
@@ -1163,6 +1169,9 @@ restore() {
         echo
         echo "Searching for HCL report on cloud..."
 
+        # Check for network connection before downloading HCL:
+        wait_for_network_connection
+
         ${CMD_CLOUD_LIST} $uuid
         error_check "Could not download HCL report from cloud."
 

Solutions you've tried

No response

@DaniilKl DaniilKl added bug Something isn't working needs review labels Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant