Project

General

Profile

Actions

Bug #4236

closed

Jenkins does not trigger a build on rebased patchset

Added by Junxiao Shi over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Build
Target version:
Start date:
08/09/2017
Due date:
% Done:

100%

Estimated time:

Description

Steps to reproduce:

  1. Post a patchset on Gerrit. Wait for Jenkins build to complete successfully.
  2. Find (or create) another Change. Wait for Jenkins build to complete successfully.
  3. Rebase the patchset in step1 onto the Change in step2, without changing another else.

Expected: Jenkins triggers a build on rebased patchset.
Actual: Jenkins does not trigger a build.

Actions #1

Updated by Junxiao Shi over 6 years ago

  • Assignee set to Eric Newberry
  • Priority changed from Normal to High

This problem has been occurring since Feb 2016, around the time when Gerrit was configured to preserve Code-Review votes on rebased patchsets. Verified votes aren't perserved, but builds aren't triggered as expected.
This problem severely slows down NFD development. It also seems to affect ndn-cxx and NLSR.

Actions #2

Updated by Eric Newberry over 6 years ago

  • Status changed from New to In Progress
Actions #3

Updated by Eric Newberry over 6 years ago

I've set NFD to also start a build on "ref update". Let's see what that does...

Actions #4

Updated by Eric Newberry over 6 years ago

Eric Newberry wrote:

I've set NFD to also start a build on "ref update". Let's see what that does...

It doesn't look like that's what we're looking for, as it appeared to cause unnecessary rebuilds. I've removed "ref update" from NFD and I'm still investigating this issue.

Actions #5

Updated by Eric Newberry over 6 years ago

  • Status changed from In Progress to Feedback

I've set builds to start for NFD on "trivial rebuilds", where there were no conflicts rebasing. This seems like what we're looking for...

Actions #6

Updated by Eric Newberry over 6 years ago

Eric Newberry wrote:

I've set builds to start for NFD on "trivial rebuilds", where there were no conflicts rebasing. This seems like what we're looking for...

It looks like ndn-cxx and NLSR already have their settings set this way.

Actions #7

Updated by Davide Pesavento over 6 years ago

What about ndn-tools?

Actions #8

Updated by Eric Newberry over 6 years ago

Davide Pesavento wrote:

What about ndn-tools?

ndn-tools is the same as ndn-cxx and NLSR.

Actions #9

Updated by Eric Newberry over 6 years ago

I looked through all of the automatic emails I received about builds not starting since the beginning of July. It looks like all the affected patchsets were for NFD. I have yet to receive an email since I changed the trivial rebuilds setting, so perhaps this fixed it?

Actions #10

Updated by Eric Newberry over 6 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 0 to 100

I still have yet to receive another automated email indicating that a build didn't start, so I'm going to assume that the issue is fixed and close it. I'll reopen it later if necessary.

Actions #11

Updated by Junxiao Shi over 6 years ago

I still have yet to receive another automated email indicating that a build didn't start

The script was no longer running due to unavailability of avenir. I have no plan of recovered it, even after avenir comes back.

close it. I'll reopen it later if necessary.

That's fine.

Actions

Also available in: Atom PDF