Ubuntu CI Services

Check if we need to upload the payload before sending to swift

Bug #1287942 reported by Evan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu CI Engine
Fix Released
High
Vincent Ladeuil
Ubuntu CI Engine phase-0
Ubuntu CI Services
Fix Released
High
Vincent Ladeuil
Ubuntu CI Services phase-0

Bug Description

10:01 PM <ev> hm, perhaps we should sha1 the deployment payload and make that the filename for the upload to swift
10:02 PM <ev> that way we can do a swift list to see if we even need to upload anything
10:02 PM <ev> rather than pushing the payload up with every deployment
10:02 PM <plars> good idea

Tags: airline

Related branches

lp:~vila/ubuntu-ci-services-itself/12897942-sha1-payload
Para Siva (community): Approve
PS Jenkins bot (community): Approve (continuous-integration)
Diff: 12 lines (+1/-1)
1 file modified
juju-deployer/deploy.py (+1/-1)
Vincent Ladeuil (vila)
Changed in ubuntu-ci-services-itself:
assignee: Evan Dandrea (ev) → Vincent Ladeuil (vila)
status: New → Confirmed
importance: Undecided → High
milestone: backlog → phase-0
Revision history for this message
Andy Doan (doanac) wrote :

you really think this is a must-have for phase0?

Revision history for this message
Vincent Ladeuil (vila) wrote :

> you really think this is a must-have for phase0?

A must-have for phase-0, no. But it itches so if I can write a patch quickly I'll do. If I don't I'll postpone. Deal ?

Revision history for this message
Vincent Ladeuil (vila) wrote :

Ok, it took slightly longer than expected.

Just sha'ing the tarball didn't work, gzip adds the tarball original name and the creation time, the tarball itself have timestamps for the dirs that change for each tarball (by design).

I resorted to calculating the sha1 for all included files.

Changed in ubuntu-ci-services-itself:
status: Confirmed → In Progress
Vincent Ladeuil (vila)
Changed in ubuntu-ci-services-itself:
status: In Progress → Fix Committed
Andy Doan (doanac)
Changed in ubuntu-ci-services-itself:
status: Fix Committed → Fix Released
Ursula Junque (ursinha)
Changed in uci-engine:
assignee: nobody → Vincent Ladeuil (vila)
importance: Undecided → High
milestone: none → phase-0
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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