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

RESOLVED FIXED

Status

SeaMonkey
Release Engineering
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: ewong, Assigned: ewong)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(5 attachments, 2 obsolete attachments)

(Assignee)

Description

2 years ago
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.
(Assignee)

Updated

2 years ago
Blocks: 450645
No longer blocks: 1294433
(Assignee)

Updated

2 years ago
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:
(Assignee)

Comment 1

2 years ago
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)

Comment 2

2 years ago
Created attachment 8829033 [details] [diff] [review]
[configs] use https instead of http [checked-in]
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #8829033 - Flags: review?(frgrahl)
(Assignee)

Comment 3

2 years ago
Created attachment 8829034 [details] [diff] [review]
[custom] use https instead of http
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+
(Assignee)

Comment 6

2 years ago
(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.
(Assignee)

Comment 8

2 years ago
Created attachment 8834723 [details] [diff] [review]
[custom] use https instead of http (v2)

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+
(Assignee)

Comment 10

a year ago
Created attachment 8837436 [details] [diff] [review]
[custom] use https instead of http (v3) [checked-in]

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+
(Assignee)

Comment 11

a year ago
https://hg.mozilla.org/build/buildbotcustom/rev/8d20f0061c7272dae4d2dab0f91b0794b7250c63
Bug 1305911 - Use https instead of http for hg.mozilla.org in buildbotcustom code. r=frg
(Assignee)

Comment 12

a year ago
Created attachment 8844368 [details] [diff] [review]
[custom] part 2 of http -> https [checked-in]

missed a few more items
Attachment #8844368 - Flags: review?(frgrahl)
Attachment #8844368 - Flags: review?(frgrahl) → review+
ewong, ping, you have review+
Flags: needinfo?(ewong)
(Assignee)

Comment 14

a year ago
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)
(Assignee)

Comment 15

a year ago
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]
(Assignee)

Comment 16

a year ago
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]
(Assignee)

Comment 17

a year ago
Created attachment 8849825 [details] [diff] [review]
[tools] change http to https [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+
(Assignee)

Comment 19

a year ago
Created attachment 8850821 [details] [diff] [review]
[custom] missed the http mention in getRepository() [checked-in]
Attachment #8850821 - Flags: review?(frgrahl)
Attachment #8850821 - Flags: review?(frgrahl) → review+
(Assignee)

Updated

a year ago
Attachment #8849825 - Attachment description: [tools] change http to https → [tools] change http to https [checked-in]
(Assignee)

Comment 20

a year ago
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]
(Assignee)

Updated

a year ago
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.