Project

General

Profile

Bug #35990

Parameter 'search' on fact_value API endpoint results in internal server error

Added by Nofar Alfassi 2 months ago. Updated 18 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2162736

Description of problem:
Searching for facts using CLI ends with error 500.

Version-Release number of selected component (if applicable):
Satellite 6.13 SNAP 6.0

How reproducible:
always

Steps to Reproduce:
1. hammer fact list --search="ipaddress"

Actual results:
Internal Server Error: the server was unable to finish the request. This may be caused by unavailability of some required service, incorrect API call or a server-side bug. There may be more information in the server's logs.

Expected results:
List of facts containing search phrase

Additional info:

production.log contains a note about missing attribute: fact_name_id

2023-01-20T11:58:45 [I|app|7c831d64] Started GET "/api/fact_values?search=ipaddress&page=1&per_page=1000" for 10.1.2.100 at 2023-01-20 11:58:45 -0500
2023-01-20T11:58:45 [I|app|7c831d64] Processing by Api::V2::FactValuesController#index as JSON
2023-01-20T11:58:45 [I|app|7c831d64]   Parameters: {"search"=>"ipaddress", "page"=>"1", "per_page"=>"1000", "apiv"=>"v2", "fact_value"=>{}}
2023-01-20T11:58:45 [I|app|7c831d64] Authorized user admin(Admin User)
2023-01-20T11:58:45 [W|app|7c831d64] Action failed
2023-01-20T11:58:45 [I|app|7c831d64] Backtrace for 'Action failed' error (ActiveModel::MissingAttributeError): missing attribute: fact_name_id
 7c831d64 | /usr/share/gems/gems/activemodel-6.1.7/lib/active_model/attribute.rb:222:in `value'
 7c831d64 | /usr/share/gems/gems/activemodel-6.1.7/lib/active_model/attribute_set/builder.rb:55:in `block in fetch_value'
 --- shrinked ---
 7c831d64 | /usr/share/gems/gems/puma-5.6.5/lib/puma/server.rb:443:in `process_client'
 7c831d64 | /usr/share/gems/gems/puma-5.6.5/lib/puma/thread_pool.rb:147:in `block in spawn_thread'
 7c831d64 | /usr/share/gems/gems/logging-2.3.1/lib/logging/diagnostic_context.rb:474:in `block in create_with_logging_context'
2023-01-20T11:58:45 [I|app|7c831d64]   Rendered api/v2/errors/custom_error.json.rabl within api/v2/layouts/error_layout (Duration: 1.2ms | Allocations: 318)
2023-01-20T11:58:45 [I|app|7c831d64]   Rendered layout api/v2/layouts/error_layout.json.erb (Duration: 1.9ms | Allocations: 534)
2023-01-20T11:58:45 [I|app|7c831d64] Completed 500 Internal Server Error in 192ms (Views: 3.6ms | ActiveRecord: 16.8ms | Allocations: 12880)

Searching facts using WebUI works correctly.
Same command worked correctly in Satellite 6.12.


Related issues

Has duplicate Foreman - Bug #34990: hammer not inhering facts value Duplicate
Has duplicate Foreman - Bug #35591: Can't use 'search' with fact_value via APIDuplicate

Associated revisions

Revision 9ed17959 (diff)
Added by Nofar Alfassi about 1 month ago

Fixes #35990 - fix search fact_values via API

fix the 500 error for searching fact_values via API

History

#1 Updated by Nofar Alfassi 2 months ago

  • Category deleted (Facts)
  • Subject changed from Can't search facts using CLI to Can't search facts using CLI
  • Project changed from Foreman to Ansible

#2 Updated by Nofar Alfassi 2 months ago

  • Project changed from Ansible to Foreman

#3 Updated by Nofar Alfassi 2 months ago

  • Assignee set to Nofar Alfassi

#4 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Description updated (diff)

I think this is a duplicate of #35591 where I already did the analysis.

#5 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Related to Feature #25591: Remove infoblox parameter named use_ranges added

#6 Updated by The Foreman Bot 2 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9596 added

#7 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases 3.6.0 added

#8 Updated by Nofar Alfassi about 1 month ago

  • Status changed from Ready For Testing to Closed

#9 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Has duplicate Bug #34990: hammer not inhering facts value added

#10 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Related to deleted (Feature #25591: Remove infoblox parameter named use_ranges)

#11 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Has duplicate Bug #35591: Can't use 'search' with fact_value via API added

#12 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Triaged changed from No to Yes
  • Category set to API
  • Subject changed from Can't search facts using CLI to Parameter 'search' on ith fact_value via API

#13 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Target version set to 3.5.2
  • Subject changed from Parameter 'search' on ith fact_value via API to Parameter 'search' on fact_value API endpoint results in internal server error

#14 Updated by The Foreman Bot 27 days ago

  • Pull request https://github.com/theforeman/foreman/pull/9641 added

#15 Updated by The Foreman Bot 18 days ago

  • Fixed in Releases 3.5.3 added

Also available in: Atom PDF