Bug 40727 – 74_radius_internetrules_priority fails sometimes

Bug 40727 - 74_radius_internetrules_priority fails sometimes


Summary:	74_radius_internetrules_priority fails sometimes

Status:	CLOSED FIXED

Product:	UCS@school
Classification:	Unclassified
Component:	ucs-test
Version:	UCS@school 4.1
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS@school 4.3 v4
Assigned To:	Jürn Brodersen
QA Contact:	Sönke Schwardt-Krummrich

URL:	http://jenkins.knut.univention.de:808...
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-02-18 12:12 CET by Florian Best
Modified:	2019-01-06 21:23 CET (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Development Internal
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2016-02-18 12:12:57 CET

[2016-02-17 23:13:03.858617]Waiting for replication...
[2016-02-17 23:13:04.027615]<closed file '<fdopen>', mode 'rb' at 0x3544c90>
[2016-02-17 23:13:05.133694]<closed file '<fdopen>', mode 'rb' at 0x3544ae0>
[2016-02-17 23:13:06.220712]Done: replication complete.
(2016-02-17 23:13:06.220750)Traceback (most recent call last):
(2016-02-17 23:13:06.220762)  File "74_radius_internetrules_priority", line 67, in <module>
(2016-02-17 23:13:06.220770)    main()
(2016-02-17 23:13:06.220779)  File "74_radius_internetrules_priority", line 64, in main
(2016-02-17 23:13:06.220795)    test_peap_auth(username, password, radius_secret, should_succeed=should_succeed)
(2016-02-17 23:13:06.220805)  File "/usr/share/ucs-test/90_ucsschool/essential/radius.py", line 33, in test_peap_auth
(2016-02-17 23:13:06.220814)    auth_result, should_succeed, username))
(2016-02-17 23:13:06.220825)essential.radius.TestFail: PEAP authentication unexpected result (False), while the expected is (True)
(2016-02-17 23:13:06.220833)User=dpl0qldx3g

Comment 1 Sönke Schwardt-Krummrich

2018-03-20 13:07:44 CET

Package: ucs-test-ucsschool
Version: 5.0.2-22A~4.3.0.201803201305
Branch: ucs_4.3-0
Scope: ucs-school-4.3

ucs-test-ucsschool (5.0.2-22)
237bb87fa091 | Bug #40727: wait in 74_radius_internetrules_priority for LDAP and S4 replication

Comment 2 Sönke Schwardt-Krummrich

2018-03-20 13:09:04 CET

ucs-test-ucsschool has been uploaded to test app center as U@S 4.3v2

Comment 3 Jürn Brodersen

2018-06-14 15:21:10 CEST

The test used a new umc connection for every internetrule and workgroup it created. If I count correctly that results in 16 module processes.

On my system this test consumed over 700M of memory. See also bug 32118.

The test might fail because the system starts swapping, which might result in a timing problem somewhere. Anyway it's probably a good idea to use just one umc connection.

[4.3 171d269ca] Bug #40727: reuse umc connection to save memory

Comment 4 Jürn Brodersen

2018-06-21 13:19:31 CEST

Found it :)

internetrule.py which is used by 74_radius_internetrules_priority did randomize the rule priority if the priority was set to zero for a new rule.

[4.3 2b54e0774] Bug #40727: Fix bug in essential/internetrule.py
[4.2 babdf02b0] Bug #40727: Fix bug in essential/internetrule.py

Comment 5 Sönke Schwardt-Krummrich

2018-12-12 18:20:31 CET

Test does not fail anymore.
Good catch.