Project

General

Profile

Actions

Bug #16501

closed

autobuild-ceph: centos7 builds are failing

Added by Greg Farnum almost 8 years ago. Updated almost 8 years ago.

Status:
Resolved
Priority:
Immediate
Assignee:
Category:
Gitbuilder
Target version:
-
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

Description

The initial failure is:
http://gitbuilder.sepia.ceph.com/gitbuilder-ceph-rpm-centos7-amd64-basic/log.cgi?log=145402f57028565a406c97a898005550be3fba79

++ git clean -fdx
++ rm -fr '/tmp/*virtualenv*'
++ cat ../../dists
+ DISTS=centos6
++ cat ../../rsync-target
+ TARGET=gitbuilder@gitbuilder.ceph.com:gitbuilder.ceph.com/ceph-rpm-centos7-x86_64-basic
++ basename gitbuilder@gitbuilder.ceph.com:gitbuilder.ceph.com/ceph-rpm-centos7-x86_64-basic
+ TARGET=ceph-rpm-centos7-x86_64-basic
++ git rev-parse HEAD
+ REV=145402f57028565a406c97a898005550be3fba79
++ git describe
+ VER=v10.2.0-2791-g145402f
++ git describe --always
+ expr index v10.2.0-2791-g145402f -
++ git describe --always
++ cut -d- -f1
+ RPM_VER=v10.2.0
++ git describe --always
++ cut -d- -f2-
++ tr - .
+ RPM_REL=2791.g145402f
+ VER=v10.2.0-2791.g145402f
++ ../branches.sh -v
++ grep 145402f57028565a406c97a898005550be3fba79
++ awk '{print $2}'
+ BRANCH=
++ basename
basename: missing operand
Try 'basename --help' for more information.
+ BRANCH=

But it moves on to

+ BUILDAREA=./rpmbuild
+ mkdir -p ./rpmbuild/SOURCES
+ mkdir -p ./rpmbuild/SRPMS
+ mkdir -p ./rpmbuild/SPECS
+ mkdir -p ./rpmbuild/RPMS
+ mkdir -p ./rpmbuild/BUILD
+ cp -a ceph-10.2.0-2826-g027c0b6.tar.bz2 ./rpmbuild/SOURCES/.
+ cp -a 'rpm/*.patch' ./rpmbuild/SOURCES
cp: cannot stat 'rpm/*.patch': No such file or directory
+ true
++ grep '^Version:' ceph.spec
++ awk '{print $2}'
grep: ceph.spec: No such file or directory
+ rpm_version=
+ [[ '' =~ .*-rc[1-9]$ ]]
+ cp ceph.spec /tmp/ceph.spec
cp: cannot stat 'ceph.spec': No such file or directory

Actions #1

Updated by Greg Farnum almost 8 years ago

  • Subject changed from autobuild-ceph: failed use of basename to autobuild-ceph: centos7 builds are failing
  • Description updated (diff)
  • Assignee set to Sage Weil

[22:01:37] <dmick> I don't know if teh cmake stuff merged yet
[22:02:11] <dmick> right, sage talked about this earlier; make-dist
[22:02:18] <dmick> (as opposed to make dist)
[22:02:25] <dmick> is not creating ceph.spec from ceph.spec.in
[22:02:36] <dmick> I started down that road but got distracted by broken -suite
[22:03:02] <dmick> ceph.spec.in used to be edited into ceph.spec by configure
[22:03:11] <gregsfortytwo> I imagine this is why all the attempts to schedule tests today are failing
[22:03:11] <dmick> make-dist.sh doesn't use configure anymore

Actions #2

Updated by Dan Mick almost 8 years ago

configure used to substitute @VAR@ in the .in files. Now cmake does, but cmake isn't involved in make-dist. I wonder if it should be, or at least a subset of it? I don't think we want to duplicate all that substitution logic outside of cmake.

There is currently no configure_file() for ceph.spec.in. I wonder if any other .in files are missing. But something would still have to run cmake in make-dist to make this work. debian/ has no @@ substitutions, but ceph.spec does.

Actions #3

Updated by Kefu Chai almost 8 years ago

  • Status changed from New to Fix Under Review
  • Assignee changed from Sage Weil to Kefu Chai

I don't think we want to duplicate all that substitution logic outside of cmake.

i duplicated part of the substitution logic in "make-dist",

see https://github.com/ceph/ceph/pull/9978

Actions #4

Updated by Kefu Chai almost 8 years ago

  • Assignee changed from Kefu Chai to Dan Mick
Actions #6

Updated by Dan Mick almost 8 years ago

I believe this to be fixed.

Actions #7

Updated by Dan Mick almost 8 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF