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

Turn on Github CI #13489

Closed
wants to merge 1 commit into from
Closed

Turn on Github CI #13489

wants to merge 1 commit into from

Conversation

@mattcaswell
Copy link
Member

@mattcaswell mattcaswell commented Nov 23, 2020

As an interim measure until we work out our longer term CI strategy
this PR enables some basic CI tests using the Github CI capability.

You can take a look at a sample of what this looks like here:
mattcaswell/temptest#5

I've just enabled a small number of builds. It is not yet clear to me what the throughput of this service is - and some builds took 1.5 hours to complete, so I don't want to overload things. The objective of this is to just try and get some CI testing back so that we can at least start committing things again.

As an interim measure until we work out our longer term CI strategy
this PR enables some basic CI tests using the Github CI capability.
@mattcaswell
Copy link
Member Author

@mattcaswell mattcaswell commented Nov 23, 2020

Note also that some builds are expected to fail. There are some issues in master at the moment. I'm not seeking to resolve them as part of this PR.

@t-j-h
Copy link
Member

@t-j-h t-j-h commented Nov 23, 2020

+1

@mattcaswell
Copy link
Member Author

@mattcaswell mattcaswell commented Nov 23, 2020

Given the lack of CI testing at the moment - are we ok with classifying this as urgent? Adding that label - sorry should have added it before. Please reconfirm with that label!

@mattcaswell mattcaswell added the urgent label Nov 23, 2020
@levitte
Copy link
Member

@levitte levitte commented Nov 23, 2020

Urgent is a good idea. My approval stands.

@rsbeckerca
Copy link

@rsbeckerca rsbeckerca commented Nov 23, 2020

To be a bit of a pain, any word on issue #13152, since we can't do GitHub CI to that platform.

@mattcaswell
Copy link
Member Author

@mattcaswell mattcaswell commented Nov 23, 2020

To be a bit of a pain, any word on issue #13152, since we can't do GitHub CI to that platform.

@levitte is working on a buildbot infrastructure which should help with that. This PR is more about dealing with the immediate lack of access to Travis.

openssl-machine pushed a commit that referenced this pull request Nov 24, 2020
As an interim measure until we work out our longer term CI strategy
this PR enables some basic CI tests using the Github CI capability.

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tim Hudson <tjh@openssl.org>
(Merged from #13489)
@mattcaswell
Copy link
Member Author

@mattcaswell mattcaswell commented Nov 24, 2020

Pushed. We'll need to try and figure out the fixes for the various failing items.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants