Actions
Bug #21580
closedCompute attributes returned from api need to be normalized
Difficulty:
Triaged:
Description
Foreman saves compute attributes as a serialized hash that comes directly from the compute attribute form without any pre-processing. Read API returns that hash as is. This approach has several negative side effects. The API response is significantly different from what foreman standardly returns and is quite difficult to use.
Following issues in compute attributes API responses should be fixed:- fileds containing IDs must be named as "*_id"
- the API must provide "*_name" alternatives for all "*_id" fields (where possible) and vice versa
- memory and storage sizes must be provided in common format and units
- empty strings saved from empty form fields must be mapped to nil
- collections must be consistently returned either as a numbered hash or an array
- field names must be in snake case
- boolean values saved as strings like "1", "0", "true", "false" must be casted to true/false
- nics_attributes vs interfaces_attributes must be consistent
Updated by Tomáš Strachota about 7 years ago
- Blocks Feature #6344: Commands for read operations around compute profiles and attributes added
Updated by The Foreman Bot about 7 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4980 added
Updated by Marek Hulán about 7 years ago
- Target version changed from 1.16.2 to 1.16.1
Updated by Marek Hulán almost 7 years ago
- Target version changed from 1.16.1 to 238
Updated by Tomáš Strachota almost 7 years ago
- Related to Bug #22037: Compute attributes create and update API miss validations added
Updated by Marek Hulán over 6 years ago
- Translation missing: en.field_release set to 353
Updated by Tomáš Strachota over 6 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 56de025f01036d037d5bca5ac263aba3e6dbe99f.
Updated by The Foreman Bot over 6 years ago
- Pull request https://github.com/theforeman/foreman/pull/5659 added
Actions