Actions
Bug #9147
closedNew NIC code causes orchestration issues with plugins due to ordering issues
Difficulty:
trivial
Triaged:
Pull request:
Description
We currently have checks for host.managed? and managed?in the dhp/dns/tftp orchestration, but they are not the first thing to be checked. as such, it's possible to call methods that don't exist for an STI host class with a new nic (eg discovery will attempt to call ip_available? on import, and fail)
We should move the managed checks to the beginning, since if a host/nic is not managed, it makes no sense to orchestrate anyway.
Updated by The Foreman Bot almost 10 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2116 added
- Pull request deleted (
)
Updated by Marek Hulán over 9 years ago
- Pull request https://github.com/theforeman/foreman/pull/2204 added
- Pull request deleted (
https://github.com/theforeman/foreman/pull/2116)
Updated by Greg Sutcliffe over 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 5440d482b0087fe29b3fb4567e0ca73f2d3af221.
Updated by Dominic Cleal over 9 years ago
- Translation missing: en.field_release set to 28
Updated by Bryan Kearney over 9 years ago
- Assignee changed from Greg Sutcliffe to Marek Hulán
Actions