Failed to sync stable/liberty for openstack projects because patch from package does not apply

Bug #1509657 reported by Dmitry Mescheryakov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
High
MOS Packaging Team

Bug Description

The CR https://review.fuel-infra.org/#/c/13033/ can not be merged because patch from package fails to apply automatically. See job https://packaging-ci.infra.mirantis.net/job/8.0-pkg-pipeline-ubuntu/2257/console

Here:

patching file requirements.txt
Hunk #1 FAILED at 15.
1 out of 1 hunk FAILED
dpkg-source: info: fuzz is not allowed when applying patches
dpkg-source: info: if patch 'no-version-for-python-tz.patch' is correctly applied by quilt, use 'quilt refresh' to update it
dpkg-source: error: LC_ALL=C patch -t -F 0 -N -p1 -u -V never -g0 -E -b -B .pc/no-version-for-python-tz.patch/ --reject-file=- < python-oslo.serialization.orig.fd9NmV/debian/patches/no-version-for-python-tz.patch gave error exit status 1
E: Failed to package source directory /srv/source/python-oslo.serialization

Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

Hi. There's nothing in the code review that makes it possible to see the patch. When I click on the patchset, it tells: "Trivial merge" with no more info. Where may I see the patch, so that I can attempt to understand?

Revision history for this message
Dmitry Mescheryakov (dmitrymex) wrote :

Hello Thomas, the reason is that the patch is a merge commit. You can do the following:
 * checkout the patch from gerrit
 * do 'git diff origin/openstack-ci/fuel-8.0/liberty..HEAD'

That will provide you with diff between current state of openstack-ci/fuel-8.0/liberty and the patch

Revision history for this message
Artem Silenkov (asilenkov) wrote :

We have to group "spec request" and "code request" before merge. Testing it right now.

Revision history for this message
Artem Silenkov (asilenkov) wrote :

ceilometer
murano
keystone
oslo.serialization

are also affected

summary: - Failed to sync stable/liberty for oslo.serialization because patch from
+ Failed to sync stable/liberty for openstack projects because patch from
package does not apply
Changed in mos:
status: Confirmed → Triaged
Revision history for this message
Artem Silenkov (asilenkov) wrote :

oslo.serialization need manual investigation, requirements.rpm file is not clean for this patch.

Changed in mos:
status: Triaged → Fix Committed
Changed in mos:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.