[Starlingx-discuss] [Stx.distro.openstack] Call for review on OpenStack Train Upgrade Patches.

Liu, ZhipengS zhipengs.liu at intel.com
Fri Nov 8 01:55:15 UTC 2019


Hi all,

So far I have not got new more comments. 
I'd like to push this 6 patches get merged soon since fully sanity test pass!

Could core reviewers final check them before adding review+2 for below 3 patches.
https://review.opendev.org/#/c/683886/   starlingx/openstack-armada-app
https://review.opendev.org/#/c/684166/   starlingx/openstack-armada-app
https://review.opendev.org/#/c/688105/   starlingx/root

Thanks!
Zhipeng
-----Original Message-----
From: Hu, Yong <yong.hu at intel.com> 
Sent: 2019年11月5日 22:47
To: starlingx-discuss at lists.starlingx.io; Alonso, Juan Carlos <juan.carlos.alonso at intel.com>; Liu, ZhipengS <zhipengs.liu at intel.com>; Lemus Contreras, Cristopher J <cristopher.j.lemus.contreras at intel.com>
Subject: [Stx.distro.openstack] Call for review on OpenStack Train Upgrade Patches.

Hi Core reviewers in Stx.distro.openstack:

The EB with containerized Train OpenStack services has *passed* the sanity, based on test results from testing team. (Thanks to @JC and @Chirstopher).

So to unblock stx.3.0 MS-3, please review the following patches in time:

     https://review.opendev.org/#/c/683886/
     https://review.opendev.org/#/c/683910/
     https://review.opendev.org/#/c/684166/
     https://review.opendev.org/#/c/687441/
     https://review.opendev.org/#/c/687197/
     https://review.opendev.org/#/c/688105/

In addition, some background info related to this upgrade:

1). Originally there was a test failure in the sanity, but later Zhipeng found the cause was from an non-standard test (cirros) image which had 2
QCOW2 headers unexpectedly and failed during the conversion from QCOW2 to RAW format. Why wasn't this issue seen with OpenStack Stein? Because the QCOW2 to RAW conversion wasn't enabled in Nova of Stein, but it is now enabled by default in Train.  Recommendation: later for the testing, we advise to use this cirros image: 
https://download.cirros-cloud.net/0.4.0/cirros-0.4.0-x86_64-disk.img, or other QCOW2 image which we assure it is with correct header.

2). these patches have to be merged *in one batch* due to the inter-dependencies.

3). after these patches are merged, need *stx.build team* to generate new docker images for Openstack Train services and *upload them to Docker Hub*.


More information about the Starlingx-discuss mailing list