Alon Bar-Lev has posted comments on this change. Change subject: core: adding a comment how upgrade should be tested ......................................................................
Patch Set 2: eli, you are not available in irc. I try to explain that README.developer is for devenv specific subjects, in order to maintain it properly, general product issues should go into other documents. the issue of reusing database when encryption keys are lost is not devenv specific hence should not go into this file. I am sorry if you think otherwise, but I showed you several scenarios that are applicable also in production environment, hence the benefit of documenting it at database specific documentation is greater than documenting it as devenv specific issue. Regardless of the above, I think the ovirt-engine component lacks documentation in-tree, while the policy is to document <something> in the wiki does not prove itself, as wiki tend to be out of sync, especially between versions. I suggest you start a README.db so you can maintain any database related note you may have. -- To view, visit http://gerrit.ovirt.org/35981 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I347967ac38ccfd3f6679c67bf128356c657efda7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches