Eli Mesika has uploaded a new change for review. Change subject: Fixing broken VMs search ......................................................................
Fixing broken VMs search The work done to support the cloud init feature removed the vm_domain column however this column was not removed from the search. Therefore , searching on VMs tab with for example "VMs:vm1" throws the following exception Position: 208: org.springframework.jdbc.BadSqlGrammarException: StatementCallback; bad SQL grammar [SELECT * FROM (SELECT * FROM vms WHERE ( vm_guid IN (SELECT vms_with_tags.vm_guid FROM vms_with_tags WHERE ( vms_with_tags.vm_pool_name ILIKE '%vm1%' OR vms_with_tags.run_on_vds_name ILIKE '%vm1%' OR vms_with_tags.vm_domain ILIKE '%vm1%' OR vms_with_tags.tag_name ILIKE '%vm1%' OR vms_with_tags.guest_cur_user_name ILIKE '%vm1%' OR vms_with_tags.vm_name ILIKE '%vm1%' OR vms_with_tags.vm_description ILIKE '%vm1%' OR vms_with_tags.quota_name ILIKE '%vm1%' OR vms_with_tags.vm_host ILIKE '%vm1%' OR vms_with_tags.vm_ip ILIKE '%vm1%' OR vms_with_tags.storage_pool_name ILIKE '%vm1%' OR vms_with_tags.vds_group_name ILIKE '%vm1%' OR vms_with_tags.vm_comment ILIKE '%vm1%' ) )) ORDER BY vm_name ASC ) as T1 OFFSET (1 -1) LIMIT 100]; nested exception is org.postgresql.util.PSQLException: ERROR: column vms_with_tags.vm_domain does not exist This patch removes the vm_domain column from the search engine code as well. Change-Id: Id4085ca3cff220442a042e0038134c75d7860e61 Signed-off-by: Eli Mesika <[email protected]> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmConditionFieldAutoCompleter.java 1 file changed, 0 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/05/23905/1 diff --git a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmConditionFieldAutoCompleter.java b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmConditionFieldAutoCompleter.java index 50c24fc..467ade0 100644 --- a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmConditionFieldAutoCompleter.java +++ b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VmConditionFieldAutoCompleter.java @@ -76,7 +76,6 @@ columnNameDict.put("STATUS", "status"); columnNameDict.put("IP", "vm_ip"); columnNameDict.put("UPTIME", "elapsed_time"); - columnNameDict.put("DOMAIN", "vm_domain"); columnNameDict.put("OS", "vm_os"); columnNameDict.put("CREATIONDATE", "vm_creation_date"); columnNameDict.put("ADDRESS", "vm_host"); -- To view, visit http://gerrit.ovirt.org/23905 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id4085ca3cff220442a042e0038134c75d7860e61 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Eli Mesika <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
