Skip to content

Commit 1fe2abb

Browse files
Gauravtalreja1evgeni
authored andcommitted
Fix corrupted_roles check: remove reference to missing 'override' column
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
1 parent ffca4d9 commit 1fe2abb

File tree

2 files changed

+7
-10
lines changed

2 files changed

+7
-10
lines changed

definitions/checks/foreman/check_corrupted_roles.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ def self.inconsistent_filter_perms
3232
filters.role_id,
3333
filters.search,
3434
filters.taxonomy_search,
35-
filters.override,
3635
filterings.id AS filtering_id,
3736
permissions.id AS permission_id,
3837
permissions.name AS permission_name,
@@ -52,8 +51,7 @@ def self.inconsistent_filter_perms
5251
first.permission_name,
5352
first.resource_type,
5453
first.search,
55-
first.taxonomy_search,
56-
first.override
54+
first.taxonomy_search
5755
FROM (#{subquery}) first JOIN (#{subquery}) second
5856
ON first.filter_id = second.filter_id AND
5957
((first.resource_type IS NOT NULL AND second.resource_type IS NULL)

definitions/procedures/foreman/fix_corrupted_roles.rb

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,30 +34,29 @@ def find_records_to_update(inconsistent_sets)
3434
def update_records(set)
3535
new_filter = create_filter set.first['role_id'],
3636
set.first['search'],
37-
set.first['taxonomy_search'],
38-
set.first['override']
37+
set.first['taxonomy_search']
3938
set.each do |item|
4039
destroy_filtering item
4140
next if !new_filter || new_filter.empty?
4241
create_filtering item, new_filter
4342
end
4443
end
4544

46-
def create_filter(role_id, search, taxonomy_search, override)
45+
def create_filter(role_id, search, taxonomy_search)
4746
feature(:foreman_database).query(
48-
create_filter_query(search, role_id, taxonomy_search, override)
47+
create_filter_query(search, role_id, taxonomy_search)
4948
).first
5049
end
5150

5251
def escape_val(value)
5352
value ? "'#{value}'" : 'NULL'
5453
end
5554

56-
def create_filter_query(search, role_id, taxonomy_search, override)
55+
def create_filter_query(search, role_id, taxonomy_search)
5756
<<-SQL
5857
WITH rows AS (
59-
INSERT INTO filters (search, role_id, taxonomy_search, override, created_at, updated_at)
60-
VALUES (#{escape_val(search)}, #{role_id}, #{escape_val(taxonomy_search)}, '#{override}', '#{Time.now}', '#{Time.now}')
58+
INSERT INTO filters (search, role_id, taxonomy_search, created_at, updated_at)
59+
VALUES (#{escape_val(search)}, #{role_id}, #{escape_val(taxonomy_search)}, '#{Time.now}', '#{Time.now}')
6160
RETURNING id
6261
)
6362
SELECT id

0 commit comments

Comments
 (0)