Adding Bob as he may have feedback based on the decoupling spec.
Al
From: Chen, Haochuan Z [mailto:haochuan.z.chen@intel.com]
Sent: Friday, November 01, 2019 9:30 AM
To: Wang, Jing (Angie); Ngo, Tee
Cc: Qi, Mingyuan; Chen, Tingjie; 'starlingx-discuss@lists.starlingx.io'
Subject: Re: [Starlingx-discuss] decouple sysinv-conductor with armada application, request to decouple the way to generate image file with armada chart
Hi Angie & Tee
Have you checked my proposal?
Thanks!
Martin, Chen
SSP, Software Engineer
021-61164330
From: Chen, Haochuan Z
Sent: Tuesday, October 29, 2019 3:18 PM
To: Wang, Jing (Angie) <Angie.Wang@windriver.com>; Tee.Ngo@windriver.com
Cc: Qi, Mingyuan <mingyuan.qi@intel.com>; Chen, Tingjie <tingjie.chen@intel.com>; 'starlingx-discuss@lists.starlingx.io' <starlingx-discuss@lists.starlingx.io>
Subject: decouple sysinv-conductor with armada application, request to decouple the way to generate image file with armada chart
Hi Angie & Tee
Currently decoupling sysinv-conductor with armada application, you introduce new class ArmadaManifestOperator.
But I find currently this class only decouple application override. For image list, it still depends on declaration in values.yaml in application’s charts.
We work to ceph containerization, plan to introduce new charts to launch ceph service.
https://review.opendev.org/#/c/681457/
The image declaration in values.yaml is not same as currently application
This chart will pull images rook/ceph:v1.1.1, with such way to declare image.
image:
prefix: rook
repository: rook/ceph
tag: v1.1.1
pullPolicy: IfNotPresent
So prefer in sysinv-conductor, also use ArmadaManifestOperator to update image list file.
Thanks!
Martin, Chen
SSP, Software Engineer
021-61164330