+starlingx-discuss Don, Some additional background: I created the initial storyboard and talked with Dariush, he is supposed to provide the background about a given patch and if it can be removed, I assume he would have checked on the history and talked with the authors. Once he gave the go ahead to try removal, Martin did the work of removing the patch and testing it in the CentOS build environment. I will wait for more details from Martin as suggested by Cindy regarding testing. Sau! On 12/3/18 7:21 AM, Xie, Cindy wrote:
Agree w/ you Don. Just read the patch and see like this is a workaround to avoid “make check” fail in certain mock config… @ Martin, can you please double check w/ the patch author on this? And your test confirmed that without this patch, the build can be successful in this mock config already?
Thx. - cindy
*From:* Penney, Don [mailto:Don.Penney@windriver.com] *Sent:* Monday, December 3, 2018 11:14 PM *To:* Xie, Cindy <cindy.xie@intel.com>; Chen, Haochuan Z <haochuan.z.chen@intel.com>; Wold, Saul <saul.wold@intel.com>; Eslimi, Dariush <Dariush.Eslimi@windriver.com>; McKenna, Jason <Jason.McKenna@windriver.com> *Cc:* Liu, ZhipengS <zhipengs.liu@intel.com>; Lin, Shuicheng <shuicheng.lin@intel.com>; Little, Scott <Scott.Little@windriver.com> *Subject:* RE: review for story 2004211 patch
I understand that goal, but this particular patch was added to deal with a build issue at the time, presumably. Was this a consideration when removal of this patch was decided upon? Was there any discussion with Scott or Jason to see if the patch may still be required?
*From:*Xie, Cindy [mailto:cindy.xie@intel.com] *Sent:* Monday, December 03, 2018 10:08 AM *To:* Penney, Don; Chen, Haochuan Z; Wold, Saul; Eslimi, Dariush; McKenna, Jason *Cc:* Liu, ZhipengS; Lin, Shuicheng *Subject:* RE: review for story 2004211 patch
Don,
The driving force is to reduce the number of patches that we have to maintain. This is the goal for non-openstack distro sub-project team.
Understand that we are not able to reduce all the patches for this particular package, thus we still need to use sRPM instead of binary RPM. However, the goodness is that we do not need to re-base those patches when we have to upgrade CentOS next time. The less patches we carry, the less upgrade effort will be.
Thanks. - cindy
*From:* Penney, Don [mailto:Don.Penney@windriver.com] *Sent:* Monday, December 3, 2018 10:45 PM *To:* Chen, Haochuan Z <haochuan.z.chen@intel.com <mailto:haochuan.z.chen@intel.com>>; Wold, Saul <saul.wold@intel.com <mailto:saul.wold@intel.com>>; Eslimi, Dariush <Dariush.Eslimi@windriver.com <mailto:Dariush.Eslimi@windriver.com>>; McKenna, Jason <Jason.McKenna@windriver.com <mailto:Jason.McKenna@windriver.com>> *Cc:* Liu, ZhipengS <zhipengs.liu@intel.com <mailto:zhipengs.liu@intel.com>>; Lin, Shuicheng <shuicheng.lin@intel.com <mailto:shuicheng.lin@intel.com>>; Xie, Cindy <cindy.xie@intel.com <mailto:cindy.xie@intel.com>> *Subject:* RE: review for story 2004211 patch
I’ll re-post my question here, since it wasn’t answered on the review itself:
What is the driver for removing this patch? Are you sure this is safe to remove in all build environments? Presumably it was added because of a build failure at the time. Adding Jason to comment on the history
You don’t seem to be removing all patches from the package, so that we could switch to just using the binary, so why remove just this one?
*From:*Chen, Haochuan Z [mailto:haochuan.z.chen@intel.com] *Sent:* Monday, December 03, 2018 12:40 AM *To:* Wold, Saul; Eslimi, Dariush; Penney, Don; McKenna, Jason *Cc:* Liu, ZhipengS; Lin, Shuicheng; Xie, Cindy *Subject:* review for story 2004211 patch
Hi folks
I have submit a patch to eliminate a meta patch, which re-enable make check in sudo package’s building.
https://review.openstack.org/#/c/621057/
Please help to review. And wait for your opinion.
Thanks!
Martin, Chen
SSG OTC, Software Engineer
021-61164330