Project

General

Profile

Actions

Bug #9615

closed

while adding alias interface, mac should be auto populated based on attached_to interface

Added by Marek Hulán about 9 years ago. Updated almost 6 years ago.

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

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1193460
Description of problem:
We don't have any validation around mac while adding an interface. I got to know we don't need mac when we add virtual interface.

Now let's say I've to configure alias of eth0, so in this case mac of alias would be same as of eth0. So UI should auto populate same mac as of attached_to interface i.e. eth0.

Or user should restrict to input mac address while defining new alias interface.

I can create alias interface without defining mac and add_interface form saved successfully. But if I do that then interface won't be configured at provisioned node.

In short, we should either of these:

1) either we should restrict user to input mac while alias interface via some validation/warning

2) Or UI should auto populate the mac of attached-to interface.

Version-Release number of selected component (if applicable):
Satellite-6.1.0-RHEL-6-20150210.0
Sat6.1.0 beta snap2

How reproducible:
always

Steps to Reproduce:
1. while creating new host, add alias interface without defining mac
2.
3.

Actual results:
alias interface won't be configured on provisioned node

Expected results:
UI should auto pick the mac of main interface to which we are attaching the alias.

Additional info:

Actions #1

Updated by Marek Hulán about 9 years ago

  • Category set to Network
  • Status changed from New to Assigned

It makes sense to modify the config template so we use parent interface in case the mac was not specified. There are use cases for all combinations.

Actions #2

Updated by The Foreman Bot about 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2222 added
  • Pull request deleted ()
Actions #3

Updated by Dominic Cleal about 9 years ago

  • translation missing: en.field_release set to 28
Actions #4

Updated by Marek Hulán about 9 years ago

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

Also available in: Atom PDF