search

tds-fdw / tds_fdw

A PostgreSQL foreign data wrapper to connect to TDS databases (Sybase and Microsoft SQL Server)
Other
381 stars 102 forks source link

Test PR for PR builder [DO NOT MERGE!] #70

Closed juliogonzalez closed 8 years ago

juliogonzalez commented 8 years ago

Just a Test PR for Jenkins PR Builder.

@GeoffMontee, do not merge this PR as it's just for testing.

juliogonzalez commented 8 years ago

Ok, BOT seems to be working fine as it started running tests for this PR :-)

juliogonzalez commented 8 years ago

Fine. Let's try to break the PR :-)

juliogonzalez commented 8 years ago

Ok, all comments were being used to trigger tests as long as the comment was being issued by an admin. Changed now to use a trigger.

juliogonzalez commented 8 years ago

Checking the trigger...

juliogonzalez commented 8 years ago

Retest this please

juliogonzalez commented 8 years ago

Testing config again...

juliogonzalez commented 8 years ago

And again... The bot should be ignorning my comments unless the passphrase is present.

juliogonzalez commented 8 years ago

Testing...

juliogonzalez commented 8 years ago

Test FAILed.

juliogonzalez commented 8 years ago

Bot installed

juliogonzalez commented 8 years ago

test this please

juliogonzalez commented 8 years ago

Test PASSed.

jenkins-juliogonzalez commented 8 years ago

Testing comments...

juliogonzalez commented 8 years ago

Interesting.... bot is installed but the job still uses my credentials. I just tried reconfiguring the job. If that doesn't work I'll need to reboot Jenkins.

juliogonzalez commented 8 years ago

retest this please

jenkins-juliogonzalez commented 8 years ago

Test FAILed.

juliogonzalez commented 8 years ago

retest this please

jenkins-juliogonzalez commented 8 years ago

Test PASSed.

jenkins-juliogonzalez commented 8 years ago

Test FAILed.

juliogonzalez commented 8 years ago

Fine!

@GeoffMontee, could you please trigger the bot to see if your rights are correctly configured at Jenkins?

Just write a comment with the same content as my last comment before the last "Test passed".

After this I'll test this for "external users". They should be able to run create PRs, but tests shouldn't run automatically. In this case either you or me will issue a comment to trigger tests after reviewing and before merging.

GeoffMontee commented 8 years ago

retest this please

jenkins-juliogonzalez commented 8 years ago

Test FAILed.

juliogonzalez commented 8 years ago

Great :-D

Final step is to test a PR from an external user (not a project member). After I confirm that it's working as expected, I'll write a small manual on how to use this, so you can see how it works (maybe you'll even start creating your own PRs :-)

GeoffMontee commented 8 years ago

Awesome. Thanks, @juliogonzalez! This PR webhook seems really useful!

juliogonzalez commented 8 years ago

Tests are complete. Closing PR.