Bug 1968542 - [master] Infra env should show the time that ISO was generated.
Summary: [master] Infra env should show the time that ISO was generated.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: assisted-installer
Version: 4.8
Hardware: Unspecified
OS: Unspecified
urgent
high
Target Milestone: ---
: ---
Assignee: Nir Magnezi
QA Contact: Yuri Obshansky
URL:
Whiteboard: AI-Team-Hive KNI-EDGE-4.8 KNI-EDGE-JU...
Depends On:
Blocks: 1969363
TreeView+ depends on / blocked
 
Reported: 2021-06-07 14:13 UTC by Michael Filanov
Modified: 2022-08-28 08:45 UTC (History)
2 users (show)

Fixed In Version: OCP-Metal-v1.0.23.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1969363 (view as bug list)
Environment:
Last Closed: 2022-08-28 08:45:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift assisted-service pull 1975 0 None open OCPBUGSM-30121: Add Image CreatedAt to InfraEnv status 2021-06-13 17:24:04 UTC

Internal Links: 1968552 1969363 1969367

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


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