Univention Bugzilla – Full Text Bug Listing |
Summary: | App Center update performance | ||
---|---|---|---|
Product: | UCS | Reporter: | Stefan Gohmann <gohmann> |
Component: | App Center | Assignee: | Dirk Wiesenthal <wiesenthal> |
Status: | CLOSED FIXED | QA Contact: | Stefan Gohmann <gohmann> |
Severity: | normal | ||
Priority: | P5 | Keywords: | interim-3 |
Version: | UCS 4.2 | ||
Target Milestone: | UCS 4.2 | ||
Hardware: | Other | ||
OS: | Linux | ||
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: | |||
Bug Depends on: | |||
Bug Blocks: | 43496, 43847, 45080 |
Description
Stefan Gohmann
2017-02-27 21:04:51 CET
I think we should simply sign the tar file and extract it in one step. (In reply to Stefan Gohmann from comment #1) > I think we should simply sign the tar file and extract it in one step. This sounds like a good idea. Extracting all.tar.gz "unsupervised" is quite fast. time univention-app update real 0m7.040s user 0m4.120s sys 0m0.356s rm -r /var/cache/univention-appcenter/appcenter.software-univention.de/ time univention-app update real 0m14.282s user 0m11.676s sys 0m1.564s Is it enough? I now added all.tar.gz files for the Test App Center to the univention-appcenter-dev package. This eases the pain of switching. Super easy to do. Downside is that the archive will soon be outdated. So this helps only if we update the archives regularly (say, 4.2-0, 4.2-1, ...) REOPEN if you think that we should sign and extract the archives right away with errata0. Downloaded archives (after a month or so) should be slow again. But keep in mind that in the current implementation, you only have to extract the "diff". It is OK for now, I'll create a new bug that we will fix it more generally. Please add a changelog entry. (In reply to Stefan Gohmann from comment #3) > It is OK for now, I'll create a new bug that we will fix it more generally. Bug #43847 Please also adjust the test case: http://jenkins.knut.univention.de:8080/job/UCS-4.2/job/UCS-4.2-0/job/AutotestJoin/75/SambaVersion=s3,Systemrolle=master/testReport/20_appcenter/60_check_appcenter_performance/test/ I would increase it to 9 seconds. Changelog entry added, threshold adjusted. (In reply to Dirk Wiesenthal from comment #6) > Changelog entry added, threshold adjusted. I didn't find a changelog commit. Maybe, it is still in your working copy? Changelog added. For tests regarding the update performance: Did you use ucr set repository/app_center/server= or univention-app dev-use-test-appcenter Because the tar.gz for the Test App Center comes with the package univention-appcenter-dev Without this package, the Test App Center cache cannot be "prefilled" with an archive. I have fixed the detection of file changes one more time in univention-appcenter (6.0.3-16) No, I didn't use univention-appcenter-dev. Anyway, the real fix will be implemented with Bug #43847. UCS 4.2 has been released: https://docs.software-univention.de/release-notes-4.2-0-en.html https://docs.software-univention.de/release-notes-4.2-0-de.html If this error occurs again, please use "Clone This Bug". |