[Starlingx-discuss] review patch to enable rook-ceph in config project.

Chen, Haochuan Z haochuan.z.chen at intel.com
Mon Jan 25 13:23:58 UTC 2021


Thanks Dan and John review for my patch. I already update and verified for review comment. Welcome more review idea!

https://review.opendev.org/c/starlingx/config/+/720637
https://review.opendev.org/c/starlingx/rook-ceph/+/771741

Thanks!

Martin, Chen
IOTG, Software Engineer
021-61164330

From: Chen, Haochuan Z
Sent: Friday, January 22, 2021 4:05 PM
To: 'Bailey, Henry Albert (Al)' <Al.Bailey at windriver.com>; 'Bart Wensley' <barton.wensley at windriver.com>; 'Church, Robert' <Robert.Church at windriver.com>; 'Chris Friesen' <chris.friesen at windriver.com>; 'Penney, Don' <Don.Penney at windriver.com>; 'Kung, John' <John.Kung at windriver.com>; 'Waines, Greg' <Greg.Waines at windriver.com>; 'Voiculeasa, Dan' <Dan.Voiculeasa at windriver.com>
Cc: Sun, Austin <austin.sun at intel.com>; Hu, Yong <yong.hu at intel.com>; Jones, Bruce E <bruce.e.jones at intel.com>; 'starlingx-discuss at lists.starlingx.io' <starlingx-discuss at lists.starlingx.io>
Subject: RE: review patch to enable rook-ceph in config project.

Thanks Dan review for my patch. Welcome more review idea!

https://review.opendev.org/c/starlingx/config/+/720637
https://review.opendev.org/c/starlingx/rook-ceph/+/771741

Thanks!

Martin, Chen
IOTG, Software Engineer
021-61164330

From: Chen, Haochuan Z
Sent: Thursday, January 21, 2021 10:32 PM
To: 'Bailey, Henry Albert (Al)' <Al.Bailey at windriver.com<mailto:Al.Bailey at windriver.com>>; 'Bart Wensley' <barton.wensley at windriver.com<mailto:barton.wensley at windriver.com>>; 'Church, Robert' <Robert.Church at windriver.com<mailto:Robert.Church at windriver.com>>; 'Chris Friesen' <chris.friesen at windriver.com<mailto:chris.friesen at windriver.com>>; 'Penney, Don' <Don.Penney at windriver.com<mailto:Don.Penney at windriver.com>>; 'Kung, John' <John.Kung at windriver.com<mailto:John.Kung at windriver.com>>; 'Waines, Greg' <Greg.Waines at windriver.com<mailto:Greg.Waines at windriver.com>>; 'Voiculeasa, Dan' <Dan.Voiculeasa at windriver.com<mailto:Dan.Voiculeasa at windriver.com>>
Cc: Sun, Austin <austin.sun at intel.com<mailto:austin.sun at intel.com>>; Hu, Yong <yong.hu at intel.com<mailto:yong.hu at intel.com>>; Jones, Bruce E <bruce.e.jones at intel.com<mailto:bruce.e.jones at intel.com>>; 'starlingx-discuss at lists.starlingx.io' <starlingx-discuss at lists.starlingx.io<mailto:starlingx-discuss at lists.starlingx.io>>
Subject: RE: review patch to enable rook-ceph in config project.

Hi Bob and Dan

I update patch for story about lifecycle. And finish verification.

https://review.opendev.org/c/starlingx/config/+/720637
https://review.opendev.org/c/starlingx/rook-ceph/+/771741

1, add lifecycle for rook-ceph
2, add a function in sysinv-conductor, update_config_for_rook_ceph, which is called in application rook-ceph hook function, in operation post stage
3, cleanup some code as function already move to rook-ceph application's hook

Thanks!

Martin, Chen
IOTG, Software Engineer
021-61164330

From: Chen, Haochuan Z
Sent: Tuesday, January 19, 2021 9:03 PM
To: Bailey, Henry Albert (Al) <Al.Bailey at windriver.com<mailto:Al.Bailey at windriver.com>>; Bart Wensley <barton.wensley at windriver.com<mailto:barton.wensley at windriver.com>>; Church, Robert <Robert.Church at windriver.com<mailto:Robert.Church at windriver.com>>; Chris Friesen <chris.friesen at windriver.com<mailto:chris.friesen at windriver.com>>; Penney, Don <Don.Penney at windriver.com<mailto:Don.Penney at windriver.com>>; Kung, John <John.Kung at windriver.com<mailto:John.Kung at windriver.com>>; Waines, Greg <Greg.Waines at windriver.com<mailto:Greg.Waines at windriver.com>>
Cc: Sun, Austin <austin.sun at intel.com<mailto:austin.sun at intel.com>>; Hu, Yong <yong.hu at intel.com<mailto:yong.hu at intel.com>>; Jones, Bruce E <bruce.e.jones at intel.com<mailto:bruce.e.jones at intel.com>>; starlingx-discuss at lists.starlingx.io<mailto:starlingx-discuss at lists.starlingx.io>
Subject: RE: review patch to enable rook-ceph in config project.

Hi

Thanks, John, AL and Penny review for my patch. I l already updated and make verification.

Please go on review my patch for config project
https://review.opendev.org/c/starlingx/config/+/720637

Thanks

Martin, Chen
IOTG, Software Engineer
021-61164330

From: Chen, Haochuan Z
Sent: Monday, December 7, 2020 2:02 PM
To: 'Bailey, Henry Albert (Al)' <Al.Bailey at windriver.com<mailto:Al.Bailey at windriver.com>>; 'Bart Wensley' <barton.wensley at windriver.com<mailto:barton.wensley at windriver.com>>; 'Church, Robert' <Robert.Church at windriver.com<mailto:Robert.Church at windriver.com>>; 'Chris Friesen' <chris.friesen at windriver.com<mailto:chris.friesen at windriver.com>>; 'Penney, Don' <Don.Penney at windriver.com<mailto:Don.Penney at windriver.com>>; 'Kung, John' <John.Kung at windriver.com<mailto:John.Kung at windriver.com>>; 'Waines, Greg' <Greg.Waines at windriver.com<mailto:Greg.Waines at windriver.com>>
Cc: Sun, Austin <austin.sun at intel.com<mailto:austin.sun at intel.com>>; Hu, Yong <yong.hu at intel.com<mailto:yong.hu at intel.com>>; Jones, Bruce E <bruce.e.jones at intel.com<mailto:bruce.e.jones at intel.com>>; 'starlingx-discuss at lists.starlingx.io' <starlingx-discuss at lists.starlingx.io<mailto:starlingx-discuss at lists.starlingx.io>>
Subject: review patch to enable rook-ceph in config project.

Hi folks

Please review my patch and bob patch to enable rook-ceph in config project.
https://review.opendev.org/c/starlingx/config/+/720637
https://review.opendev.org/c/starlingx/config/+/758935

BR!

Martin, Chen
IOTG, Software Engineer
021-61164330

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20210125/8780a063/attachment.html>


More information about the Starlingx-discuss mailing list