Bug #17300
closedAttached devices not saved when editing bond network interfaces
Description
So we upgraded to 1.13.1, found out bonding was broke. I found this PR and applied it to my 1.13.1 instance.
https://github.com/theforeman/foreman/pull/3994
Once i applied that, the form shows bond details, however when i add two interfaces to the bond and click save, the next screen is blank where it should show the interfaces.
Updated by Byron Miller about 8 years ago
unfortunately i can't post an awesome video i made to show this issue as it has data that our security team says is confidential/protected so i'm trying to piece together screenshots. I'm provisioning another host right now to manually bond so that i can create a showcase of logging to show how editing a bond causes the data to disappear since i don't want to break any of my working hosts to demo the edit.
Updated by Dominic Cleal about 8 years ago
- Category set to Network
Can you be more specific about what's blank? Is this the attached devices field - it isn't saving changes in this field for an existing bond?
Or are you adding entirely new interfaces to the list and it isn't showing up when the popup window closes?
I think I see an issue with editing attached devices, but I'd like to check that this is the same issue you're experiencing.
Updated by Byron Miller about 8 years ago
I'm having multiple problems but this one is specific to an existing host that has a bond. If i click edit on the host and try and change the attached interfaces that are associated to the bond in the interfaces field, my changes disappear and the field is blank when i save it. I don't see the interfaces when i then go and preview the kickstart templates either.
I don't see any errors either when this happens. its like its just ignored. I'm working on a test case to try and re-produce this on a non production system but i'm having another problem that discovered hosts don't show the bond drop down even when the bond drop down is selected (and the patch you contributed is applied) so i'm trying to keep those separate.
Updated by Dominic Cleal about 8 years ago
- Subject changed from Bonds were broke in 1.13.1, applied the fix here: https://github.com/theforeman/foreman/pull/3994 but now the form is blank when i try and save edits on existing hosts. to Attached devices not saved when editing bond network interfaces
- Status changed from New to Assigned
- Assignee set to Dominic Cleal
Updated by Dominic Cleal about 8 years ago
Thanks, that confirms it's the same issue I saw here when editing the attached devices field.
The value's being ignored as the parameter filter expects it to be an array (the documented input from the API), but it's a string from the UI. It should permit both.
Updated by The Foreman Bot about 8 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4013 added
Updated by Dominic Cleal about 8 years ago
- Related to Feature #3917: Add strong_parameters to foreman added
Updated by Dominic Cleal about 8 years ago
- Translation missing: en.field_release set to 203
Updated by Dominic Cleal about 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 18780e5cb7f7d0fbcf97b99426217730b1a54635.