URL: https://github.com/freeipa/freeipa/pull/4156
Author: wladich
 Title: #4156: [Backport][ipa-4-6] Add test_winsyncmigrate to nightly builds
Action: opened

PR body:
"""
This is a manual backport of #4113

The test suite test_winsyncmigrate was missing in nightly definitions
because CI was lacking configuration needed for establishing winsync
agreement: the Certificate Authority needs to be configured on
Windows AD instance. Now that PR-CI is updated to include said changes, we
can start executing this test suite. It is not reasonable to add it to
gating as this suite is time consuming just like other tests requiring
provisioning of AD instances.

PRs with changes required for executing the test suite:
freeipa/freeipa-pr-ci#323
freeipa/freeipa-pr-ci#335

Stability of the test suite was checked on a private prci runner: wladich#5
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/4156/head:pr4156
git checkout pr4156
From 1bb3d3c23a0ac112ec202e1333bb958909ff8853 Mon Sep 17 00:00:00 2001
From: Sergey Orlov <sor...@redhat.com>
Date: Thu, 23 Jan 2020 16:55:27 +0100
Subject: [PATCH 1/3] ipatests: add test_winsyncmigrate suite to nightly runs

The test suite test_winsyncmigrate was missing in nightly definitions
because CI was lacking configuration needed for establishing winsync
agreement: the Certificate Authority needs to be configured on
Windows AD instance. Now that PR-CI is updated to include said changes, we
can start executing this test suite. It is not reasonable to add it to
gating as this suite is time consuming just like other tests requiring
provisioning of AD instances.
---
 ipatests/prci_definitions/nightly_ipa-4-6.yaml | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/ipatests/prci_definitions/nightly_ipa-4-6.yaml b/ipatests/prci_definitions/nightly_ipa-4-6.yaml
index 62d464d981..d8f975d162 100644
--- a/ipatests/prci_definitions/nightly_ipa-4-6.yaml
+++ b/ipatests/prci_definitions/nightly_ipa-4-6.yaml
@@ -27,6 +27,10 @@ topologies:
     name: master_3repl_1client
     cpu: 6
     memory: 12900
+  ad_master: &ad_master
+    name: ad_master
+    cpu: 4
+    memory: 12000
 
 jobs:
   fedora-27/build:
@@ -1074,3 +1078,15 @@ jobs:
         template: *ci-master-f27
         timeout: 3600
         topology: *master_1repl
+
+  fedora-27/test_winsyncmigrate:
+    requires: [fedora-27/build]
+    priority: 50
+    job:
+      class: RunADTests
+      args:
+        build_url: '{fedora-27/build_url}'
+        test_suite: test_integration/test_winsyncmigrate.py
+        template: *ci-master-f27
+        timeout: 4800
+        topology: *ad_master

From 9e2ed97826f13c64a9a49d1e707ff8e833ab51ee Mon Sep 17 00:00:00 2001
From: Sergey Orlov <sor...@redhat.com>
Date: Thu, 23 Jan 2020 16:58:23 +0100
Subject: [PATCH 2/3] Add convenient template for temp commits

---
 ipatests/prci_definitions/temp_commit.yaml | 70 ++++++++++++++++++++++
 1 file changed, 70 insertions(+)
 create mode 100644 ipatests/prci_definitions/temp_commit.yaml

diff --git a/ipatests/prci_definitions/temp_commit.yaml b/ipatests/prci_definitions/temp_commit.yaml
new file mode 100644
index 0000000000..408e598453
--- /dev/null
+++ b/ipatests/prci_definitions/temp_commit.yaml
@@ -0,0 +1,70 @@
+#
+# Template for temporary test commit
+#
+# $ ln -sf ipatests/prci_definitions/temp_commit.yaml .freeipa-pr-ci.yaml
+#
+
+topologies:
+  build: &build
+    name: build
+    cpu: 2
+    memory: 3800
+  master_3client: &master_3client
+    name: master_3client
+    cpu: 5
+    memory: 10150
+  master_1repl: &master_1repl
+    name: master_1repl
+    cpu: 4
+    memory: 6450
+  master_1repl_1client: &master_1repl_1client
+    name: master_1repl_1client
+    cpu: 4
+    memory: 7400
+  ipaserver: &ipaserver
+    name: ipaserver
+    cpu: 2
+    memory: 2400
+  master_2repl_1client: &master_2repl_1client
+    name: master_2repl_1client
+    cpu: 5
+    memory: 10150
+  master_3repl_1client: &master_3repl_1client
+    name: master_3repl_1client
+    cpu: 6
+    memory: 12900
+  ad_master_2client: &ad_master_2client
+    name: ad_master_2client
+    cpu: 4
+    memory: 12000
+  ad_master: &ad_master
+   name: ad_master
+   cpu: 4
+   memory: 12000
+
+jobs:
+  fedora-27/build:
+    requires: []
+    priority: 100
+    job:
+      class: Build
+      args:
+        git_repo: '{git_repo}'
+        git_refspec: '{git_refspec}'
+        template: &ci-master-f27
+          name: freeipa/ci-ipa-4-6-f27
+          version: 1.0.3
+        timeout: 1800
+        topology: *build
+
+  fedora-27/temp_commit:
+    requires: [fedora-27/build]
+    priority: 50
+    job:
+      class: RunPytest
+      args:
+        build_url: '{fedora-27/build_url}'
+        test_suite: test_integration/test_REPLACEME.py
+        template: *ci-master-f27
+        timeout: 3600
+        topology: *master_1repl_1client

From 84402bc5aff9063807096a8fe5478ef7a9119ae8 Mon Sep 17 00:00:00 2001
From: Sergey Orlov <sor...@redhat.com>
Date: Thu, 23 Jan 2020 16:59:10 +0100
Subject: [PATCH 3/3] temp commit

---
 ipatests/prci_definitions/temp_commit.yaml | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/ipatests/prci_definitions/temp_commit.yaml b/ipatests/prci_definitions/temp_commit.yaml
index 408e598453..9b13df5960 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_winsyncmigrate:
     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_winsyncmigrate.py
         template: *ci-master-f27
-        timeout: 3600
-        topology: *master_1repl_1client
+        timeout: 4800
+        topology: *ad_master
_______________________________________________
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