Closed Bug 1305911 Opened 8 years ago Closed 8 years ago

change all SeaMonkey automation steps to clone off/check for https:// and not http:

Categories

(SeaMonkey :: Release Engineering, defect)

Product:
SeaMonkey
Component:
Release Engineering
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: ewong, Assigned: ewong)

References

Details

Attachments

(5 files, 2 obsolete files)

[configs] use https instead of http [checked-in]
4.68 KB, patch
frg
: review+
Details | Diff | Splinter Review
[custom] use https instead of http (v3) [checked-in]
13.28 KB, patch
frg
: review+
Details | Diff | Splinter Review
[custom] part 2 of http -> https [checked-in]
18.88 KB, patch
frg
: review+
Details | Diff | Splinter Review
[tools] change http to https [checked-in]
13.17 KB, patch
frg
: review+
Details | Diff | Splinter Review
[custom] missed the http mention in getRepository() [checked-in]
1.07 KB, patch
frg
: review+
Details | Diff | Splinter Review
we're busting with 2.46 due to finger print issues and since we use http for some steps but client.py uses https, I figured it's better to use https throughout the whole process.
Blocks: 450645
No longer blocks: SM2.46
Summary: change release steps to clone off https:// and not http: → change all SeaMonkey automation steps to clone off/check for https:// and not http:
We need to move to https for automation as bug 450645 will 301 all http requests. [fwiw.. we solved the 2.46 blockage by updating the fingerprints.]
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #8829033 - Flags: review?(frgrahl)
Attachment #8829034 - Flags: review?(frgrahl)
Comment on attachment 8829033 [details] [diff] [review] [configs] use https instead of http [checked-in] ok
Attachment #8829033 - Flags: review?(frgrahl) → review+
Comment on attachment 8829034 [details] [diff] [review] [custom] use https instead of http oki too for the https stuff. > Watch https://tbpl.mozilla.org/?tree=Try&rev=%(revision)s for your results to come in. https://tbpl.mozilla.org is no longer valid but I think try is broken for SeaMonkey anyway.
Attachment #8829034 - Flags: review?(frgrahl) → review+
(In reply to Frank-Rainer Grahl from comment #5) > Comment on attachment 8829034 [details] [diff] [review] > [custom] use https instead of http > > oki too for the https stuff. > > > Watch https://tbpl.mozilla.org/?tree=Try&rev=%(revision)s for your results to come in. > > https://tbpl.mozilla.org is no longer valid but I think try is broken for > SeaMonkey anyway. we don't use that try section anyway.
I think I forgot some other items.
Attachment #8829034 - Attachment is obsolete: true
Attachment #8834723 - Flags: review?(frgrahl)
Comment on attachment 8834723 [details] [diff] [review] [custom] use https instead of http (v2) >> https://localhost Does this work? r+ if you think yes.
Attachment #8834723 - Flags: review?(frgrahl) → review+
https://localhost doesn't work and tbh, really is not necessary.
Attachment #8834723 - Attachment is obsolete: true
Attachment #8837436 - Flags: review?(frgrahl)
Attachment #8837436 - Flags: review?(frgrahl) → review+
https://hg.mozilla.org/build/buildbotcustom/rev/8d20f0061c7272dae4d2dab0f91b0794b7250c63 Bug 1305911 - Use https instead of http for hg.mozilla.org in buildbotcustom code. r=frg
missed a few more items
Attachment #8844368 - Flags: review?(frgrahl)
Attachment #8844368 - Flags: review?(frgrahl) → review+
ewong, ping, you have review+
Flags: needinfo?(ewong)
Comment on attachment 8844368 [details] [diff] [review] [custom] part 2 of http -> https [checked-in] Pushed to buildbotcustom: https://hg.mozilla.org/build/buildbotcustom/rev/ddd1b696cc27
Attachment #8844368 - Attachment description: [custom] part 2 of http -> https → [custom] part 2 of http -> https [checked-in]
Flags: needinfo?(ewong)
Comment on attachment 8837436 [details] [diff] [review] [custom] use https instead of http (v3) [checked-in] Pushed to buildbotcustom: https://hg.mozilla.org/build/buildbotcustom/rev/8d20f0061c72
Attachment #8837436 - Attachment description: [custom] use https instead of http (v3) → [custom] use https instead of http (v3) [checked-in]
Comment on attachment 8829033 [details] [diff] [review] [configs] use https instead of http [checked-in] Pushed to buildbot-configs: https://hg.mozilla.org/build/buildbot-configs/rev/52360c824e78
Attachment #8829033 - Attachment description: [configs] use https instead of http → [configs] use https instead of http [checked-in]
this patch just covers the scripts/files that we use and *could potentially* use (i.e. release_repacks.sh).
Attachment #8849825 - Flags: review?(frgrahl)
Comment on attachment 8849825 [details] [diff] [review] [tools] change http to https [checked-in] looks good.
Attachment #8849825 - Flags: review?(frgrahl) → review+
Attachment #8850821 - Flags: review?(frgrahl) → review+
Attachment #8849825 - Attachment description: [tools] change http to https → [tools] change http to https [checked-in]
Comment on attachment 8850821 [details] [diff] [review] [custom] missed the http mention in getRepository() [checked-in] Pushed to buildbotcustom: https://hg.mozilla.org/build/buildbotcustom/rev/37fa3b7dd930
Attachment #8850821 - Attachment description: [custom] missed the http mention in getRepository() → [custom] missed the http mention in getRepository() [checked-in]
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: