Bug 42231 - Migrate ucs-overview entries to new 4.2 portal entries
Migrate ucs-overview entries to new 4.2 portal entries
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Portal
UCS 4.2
Other Linux
: P5 normal (vote)
: UCS 4.2
Assigned To: Dirk Wiesenthal
Erik Damrose
: interim-2
: 37897 (view as bug list)
Depends on:
Blocks: 42233 43579 43932
  Show dependency treegraph
 
Reported: 2016-09-02 14:54 CEST by Alexander Kläser
Modified: 2017-04-04 18:29 CEST (History)
4 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

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2016-09-02 14:54:23 CEST
We will need to migrate UCR variable entries for ucs-overview page to new UCS 4.2 portal entries.
Comment 1 Florian Best univentionstaff 2016-09-13 14:32:59 CEST
*** Bug 37897 has been marked as a duplicate of this bug. ***
Comment 2 Florian Best univentionstaff 2017-02-21 15:04:39 CET
Ideas regarding this was:
a listener module which looks for objectType=univentionApp changes triggers in the postrun() a scripts which writes a static json file.
This json file is fetched in the portal via requirement "umc/json!/univention/portal/entries.json". The whole anonymous UMC module can and should for performance reasons be removed (it takes at least 1 second to make the connection to the module process which costs time in rendering).
Comment 3 Alexander Kläser univentionstaff 2017-02-21 15:56:27 CET
We need some sort of documentation for ISVs in order to let them know how to migrate from UCR variables to LDAP entries.
Comment 4 Dirk Wiesenthal univentionstaff 2017-03-01 02:17:56 CET
Using the UCR module of Bug#42234 to migrate all existing variables.
Comment 5 Florian Best univentionstaff 2017-03-02 18:38:27 CET
When opening the module there is a pop up:

Die folgenden leeren Eigenschaften wurdem im Formular auf Vorgabewerte gesetzt. Die Werte werden beim Speichern angewendet.
Allgemein - Erweitert - Authorisation: Visible for Admins only

Can you set a default value? Or add an empty value if possible.
Comment 6 Florian Best univentionstaff 2017-03-02 18:46:19 CET
(In reply to Florian Best from comment #5)
> When opening the module there is a pop up:
> 
> Die folgenden leeren Eigenschaften wurdem im Formular auf Vorgabewerte
> gesetzt. Die Werte werden beim Speichern angewendet.
> Allgemein - Erweitert - Authorisation: Visible for Admins only
> 
> Can you set a default value? Or add an empty value if possible.
→ This is part of Bug #42175
Comment 7 Erik Damrose univentionstaff 2017-03-08 17:20:00 CET
reopen:

* the ucs master overview entry from other roles is converted as-is, so the link to the master is limited to the portal cn=domain,cn=portal,, but i think it should be the local portal... Maybe we could filter out the old overview variables, and set a better link on local portals?

r77283 35univention-portal.inst
* The naming of all UMC links should be tuned. DC-Master and local UMC links do not differ enough in my opinion "System- and domain settings" and "System settings" - and the long description is too long to fit into its area. Also recheck the translations for local umc link
Comment 8 Alexander Kläser univentionstaff 2017-03-09 14:19:12 CET
Would it be possible to ignore the following UCR overview variables, as well:
  ucs/web/overview/entries/service/passwordreset.*
  ucs/web/overview/entries/service/passwordchange.*
Comment 9 Dirk Wiesenthal univentionstaff 2017-03-10 03:15:52 CET
(In reply to Erik Damrose from comment #7)
> reopen:
> 
> * the ucs master overview entry from other roles is converted as-is, so the
> link to the master is limited to the portal cn=domain,cn=portal,, but i
> think it should be the local portal... Maybe we could filter out the old
> overview variables, and set a better link on local portals?

Fixed in r77553. I guess. If not, please point me to a portal where this happens.

> 
> r77283 35univention-portal.inst
> * The naming of all UMC links should be tuned. DC-Master and local UMC links
> do not differ enough in my opinion "System- and domain settings" and "System
> settings" - and the long description is too long to fit into its area. Also
> recheck the translations for local umc link

Fixed translation. I open a new bug for the CSS, because this is generic to the portal.

(In reply to Alexander Kläser from comment #8)
> Would it be possible to ignore the following UCR overview variables, as well:
>   ucs/web/overview/entries/service/passwordreset.*
>   ucs/web/overview/entries/service/passwordchange.*

Done.
Comment 10 Florian Best univentionstaff 2017-03-14 14:56:12 CET
I fixed the cleanup so that the french variables are also removed:

univention-self-service (2.0.9-7):
r77693 | Bug #42231: remove obsolete UCR entries
r77692 | Bug #42231: remove obsolete UCR entries

univention-management-console (9.0.61-2):
r77691 | Bug #42231: remove obsolete entries
Comment 11 Erik Damrose univentionstaff 2017-03-14 17:16:09 CET
OK: master overview links
OK: translation
OK: ignore passwordreset passwordchange

In the future, please name the new bug number if you create a follow up/split bug, makes tracking a lot easier

Verified
Comment 12 Florian Best univentionstaff 2017-03-17 14:05:37 CET
BTW: During the upgrade to UCS 4.2 I see:

Unsetting ucs/web/overview/entries/admin/umc/label
Unsetting ucs/web/overview/entries/admin/umc/label/de
Unsetting ucs/web/overview/entries/admin/umc/description
Unsetting ucs/web/overview/entries/admin/umc/description/de
Script: /etc/univention/templates/scripts/ucs-school-ucr-l10n-fr
File: /var/www/univention/meta.json
Module: create_portal_entries
Traceback (most recent call last):
  File "/usr/share/univention-directory-manager-tools/univention-cli-server", line 222, in doit
    output = univention.admincli.admin.doit(arglist)
  File "/usr/lib/pymodules/python2.7/univention/admincli/admin.py", line 406, in doit
    out = _doit(arglist)
  File "/usr/lib/pymodules/python2.7/univention/admincli/admin.py", line 534, in _doit
    co = univention.admin.config.config(configRegistry['ldap/master'])
  File "/usr/lib/pymodules/python2.7/univention/admin/config.py", line 40, in __init__
    base = univention.admin.uldap.getBaseDN(host)
  File "/usr/lib/pymodules/python2.7/univention/admin/uldap.py", line 136, in getBaseDN
    result = lo.search_s('', ldap.SCOPE_BASE, 'objectClass=*', ['NamingContexts'])
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 559, in search_s
    return self.search_ext_s(base,scope,filterstr,attrlist,attrsonly,None,None,timeout=self.timeout)
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 920, in search_ext_s
    return self._apply_method_s(SimpleLDAPObject.search_ext_s,*args,**kwargs)
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 867, in _apply_method_s
    self.reconnect(self._uri,retry_max=self._retry_max,retry_delay=self._retry_delay)
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 832, in reconnect
    self._apply_last_bind()
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 805, in _apply_last_bind
    SimpleLDAPObject.simple_bind_s(self,'','')
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 214, in simple_bind_s
    msgid = self.simple_bind(who,cred,serverctrls,clientctrls)
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 208, in simple_bind
    return self._ldap_call(self._l.simple_bind,who,cred,RequestControlTuples(serverctrls),RequestControlTuples(clientctrls))
  File "/usr/lib/python2.7/dist-packages/ldap/ldapobject.py", line 106, in _ldap_call
    result = func(*args,**kwargs)
SERVER_DOWN: {'desc': "Can't contact LDAP server"}
Comment 13 Erik Damrose univentionstaff 2017-03-17 14:17:30 CET
This bug is already verified. If you think it is worth reopening, go ahead. Or create a new bug.
Comment 14 Stefan Gohmann univentionstaff 2017-04-04 18:29:47 CEST
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".