Project

General

Profile

Bug #22423

Option --scan-revisions did not work so well

Added by Gianluca Petrillo almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Vito Di Benedetto
Target version:
Continuous Integration (CI) - v1_9_0_RC
Start date:
04/19/2019
Due date:
% Done:

100%

Estimated time:
Duration:

Description

My working area was icarusbuild01.fnal.gov:/icarus/app/users/petrillo/LArSoft/develop/c2_prof, with content in srcs:

develop [icarus_data]
develop [icaruscode]
develop [icarusutil]
develop [larana]
develop [larcore]
feature/gp_moreGeometryFeatures [larcorealg]
develop [larcoreobj]
develop [lardataalg]
feature/gp_refreshLArVoxelReadout [larsim]

The output of trigger --build-delay 0 --scan-revisions was:

using revision:icarus_data@914e5c6
icaruscode@
icarusutil@
larana@
larcore@
larcorealg@
larcoreobj@
lardataalg@
larsim@1e4cef21
Triggering CI build

and the resulting failure can be contemplated in the LArCI dashboard : http://dbweb5.fnal.gov:8080/LarCI/app/build_detail/phase_details?build_id=lar_ci/5339&platform=Linux%203.10.0-957.5.1.el7.x86_64&phase=checkout&buildtype=slf7%20e17:prof.

I am using a current version of lar_ci, v3_6_0.

Associated revisions

Revision fca7287e (diff)
Added by Vito Di Benedetto over 1 year ago

fix scan_for_revisions function for #22423

History

#1 Updated by Vito Di Benedetto over 1 year ago

  • Target version set to v1_9_0_RC
  • Assignee set to Vito Di Benedetto
  • Status changed from New to Work in progress

#2 Updated by Vito Di Benedetto over 1 year ago

  • % Done changed from 0 to 90

Gianluca, thanks to report this issue.
It has been fixed into generic_ci develop branch.
Next CI release candidate is planned for the end of June. At that time there will be a test UPS package available through CVMFS.
If you think it will be useful to have this fix available earlier, I'll be happy to make a new UPS package in coming days.

#3 Updated by Gianluca Petrillo over 1 year ago

Nope; take your time.

#4 Updated by Vito Di Benedetto over 1 year ago

  • % Done changed from 90 to 100

#5 Updated by Vito Di Benedetto over 1 year ago

  • Status changed from Work in progress to Resolved

CI v1_9_0 declared current on Monday has this issue fixed.

#6 Updated by Vito Di Benedetto over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF