Skip to content

ci: remove travis - #4790

Merged
ethomson merged 2 commits into
masterfrom
ethomson/travis
Sep 3, 2018
Merged

ci: remove travis#4790
ethomson merged 2 commits into
masterfrom
ethomson/travis

Conversation

@ethomson

Copy link
Copy Markdown
Member

Travis has started erroring:

WARNING: The following packages cannot be authenticated!
  libssh2-1 libcurl4-gnutls-dev libcurl3-gnutls curl libcurl3 libssh2-1-dev
E: There were unauthenticated packages and -y was used without --allow-unauthenticated

I... don't have any idea why. And, to be completely honest, I don't have a strong interesting in investigating. At this point, VSTS supports Linux, WIndows and macOS builds in a single place and at roughly the same speed as Travis. Travis isn't giving us much additional benefit and I don't want to spend a lot of time to try to figure this out.

Objections?

@pks-t

pks-t commented Aug 31, 2018 via email

Copy link
Copy Markdown
Member

@ethomson

Copy link
Copy Markdown
Member Author

The only issue I have right now with VSTS is that I don't have any access to trigger builds or otherwise do anything to the environment. Does it require a Microsoft account to get access?

It does. Do you have one?

It looks like the identity team is starting to integrate other ways to sign in to Microsoft services but it sounds like it's Not Quite Ready Yet.

One thing that I was thinking was to hook Probot up so that you could do a /rebuild in a PR comment or over email and have it do some magic. This might be useful if you were not interested in creating a Microsoft account.

@tiennou

tiennou commented Aug 31, 2018

Copy link
Copy Markdown
Contributor

Since we're discussing access, I created an account with my GitHub's profile email a few weeks ago.

Though I wouldn't mind using Probot for that 😉.

@ethomson

ethomson commented Sep 2, 2018

Copy link
Copy Markdown
Member Author

I decided to whip up a Probot script after all. I need to figure out how/where to deploy it but then I'll continue to move forward with this.

@pks-t

pks-t commented Sep 3, 2018 via email

Copy link
Copy Markdown
Member

@ethomson

ethomson commented Sep 3, 2018

Copy link
Copy Markdown
Member Author

/rebuild

@libgit2-azure-pipelines

Copy link
Copy Markdown

Sorry @ethomson, an error occurred while trying to requeue the build.

@ethomson

ethomson commented Sep 3, 2018

Copy link
Copy Markdown
Member Author

/rebuild

@libgit2-azure-pipelines

Copy link
Copy Markdown

Sorry @ethomson, an error occurred while trying to requeue the build.

@ethomson

ethomson commented Sep 3, 2018

Copy link
Copy Markdown
Member Author

/rebuild

@libgit2-azure-pipelines

Copy link
Copy Markdown

Okay, @ethomson, I started to rebuild this pull request.

@ethomson

ethomson commented Sep 3, 2018

Copy link
Copy Markdown
Member Author

Okay, @ethomson, I started to rebuild this pull request.

😅

@ethomson

ethomson commented Sep 3, 2018

Copy link
Copy Markdown
Member Author

So I did create a GitHub App, based on Probot.

If you're curious, it's https://github.com/ethomson/probot-vsts-build, and I intend to write up something more proper. But in a nutshell, it will listen for issue comments that are /rebuild (and only that, although whitespace is trimmed). It will then ensure that the person who issued the command is a collaborator on the project (per the GitHub API notion, which basically means that they've been added to a team). Then it looks up the corresponding projects in VSTS, and looks for build definitions in those projects that are set to build pull requests in this repository. (Since a GitHub App is per-account, this lets us use it on other repositories in the libgit2 project). Finally, it will look at builds that were queued for this particular pull request within each of those build definitions, and re-queues them.

(This is true whether the builds passed or failed.)

@ethomson
ethomson merged commit 045cc32 into master Sep 3, 2018
@pks-t

pks-t commented Sep 6, 2018

Copy link
Copy Markdown
Member

/rebuild

@libgit2-azure-pipelines

Copy link
Copy Markdown

Okay, @pks-t, I started to rebuild this pull request.

@pks-t

pks-t commented Sep 6, 2018

Copy link
Copy Markdown
Member

That's great! It'd be even greater if the bot comment also linked to the new build that is being performed. No idea if it has all required information for that readily available, though

@ethomson

ethomson commented Sep 6, 2018

Copy link
Copy Markdown
Member Author

That's a good idea - let me see what I can do.

@pks-t

pks-t commented Sep 6, 2018

Copy link
Copy Markdown
Member

Thanks a lot for working on this! It really improves our infrastructure by a whole lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants