promoter script is not comparing timestamps correctly when folding hashes

Bug #1768090 reported by Gabriele Cerami
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Gabriele Cerami

Bug Description

When the promoter looks at the timestamp to choose only to promote hashes older than the one previously promoted, it's comparing timestamps from two different promotions.

So for example, when trying to promote to current-tripleo-rdo we look at the last promotion timestamp, then try to find timestamps older than this among the current-tripleo promotions.
So, if the latest promotion to current-tripleo-rdo for hash A happened after a current-tripleo promotion for hash B (and it's happening only lately that we have so many promotions) then promotion to current-tripleo-rdo for hash B will be skipped.

We need to start also confronting the hash and be sure we are comparing timestamps at the same level (current-tripleo with current-tripleo)

summary: - promoter script is not confronting timestamps correctly when folding
+ promoter script is not comparing timestamps correctly when folding
hashes
Revision history for this message
Gabriele Cerami (gcerami) wrote :

I'll have to rewrite the get_latest_hashes function to do a comparison based on hashes instead of just timestamps.

Revision history for this message
Gabriele Cerami (gcerami) wrote :

Proposed patch https://review.rdoproject.org/13604, tested in the promoter server, stage environment with dry run, returns correct list of hashes.

Changed in tripleo:
status: Triaged → Fix Committed
importance: Undecided → High
Revision history for this message
Gabriele Cerami (gcerami) wrote :
Revision history for this message
Matt Young (halcyondude) wrote :

https://review.rdoproject.org/r/#/c/13604 still needs workflow, it's not released/commited

Changed in tripleo:
status: Fix Committed → In Progress
Revision history for this message
Matt Young (halcyondude) wrote :

flipped status back to in progress, we don't have "releases" of CI, and with this state it's not been showing up for triage.

Changed in tripleo:
milestone: rocky-2 → rocky-3
Arx Cruz (arxcruz)
Changed in tripleo:
status: In Progress → Fix Released
tags: removed: promotion-blocker
tags: added: promotion-blocker
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.