[Xen-devel] [ovmf test] 128979: regressions - FAIL

[osstest service owner <osstest-admin@xxxxxxxxxxxxxx>]

flight 128979 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/128979/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 test-amd64-i386-xl-qemuu-ovmf-amd64 10 debian-hvm-install fail REGR. vs. 128974
 test-amd64-amd64-xl-qemuu-ovmf-amd64 10 debian-hvm-install fail REGR. vs. 
128974

version targeted for testing:
 ovmf                 ea98a8254ac9cf54e6c15ebfa65f582241ab4a53
baseline version:
 ovmf                 0019375fbc89e4d7cfebe29e288b74731bd9f456

Last test of basis   128974  2018-10-25 11:27:00 Z    0 days
Testing same since   128976  2018-10-25 18:40:59 Z    0 days    2 attempts

------------------------------------------------------------
People who touched revisions under test:
  Carsey, Jaben <jaben.carsey@xxxxxxxxx>
  Jaben Carsey <jaben.carsey@xxxxxxxxx>

jobs:
 build-amd64-xsm                                              pass    
 build-i386-xsm                                               pass    
 build-amd64                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          pass    
 build-i386-libvirt                                           pass    
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         fail    
 test-amd64-i386-xl-qemuu-ovmf-amd64                          fail    


------------------------------------------------------------
sg-report-flight on osstest.test-lab.xenproject.org
logs: /home/logs/logs
images: /home/logs/images

Logs, config files, etc. are available at
    http://logs.test-lab.xenproject.org/osstest/logs

Explanation of these reports, and of osstest in general, is at
    http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master
    http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master

Test harness code can be found at
    http://xenbits.xen.org/gitweb?p=osstest.git;a=summary


Not pushing.

------------------------------------------------------------
commit ea98a8254ac9cf54e6c15ebfa65f582241ab4a53
Author: Carsey, Jaben <jaben.carsey@xxxxxxxxx>
Date:   Wed Oct 24 01:29:24 2018 +0800

    BaseTools/GenFds: create and use new variable in FdfParser
    
    replace lots of '}' and "}" with a shared new consistent variable.
    
    Cc: Bob Feng <bob.c.feng@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit ca957eb5cd51251639d905853c0ccc6f0808eb71
Author: Carsey, Jaben <jaben.carsey@xxxxxxxxx>
Date:   Wed Oct 24 01:29:23 2018 +0800

    BaseTools/GenFds: refactor FdfParser warnings
    
    make functions for common error messages
    refactor to use these functions
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 5a6a268d51079ce66e09db5d5b6842d499256cb9
Author: Jaben Carsey <jaben.carsey@xxxxxxxxx>
Date:   Wed Oct 24 01:29:22 2018 +0800

    BaseTools/GenFds: remove MacroDict parameter
    
    The MacroDict parameter goes around in circles through 4 functions without 
use.
    1. GetSectionData calls into GetLeafSection, otherwise doesn?t use MacroDict
    2. GetLeafSection calls into GetFileStatement, otherwise doesn?t use 
MacroDict
    3. GetFileStatement calls into GetFilePart, otherwise doesn?t use MacroDict
    4. GetFilePart calls into GetSectionData, otherwise doesn?t use MacroDict
            Go to 1 and repeat forever.
    
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 9ffaaac2e5e7c5e21ad4ff2d0f9ac5d84878e57d
Author: Carsey, Jaben <jaben.carsey@xxxxxxxxx>
Date:   Wed Oct 24 01:29:21 2018 +0800

    Basetools/GenFds: refactor class FV
    
    1) initialize UiFvName via __init__ parameter. No change to default 
behavior.
    2) initialize 3 empty lists in __init__. Curently not guarenteed 
initialized.
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 5a264f28ee01e58f1dee19efd7a590942cf2fb21
Author: Carsey, Jaben <jaben.carsey@xxxxxxxxx>
Date:   Wed Oct 24 01:29:20 2018 +0800

    BaseTools/GenFds: change objects to sets
    
    Change lists and tuples used solely for "in" testing to sets.
    These operations are not order dependent.
    fixed some line length for PEP8 compliance on some.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

commit 9e47e6f90880e090cd81f585abd7a6c553fbce10
Author: Carsey, Jaben <jaben.carsey@xxxxxxxxx>
Date:   Wed Oct 24 01:29:19 2018 +0800

    BaseTools/GenFds: cleanup GenFds
    
    1) remove wildcard imports and use explicit imports
    2) refactor to use shared variables from Common/DataType
    3) rename to not shadow imports
    4) don't assign a variable in a loop (just do final assignment)
    5) remove spaces, parens, unused or commented out code, etc.
    6) merge unnecessary parent classes into child
    7) refactor to share DXE and PEI apriori GUIDs from one place
      this includes changes to Build and EOT files
    8) for PEP8, dont use __ for custom methods.
    
    Cc: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Cc: Bob C Feng <bob.c.feng@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Jaben Carsey <jaben.carsey@xxxxxxxxx>
    Reviewed-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel