Bug #13968
closedShow errors when creating proxy even if /features returns an array
Description
When using an old version of Foreman, say '1.10' against a newer version of the proxy, say one that contains the 'logs' feature, the /features call will return an array ['logs'].
If you try to add a smart-proxy that only has this feature, the check we make `reply.is_a?(Array) and reply.any?` is not enough. We should verify we have the features in the db.
Currently the problem is:
- I try to add new smart-proxy with features ['logs'] to old foreman without this feature.
- Submit, it won't save. But it won't throw any error either.
We should display an error and say 'the features in this proxy are not recognized by foreman'.
Updated by The Foreman Bot over 10 years ago
- Status changed from New to Ready For Testing
- Assignee set to Daniel Lobato Garcia
- Pull request https://github.com/theforeman/foreman/pull/3270 added
Updated by Dominic Cleal over 10 years ago
- Has duplicate Feature #14669: Show something like "No supported features available on proxy" instead of quietly do nothing and just write in log "Failed to save" added
Updated by Dominic Cleal almost 10 years ago
- Status changed from Ready For Testing to New
- Assignee deleted (
Daniel Lobato Garcia) - Pull request deleted (
https://github.com/theforeman/foreman/pull/3270)
PR closed due to inactivity, but the bug is probably still valid.
Updated by The Foreman Bot almost 10 years ago
- Status changed from New to Ready For Testing
- Assignee set to Shlomi Zadok
- Pull request https://github.com/theforeman/foreman/pull/3895 added
Updated by Marek Hulán almost 10 years ago
- Target version changed from 1.5.3 to 1.5.0
Updated by The Foreman Bot almost 10 years ago
- Pull request https://github.com/Katello/katello/pull/6377 added
Updated by Shlomi Zadok almost 10 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 61802878d0d8a6b1bf2c6dfca8bfebdc7a8dc7bf.
Updated by Dominic Cleal almost 10 years ago
- Translation missing: en.field_release set to 189
Updated by The Foreman Bot almost 10 years ago
- Pull request https://github.com/theforeman/foreman/pull/3946 added