Bug 1968542

Summary: [master] Infra env should show the time that ISO was generated.
Product: OpenShift Container Platform Reporter: Michael Filanov <mfilanov>
Component: assisted-installerAssignee: Nir Magnezi <nmagnezi>
assisted-installer sub component: assisted-service QA Contact: Yuri Obshansky <yobshans>
Status: CLOSED CURRENTRELEASE Docs Contact:
Severity: high    
Priority: urgent CC: fpercoco, nmagnezi
Version: 4.8Keywords: Triaged
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: AI-Team-Hive KNI-EDGE-4.8 KNI-EDGE-JUKE-4.8
Fixed In Version: OCP-Metal-v1.0.23.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1969363 (view as bug list) Environment:
Last Closed: 2022-08-28 08:45:59 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:    
Bug Blocks: 1969363    

Description Michael Filanov 2021-06-07 14:13:29 UTC 
Description of problem:

When ISO is generated we need to know when it happened, it will indicate when was the last time that the configuration changed. this way BMAC can know if it should use the ISO or wait a bit longer because it may get more configuration changes. 


Version-Release number of selected component (if applicable):
4.8.0


How reproducible:


Steps to Reproduce:
1. create cluster deployment
2. create infra env
3. take a look if condition show the generation time.
4. make some changes and see if the time in the condition is updated. 

Actual results:



Expected results:


Additional info:
Comment 1 Flavio Percoco 2021-06-08 07:14:41 UTC 
I brought this back up a couple of weeks ago and, during that discussion, the idea of using the Conditions (adding a field to them) was not considered valid as it would "abuse" the real scope of the Conditions. We may have to add a field to the status directly, or a struct like the BMH has:

```
image:
    URL: ...
    updatedAt: ...
```
Comment 3 Flavio Percoco 2021-06-08 07:18:53 UTC 
We need to switch this bug to track master and then clone it for `4.8` so it can be backported

/cc @nmagnezi
Comment 4 Nir Magnezi 2021-06-10 13:23:11 UTC 
(In reply to Flavio Percoco from comment #3)
> We need to switch this bug to track master and then clone it for `4.8` so it
> can be backported
> 
> /cc @nmagnezi

cloned by Ronnie Lazar --> Bug 1969363
Comment 5 Nir Magnezi 2021-06-20 13:18:57 UTC 
https://github.com/openshift/assisted-service/pull/1975 merged to master