Alex Lourie has posted comments on this change. Change subject: packaging: Do not allow using ' in passwords ......................................................................
Patch Set 1: Alon First, there's no relation to root password, it is only for the engine itself/the DB. Second, you're right about the existing passwords. Currently, if the password includes it the installation will fail and leave DB in bad state. I prefer we at least do not fail so harsh (thus the fix), in the future patch I intend to fix it fully to allow using ' in passwords. -- To view, visit http://gerrit.ovirt.org/8723 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7fa3cf2e17ff8f5da670f901da7c681b8be86e8c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alou...@redhat.com> Gerrit-Reviewer: Alex Lourie <alou...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com> Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
