Closed Bug 1305911 Opened 8 years ago Closed 7 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: 7 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: