Bug 1255484

Summary: [RFE] Make subnet an optional field
Product: Red Hat Satellite Reporter: Karthick Murugadhas <kmurugad>
Component: ProvisioningAssignee: Lukas Zapletal <lzap>
Status: CLOSED ERRATA QA Contact: Shimon Shtein <sshtein>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.0.8CC: bbuckingham, bkearney, cwelton, ktordeur, ldelouw, lzap, mhulan, mwelk, ohadlevy, pmoravec, pmutha, rplevka, sshtein, sthirugn, xdmoon
Target Milestone: UnspecifiedKeywords: FutureFeature, Reopened, Triaged
Target Release: Unused   
Hardware: All   
OS: Linux   
URL: http://projects.theforeman.org/issues/15133
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-02-21 12:30:53 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Verification screenshot none

Comment 17 Lukas Zapletal 2016-05-16 19:31:09 UTC
I think there was a misunderstanding on my side. RFE is was to make Subnet an optional field (therefore I am also renaming the subject).

Subnet field was and is (in 6.2) required field for Managed and/or Provision interfaces. It defines whether to use DHCP or Static configuration for Managed interface and defines other required services for Provision interface like Puppet Master/CA, TFTP, Reverse DNS and Discovery. These are essential associations which are made through Subnet, for this reason this is mandatory field and we unlikely change that.

For unmanaged (non-provision) interfaces it should be technically possible not to require Subnet field, but we don't support that. I was under impression that version 6.2 supports that, but that was not the case. But if I understand customer requirements correctly, in both cases they would like to have the field optional for Provision interfaces.

While it might be tempting to do a workaround by creating a "fake" Subnet(s), I'd not recommend that. If subnet address and netmask does not match, Satellite will be unable to find the subnet for particular IP address (e.g. discovery finds Capsule for communication).

Ohad, can you confirm my opinion? Or do you see there is a way of turning Subnet field into non-mandatory one?

Comment 18 Ohad Levy 2016-05-17 07:41:29 UTC
I tend to agree with Lukas, however I wonder if we could simplify the usage cases where tftp is only used (e.g. no dhcp/dns, where ip address management is not as important), without adding further confusion / more complex setup.

Having said that, we already have infoblox support in the upstream, and would expect that being added downstream, so in the case of using infoblox and satellite, you would actually need to use the current subnet anyway.

Comment 19 Lukas Zapletal 2016-05-17 13:27:40 UTC
Can you elaborate on that Ohad? Do you suggest to make the subnet field optional when DHCP Capsule is not associated? If so, do you think this should still go into 6.2 GA release?

Comment 23 errata-xmlrpc 2016-07-27 08:54:54 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:1500

Comment 25 Lukas Zapletal 2017-05-18 07:29:00 UTC
I am making this bug public, keeping the initial internal discussion private tho.

Anyway, I am moving this on to ON_QA for 6.3. I am now able to create host without any subnet. When you select domain, subnet is automatically selected it there is one associated, but the dropdown can be switched back to "Please select" which is blank value. I tested this with one or two subnets, all works just fine.

QA: Please just verify, steps are above.

Comment 26 Lukas Zapletal 2017-05-18 07:34:05 UTC
I also created small change upstream to change the confusing label for blank value: http://projects.theforeman.org/issues/19583

Comment 27 Shimon Shtein 2017-08-16 17:40:08 UTC
Created attachment 1314259 [details]
Verification screenshot

Verified.

Screenshot attached. Works both for domain without any subnet and for domain with single subnet (with deselecting it).

Sat version: 6.3.0-16.0.beta.el7sat

Comment 28 Luc de Louw 2017-09-20 14:10:45 UTC
Is it planned to also manually provide a (default) gateway?

Thanks,

Luc

Comment 31 errata-xmlrpc 2018-02-21 12:30:53 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:0336