{"thread":{"id":"46982","subject":"Is t5601 flaky for anybody else?","startedAt":"2017-10-17T05:04:17Z","lastAt":"2017-10-22T09:36:38Z","messageCount":5,"participants":["Junio C Hamano","Brandon Williams","Jeff King","Kaartic Sivaraam"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"330521","messageId":"xmqq376ipdpx.fsf@gitster.mtv.corp.google.com","threadId":"46982","inReplyTo":null,"subject":"Is t5601 flaky for anybody else?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-10-17T05:04:10Z","receivedAt":"2017-10-17T05:04:17Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"I seem to be seeing sporadic errors with this test, and today I got\nannoyed enough to do\n\n\tcd t && while sh t5601-clone.sh -i -v; do :; done\n\nI saw an error from \"cp\" saying \"plink.exe - text file busy\" or\nsomething like that at around test #45; here is an workaround that\nseems to work (the above loop is spinning without problem for\nseveral minutes now).\n\n t/t5601-clone.sh | 2 ++\n 1 file changed, 2 insertions(+)\n\ndiff --git a/t/t5601-clone.sh b/t/t5601-clone.sh\nindex 9c56f771b6..50e40abb11 100755\n--- a/t/t5601-clone.sh\n+++ b/t/t5601-clone.sh\n@@ -308,6 +308,7 @@ test_expect_success 'clone checking out a tag' '\n \n setup_ssh_wrapper () {\n \ttest_expect_success 'setup ssh wrapper' '\n+\t\trm -f \"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n \t\tcp \"$GIT_BUILD_DIR/t/helper/test-fake-ssh$X\" \\\n \t\t\t\"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n \t\tGIT_SSH=\"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n@@ -318,6 +319,7 @@ setup_ssh_wrapper () {\n }\n \n copy_ssh_wrapper_as () {\n+\trm -f \"${1%$X}$X\" &&\n \tcp \"$TRASH_DIRECTORY/ssh-wrapper$X\" \"${1%$X}$X\" &&\n \tGIT_SSH=\"${1%$X}$X\" &&\n \texport GIT_SSH\n"},{"id":"330552","messageId":"20171017171046.GA48544@google.com","threadId":"46982","inReplyTo":"xmqq376ipdpx.fsf@gitster.mtv.corp.google.com","subject":"Re: Is t5601 flaky for anybody else?","fromName":"Brandon Williams","fromEmail":"bmwill@google.com","sentAt":"2017-10-17T17:10:46Z","receivedAt":"2017-10-17T17:10:54Z","isPatch":false,"sender":{"key":"bwilliams.eng@gmail.com","avatar":null},"body":"On 10/17, Junio C Hamano wrote:\n> I seem to be seeing sporadic errors with this test, and today I got\n> annoyed enough to do\n> \n> \tcd t && while sh t5601-clone.sh -i -v; do :; done\n> \n> I saw an error from \"cp\" saying \"plink.exe - text file busy\" or\n> something like that at around test #45; here is an workaround that\n> seems to work (the above loop is spinning without problem for\n> several minutes now).\n\nI haven't noticed any issues myself but maybe this has something to do\nwith my changes to this test in the 'bw/protocol-v1' topic?\n\n> \n>  t/t5601-clone.sh | 2 ++\n>  1 file changed, 2 insertions(+)\n> \n> diff --git a/t/t5601-clone.sh b/t/t5601-clone.sh\n> index 9c56f771b6..50e40abb11 100755\n> --- a/t/t5601-clone.sh\n> +++ b/t/t5601-clone.sh\n> @@ -308,6 +308,7 @@ test_expect_success 'clone checking out a tag' '\n>  \n>  setup_ssh_wrapper () {\n>  \ttest_expect_success 'setup ssh wrapper' '\n> +\t\trm -f \"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n>  \t\tcp \"$GIT_BUILD_DIR/t/helper/test-fake-ssh$X\" \\\n>  \t\t\t\"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n>  \t\tGIT_SSH=\"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n> @@ -318,6 +319,7 @@ setup_ssh_wrapper () {\n>  }\n>  \n>  copy_ssh_wrapper_as () {\n> +\trm -f \"${1%$X}$X\" &&\n>  \tcp \"$TRASH_DIRECTORY/ssh-wrapper$X\" \"${1%$X}$X\" &&\n>  \tGIT_SSH=\"${1%$X}$X\" &&\n>  \texport GIT_SSH\n\n-- \nBrandon Williams\n"},{"id":"330572","messageId":"xmqqshehmqrg.fsf_-_@gitster.mtv.corp.google.com","threadId":"46982","inReplyTo":"20171017171046.GA48544@google.com","subject":"Re* Is t5601 flaky for anybody else?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-10-17T21:02:59Z","receivedAt":"2017-10-17T21:03:11Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Brandon Williams <bmwill@google.com> writes:\n\n> I haven't noticed any issues myself but maybe this has something to do\n> with my changes to this test in the 'bw/protocol-v1' topic?\n\nAs I've seen this on 'master', too, I suspect the topic has nothing\nto do with it.\n\nHere is what I have on 'pu'.\n\n\n-- >8 --\nFrom: Junio C Hamano <gitster@pobox.com>\nDate: Tue, 17 Oct 2017 14:04:43 +0900\nSubject: [PATCH] t5601: rm the target file of cp that could still be executing\n\n\"while sh t5601-clone.sh; do :; done\" seems to fail sporadically at\naround test #45 where fake-ssh wrapper is copied create plink.exe,\nwith an error message that says the \"text is busy\".\n\nI have a mild suspicion that the root cause of the bug is that the\nfake SSH process from the previous test is still running by the time\nthe next test wants to replace it with a new binary, but in the\nmeantime, removing the target that could still be executing before\ncopying something else over seems to work it around.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n t/t5601-clone.sh | 2 ++\n 1 file changed, 2 insertions(+)\n\ndiff --git a/t/t5601-clone.sh b/t/t5601-clone.sh\nindex 9c56f771b6..50e40abb11 100755\n--- a/t/t5601-clone.sh\n+++ b/t/t5601-clone.sh\n@@ -308,6 +308,7 @@ test_expect_success 'clone checking out a tag' '\n \n setup_ssh_wrapper () {\n \ttest_expect_success 'setup ssh wrapper' '\n+\t\trm -f \"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n \t\tcp \"$GIT_BUILD_DIR/t/helper/test-fake-ssh$X\" \\\n \t\t\t\"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n \t\tGIT_SSH=\"$TRASH_DIRECTORY/ssh-wrapper$X\" &&\n@@ -318,6 +319,7 @@ setup_ssh_wrapper () {\n }\n \n copy_ssh_wrapper_as () {\n+\trm -f \"${1%$X}$X\" &&\n \tcp \"$TRASH_DIRECTORY/ssh-wrapper$X\" \"${1%$X}$X\" &&\n \tGIT_SSH=\"${1%$X}$X\" &&\n \texport GIT_SSH\n-- \n2.15.0-rc1-178-ge1264d9eb8\n\n"},{"id":"330575","messageId":"20171017210833.n5vgly7z2zi5dhuh@sigill.intra.peff.net","threadId":"46982","inReplyTo":"xmqqshehmqrg.fsf_-_@gitster.mtv.corp.google.com","subject":"Re: Re* Is t5601 flaky for anybody else?","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-10-17T21:08:34Z","receivedAt":"2017-10-17T21:08:40Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Oct 18, 2017 at 06:02:59AM +0900, Junio C Hamano wrote:\n\n> Brandon Williams <bmwill@google.com> writes:\n> \n> > I haven't noticed any issues myself but maybe this has something to do\n> > with my changes to this test in the 'bw/protocol-v1' topic?\n> \n> As I've seen this on 'master', too, I suspect the topic has nothing\n> to do with it.\n> \n> Here is what I have on 'pu'.\n\nFWIW, I can't replicate the problem on either \"master\" or \"pu\". I wonder\nwhy.\n\n-Peff\n"},{"id":"330800","messageId":"1508664986.6715.4.camel@gmail.com","threadId":"46982","inReplyTo":"20171017210833.n5vgly7z2zi5dhuh@sigill.intra.peff.net","subject":"Re: Re* Is t5601 flaky for anybody else?","fromName":"Kaartic Sivaraam","fromEmail":"kaartic.sivaraam@gmail.com","sentAt":"2017-10-22T09:36:26Z","receivedAt":"2017-10-22T09:36:38Z","isPatch":false,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Tue, 2017-10-17 at 17:08 -0400, Jeff King wrote:\n> On Wed, Oct 18, 2017 at 06:02:59AM +0900, Junio C Hamano wrote:\n> \n> FWIW, I can't replicate the problem on either \"master\" or \"pu\". I wonder\n> why.\n> \n\nNeither can I. I tried running t5601 on \"master\" and on \"pu\" after\nreverting the patch found in this thread. Both ran successfully without\nissues.\n\n-- \nKaartic\n"}]}