|
 |
bbecb6 |
From 88b9be29036a3580a8bccd31986fc30faa9852df Mon Sep 17 00:00:00 2001
|
|
|
bbecb6 |
From: mbhalodi <mbhalodi@redhat.com>
|
|
|
bbecb6 |
Date: Tue, 14 Feb 2023 15:04:58 +0530
|
|
|
bbecb6 |
Subject: [PATCH] ipatests: ensure that ipa automember-rebuild prints a warning
|
|
|
bbecb6 |
|
|
|
bbecb6 |
ipa automember-rebuild now prints a warning about CPU usage.
|
|
|
bbecb6 |
Ensure that the warning is properly displayed.
|
|
|
bbecb6 |
|
|
|
bbecb6 |
Related: https://pagure.io/freeipa/issue/9320
|
|
|
bbecb6 |
|
|
|
bbecb6 |
Signed-off-by: mbhalodi <mbhalodi@redhat.com>
|
|
|
bbecb6 |
Reviewed-By: Florence Blanc-Renaud <flo@redhat.com>
|
|
|
bbecb6 |
---
|
|
|
bbecb6 |
ipatests/test_integration/test_automember.py | 13 ++++++++++---
|
|
|
bbecb6 |
1 file changed, 10 insertions(+), 3 deletions(-)
|
|
|
bbecb6 |
|
|
|
bbecb6 |
diff --git a/ipatests/test_integration/test_automember.py b/ipatests/test_integration/test_automember.py
|
|
|
bbecb6 |
index f013964140714db046a1aa6a92409244b2137727..7acd0d7bf895fec970f2bda8b54f4496280525b6 100644
|
|
|
bbecb6 |
--- a/ipatests/test_integration/test_automember.py
|
|
|
bbecb6 |
+++ b/ipatests/test_integration/test_automember.py
|
|
|
bbecb6 |
@@ -10,6 +10,9 @@ from ipapython.dn import DN
|
|
|
bbecb6 |
from ipatests.pytest_ipa.integration import tasks
|
|
|
bbecb6 |
from ipatests.test_integration.base import IntegrationTest
|
|
|
bbecb6 |
|
|
|
bbecb6 |
+msg = ('IMPORTANT: In case of a high number of users, hosts or '
|
|
|
bbecb6 |
+ 'groups, the operation may require high CPU usage.')
|
|
|
bbecb6 |
+
|
|
|
bbecb6 |
|
|
|
bbecb6 |
class TestAutounmembership(IntegrationTest):
|
|
|
bbecb6 |
"""Tests for autounmembership feature.
|
|
|
bbecb6 |
@@ -206,11 +209,13 @@ class TestAutounmembership(IntegrationTest):
|
|
|
bbecb6 |
assert self.is_user_member_of_group(user2, group1)
|
|
|
bbecb6 |
|
|
|
bbecb6 |
# Running automember-build so that user is part of correct group
|
|
|
bbecb6 |
- self.master.run_command(['ipa', 'automember-rebuild',
|
|
|
bbecb6 |
- '--users=%s' % user2])
|
|
|
bbecb6 |
+ result = self.master.run_command(['ipa', 'automember-rebuild',
|
|
|
bbecb6 |
+ '--users=%s' % user2])
|
|
|
bbecb6 |
assert self.is_user_member_of_group(user2, group2)
|
|
|
bbecb6 |
assert not self.is_user_member_of_group(user2, group1)
|
|
|
bbecb6 |
|
|
|
bbecb6 |
+ assert msg in result.stdout_text
|
|
|
bbecb6 |
+
|
|
|
bbecb6 |
finally:
|
|
|
bbecb6 |
# testcase cleanup
|
|
|
bbecb6 |
self.remove_user_automember(user2, raiseonerr=False)
|
|
|
bbecb6 |
@@ -240,12 +245,14 @@ class TestAutounmembership(IntegrationTest):
|
|
|
bbecb6 |
assert self.is_host_member_of_hostgroup(host2, hostgroup1)
|
|
|
bbecb6 |
|
|
|
bbecb6 |
# Running the automember-build so host is part of correct hostgroup
|
|
|
bbecb6 |
- self.master.run_command(
|
|
|
bbecb6 |
+ result = self.master.run_command(
|
|
|
bbecb6 |
['ipa', 'automember-rebuild', '--hosts=%s' % host2]
|
|
|
bbecb6 |
)
|
|
|
bbecb6 |
assert self.is_host_member_of_hostgroup(host2, hostgroup2)
|
|
|
bbecb6 |
assert not self.is_host_member_of_hostgroup(host2, hostgroup1)
|
|
|
bbecb6 |
|
|
|
bbecb6 |
+ assert msg in result.stdout_text
|
|
|
bbecb6 |
+
|
|
|
bbecb6 |
finally:
|
|
|
bbecb6 |
# testcase cleanup
|
|
|
bbecb6 |
self.remove_host_automember(host2, raiseonerr=False)
|
|
|
bbecb6 |
--
|
|
|
bbecb6 |
2.39.1
|
|
|
bbecb6 |
|