-
Notifications
You must be signed in to change notification settings - Fork 280
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
Updating linux pipeline from BASE folder #2022
Updating linux pipeline from BASE folder #2022
Conversation
da25667
to
b5c3d17
Compare
BASE/.vsts/linux-build.yml
Outdated
@@ -1,47 +1,50 @@ | |||
pool: | |||
vmImage: 'ubuntu-16.04' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets move to newer version. 16 is getting older.. 18.04.5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i tried...using "ubuntu-latest" but two tests started to fail in MetricsTest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:D lets pick 18.04.5 and try. If it fails, then we have an issue with latest linux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
today, ubuntu-latest = 18.04
25d3b04
to
2843513
Compare
changing to ubuntu-latest (18.04 for now) reverting moving to ubuntu-latest Remove EmptyApp tests as its covered eslewhere (#2025) * Remove EmptyApp tests as its covered eslewhere * cleanup Ingestion service data delivery status (#1887) * Added eventhandler to transmission * Updated public API * Modified changelog.md * Added more tests * Fix API * Added coverage for timeout * Update to API * Modified comment in transmission. * PR Comments * PR feedback * Fix test * Added TransmissionStatusEvent to InMemoryChannel. * Remove Inmemory change Co-authored-by: Cijo Thomas <cithomas@microsoft.com> Co-authored-by: Timothy Mothra <tilee@microsoft.com> update version of Microsoft.AspNetCore.Hosting to 2.1.0 (and fix major test bug) (#2026) * Refactor tests and fix several issues in one go * remove unwanted changes * update changelog Co-authored-by: Timothy Mothra Lee <tilee@microsoft.com> changing to posix updating orderby removing unused usings and space updating trx folder path udoing
a5b45a7
to
c174e0c
Compare
Fix Issue # .
Changes
(Please provide a brief description of the changes here.)
Checklist
For significant contributions please make sure you have completed the following items:
The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:
Notes for reviewers:
/AzurePipelines run
will queue all builds/AzurePipelines run <pipeline-name>
will queue a specific build