Merge ~ubuntu-release/autopkgtest/+git/development:flock-less into ~ubuntu-release/autopkgtest/+git/development:master

Proposed by Brian Murray
Status: Merged
Merge reported by: Brian Murray
Merged at revision: 1c14d445499f0b0ccc948db5e8a9f3be40d10356
Proposed branch: ~ubuntu-release/autopkgtest/+git/development:flock-less
Merge into: ~ubuntu-release/autopkgtest/+git/development:master
Diff against target: 19 lines (+6/-2)
1 file modified
ssh-setup/nova (+6/-2)
Reviewer Review Type Date Requested Status
Paride Legovini (community) Approve
Steve Langasek Needs Fixing
Review via email: mp+435769@code.qastaging.launchpad.net

Description of the change

The creation of instances for running autopkgtests was taking an extraordinarily long period of time because we had up to 70 processes waiting for the lock file to be released. This reduces the period of time which we wait and creates per architecture lock files.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) :
review: Needs Fixing
Revision history for this message
Paride Legovini (paride) wrote :

As discussed in standup a nicer command would be:

  openstack image show "$imageuuid" -c properties -f json | jq --raw-output .properties.architecture

but we don't have jq installed. We should probably add it to the charm: it's a useful tool and adding it would be a simple exercise on changing the charm dependencies.

In any case this is good to land, so approving.

review: Approve
Revision history for this message
Paride Legovini (paride) wrote :

(Not merging in case you want to add the jq command as a comment.)

Revision history for this message
Brian Murray (brian-murray) wrote :

Merged but with a squash so manually marking this as merged.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches