Univention Bugzilla – Full Text Bug Listing |
Summary: | Installation succeeds with error messages if something was written to stderr | ||
---|---|---|---|
Product: | UCS | Reporter: | Dirk Wiesenthal <wiesenthal> |
Component: | App Center | Assignee: | Dirk Wiesenthal <wiesenthal> |
Status: | CLOSED FIXED | QA Contact: | Felix Botner <botner> |
Severity: | normal | ||
Priority: | P5 | CC: | botner |
Version: | UCS 4.1 | ||
Target Milestone: | UCS 4.1-4-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
What kind of report is it?: | Bug Report | What type of bug is this?: | 5: Major Usability: Impairs usability in key scenarios |
Who will be affected by this bug?: | 1: Will affect a very few 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.057 | Enterprise Customer affected?: | |
School Customer affected?: | ISV affected?: | Yes | |
Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
Ticket number: | Bug group (optional): | ||
Max CVSS v3 score: |
Description
Dirk Wiesenthal
2017-01-20 16:05:38 CET
We should also lower the log level in some cases, specifically when an ini file is not readable. Ok, as discussed, i give this to you: Just changing the "log level" in the java code (remove res.level == 'WARNING' in AppDetailsPage.js@callInstaller) removes the stdout messages, but also in case of an actual failure. What we want is, no message on success=true, "all" message on success=false. I have added some more test in 40_app_umc_install_latest_appbox (check installation if setup script writes to stderr but returns 0, check installation if setup script writes to stderr and returns 1). Please update install_via_umc in tests/80_docker/dockertest.py according to the fix here. Currently install_via_umc treats 'WARNING', 'ERROR', 'CRITICAL as errors. Fixed in univention-appcenter 5.0.23-59.283.201702180315 by only using ERROR, CRITICAL, not WARNING. This means, to show an error, you have to do MODULE.error in UMC or self.fatal() or raise Abort() - self.warn() will not longer show up. I have raised and lowered some error messages. Also put that into ucs-test 6.0.37-53.1595.201702180320 OK - only critical errors are show OK - merged to 4.2-0 OK - test OK - yaml |