Martin Peřina has posted comments on this change. Change subject: db: add config key with same value to multiple versions ......................................................................
Patch Set 3: (1 comment) http://gerrit.ovirt.org/#/c/35114/3/packaging/dbscripts/common_sp.sql File packaging/dbscripts/common_sp.sql: Line 606: returns void Line 607: AS $procedure$ Line 608: declare Line 609: m varchar; Line 610: arr varchar[] := array['3.0', '3.1', '3.2', '3.3', '3.4', '3.5', '3.6']; Please add info here and to org.ovirt.engine.core.compat.Version that this array and Version content should be in sync. Btw if 3.6 is here, we should add it also to Version, right? Line 611: begin Line 612: FOREACH m IN ARRAY arr Line 613: LOOP Line 614: perform fn_db_add_config_value(v_option_name, v_val, m); -- To view, visit http://gerrit.ovirt.org/35114 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I432038577c1e695d2821471a607ab842ea64c97b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Liran Zelkha <lzel...@redhat.com> Gerrit-Reviewer: Martin Peřina <mper...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches