[Starlingx-discuss] decouple sysinv-conductor with armada application, request to decouple the way to generate image file with armada chart

Chen, Haochuan Z haochuan.z.chen at intel.com
Mon Nov 4 06:19:03 UTC 2019


Thanks Tee.


Hi Bob

I think my suggestion will be accepted. And it is align with this item in the spec.

Have the platform be application agnostic. It should not contain specific application knowledge

Thanks!

Martin, Chen
SSP, Software Engineer
021-61164330

From: Ngo, Tee <Tee.Ngo at windriver.com>
Sent: Friday, November 1, 2019 11:01 PM
To: Bailey, Henry Albert (Al) <Al.Bailey at windriver.com>; Chen, Haochuan Z <haochuan.z.chen at intel.com>; Wang, Jing (Angie) <Angie.Wang at windriver.com>; Church, Robert <Robert.Church at windriver.com>
Cc: Qi, Mingyuan <mingyuan.qi at intel.com>; Chen, Tingjie <tingjie.chen at intel.com>; 'starlingx-discuss at lists.starlingx.io' <starlingx-discuss at lists.starlingx.io>
Subject: RE: decouple sysinv-conductor with armada application, request to decouple the way to generate image file with armada chart

Martin,

Bob designed and implemented the ArmadaManifestOperator and is also a Ceph SME. He's in the best position to provide you feedback on the proposed changes.

Tee

From: Bailey, Henry Albert (Al)
Sent: November-01-19 9:39 AM
To: Chen, Haochuan Z; Wang, Jing (Angie); Ngo, Tee; Church, Robert
Cc: Qi, Mingyuan; Chen, Tingjie; 'starlingx-discuss at lists.starlingx.io'
Subject: RE: decouple sysinv-conductor with armada application, request to decouple the way to generate image file with armada chart

Adding Bob as he may have feedback based on the decoupling spec.
https://opendev.org/starlingx/specs/src/branch/master/doc/source/specs/stx-3.0/approved/containerization-2006537-decoupling-container-apps.rst

Al

From: Chen, Haochuan Z [mailto:haochuan.z.chen at intel.com]
Sent: Friday, November 01, 2019 9:30 AM
To: Wang, Jing (Angie); Ngo, Tee
Cc: Qi, Mingyuan; Chen, Tingjie; 'starlingx-discuss at 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 at windriver.com<mailto:Angie.Wang at windriver.com>>; Tee.Ngo at windriver.com<mailto:Tee.Ngo at windriver.com>
Cc: Qi, Mingyuan <mingyuan.qi at intel.com<mailto:mingyuan.qi at intel.com>>; Chen, Tingjie <tingjie.chen at intel.com<mailto:tingjie.chen at intel.com>>; 'starlingx-discuss at lists.starlingx.io' <starlingx-discuss at lists.starlingx.io<mailto:starlingx-discuss at 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.
https://review.opendev.org/#/c/681457/5/stx-platform-helm/stx-platform-helm/helm-charts/rook-ceph/values.yaml
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20191104/4ce27e43/attachment-0001.html>


More information about the Starlingx-discuss mailing list