URL: https://github.com/freeipa/freeipa/pull/4945
Author: fcami
 Title: #4945: re-enable test_sss_ssh_authorizedkeys
Action: opened

PR body:
"""
test_sss_ssh_authorizedkeys was disabled but recent test runs show it is 
working properly.
Run in multiple times to see if it works.
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/4945/head:pr4945
git checkout pr4945
From 69de6c4abb70492ad4d9472b38c56981f3e0c25e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <[email protected]>
Date: Thu, 23 Jul 2020 15:11:13 +0200
Subject: [PATCH 1/3] ipatests: re-enable test_sss_ssh_authorizedkeys
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Re-enable test_sss_ssh_authorizedkeys.

Related: https://pagure.io/freeipa/issue/8151
Signed-off-by: François Cami <[email protected]>
---
 ipatests/test_integration/test_commands.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/ipatests/test_integration/test_commands.py b/ipatests/test_integration/test_commands.py
index bacde50416..bfb637cae5 100644
--- a/ipatests/test_integration/test_commands.py
+++ b/ipatests/test_integration/test_commands.py
@@ -1023,7 +1023,6 @@ def is_tls_version_enabled(tls_version):
         assert is_tls_version_enabled('tls1_2')
         assert is_tls_version_enabled('tls1_3')
 
-    @pytest.mark.skip(reason='https://pagure.io/freeipa/issue/8151')
     def test_sss_ssh_authorizedkeys(self):
         """Login via Ssh using private-key for ipa-user should work.
 

From d5421a30fc1c7f66214b8b11e5dd1021fdcdf710 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <[email protected]>
Date: Thu, 23 Jul 2020 15:13:04 +0200
Subject: [PATCH 2/3] ipatests: test_sss_ssh_authorizedkeys
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Add debug information to the ssh invocation.

Related: https://pagure.io/freeipa/issue/8151
Signed-off-by: François Cami <[email protected]>
---
 ipatests/test_integration/test_commands.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipatests/test_integration/test_commands.py b/ipatests/test_integration/test_commands.py
index bfb637cae5..184a70d8d3 100644
--- a/ipatests/test_integration/test_commands.py
+++ b/ipatests/test_integration/test_commands.py
@@ -1067,7 +1067,7 @@ def test_sss_ssh_authorizedkeys(self):
             assert ssh_pub_key in result.stdout_text
             # login to the system
             self.master.run_command(
-                ['ssh', '-o', 'PasswordAuthentication=no',
+                ['ssh', '-v', '-o', 'PasswordAuthentication=no',
                  '-o', 'IdentitiesOnly=yes', '-o', 'StrictHostKeyChecking=no',
                  '-o', 'ConnectTimeout=10', '-l', user, '-i', user_key,
                  self.master.hostname, 'true'])

From 022d7ca3f8b82857584ea8a04a405d17252626e3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <[email protected]>
Date: Thu, 23 Jul 2020 18:19:04 +0200
Subject: [PATCH 3/3] temp commit

---
 .freeipa-pr-ci.yaml                        |   2 +-
 ipatests/prci_definitions/temp_commit.yaml | 187 ++++++++++++++++++++-
 2 files changed, 185 insertions(+), 4 deletions(-)

diff --git a/.freeipa-pr-ci.yaml b/.freeipa-pr-ci.yaml
index abcf8c5b63..8065669008 120000
--- a/.freeipa-pr-ci.yaml
+++ b/.freeipa-pr-ci.yaml
@@ -1 +1 @@
-ipatests/prci_definitions/gating.yaml
\ No newline at end of file
+ipatests/prci_definitions/temp_commit.yaml
\ No newline at end of file
diff --git a/ipatests/prci_definitions/temp_commit.yaml b/ipatests/prci_definitions/temp_commit.yaml
index e337068145..9c09f845f0 100644
--- a/ipatests/prci_definitions/temp_commit.yaml
+++ b/ipatests/prci_definitions/temp_commit.yaml
@@ -61,14 +61,195 @@ jobs:
         timeout: 1800
         topology: *build
 
-  fedora-latest/temp_commit:
+  fedora-latest/test_commands0:
     requires: [fedora-latest/build]
     priority: 50
     job:
       class: RunPytest
       args:
         build_url: '{fedora-latest/build_url}'
-        test_suite: test_integration/test_REPLACEME.py
+        test_suite: test_integration/test_commands.py
         template: *ci-master-latest
         timeout: 3600
-        topology: *master_1repl_1client
+        topology: *master_1repl
+
+  fedora-latest/test_commands1:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands2:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands3:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands4:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands5:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands6:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands7:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands8:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands9:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands10:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands11:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands12:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands13:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands14:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+   
+  fedora-latest/test_commands15:
+    requires: [fedora-latest/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-latest/build_url}'
+        test_suite: test_integration/test_commands.py
+        template: *ci-master-latest
+        timeout: 3600
+        topology: *master_1repl
+
_______________________________________________
FreeIPA-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/[email protected]

Reply via email to