Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1972262

Summary:	[4.8.0] "baremetalhost.metal3.io/detached" uses boolean value where string is expected
Product:	OpenShift Container Platform	Reporter:	Mat Kowalski <mko>
Component:	assisted-installer	Assignee:	Mat Kowalski <mko>
assisted-installer sub component:	assisted-service	QA Contact:	Yuri Obshansky <yobshans>
Status:	CLOSED ERRATA	Docs Contact:
Severity:	medium
Priority:	medium	CC:	alazar, aos-bugs, ccrum, nshidlin
Version:	4.8	Keywords:	Triaged
Target Milestone:	---
Target Release:	4.8.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	AI-Team-Platform
Fixed In Version:		Doc Type:	No Doc Update
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-07-27 23:12:54 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:
Bug Depends On:	1972272
Bug Blocks:

Description Mat Kowalski 2021-06-15 14:27:02 UTC

In the baremetal-operator the "baremetalhost.metal3.io/detached" annotation is a string value with the following behaviour

- existence of the label defines host as detached
- missing label defines the host as not detached

This has the following consequence - the value of the annotation is always ignored (this was a design decision).

In assisted-service we are setting value to "true" what may create an implicit assumption that the value is a boolean and setting it to "false" would reverse the behaviour. This is not the case and `baremetalhost.metal3.io/detached: "false"` still causes the host to be treated as detached.

The issue has been already fixed on master via https://github.com/openshift/assisted-service/pull/1974 so that we are setting `baremetalhost.metal3.io/detached: "assisted-service-controller"`

Comment 4 nshidlin 2021-06-29 17:37:36 UTC

this bug doesn't block OCP4.8.0 -release -  the fix will be delivered the the ACM channel on the AI-operator add verified then

Comment 6 Chad Crum 2021-07-09 18:11:23 UTC

I have validated the fix.

- hub cluster 4.8.0-fc.7
- acm ds snapshot 2.3.0-DOWNSTREAM-2021-07-06-05-34-53


Steps:
- Deployed a SNO cluster using standard spoke cluster CRDs
- Validated that "assisted-service-controller", rather than "true", is set in the bmh annotation of a deployed spoke cluster


apiVersion: v1
items:
- apiVersion: metal3.io/v1alpha1
  kind: BareMetalHost
  metadata:
    annotations:
      baremetalhost.metal3.io/detached: assisted-service-controller
      inspect.metal3.io: disabled

Comment 9 errata-xmlrpc 2021-07-27 23:12: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 (Moderate: OpenShift Container Platform 4.8.2 bug fix and security update), 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-2021:2438