Project

General

Profile

Bug #13704

Error when updating host location by name through API

Added by David Barbion over 5 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Hi,

I would like to update a host location by providing "location_name" by it results in error: Here is a simple curl command I use:

$ curl -s -X PUT -H "Content-Type: application/json" -k -u hammer:hammer \
  -d '{"host": { "location_name": "mag003"}}' \
  https://foreman.example.com/api/hosts/host.example.com
{
  "error": {"message":"undefined method `id' for nil:NilClass"}
}

When I provide "location_id", it works.

production.log production.log 13.5 KB Backtrace of error David Barbion, 02/15/2016 06:15 AM

Related issues

Related to Foreman - Bug #16813: Confusing error message on association not found errorClosed2016-10-06

Associated revisions

Revision 65af0932 (diff)
Added by Shlomi Zadok over 4 years ago

Fixes #13704 - Defensive association name setter
When setting an association by name
(e.g., hostgroup_name, organization_name)
and the name does not exist, we raise a not found error

Revision 872acf80 (diff)
Added by Shlomi Zadok over 4 years ago

Fixes #13704 - Defensive association name setter
When setting an association by name
(e.g., hostgroup_name, organization_name)
and the name does not exist, we raise a not found error

(cherry picked from commit 65af0932ee5ed8cf9a1ad1ac3c55958fcea77ab7)

History

#1 Updated by David Barbion over 5 years ago

This problem occurs when providing an inexistent location.

In my case, it works if I provide full location_name:

curl -s -X PUT -H "Content-Type: application/json" -k -u hammer:hammer \
  -d '{"host": { "location_name": "Full/path/to/mag003"}}' \
  https://foreman.example.com/api/hosts/host.example.com

#2 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Shlomi Zadok
  • Pull request https://github.com/theforeman/foreman/pull/3851 added

#3 Updated by Shlomi Zadok over 4 years ago

  • Target version set to 1.5.3

#4 Updated by Shlomi Zadok over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal over 4 years ago

  • Legacy Backlogs Release (now unused) set to 190

#6 Updated by Marek Hulán over 4 years ago

  • Related to Bug #16813: Confusing error message on association not found error added

#7 Updated by Marek Hulán over 4 years ago

  • Bugzilla link set to 1379006

Also available in: Atom PDF