Project

General

Profile

Actions

Feature #1882

open

Automatically import available subnets from the smart proxy

Added by Sam Kottler over 11 years ago. Updated almost 11 years ago.

Status:
New
Priority:
Normal
Assignee:
Sam Kottler
Category:
Proxy gateway
Target version:
-
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

It should be possible to import subnets for the use to make available. The potential values could be imported based on the subnets managed by the DHCP server that the proxy is connected to.

Actions #1

Updated by Greg Sutcliffe over 11 years ago

I thought we did this already? Is there not an "Import Subnets" button associated with a DHCP-managing proxy?

Actions #2

Updated by Sam Kottler over 11 years ago

Hadn't seen that before...

We should add it to the subnets page to keep the UI consistent with how we handle classes.

Actions #3

Updated by Ohad Levy over 11 years ago

+1, so its just to add a button to the subnets ui?

Actions #4

Updated by Sam Kottler over 11 years ago

  • Category set to Proxy gateway
  • Target version set to 1.1

Exactly, @Ohad Levy. I will submit a pull request for it soon; tagging for 1.1.

Actions #5

Updated by Ohad Levy over 11 years ago

  • Assignee set to Sam Kottler
Actions #6

Updated by Sam Kottler over 11 years ago

  • Status changed from New to Feedback
  • Priority changed from Low to Normal

I've opened a pull request for this at https://github.com/theforeman/foreman/pull/307.

Actions #7

Updated by Ohad Levy over 11 years ago

commented on the PR

Actions #8

Updated by Mikael Fridh over 11 years ago

I was previously looking at parsing isc dhcpd config to automatically populate the range fields etc, could be done in a naive way rather easily, but getting all the edge cases right didn't really feel simple enough to be worth it. What I mean is, if there is a range from to use that, if there is a group{} with deny unknown-clients and a range inside use that, skip the range with permit unknown-clients etc.

Now I'm thinking... maybe, as a helper to the operator we could present a text representation of the actual subnet {} declaration somewhere in the UI.

Actions #9

Updated by Sam Kottler over 11 years ago

  • Target version changed from 1.1 to 1.2.0

I'm moving this to 1.2 because it seems like there is more work that's needed beyond just adding a new button.

Actions #10

Updated by Ohad Levy about 11 years ago

  • Target version deleted (1.2.0)
Actions #11

Updated by Benjamin Papillon almost 11 years ago

  • Status changed from Feedback to New
Actions

Also available in: Atom PDF