Bug 44250 - 11_squidguard_test_dbtemp_option.test: Unexpected number of temporary backing files
11_squidguard_test_dbtemp_option.test: Unexpected number of temporary backing...
Status: REOPENED
Product: UCS@school
Classification: Unclassified
Component: ucs-test
UCS@school 5.0
Other Linux
: P5 normal (vote)
: UCS@school 4.2 vXXX
Assigned To: UCS@school maintainers
:
Depends on:
Blocks: 44297
  Show dependency treegraph
 
Reported: 2017-04-03 21:27 CEST by Sönke Schwardt-Krummrich
Modified: 2023-10-23 12:29 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.137
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

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2017-04-03 21:27:55 CEST
11_squidguard_test_dbtemp_option.test tries to check for temporary backing files of squidguard. Somehow the newest version of squidguard does not create any.

[2017-04-03 06:16:40.312329] 2017-04-03 06:16:39 [7365] Added UserList source: jane999
[2017-04-03 06:16:40.312337] 2017-04-03 06:16:39 [7365] Added UserList source: mydomain\jane999
[2017-04-03 06:16:40.312346] 2017-04-03 06:16:39 [7365] init domainlist /tmp/tmp.ucs-test.squidguard.M6TJEd//whitelisted-domains
[2017-04-03 06:16:40.312360] 2017-04-03 06:16:39 [7365] INFO: loading dbfile /tmp/tmp.ucs-test.squidguard.M6TJEd//whitelisted-domains.db
[2017-04-03 06:16:40.312371] 2017-04-03 06:16:39 [7365] init urllist /tmp/tmp.ucs-test.squidguard.M6TJEd//whitelisted-urls
[2017-04-03 06:16:40.312380] 2017-04-03 06:16:39 [7365] INFO: loading dbfile /tmp/tmp.ucs-test.squidguard.M6TJEd//whitelisted-urls.db
[2017-04-03 06:16:40.312389] 2017-04-03 06:16:39 [7365] INFO: squidGuard 1.5 started (1491214599.124)
[2017-04-03 06:16:40.312397] 2017-04-03 06:16:39 [7365] INFO: squidGuard ready for requests (1491214599.663)
[2017-04-03 06:16:40.312405] 2017-04-03 06:16:39 [7365] INFO: squidGuard stopped (1491214599.663)
[2017-04-03 06:16:40.312413] Counts: /var/tmp: 0 ==> 0    /tmp/tmp.ucs-test.squidguard.M6TJEd: 0 ==> 0
[2017-04-03 06:16:40.312420] ### FAIL ###
[2017-04-03 06:16:40.312440] Unexpected number of temporary backing files
[2017-04-03 06:16:40.312450] ###      ###

http://jenkins.knut.univention.de:8080/job/UCSschool%204.2/job/UCSschool%204.2%20Singleserver/ImportTests=NoImportTests,SambaVersion=s4/lastCompletedBuild/testReport/90_ucsschool/11_squidguard_test_dbtemp_option/test/
Comment 1 Daniel Tröder univentionstaff 2017-04-05 15:51:04 CEST
r78668: deactivated the test.

Squidguard now uses "safe temporary files": (open(); delete(); use();) No BDB files are left.
The dbtemp option does still work though:

When running 11_squidguard_test_dbtemp_option on 4.1r2 with configuration:
----
dbtemp /tmp/tmp.ucs-test.squidguard.Bug7uy
----

# ps ax| grep guard
# lsof -p 5945
squidGuar 10557            root    5u      REG              253,0   2392064    4971033 /tmp/tmp.ucs-test.squidguard.Bug7uy/BDB10557

==========================

When running 11_squidguard_test_dbtemp_option on 4.2 with configuration:
----
dbtemp /tmp/tmp.ucs-test.squidguard.Okfv5S
----

# ps ax | grep guar
# lsof -p 7109
squidGuar 7109 root    6u   REG  253,0  2383872 4971571 /tmp/tmp.ucs-test.squidguard.Okfv5S/BDB07109 (deleted)


When running 11_squidguard_test_dbtemp_option on 4.2 with "dbtemp" unset:

squidGuar 7320 root    6u   REG  253,0  2383872  523643 /var/tmp/BDB07320 (deleted)

==========================

Bug #44297 was created to deal with the ucs-school-webproxy package.
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2017-04-05 16:23:24 CEST
We should adapt the test: 
the script should check that no temporary backing files are "generated" (== non-deleted temporary backing files).
Comment 3 Michel Smidt 2020-07-09 14:38:18 CEST
This issue has been filed against UCS@school 4.2.

UCS@school 4.2 is out of maintenance and many UCS@school components have changed in later releases. Thus, this issue is now being closed.

If this issue still occurs in newer UCS@school versions, please reopen it and update the UCS@school version. In this case please provide detailed information on how this issue is affecting you.
Comment 4 J Leadbetter univentionstaff 2023-10-23 12:29:48 CEST
This test is still failing in 5.0-5. Reopening. I have created this Automated Tests issue in gitlab to handle this as part of Test Team: https://git.knut.univention.de/univention/ucsschool/-/issues/1137