Closed Bug 1068542 Opened 10 years ago Closed 10 years ago

Update remote SSL tests for EV certs to make use of https://ssl-ev.mozqa.com/

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect, P1)

Product:
Mozilla QA Graveyard
Component:
Mozmill Tests
Type:
defect

Tracking

(firefox34 fixed, firefox35 fixed, firefox36 fixed, firefox37 fixed, firefox-esr31 fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox34 --- fixed
firefox35 --- fixed
firefox36 --- fixed
firefox37 --- fixed
firefox-esr31 --- fixed

People

(Reporter: whimboo, Assigned: daniela.domnici)

Details

(Whiteboard: [sprint])

Attachments

(3 files, 1 obsolete file)

patch_V2
3.97 KB, patch
AndreeaMatei
: review+
Details | Diff | Splinter Review
esr31_V1
3.04 KB, patch
AndreeaMatei
: review+
Details | Diff | Splinter Review
release_V1
3.94 KB, patch
AndreeaMatei
: review+
Details | Diff | Splinter Review 
With bug 891288 we accidentally got an EV cert created. As discussed over there lets make use of our own EV cert, which is reachable via https://ssl-ev.mozqa.com/

So we should update all of our mozmill tests, which are doing specific EV checks, from externally hosted domains to mozqa.com. That way we can also limit the dependency on the addons.mozilla.org server status.

This is one more bug to let us reduce the failure rate for tests.
Whiteboard: [sprint]
Assignee: nobody → daniela.domnici
Status: NEW → ASSIGNED
Attached patch patch_V1 (obsolete) — — Splinter Review 
Update remote SSL tests with EV certification.
Attachment #8529035 - Flags: review?(andrei.eftimie)
Attachment #8529035 - Flags: review?(andreea.matei)
Comment on attachment 8529035 [details] [diff] [review]
patch_V1

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

Looks good, couldn't test as it doesn't apply anymore. Please update and retest.
Attachment #8529035 - Flags: review?(andrei)
Attachment #8529035 - Flags: review?(andreea.matei)
Attachment #8529035 - Flags: review-
Attached patch patch_V2 — — Splinter Review 

        Attachment #8531224 -
        Flags: review?(mihaela.velimiroviciu)

        Attachment #8531224 -
        Flags: review?(andreea.matei)

    
    

    
  
Comment on attachment 8531224 [details] [diff] [review]
patch_V2

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

Looks good, tests well.
http://hg.mozilla.org/qa/mozmill-tests/rev/655bcf44d8a7 (default)

Lets backport this as it brings stability to our tests.

        Attachment #8531224 -
        Flags: review?(mihaela.velimiroviciu)

        Attachment #8531224 -
        Flags: review?(andreea.matei)

        Attachment #8531224 -
        Flags: review+

    
    

    
  
Don't forget to obsolete patches that got r- when adding a new one.

          status-firefox34:
          --- → affected

          status-firefox35:
          --- → affected

          status-firefox36:
          --- → affected

          status-firefox37:
          --- → fixed

          status-firefox-esr31:
          --- → affected

        Attachment #8529035 -
        Attachment is obsolete: true

    
    

    
  
The attachment #8531224 [details] [diff] [review] was applied on nightly, aurora and beta. And for esr31 and release I have applied new patches.

    
    

    
  

      
      
      
      

        Attached patch
          
          
        esr31_V1
        — 
          — Splinter Review
      

    


  

        Attachment #8531224 -
        Attachment is obsolete: true

        Attachment #8531600 -
        Flags: review?(mihaela.velimiroviciu)

        Attachment #8531600 -
        Flags: review?(andreea.matei)

    
    

    
  

      
      
      
      

        Attached patch
          
          
        release_V1
        — 
          — Splinter Review
      

    


  

        Attachment #8531601 -
        Flags: review?(mihaela.velimiroviciu)

        Attachment #8531601 -
        Flags: review?(andreea.matei)

        Attachment #8531224 -
        Attachment is obsolete: false

        Attachment #8531601 -
        Flags: review?(mihaela.velimiroviciu)

        Attachment #8531601 -
        Flags: review?(andreea.matei)

        Attachment #8531601 -
        Flags: review+

    
    

    
  
Comment on attachment 8531600 [details] [diff] [review]
esr31_V1

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

http://hg.mozilla.org/qa/mozmill-tests/rev/88bf9c1b4bc6 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/46ed29017b5e (esr31)

Thanks Daniela!

        Attachment #8531600 -
        Flags: review?(mihaela.velimiroviciu)

        Attachment #8531600 -
        Flags: review?(andreea.matei)

        Attachment #8531600 -
        Flags: review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago

          status-firefox34:
          affected → fixed

          status-firefox-esr31:
          affected → fixed
Resolution: --- → FIXED

    
  
Product: Mozilla QA → Mozilla QA Graveyard

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: