URL: https://github.com/freeipa/freeipa/pull/4468
Author: fcami
 Title: #4468: [Backport][ipa-4-8] pr-ci templates: update test_fips timeouts
Action: opened

PR body:
"""
MANUAL CHERRY-PICK of:

- b205bf8aca09efca90cacc28a2fcf148e90f53af
- 7141e0702fe778f66946ae0f62eddd0ac1bd991e

    pr-ci templates: update test_fips timeouts

    test_fips takes between 45 and ~80 mins to run.
    The templates' timeout was 3600s which is too short for
    successful execution. 7200s should do.
    
    Fixes: https://pagure.io/freeipa/issue/8247
    Signed-off-by: François Cami <fc...@redhat.com>
    Reviewed-By: Armando Neto <abiag...@redhat.com>
+

    Remove Fedora repository fastmirror selection
    
    Fast mirror selection somehow stopped working. If disabled, the
    difference is around 20 seconds for the 'Prepare build environment' step
    (2:49 versus 3:09), so while we are saving, currently it is not a lot.
    
    Also remove explicit nodejs stream choice, it seems to be not needed
    anymore (again).
    
    Signed-off-by: Alexander Bokovoy <aboko...@redhat.com>
    Reviewed-By: Armando Neto <abiag...@redhat.com>

"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/4468/head:pr4468
git checkout pr4468
From b205bf8aca09efca90cacc28a2fcf148e90f53af Mon Sep 17 00:00:00 2001
From: Alexander Bokovoy <aboko...@redhat.com>
Date: Fri, 27 Mar 2020 20:04:36 +0200
Subject: [PATCH 1/2] Remove Fedora repository fastmirror selection

Fast mirror selection somehow stopped working. If disabled, the
difference is around 20 seconds for the 'Prepare build environment' step
(2:49 versus 3:09), so while we are saving, currently it is not a lot.

Also remove explicit nodejs stream choice, it seems to be not needed
anymore (again).

Signed-off-by: Alexander Bokovoy <aboko...@redhat.com>
Reviewed-By: Armando Neto <abiag...@redhat.com>
---
 ipatests/azure/templates/prepare-build-fedora.yml | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/ipatests/azure/templates/prepare-build-fedora.yml b/ipatests/azure/templates/prepare-build-fedora.yml
index 643910f085..9de1be101e 100644
--- a/ipatests/azure/templates/prepare-build-fedora.yml
+++ b/ipatests/azure/templates/prepare-build-fedora.yml
@@ -2,18 +2,6 @@ steps:
 - script: |
     set -e
     sudo rm -rf /var/cache/dnf/*
-    echo "dnf.conf: enable fastestmirror, use 8 download workers, lower timeout to fail faster, and add more retries"
-    sudo tee -a /etc/dnf/dnf.conf <<EOF > /dev/null
-    fastestmirror = True
-    max_parallel_downloads = 8
-    timeout = 8
-    retries = 20
-    EOF
-    echo "Fedora mirror metalink content:"
-    for metalink in $(sudo dnf repolist -v |grep Repo-metalink | awk '{print $2}' ) ; do echo '###############' ; echo '####' ; echo $metalink ; echo '####' ; curl $metalink ; done
-    echo "Fastestmirror results:"
-    sudo cat /var/cache/dnf/fastestmirror.cache
-    sudo dnf -y module enable nodejs:12
     sudo dnf makecache || :
     echo "Installing base development environment"
     sudo dnf install -y \
@@ -29,5 +17,6 @@ steps:
         python3-pyyaml \
 
     echo "Installing FreeIPA development dependencies"
+    sudo dnf builddep -y freeipa
     sudo dnf builddep -y --skip-broken -D "with_wheels 1" -D "with_lint 1" --spec freeipa.spec.in --best --allowerasing --setopt=install_weak_deps=False
   displayName: Prepare build environment

From 7141e0702fe778f66946ae0f62eddd0ac1bd991e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <fc...@redhat.com>
Date: Sat, 28 Mar 2020 11:57:16 +0100
Subject: [PATCH 2/2] pr-ci templates: update test_fips timeouts
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

test_fips takes between 45 and ~80 mins to run.
The templates' timeout was 3600s which is too short for
successful execution. 7200s should do.

Fixes: https://pagure.io/freeipa/issue/8247
Signed-off-by: François Cami <fc...@redhat.com>
Reviewed-By: Armando Neto <abiag...@redhat.com>
---
 ipatests/prci_definitions/nightly_ipa-4-8_latest.yaml   | 2 +-
 ipatests/prci_definitions/nightly_ipa-4-8_previous.yaml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/ipatests/prci_definitions/nightly_ipa-4-8_latest.yaml b/ipatests/prci_definitions/nightly_ipa-4-8_latest.yaml
index 06460dc508..ca31fbf1cd 100644
--- a/ipatests/prci_definitions/nightly_ipa-4-8_latest.yaml
+++ b/ipatests/prci_definitions/nightly_ipa-4-8_latest.yaml
@@ -172,7 +172,7 @@ jobs:
         build_url: '{fedora-latest-ipa-4-8/build_url}'
         test_suite: test_integration/test_fips.py
         template: *ci-ipa-4-8-latest
-        timeout: 3600
+        timeout: 7200
         topology: *master_1repl_1client
 
   fedora-latest-ipa-4-8/test_forced_client_enrolment:
diff --git a/ipatests/prci_definitions/nightly_ipa-4-8_previous.yaml b/ipatests/prci_definitions/nightly_ipa-4-8_previous.yaml
index ba28caf6a5..3a263cb29e 100644
--- a/ipatests/prci_definitions/nightly_ipa-4-8_previous.yaml
+++ b/ipatests/prci_definitions/nightly_ipa-4-8_previous.yaml
@@ -172,7 +172,7 @@ jobs:
         build_url: '{fedora-previous-ipa-4-8/build_url}'
         test_suite: test_integration/test_fips.py
         template: *ci-ipa-4-8-previous
-        timeout: 3600
+        timeout: 7200
         topology: *master_1repl_1client
 
   fedora-previous-ipa-4-8/test_forced_client_enrolment:
_______________________________________________
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