Closed Bug 1238798 Opened 8 years ago Closed 8 years ago

Update puppet yasm-dmg.sh for yasm 1.3.0

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rillian, Assigned: rillian)

References

Details

Attachments

(2 files)

In support of bug 1224408, update the puppet script for building yasm for the mac builders.
Attachment #8706686 - Flags: review?(dustin)
Comment on attachment 8706686 [details] [diff] [review]
Update yasm-dmg.sh for yasm 1.3.0

Review of attachment 8706686 [details] [diff] [review]:
-----------------------------------------------------------------

::: modules/packages/manifests/yasm-dmg.sh
@@ +20,5 @@
>  
> +VERSION=1.3.0
> +SHA2=3dce6601b495f5b3d45b59f7d2492a340ee7e84b5beca17e48f862502bd5603f
> +curl -LO https://www.tortall.net/projects/yasm/releases/yasm-$VERSION.tar.gz
> +CHECKSUM=$(shasum -a 256 yasm-$VERSION.tar.gz | cut -d ' ' -f 1)

:heart:
Attachment #8706686 - Flags: review?(dustin) → review+
Incidentally, mozreview is set up for this repo too if you'd like to use that.

Now, to figure out how to upgrade this.  It's easy enough to change the version number in the puppet manifests, and as machines reboot they will pick up the new version.  But that doesn't give us a per-tree approach, nor does it allow us to make a crisp transition.  How do you think we should proceed?
Flags: needinfo?(giles)
Thanks. yasm-1.3.0.dmg looks fine to me.

From what you say, to proceed, I suggest we:

1. Checkout one of the builders, install the pkg manually, and verify it works in the actual build environment.
2. Bump the puppet manifest to point to the new package, reboot and verify at least one machine.
3. Schedule all the machines to reboot so we get timely rollout.

I'll see about checking out a machine unless you have perms to do that directly.
Depends on: 1239174
Looks like step 1 is well underway.  I'll send a review req for the version bump your way.

Note that we don't have a way to schedule reboots.  Instead, machines reboot after finishing each job, and when they've been idle for a while.  In general, that's at least once a day.  But we'll have mixed results for a while.
Ok, mixed results over a day is fine. I just didn't want things to take weeks.
Comment on attachment 8707485 [details]
MozReview Request: Bug 1238798: bump yasm to 1.3.0 on Darwin; r?rillian

https://reviewboard.mozilla.org/r/30729/#review27545

I wondered if that was all that was required. Thanks for doing this bit!
Attachment #8707485 - Flags: review?(giles) → review+
The yasm 1.3.0 pkg installed manually on the loaner machine, and I was able to run yasm --version. Just waiting for a firefox build to complete for final confirmation.
Flags: needinfo?(giles)
Build with the new yasm confirmed working on a loaner machine. If you merge the puppet update, I'll check deployment.
Flags: needinfo?(dustin)
OK, it's landed
Flags: needinfo?(dustin)
Saw the first yasm 1.3.0 on and inbound build. Looks like the rollout is happening. Thanks for your help, Dustin!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: