Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please ignore: dotnet jenkins CI WIP #11646

Closed
wants to merge 13 commits into from

Conversation

RyanCavanaugh
Copy link
Member

Doing some setup work on the .net Jenkins CI since Travis seems to be 👻

@dilijev
Copy link
Contributor

dilijev commented Oct 19, 2016

@mmitche Does the initial commit of the configuration script need to be merged in before it's possible to request a CI test, or is something else wrong here?

@mmitche
Copy link
Member

mmitche commented Oct 19, 2016

No it should work, let me check on this.

@mmitche
Copy link
Member

mmitche commented Oct 19, 2016

DId you put the webhooks in from https://github.com/dotnet/dotnet-ci/blob/master/docs/CI-SETUP.md?

@RyanCavanaugh
Copy link
Member Author

I had the wrong Webhook set up (didn't realize I needed to change it when the config file changed to CI2). Updating now

@mmitche
Copy link
Member

mmitche commented Oct 19, 2016

Ahh sorry. So you added both yes? Anyway you can add me as an admin quick and I'll take a look? It looks like you have the right stuff

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@mmitche
Copy link
Member

mmitche commented Oct 19, 2016

@dotnet-bot test this please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test this please

@dilijev
Copy link
Contributor

dilijev commented Oct 19, 2016

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

Why does the bot ignore me? 🙁

@dilijev
Copy link
Contributor

dilijev commented Oct 19, 2016

It's not ignoring you, you're just asking for a test that doesn't exist yet :P
"test this please" is a trigger that only works once the Jenkins config is merged in, whereas what I typed tests the CI configuration in the current PR.

@RyanCavanaugh
Copy link
Member Author

When Matt wrote it, it triggered an actual test run. Or do the commands need to be invoked serially?

@dilijev
Copy link
Contributor

dilijev commented Oct 19, 2016

No, they are independent. I would think @mmitche request for "test this please" didn't actually do anything but he might have done something in the backend to wake @dotnet-bot up and pay attention to your request, and set that off with a "test this please"... or maybe there was a delay for a few minutes, but honestly I don't know.

You should try again! (No worries about the extra requests, this is a drop in the bucket.)

@mmitche
Copy link
Member

mmitche commented Oct 20, 2016

Oh, I had disabled the generated jobs agian.

@mmitche
Copy link
Member

mmitche commented Oct 20, 2016

@dotnet-bot test ci please

@mmitche
Copy link
Member

mmitche commented Oct 20, 2016

@dotnet-bot test this please (enabled jobs for testing)

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test this please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test this please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test this please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

1 similar comment
@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test this please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test ci please

@RyanCavanaugh
Copy link
Member Author

@dotnet-bot test this please

@RyanCavanaugh
Copy link
Member Author

Closing in favor of #11763

@RyanCavanaugh RyanCavanaugh deleted the jenkins branch October 20, 2016 23:25
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants