Project

General

Profile

Actions

Bug #17300

closed

Attached devices not saved when editing bond network interfaces

Added by Byron Miller over 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Network
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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.


Related issues 1 (0 open1 closed)

Related to Foreman - Feature #3917: Add strong_parameters to foremanClosedDominic Cleal12/19/2013Actions
Actions #1

Updated by Byron Miller over 7 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.

Actions #2

Updated by Dominic Cleal over 7 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.

Actions #3

Updated by Byron Miller over 7 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.

Actions #4

Updated by Dominic Cleal over 7 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
Actions #5

Updated by Dominic Cleal over 7 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.

Actions #6

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/4013 added
Actions #7

Updated by Dominic Cleal over 7 years ago

  • Related to Feature #3917: Add strong_parameters to foreman added
Actions #8

Updated by Dominic Cleal over 7 years ago

  • translation missing: en.field_release set to 203
Actions #9

Updated by Dominic Cleal over 7 years ago

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

Also available in: Atom PDF