Bug #12575

Fix basebox:clean_old

Added by intrigeri 2017-05-22 07:47:47 . Updated 2017-06-12 16:12:16 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Build system
Target version:
Start date:
2017-05-22
Due date:
% Done:

100%

Feature Branch:
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:
289

Description

See parent ticket for details.


Subtasks


Related issues

Blocks Tails - Feature #12576: Have Jenkins use basebox:clean_old instead of basebox:clean_all Resolved 2017-05-22

History

#1 Updated by intrigeri 2017-05-22 07:48:50

  • blocks Feature #12576: Have Jenkins use basebox:clean_old instead of basebox:clean_all added

#2 Updated by anonym 2017-05-25 13:27:14

  • Status changed from Confirmed to In Progress
  • Assignee changed from anonym to bertagaz
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA

Since we don’t run this code anywhere I pushed the fix straight to feature/stretch: see commit:d0f0179e2738703090bf246653b665c1d2977b99.

#3 Updated by intrigeri 2017-05-27 08:58:14

  • blocked by deleted (Feature #12576: Have Jenkins use basebox:clean_old instead of basebox:clean_all)

#4 Updated by intrigeri 2017-05-27 08:58:46

#5 Updated by intrigeri 2017-05-27 08:59:05

#6 Updated by intrigeri 2017-05-27 09:01:23

  • blocks Feature #12576: Have Jenkins use basebox:clean_old instead of basebox:clean_all added

#7 Updated by intrigeri 2017-05-27 09:01:59

  • Priority changed from High to Elevated

(Same rationale as Feature #12576.)

#8 Updated by bertagaz 2017-05-30 15:27:14

  • Status changed from In Progress to Fix committed
  • Assignee deleted (bertagaz)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

anonym wrote:
> Since we don’t run this code anywhere I pushed the fix straight to feature/stretch: see commit:d0f0179e2738703090bf246653b665c1d2977b99.

Works better now indeed. I’ve also pushed a commit on top (see commit:fc1bba108640e6057bcd0a34f052de1aebab5461) that lowers the cleaning of baseboxes to those that are older than 4 months, as we discussed in Bug #12531#note-20.

#9 Updated by bertagaz 2017-05-30 16:28:13

  • Status changed from Fix committed to In Progress

Applied in changeset commit:fc1bba108640e6057bcd0a34f052de1aebab5461.

#10 Updated by intrigeri 2017-05-30 16:44:59

  • Assignee set to bertagaz

Shouldn’t this be “fix committed”?

#11 Updated by bertagaz 2017-05-30 17:05:50

  • Status changed from In Progress to Fix committed

intrigeri wrote:
> Shouldn’t this be “fix committed”?

Right, the commit did change it back after I put it the right way.

#12 Updated by bertagaz 2017-05-30 17:06:30

  • Assignee deleted (bertagaz)

#13 Updated by intrigeri 2017-06-12 16:12:16

  • Status changed from Fix committed to Resolved