CEPH-Devel archive on lore.kernel.org
 help / color / Atom feed
* "Signed-off-by" jenkins job for PRs now failing
@ 2020-10-07  8:23 Alfonso Martinez Hidalgo
  2020-10-08 14:28 ` Ken Dreyer
  0 siblings, 1 reply; 2+ messages in thread
From: Alfonso Martinez Hidalgo @ 2020-10-07  8:23 UTC (permalink / raw)
  To: Ceph Devel

Hi All,

It seems that this job has started to fail on several PRs.
Does anyone have privileges to fix this error?

============================= test session starts ==============================
platform linux2 -- Python 2.7.12, pytest-4.6.11, py-1.9.0,
pluggy-0.13.1 -- /tmp/venv.4M9HM0FOIP/bin/python2.7
cachedir: .pytest_cache
rootdir: /home/jenkins-build/build/workspace/ceph-pr-commits
collecting ... collected 2 items / 1 deselected / 1 selected

ceph-build/ceph-pr-commits/build/test_commits.py::TestCommits::test_signed_off_by
Running command: git fetch origin
+refs/heads/master:refs/remotes/origin/master
Running command: git log -z --no-merges origin/master..HEAD
FAILED

=================================== FAILURES ===================================
________________________ TestCommits.test_signed_off_by ________________________

self = <test_commits.TestCommits object at 0x7f95779c8190>

    @pytest.mark.code_test
    def test_signed_off_by(self):
        signed_off_regex = r'Signed-off-by: \S.* <[^@]+@[^@]+\.[^@]+>'
        # '-z' puts a '\0' between commits, see later split('\0')
        check_signed_off_commits = 'git log -z --no-merges origin/%s..%s' % (
            self.target_branch, self.source_branch)
        wrong_commits = list(filterfalse(
            re.compile(signed_off_regex).search,
>           self.command(check_signed_off_commits).split('\0')))

ceph-build/ceph-pr-commits/build/test_commits.py:63:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

cls = <class 'test_commits.TestCommits'>
command = 'git log -z --no-merges origin/master..HEAD'

    @classmethod
    def command(cls, command):
        print("Running command:", command)
>       return check_output(shlex.split(command), cwd=cls.ceph_checkout).decode()
E       UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in
position 68: ordinal not in range(128)

ceph-build/ceph-pr-commits/build/test_commits.py:29: UnicodeDecodeError
- generated xml file:
/home/jenkins-build/build/workspace/ceph-pr-commits/report.xml -
==================== 1 failed, 1 deselected in 0.60 seconds ====================
Build step 'Execute shell' marked build as failure


Regards,
-- 

Alfonso Martínez
Senior Software Engineer, Ceph Storage
Red Hat


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: "Signed-off-by" jenkins job for PRs now failing
  2020-10-07  8:23 "Signed-off-by" jenkins job for PRs now failing Alfonso Martinez Hidalgo
@ 2020-10-08 14:28 ` Ken Dreyer
  0 siblings, 0 replies; 2+ messages in thread
From: Ken Dreyer @ 2020-10-08 14:28 UTC (permalink / raw)
  To: Alfonso Martinez Hidalgo; +Cc: Ceph Devel

I thought we were going to switch over to https://github.com/apps/dco
? Is that still the case?

- Ken

On Wed, Oct 7, 2020 at 2:25 AM Alfonso Martinez Hidalgo
<almartin@redhat.com> wrote:
>
> Hi All,
>
> It seems that this job has started to fail on several PRs.
> Does anyone have privileges to fix this error?
>
> ============================= test session starts ==============================
> platform linux2 -- Python 2.7.12, pytest-4.6.11, py-1.9.0,
> pluggy-0.13.1 -- /tmp/venv.4M9HM0FOIP/bin/python2.7
> cachedir: .pytest_cache
> rootdir: /home/jenkins-build/build/workspace/ceph-pr-commits
> collecting ... collected 2 items / 1 deselected / 1 selected
>
> ceph-build/ceph-pr-commits/build/test_commits.py::TestCommits::test_signed_off_by
> Running command: git fetch origin
> +refs/heads/master:refs/remotes/origin/master
> Running command: git log -z --no-merges origin/master..HEAD
> FAILED
>
> =================================== FAILURES ===================================
> ________________________ TestCommits.test_signed_off_by ________________________
>
> self = <test_commits.TestCommits object at 0x7f95779c8190>
>
>     @pytest.mark.code_test
>     def test_signed_off_by(self):
>         signed_off_regex = r'Signed-off-by: \S.* <[^@]+@[^@]+\.[^@]+>'
>         # '-z' puts a '\0' between commits, see later split('\0')
>         check_signed_off_commits = 'git log -z --no-merges origin/%s..%s' % (
>             self.target_branch, self.source_branch)
>         wrong_commits = list(filterfalse(
>             re.compile(signed_off_regex).search,
> >           self.command(check_signed_off_commits).split('\0')))
>
> ceph-build/ceph-pr-commits/build/test_commits.py:63:
> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>
> cls = <class 'test_commits.TestCommits'>
> command = 'git log -z --no-merges origin/master..HEAD'
>
>     @classmethod
>     def command(cls, command):
>         print("Running command:", command)
> >       return check_output(shlex.split(command), cwd=cls.ceph_checkout).decode()
> E       UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in
> position 68: ordinal not in range(128)
>
> ceph-build/ceph-pr-commits/build/test_commits.py:29: UnicodeDecodeError
> - generated xml file:
> /home/jenkins-build/build/workspace/ceph-pr-commits/report.xml -
> ==================== 1 failed, 1 deselected in 0.60 seconds ====================
> Build step 'Execute shell' marked build as failure
>
>
> Regards,
> --
>
> Alfonso Martínez
> Senior Software Engineer, Ceph Storage
> Red Hat
>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-07  8:23 "Signed-off-by" jenkins job for PRs now failing Alfonso Martinez Hidalgo
2020-10-08 14:28 ` Ken Dreyer

CEPH-Devel archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/ceph-devel/0 ceph-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ceph-devel ceph-devel/ https://lore.kernel.org/ceph-devel \
		ceph-devel@vger.kernel.org
	public-inbox-index ceph-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.ceph-devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git