Explicitly set 644 permissions on ssh config files (#2231)

* explicitly set 644 permissions on ssh config files * bring back sros tests
srl-labs · Oct 14, 2024 · 37a111e · 37a111e
1 parent d812ba1
commit 37a111e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/.github/workflows/cicd.yml b/.github/workflows/cicd.yml
@@ -314,13 +314,13 @@ jobs:
     with:
       py_ver: ${{ needs.file-changes.outputs.py_ver }}
 
-  # sros-tests:
-  #   uses: ./.github/workflows/sros-tests.yml
-  #   needs:
-  #     - file-changes
-  #     - build-containerlab
-  #   with:
-  #     py_ver: ${{ needs.file-changes.outputs.py_ver }}
+  sros-tests:
+    uses: ./.github/workflows/sros-tests.yml
+    needs:
+      - file-changes
+      - build-containerlab
+    with:
+      py_ver: ${{ needs.file-changes.outputs.py_ver }}
 
   fortigate-tests:
     uses: ./.github/workflows/fortigate-tests.yml
@@ -350,7 +350,7 @@ jobs:
       - ixiac-one-basic-tests
       - vxlan-tests
       - kind-tests
-      # - sros-tests
+      - sros-tests
       - fortigate-tests
       - cisco_iol-tests
 

diff --git a/clab/sshconfig.go b/clab/sshconfig.go
@@ -89,7 +89,7 @@ func (c *CLab) addSSHConfig() error {
 		return err
 	}
 
-	f, err := os.Create(c.TopoPaths.SSHConfigPath())
+	f, err := os.OpenFile(c.TopoPaths.SSHConfigPath(), os.O_CREATE|os.O_WRONLY|os.O_TRUNC, 0644)
 	if err != nil {
 		return err
 	}