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

Reply via email to