The schema for the dna plugin was changed with the fix for 47379. But the ldif was not added to the schema update script.
when fixing this we should also check if there are other schema changes which could be missed during upgrade
The following schema files need to be added to the upgrade script:
10dna-plugin.ldif 50ns-mail.ldif 60qmail.ldif 60radius.ldif 60samba3.ldif 60posix-winsync-plugin.ldif 60sabayon.ldif 60nis.ldif 60sendmail.ldif
ack. just wondering if it wouldn't be better if the schema files would be listed in order 00xx .. 02xx 10xx ....
Replying to [comment:6 lkrispen]:
I was under the impression the numbering was only used for startup purposes, but just in case I redid the patch. New patch attached.
revision 0001-Ticket-47861-Certain-schema-files-are-not-replaced-d.patch
I think functionally the order is irrelevant, just easier to check if the file is already present in the list if you change a schema file
git merge ticket47861 Updating 65216ab..fe59338 Fast-forward ldap/admin/src/scripts/60upgradeschemafiles.pl | 2 +-
65216ab..fe59338 master -> master commit fe59338
da318fa..daffe97 389-ds-base-1.3.2 -> 389-ds-base-1.3.2 commit daffe97
722117e..ba60488 389-ds-base-1.3.1 -> 389-ds-base-1.3.1 commit ba604880431bc610c5dff92356b7c897d495f0a6
7dc69db..f1551fe 389-ds-base-1.2.11 -> 389-ds-base-1.2.11 commit f1551fe
Metadata Update from @mreynolds: - Issue assigned to mreynolds - Issue set to the milestone: 1.3.2.20
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/1192
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Log in to comment on this ticket.