URL: https://github.com/freeipa/freeipa/pull/4411
Author: flo-renaud
 Title: #4411: [ipa-4-6] ipatests: fix KeyError in test_sssd
Action: opened

PR body:
"""
The test was missing a backport defining users['ipa'].
Fixes: https://pagure.io/freeipa/issue/8238
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/4411/head:pr4411
git checkout pr4411
From ce96600c095dcc3a84b221c3a640fb705efde898 Mon Sep 17 00:00:00 2001
From: Florence Blanc-Renaud <f...@redhat.com>
Date: Mon, 23 Mar 2020 09:31:11 +0100
Subject: [PATCH 1/2] ipatests: fix KeyError in test_sssd

The test was missing a backport defining users['ipa'].
Fixes: https://pagure.io/freeipa/issue/8238
---
 ipatests/test_integration/test_sssd.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/ipatests/test_integration/test_sssd.py b/ipatests/test_integration/test_sssd.py
index 2f6de342ae..57b3dada08 100644
--- a/ipatests/test_integration/test_sssd.py
+++ b/ipatests/test_integration/test_sssd.py
@@ -27,6 +27,10 @@ class TestSSSDWithAdTrust(IntegrationTest):
     num_clients = 1
 
     users = {
+        'ipa': {
+            'name': 'user1',
+            'password': 'SecretUser1',
+        }
         'ad': {
             'name_tmpl': 'testuser@{domain}',
             'password': 'Secret123'

From d30fc89ebabc89edf265b801f1888f2261fdba72 Mon Sep 17 00:00:00 2001
From: Florence Blanc-Renaud <f...@redhat.com>
Date: Mon, 23 Mar 2020 09:33:30 +0100
Subject: [PATCH 2/2] Temp commit to run test_sssd please remove before pushing

---
 .freeipa-pr-ci.yaml                        |  2 +-
 ipatests/prci_definitions/temp_commit.yaml | 10 +++++-----
 2 files changed, 6 insertions(+), 6 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 408e598453..e7f9993643 100644
--- a/ipatests/prci_definitions/temp_commit.yaml
+++ b/ipatests/prci_definitions/temp_commit.yaml
@@ -57,14 +57,14 @@ jobs:
         timeout: 1800
         topology: *build
 
-  fedora-27/temp_commit:
+  fedora-27/test_sssd:
     requires: [fedora-27/build]
     priority: 50
     job:
-      class: RunPytest
+      class: RunADTests
       args:
         build_url: '{fedora-27/build_url}'
-        test_suite: test_integration/test_REPLACEME.py
+        test_suite: test_integration/test_sssd.py
         template: *ci-master-f27
-        timeout: 3600
-        topology: *master_1repl_1client
+        timeout: 4800
+        topology: *ad_master_2client
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org
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/freeipa-devel@lists.fedorahosted.org

Reply via email to