Project

General

Profile

Actions

Bug #19476

closed

Smart proxy features list should be returned from #refresh API call

Added by Dominic Cleal over 7 years ago. Updated about 6 years ago.

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

Description

When issuing PUT /api/smart_proxies/:id/refresh to Api::V2::SmartProxiesController#refresh, the API response does not contain the feature list:

[ INFO 2017-05-08T15:56:04 API] Server: http://localhost:3000/
[ INFO 2017-05-08T15:56:04 API] PUT /api/smart_proxies/6/refresh
[DEBUG 2017-05-08T15:56:04 API] Params: {}
[DEBUG 2017-05-08T15:56:04 API] Headers: {}
[DEBUG 2017-05-08T15:56:04 API] Using authenticator: HammerCLIForeman::Api::InteractiveBasicAuth
[DEBUG 2017-05-08T15:56:04 API] Response: {
              "id" => 6,
            "name" => "cobalt",
             "url" => "http://cobalt:8443",
      "created_at" => "2013-10-07T10:27:13.104Z",
      "updated_at" => "2017-05-08T14:55:36.698Z",
    "expired_logs" => "1469699580" 
}

The response should match the GET request with a full list of features:

[ INFO 2017-05-08T15:57:47 API] Server: http://localhost:3000/
[ INFO 2017-05-08T15:57:47 API] GET /api/smart_proxies/6
[DEBUG 2017-05-08T15:57:47 API] Params: {}
[DEBUG 2017-05-08T15:57:47 API] Headers: {
    :params => {}
}
[DEBUG 2017-05-08T15:57:47 API] Using authenticator: HammerCLIForeman::Api::InteractiveBasicAuth
[DEBUG 2017-05-08T15:57:47 API] Response: {
    "created_at" => "2013-10-07 10:27:13 UTC",
    "updated_at" => "2017-05-08 14:55:36 UTC",
          "name" => "cobalt",
            "id" => 6,
           "url" => "http://cobalt:8443",
      "features" => [
        [0] {
            "name" => "TFTP",
              "id" => 1
        },
        [1] {
            "name" => "DNS",
              "id" => 2
        },
        [2] {
            "name" => "BMC",
              "id" => 6
        },
        [3] {
            "name" => "Logs",
              "id" => 13
        }
    ]
}

This will allow for the refreshed features to be validated.

Actions #1

Updated by The Foreman Bot over 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4511 added
Actions #2

Updated by Dominic Cleal over 7 years ago

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

Updated by Marek Hulán over 7 years ago

  • Translation missing: en.field_release set to 240
Actions

Also available in: Atom PDF