This is a list of all comments for UI-DEN-15918-2.

Review Summary: No summary


----------------------------------------
File: scripts/generate_config_files_sha.sh
Revision Comment by Behrouz NematiPour on 28 July 2023, 02:09 
https://devapps.diality.us/cru/UI-DEN-15918-2#c18301

  it will be /home/denali/.config/settings.crc

  Reply by vduong on 28 July 2023, 06:45
  > Added

    Reply by Behrouz NematiPour on 01 August 2023, 18:50
    > RESOLVED

Revision Comment by Behrouz NematiPour on 28 July 2023, 02:05 
https://devapps.diality.us/cru/UI-DEN-15918-2#c18299

  the Settings/Systems.conf cannot be checked for checksum,
  it is user preference and will change, therefore will fail the
  checksum.

  Reply by vduong on 28 July 2023, 06:48
  > Done. Added all *.dflt as part of the filter

    Reply by Behrouz NematiPour on 01 August 2023, 18:39
    > RESOLVED

Revision Comment by Behrouz NematiPour on 28 July 2023, 02:07 
https://devapps.diality.us/cru/UI-DEN-15918-2#c18300

  Please check Settings/Systems.dflt instead.

  Reply by vduong on 28 July 2023, 06:45
  > Done, I excluded the System.conf file specifically and added
  > all *.dflt  to the filter

    Reply by Behrouz NematiPour on 01 August 2023, 18:49
    > RESOLVED

Revision Comment by Behrouz NematiPour on 28 July 2023, 02:12 
https://devapps.diality.us/cru/UI-DEN-15918-2#c18302

  not clear what this does.
  I think we should look for a FAILED keyword here.

  Reply by vduong on 28 July 2023, 05:38
  > This is for stripping the output file. The I am assuming that
  > we are running this script on the build server, which may not
  > give us the right path. This is why the script is also asking
  > for the "device path" 
  > 
  > The output file looks like this:
  > <HASH>        <current path of conf file in server>

    Reply by Behrouz NematiPour on 01 August 2023, 19:19
    > RESOLVED


---

ID: UI-DEN-15918-2 https://devapps.diality.us/cru/UI-DEN-15918-2

Title: UI-DEN-15918_UI VD S102 - Scripts - Post Settings Check- [ READY ]
Statement of Objectives:


State: Closed

Summary: 

Author: vduong
Moderator: vduong
Reviewers: (4 active, 3 completed*)
     Tiffany Mejia   (*)
     jpaguio   (*)
     Behrouz NematiPour   (*)
     Sean Nash
     Michael Garthwaite
     Dara Navaei
     msuleiman