Skip to content

Commit

Permalink
smoke: kill all testuser processes before smoketest
Browse files Browse the repository at this point in the history
  • Loading branch information
@joaomoreno
joaomoreno committed Oct 4, 2017
1 parent 08e6c5d commit d78d5da
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions build/tfs/linux/build.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ function smoketest {
sudo -i -u testuser -- sh -c 'git config --global user.name "VS Code Agent" && git config --global user.email "monacotools@microsoft.com"'
chown -R testuser $SCREENSHOTS

ps -o pid= -u testuser | xargs sudo kill -9
DISPLAY=:10 sudo -i -u testuser -- sh -c "cd $BUILD_SOURCESDIRECTORY/test/smoke && ./node_modules/.bin/mocha --build $AGENT_BUILDDIRECTORY/VSCode-linux-$ARCH --screenshots $SCREENSHOTS"
}

Expand Down

3 comments on commit d78d5da

@michelkaporin
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should have done it first before trying to investigate the root issue of some things you raise in #34203...

@joaomoreno
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eheh hey man! It seems you only scratched the surface huh? 😆

@michelkaporin
Copy link
Contributor

@michelkaporin michelkaporin commented on d78d5da Oct 6, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joaomoreno hey! yeah.. I see you do a good painful job over the smoke test, nice work! 👍 i'll bombard you with questions on that when I will visit you (not sure yet when) 😄

Please sign in to comment.