Opened 2 years ago

Last modified 6 months ago

#20351 assigned enhancement

Turn the updateindex module into a function that runs after each module run

Reported by: iwakeh Owned by: metrics-team
Priority: Medium Milestone: CollecTor 1.7.0
Component: Metrics/CollecTor Version:
Severity: Normal Keywords: metrics-2018
Cc: metrics-team Actual Points:
Parent ID: #20518 Points:
Reviewer: Sponsor:

Description

Once, #20350 is in place. updateindex can be plainly added as a function that runs after each module run.

Child Tickets

Change History (11)

comment:1 Changed 2 years ago by iwakeh

Parent ID: #20518

comment:2 Changed 14 months ago by karsten

Summary: updateindex should become a functionTurn the updateindex module into a function that runs after each module run

Attempt to make the summary a bit clearer.

comment:3 Changed 14 months ago by karsten

Keywords: metrics-2018 added

comment:4 Changed 14 months ago by karsten

Owner: set to metrics-team
Status: newassigned

comment:5 Changed 8 months ago by iwakeh

Owner: changed from metrics-team to iwakeh
Status: assignedaccepted

comment:6 Changed 8 months ago by irl

Cc: metrics-team added

Adding metrics-team to cc

comment:7 Changed 7 months ago by iwakeh

Status: acceptedneeds_review

Please review this patch branch. I chose warn-level for index-update failures as these are attempted after each module run.

(review is not urgent as this should be deployed together with #20350)

comment:8 Changed 7 months ago by iwakeh

Milestone: CollecTor 1.6.0

comment:9 Changed 7 months ago by karsten

Status: needs_reviewneeds_information

I just did a quick review of commit 7aa8984, and I think this can work. I did not try it out, though. We'll indeed have to do #20350 first before merging this. Not sure if this is the right thing to do, but I'm setting this ticket to needs_information until #20350 is resolved. Thanks!

comment:10 Changed 6 months ago by iwakeh

Owner: changed from iwakeh to metrics-team
Status: needs_informationassigned

Re-assigning to metrics-team as the patch is provided as well as reviewed and the ticket is waiting for the implementation of #20350.

comment:11 Changed 6 months ago by iwakeh

Milestone: CollecTor 1.6.0CollecTor 1.7.0

Moved to new milestone.

Note: See TracTickets for help on using tickets.