Bug 46712 – [4.3] Missing log summary after dryrun

Bug 46712 - [4.3] Missing log summary after dryrun


Summary:	[4.3] Missing log summary after dryrun

Status:	CLOSED FIXED

Product:	UCS@school
Classification:	Unclassified
Component:	HTTP-API (Kelvin)
Version:	unspecified
Hardware:	Other Mac OS X 10.1

Importance:	P5 normal (vote)
Target Milestone:	UCS@school 4.3 v5
Assigned To:	Daniel Tröder
QA Contact:	Jürn Brodersen

URL:
Keywords:

Depends on:
Blocks:	45019 46838 47465
	Show dependency tree / graph

Reported:	2018-03-21 12:51 CET by Michel Smidt
Modified:	2018-09-11 11:34 CEST (History)
CC List:	2 users (show)

See Also:	45165
What kind of report is it?:	Bug Report
What type of bug is this?:	2: Improvement: Would be a product improvement
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.023
Enterprise Customer affected?:
School Customer affected?:	Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	Usability
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 Michel Smidt 2018-03-21 12:51:01 CET

Currently a dryrun in the Web-Import do not show a summary like X created, Y modified, Z deleted and 0 errors.

Comment 1 Daniel Tröder

2018-07-03 12:37:09 CEST

The textual summary as displayed on the cmdline is now shown in the UMC import module.

Code is in branch dtroeder/46712_log.summary.after.dryrun, reopen/mail to have it merged and built in 4.3.

[dtroeder/46712_log.summary.after.dryrun 848bae5ca] Bug #46712: display summary after dry-run in UMC import module

Comment 2 Daniel Tröder

2018-07-06 11:24:13 CEST

Merged to 4.3 and built.

[4.3] 0006d0e19 Bug #46712: display summary after dry-run in UMC import module
[4.3] ea0db55b5 Bug #46712: changelog
[4.3] fcea31e51 Bug #46712: advisory

ucs-school-import (16.0.2-23)

Comment 3 Jürn Brodersen

2018-08-07 11:11:32 CEST

Reopen
I would like a short help text where to find the full import log (/var/lib/ucs-school-import/jobs/$YEAR/$job_id). That would be especially handy if an error had happened.

Set this bug to resolved again if you think that is outside the scope of this bug.

Otherwise the changes look good :)

What I tested:
Import with errors: Job id is now shown -> OK
Import without errors: Summary is shown -> OK

Comment 4 Daniel Tröder

2018-08-07 12:21:34 CEST

(In reply to Jürn Brodersen from comment #3)
> I would like a short help text where to find the full import log
> (/var/lib/ucs-school-import/jobs/$YEAR/$job_id). That would be especially
> handy if an error had happened.
> 
> Set this bug to resolved again if you think that is outside the scope of
> this bug.
Bug #47465 will handle this.

Comment 5 Jürn Brodersen

2018-08-07 13:36:47 CEST

(In reply to Daniel Tröder from comment #4)
> (In reply to Jürn Brodersen from comment #3)
> > I would like a short help text where to find the full import log
> > (/var/lib/ucs-school-import/jobs/$YEAR/$job_id). That would be especially
> > handy if an error had happened.
> > 
> > Set this bug to resolved again if you think that is outside the scope of
> > this bug.
> Bug #47465 will handle this.

OK
-> verified

Comment 6 Sönke Schwardt-Krummrich

2018-09-11 11:34:42 CEST

UCS@school 4.3 v5 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.3v5-de.html

If this error occurs again, please clone this bug.