[Starlingx-discuss] 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/... 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
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. https://review.opendev.org/#/c/681457/5/stx-platform-helm/stx-platform-helm/... 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
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... 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. https://review.opendev.org/#/c/681457/5/stx-platform-helm/stx-platform-helm/... 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
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@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... 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. https://review.opendev.org/#/c/681457/5/stx-platform-helm/stx-platform-helm/... 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
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@windriver.com> Sent: Friday, November 1, 2019 11:01 PM To: Bailey, Henry Albert (Al) <Al.Bailey@windriver.com>; Chen, Haochuan Z <haochuan.z.chen@intel.com>; Wang, Jing (Angie) <Angie.Wang@windriver.com>; Church, Robert <Robert.Church@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: 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@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... 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<mailto:Angie.Wang@windriver.com>>; Tee.Ngo@windriver.com<mailto:Tee.Ngo@windriver.com> Cc: Qi, Mingyuan <mingyuan.qi@intel.com<mailto:mingyuan.qi@intel.com>>; Chen, Tingjie <tingjie.chen@intel.com<mailto:tingjie.chen@intel.com>>; 'starlingx-discuss@lists.starlingx.io' <starlingx-discuss@lists.starlingx.io<mailto: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. https://review.opendev.org/#/c/681457/5/stx-platform-helm/stx-platform-helm/... 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
participants (3)
-
Bailey, Henry Albert (Al)
-
Chen, Haochuan Z
-
Ngo, Tee