Update patch set 2

Patch Set 2:

(1 comment)

Patch-set: 2
Attention: {"person_ident":"Gerrit User 14826 \u003c14826@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_35264\u003e replied on the change"}
Attention: {"person_ident":"Gerrit User 35264 \u003c35264@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_35264\u003e replied on the change"}

0b3e4935732bfd12152c3b3cea77971251a3f839

@@ -16,6 +16,24 @@
       "message": "Do you know where this dependency comes from? IIRC tenks does not directly use it, and any indirect usage (e.g. ansible) should use a dependency in the other package. However, I don\u0027t think ansible uses paramiko by default - it uses a native SSH client.",
       "revId": "0b3e4935732bfd12152c3b3cea77971251a3f839",
       "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "8738e517_210deb68",
+        "filename": "requirements.txt",
+        "patchSetId": 2
+      },
+      "lineNbr": 9,
+      "author": {
+        "id": 35264
+      },
+      "writtenOn": "2024-03-05T12:11:45Z",
+      "side": 1,
+      "message": "My thought process for this was:\nvbmcd not starting\nJournal says it\u0027s missing the cryptography package\nCryptography package exists in the tenks venv but the internet says paramiko is also required\npip install paramiko in the tenks venv solves the problem so slap paramiko in the requirements file.\n\nUnfortunately the error logs are long gone, but I now see how requirements are handled in other roles so I\u0027ll move it to the virtualbmc-daemon role which I think is where it\u0027s required.",
+      "parentUuid": "00193293_b37d755f",
+      "revId": "0b3e4935732bfd12152c3b3cea77971251a3f839",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
     }
   ]
 }