Bug 1290807 - [RFE] Do not delete facts during discovered provisioning
[RFE] Do not delete facts during discovered provisioning
Status: CLOSED UPSTREAM
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Discovery Plugin (Show other bugs)
Unspecified
Unspecified Unspecified
unspecified Severity medium (vote)
: Unspecified
: --
Assigned To: Lukas Zapletal
Katello QA List
http://projects.theforeman.org/issues...
: FutureFeature, Triaged
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2015-12-11 09:19 EST by John Fulton
Modified: 2017-04-18 10:49 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-04-18 07:15:54 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Bugzilla 1272238 None None None Never
Foreman Issue Tracker 12244 None None None Never

  None (edit)
Description John Fulton 2015-12-11 09:19:19 EST
In Red Hat BZ 1272238 Foreman was modified so that Puppet facts don't have to be lost when a host is set to the build state. This is a request for the same functionality to be added to Satellite 6.x. 

As per Lukas Zapletal's comment [1] in BZ 1272238 an upstream issue [2] exists to track this, though I understand that the codebase is now a little different and the correct method to put the code in has changed. 

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1272238#c8
[2] http://projects.theforeman.org/issues/12244
Comment 2 Lukas Zapletal 2015-12-14 04:03:19 EST
For the record, in the latest Discovery 4.1 (part of Foreman 1.10 and Satellite 6.1.5) we do not delete facts with "discovery_" prefix. This can be used as a workaround (report all facts which must be preserved as discovery_xyz).

Valid RFE, we are tracking this already as http://projects.theforeman.org/issues/12244 upstream.
Comment 4 Lukas Zapletal 2016-05-13 03:24:20 EDT
This is a future RFE, we are discussing redesigning discovered hosts which should provide this feature.
Comment 9 Lukas Zapletal 2017-04-18 07:15:54 EDT
Hello,

this is already implemented in Satellite 6.2. There is a new setting called "Clean all facts" which can be set to false to prevent the unwanted behavior. If you want to delete just some facts, set it to true and name all facts to be kept "discovery_*" and they will survive provisioning (but will be deleted on puppet checkin).

This behavior will be the same in Satellite 6.3 (where Discovery 9.0 is planned).
Comment 10 Lukas Zapletal 2017-04-18 10:49:36 EDT
Correction, this was implemented upstream and it will be available in Satellite 6.3.

Note You need to log in before you can comment on or make changes to this bug.