Hi Bruce and all,
Below is some update for openstack-placement containerized task.
The placement API works now. I have tested application apply and VM creation. Both are pass!
Besides that, I will do some basic test on multi-node as well.
I’d like to seek support from nova experts about how to test nova-placement in depth.
If possible, can we involve test team to help do sanity test before getting these patches merged.
4 below patches submitted for review (662229 is openstack-helm upstream patch)
1) Add stx-placement docker image directives files
https://review.opendev.org/#/c/661679/
2) WIP: add placement chart ( submitted to openstack-helm project)
https://review.opendev.org/#/c/662229/
3) Add placement chart patch to openstack-helm
https://review.opendev.org/#/c/662371/
4) Add placement chart to armada system
https://review.opendev.org/#/c/662614/
Your comments are appreciated!
Thanks!
Zhipeng
Hi Gerry,
Thanks for your great comments!
For step 2, I already submitted SB and a patch to openstack-helm project
https://review.opendev.org/#/c/662229/
Meanwhile, I will add these charts as a patch to openstack-helm.
For step 3, I will submit 1 patch of adding placement chart to armada system
For step 4, I will submit 1 patch to openstack-helm to change nova chart to turn off the placement of nova.
Thanks!
Zhipeng
From: Kopec, Gerald (Gerry) [mailto:Gerry.Kopec@windriver.com]
Sent: 2019年5月31日
6:41
To: Liu, ZhipengS <zhipengs.liu@intel.com>
Cc: starlingx-discuss@lists.starlingx.io
Subject: RE: [Starlingx-discuss] About task of integrating containerized Nova Placement into StarlingX
Hi Zhipeng,
Some initial thoughts below…
Gerry
From: Liu, ZhipengS [mailto:zhipengs.liu@intel.com]
Sent: Tuesday, May 28, 2019 10:07 AM
To: Kopec, Gerald (Gerry)
Cc: starlingx-discuss@lists.starlingx.io
Subject: [Starlingx-discuss] About task of integrating containerized Nova Placement into StarlingX
Hi Gerry,
I'm working on the task of integrating containerized Nova Placement into StarlingX
https://storyboard.openstack.org/#!/story/2005750
I really need your guide for the task.
I already had high level understanding for the task.
It should include below steps:
1) Create new container image for Placement
https://review.opendev.org/#/c/661679 Done
2) Add Helm Chart related yaml configuration for Placement(need time for detail digest)
Need to move related configuration out of Nova, and add some new configurations
Now, I'm Working on the patch of adding chart for placement
Created below folder
stx-config/kubernetes/applications/stx-openstack/stx-openstack-helm/stx-openstack-helm/openstack-placement
Then I will create related chart yaml files. I mainly refer to the ones in openstack-helm/nova/
[GK] Don’t think this should go in starlingx. Should be with openstack-helm. Looks like you’ve got that started with
https://review.opendev.org/#/c/662229/. It would be based on what is in openstack-helm/nova related to placement.
For initial deployment within starlingx, you’ll probably have to add a patch with your changes to:
stx-upstream/openstack/openstack-helm/files/
3) Start placement container in controller and make sure placement service started successfully.
[GK] You’ll need to add placement to the armada manifest so it will get started on stx-openstack application-apply. I would assume it would go with the compute-kit. See:
stx-config/kubernetes/applications/stx-openstack/stx-openstack-helm/stx-openstack-helm/manifests/manifest.yaml
Also you’ll need to setup up dynamic overrides for placement in sysinv/helm, see nova example:
stx-config/sysinv/sysinv/sysinv/sysinv/helm/nova.py
Placement needs to be included in sysinv generated helm override yaml files and provided as input to armada on application-apply.
4) Do related modification for current Nova yaml file.
One patch for modifying chart for nova.
[GK] I’m hoping it’s possible to turn off the placement capability in openstack-helm/nova via overrides, but maybe a openstack-helm nova chart change will be required.
5) Integrate to starlingx and start separate container. Debug and Sanity test.
[GK] You’ll also need to include the placement helm chart in the build helm chart tarball.
It should be OK to finish related patches in next 3 weeks.
But for code review/merge, it may have some risk.
Your comments are appreciated!
Thanks!
Zhipeng