← Back to team overview

launchpad-dev team mailing list archive

Warning: For a few hours, Please merge devel into your branch before running ec2 test

 

Hi everyone,

Just got some feedback on the ec2 fix that landed yesterday. For the next few hours, you MUST merge 'devel' into the branch you will be testing with ec2 before running the test suite, or you may find that ec2 passes without running any tests and without failing!

This bug affects people who have the 'ec2' command linked to 'lp-branches/devel/utilities/ec2', and then run 'ec2' in a branch that has not had the latest 'devel' merged in.

As a workaround, you can:

   a) merge 'devel' into your branch

b) run the 'utilities/ec2' script from within the branch you are currently testing.


We are working on a fix now.


The error happens because the utilities/ec2 script imports _pythonpath.py from the local branch directory. Even if you run the 'ec2' command from via a symlink, if you run it in the local branch, it will actually pick up the *local* _pythonpath, and start running the older, local copy of ec2test: call it v1. When your branch is copied up to ec2, test_on_merge will check out a fresh copy of devel, and then merge your branch into it. That merge updates your Makefile to the latest version: v2. The ec2 runner then calls the v2 Makefile with your v1 remote.py: fail. The detailed test log will say SUCCESS with a single message:

  "make: Nothing to be done for `ec2_check'."


I'll keep everyone posted in this thread.


Maris



Follow ups