> On Nov. 4, 2013, 8:48 a.m., Prasanna Santhanam wrote: > > Also your patch failed to apply for me on latest master: > > > > ~/workspace/cloudstack/incubator-cloudstack(branch:master) ยป git am > > patch/element.patch > > Applying: Added a verifyElementInList utility function > > /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:27: > > trailing whitespace. > > @Description: > > /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:33: > > trailing whitespace. > > at a given pos > > /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:34: > > trailing whitespace. > > @Input: > > /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:36: > > trailing whitespace. > > II : Element to verify whether it exists in the list > > error: patch failed: tools/marvin/marvin/codes.py:39 > > error: tools/marvin/marvin/codes.py: patch does not apply > > error: patch failed: tools/marvin/marvin/integration/lib/utils.py:353 > > error: tools/marvin/marvin/integration/lib/utils.py: patch does not apply > > Patch failed at 0001 Added a verifyElementInList utility function > > When you have resolved this problem run "git am --resolved". > > If you would prefer to skip this patch, instead run "git am --skip". > > To restore the original branch and stop patching run "git am --abort". > >
This patch already is applied to master and i could see it there, may be the status was not changed or not marked with ship it. - Santhosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15175/#review28089 ----------------------------------------------------------- On Nov. 1, 2013, 7:44 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15175/ > ----------------------------------------------------------- > > (Updated Nov. 1, 2013, 7:44 a.m.) > > > Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > The purpose is to verify a given element in list at a given position with few > error checks like list type,empty list and position > Returns appropriate codes based upon the inputs. Can be used under tests > instead of multiple asserts > > Earlier, there was a list validity function, this is on top of that. > > Need to go to both master and 4.2 > > > Diffs > ----- > > tools/marvin/marvin/codes.py bd01ad3 > tools/marvin/marvin/integration/lib/utils.py b6e38ec > > Diff: https://reviews.apache.org/r/15175/diff/ > > > Testing > ------- > > Tested. > > >>> from utils import verifyElementInList > >>> a=[1,2,3] > >>> verifyElementInList(a,2) > [0, 'ELEMENT NOT FOUND IN THE INPUT'] > >>> verifyElementInList(a,1) > [1, None] > >>> verifyElementInList(a,3) > [0, 'ELEMENT NOT FOUND IN THE INPUT'] > > > Thanks, > > Santhosh Edukulla > >