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

Set up CI with Azure Pipelines #1031

Merged
merged 5 commits into from
Mar 11, 2019
Merged

Set up CI with Azure Pipelines #1031

merged 5 commits into from
Mar 11, 2019

Conversation

gdude2002
Copy link
Contributor

This is just the initial file, we'll need to modify this.

@Pr0methean
Copy link
Contributor

Pr0methean commented Mar 6, 2019

Make sure the environment doesn't require software to be "cgroup-aware", as many newer container platforms do. This means commands such as free and top must report only our share of the host's memory. We have something somewhere in our build pipeline that's not cgroup-aware and runs out of memory on the CircleCI container platform (so we're now using their traditional VM instead), and I found it to be beyond reach of JVM args; maybe that's happening here too.

@gdude2002
Copy link
Contributor Author

Well, so far so good! I think we'll be alright, given that this runs just fine without the memory limiting specified in the CircleCI env vars. Still more work to do to finish this thing off, though.

@gdude2002 gdude2002 changed the title Set up CI with Azure Pipelines [WIP] Set up CI with Azure Pipelines Mar 8, 2019
@gdude2002 gdude2002 self-assigned this Mar 8, 2019
@gdude2002 gdude2002 changed the title [WIP] Set up CI with Azure Pipelines Set up CI with Azure Pipelines Mar 11, 2019
@gdude2002
Copy link
Contributor Author

This should now be functionally equivalent to the CircleCI pipeline.

@mastercoms mastercoms merged commit 9731f5a into dev Mar 11, 2019
@mastercoms mastercoms deleted the azure-pipelines branch March 11, 2019 10:57
@Paulomart
Copy link
Member

Why is the trigger set to master? Are feature branches no longer tested?

@gdude2002
Copy link
Contributor Author

Oh right, I forgot to update that from the template. I'll fix it in #1032

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

Successfully merging this pull request may close these issues.

None yet

4 participants