New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #27840 - Setting to toggle host profile stealing #8344
Conversation
There were the following issues with the commit message:
If you don't have a ticket number, please create an issue in Redmine. More guidelines are available in Coding Standards or on the Foreman wiki. This message was auto-generated by Foreman's prprocessor |
4de2701
to
f0f6433
Compare
app/models/setting/content.rb
Outdated
@@ -137,6 +137,9 @@ def self.load_defaults | |||
N_("If hosts fail to register because of duplicate DMI UUIDs " \ | |||
"add their comma-separated values here. Subsequent registrations will generate a unique DMI UUID for the affected hosts."), | |||
[], N_('Host Duplicate DMI UUIDs')), | |||
self.set('host_profile_steal', N_("Allow new Host registrations to overtake registered profiles with matching hostname " \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about "host_profile_assume" and: "Allow new Host registrations to assume registered profiles with matching hostname as long as the registering DMI UUID is unique."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated!
@mccun934 ACK ? |
@@ -137,6 +137,9 @@ def self.load_defaults | |||
N_("If hosts fail to register because of duplicate DMI UUIDs " \ | |||
"add their comma-separated values here. Subsequent registrations will generate a unique DMI UUID for the affected hosts."), | |||
[], N_('Host Duplicate DMI UUIDs')), | |||
self.set('host_profile_assume', N_("Allow new Host registrations to assume registered profiles with matching hostname " \ | |||
"as long as the registering DMI UUID is unique."), | |||
true, N_('Host Profile Assume')), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trying to understand this wording. "as long as registering DMI uuid is unique". Do you mean 'unique other than the host its taking' ? I assume it already would take the host's profile if its uuid matched that, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct! It's kind of a strange behavior to articulate. Let me know if you have a suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"as long as the registering DMI UUID is not used by another host" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with that, updating
ad92239
to
ef35d8a
Compare
@jlsherrill updated! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, ACK based on @mccun934's testing
To test:
subscription-manager clean
on the client and change its DMI UUID:To test the reverse: toggle the new Host Profile Assume setting to false. Repeating the test (with a new UUID) will result in a registration error