Use GitHub workflow for CI on PRs and master builds #231
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I have several PR's where I'm waiting on a release. Anything I may be able to assist with here to help get your builds running? |
NathanielRN
approved these changes
Mar 7, 2022
jon-armen
approved these changes
Mar 7, 2022
You can see the run in this PR itself or on my fork here. @jon-armen |
Co-authored-by: Nathaniel Ruiz Nowell <enruizno@uwaterloo.ca>
Co-authored-by: Nathaniel Ruiz Nowell <enruizno@uwaterloo.ca>
jon-armen
reviewed
Mar 7, 2022
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| version: [net452, netcoreapp2.0, netcoreapp3.1] |
There was a problem hiding this comment.
Any reason we're targeting netcoreapp2.0 out of support? Why not target 2.1
Suggested change
| version: [net452, netcoreapp2.0, netcoreapp3.1] | |
| version: [net452, netcoreapp2.1, netcoreapp3.1] |
There was a problem hiding this comment.
Never mind for now, I see that the test projects are currently referencing netcoreapp2.0. Much better to get CI up and running any make those changes in another pr.
There was a problem hiding this comment.
Yes. We should update our targeted frameworks to the supported ones for the whole SDK some time soon.
There was a problem hiding this comment.
I've got a PR out there to do just that. I'll update it once we have this merged.
srprash
added a commit
that referenced
this pull request
Mar 21, 2022
* Create stale.yml (#136) * Release commit for v2.9.0 (#137) * Added 2 seconds timeout for EC2 metadata requests (#138) * Update NOTICE so it only has third party licenses and no proprietary notice (#139) * Fixing link to EFCore usage section (#143) * Added EF Core installation instruction step (#145) * Fixing race condition for initializing and running timer in RulePoller (#154) * Bump AWSSDK.Core dependency version (#155) * Bump AWSSDK.Core dependency version * Bumped AWSSDK.Core version as range. * Removed redundant dependency * Applied HttpClient to get sampling info (#159) * Applied HttpWebRequest to get sampling info and removed dependency on AWSSDK.XRay * Fixed lock issue and switched to HttpClient * Added and modified documentations. * Added documentations. * Set ExpectContinue header as false by default * Adding limitation for EFCore with multiple DbCommandInterceptor (#164) * Adding limitation for EFCore with multiple DbCommandInterceptor * Adding doc ref for AsyncLocal * Updated xraycontext to use a concurrent dictionary (#157) To avoid concurrent thread access to the dictionary causing an exception * Enable tracing S3 id pairs and fix SNS naming issue. (#168) * Enable tracing S3 id pairs and fix SNS naming issue. * Changed AWSSDK.Core version * Changed parameter names and updated comments. * Set parameters as default * Fixed appvoyer building configuration issue. (#173) * Add tracing support for EntityFramework 6 (.NET Framework) (#171) * Added support for EntityFramework 6 for .NET framework. * Added release workflow (#179) * Added release workflow * Added space line * Update configuration * Switching out AppVeyor for GH workflow (#181) * Switching out AppVeyor for GH workflow * Running workflow on linux as well. building wth release config * Revert "Switching out AppVeyor for GH workflow (#181)" (#185) This reverts commit 34134f0. * Added public key token (#186) * Release commit for v2.10.0 (#187) * Updated API Doc (#188) * Add smoke test of distribution channel (#189) * Add smoke test of distribution channel * Updated AWSXRayRecorder.sln * Points to the latest version * Publish metric on distribution availability (#190) * Run test on ubuntu (#191) * Bump System.Net.Http from 4.3.3 to 4.3.4 in /sdk/src/Core (#193) Bumps System.Net.Http from 4.3.3 to 4.3.4. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Release commit for v2.10.1 (#194) * Update continuous-monitoring.yml * Increase continuous monitoring to 10 minutes * Add netcoreapp3.1 target framework Add a target framework moniker for .NET Core 3.1. * Create CODEOWNERS * Update appveyor.yml (#226) * Update appveyor.yml * Update appveyor.yml * Update .netappcore version for smoke tests (#228) * Update continuous-monitoring.yml * Update netcoreapp version in .csproj file * Update continuous-monitoring.yml * Update AWSXRayRecorder.SmokeTests.csproj * Use GitHub workflow for CI on PRs and master builds (#231) * Update continuous-monitoring.yml * Add Source Link support for enhanced debugging experience (#229) * Add Source Link support for enhanced debugging experience * NetFramework does not support embedded pdb Co-authored-by: Jon Armen <jarmen@bstglobal.com> * Finding the same exception in different subsegments should reuse the same cause id (#210) Co-authored-by: Jon Armen <jarmen@bstglobal.com> * Removed blocking call which was producing deadlocks from EC2Plugin. (#234) * Adding 3.x for continous build Co-authored-by: srprash <50466688+srprash@users.noreply.github.com> Co-authored-by: Anuraag Agrawal <aanuraag@amazon.co.jp> Co-authored-by: Lu Peng <61207760+lupengamzn@users.noreply.github.com> Co-authored-by: Stephen D'Olier <stephen@dolier.net> Co-authored-by: Bhautik Pipaliya <56270044+bhautikpip@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: William Armiros <54150514+willarmiros@users.noreply.github.com> Co-authored-by: martincostello <martin@martincostello.com> Co-authored-by: Jon Armen <jarmen@bstglobal.com> Co-authored-by: Scott Munro <scott.munro@yahoo.com>
jon-armen
added a commit
to jon-armen/aws-xray-sdk-dotnet
that referenced
this pull request
Apr 15, 2022
* Create stale.yml (aws#136) * Release commit for v2.9.0 (aws#137) * Added 2 seconds timeout for EC2 metadata requests (aws#138) * Update NOTICE so it only has third party licenses and no proprietary notice (aws#139) * Fixing link to EFCore usage section (aws#143) * Added EF Core installation instruction step (aws#145) * Fixing race condition for initializing and running timer in RulePoller (aws#154) * Bump AWSSDK.Core dependency version (aws#155) * Bump AWSSDK.Core dependency version * Bumped AWSSDK.Core version as range. * Removed redundant dependency * Applied HttpClient to get sampling info (aws#159) * Applied HttpWebRequest to get sampling info and removed dependency on AWSSDK.XRay * Fixed lock issue and switched to HttpClient * Added and modified documentations. * Added documentations. * Set ExpectContinue header as false by default * Adding limitation for EFCore with multiple DbCommandInterceptor (aws#164) * Adding limitation for EFCore with multiple DbCommandInterceptor * Adding doc ref for AsyncLocal * Updated xraycontext to use a concurrent dictionary (aws#157) To avoid concurrent thread access to the dictionary causing an exception * Enable tracing S3 id pairs and fix SNS naming issue. (aws#168) * Enable tracing S3 id pairs and fix SNS naming issue. * Changed AWSSDK.Core version * Changed parameter names and updated comments. * Set parameters as default * Fixed appvoyer building configuration issue. (aws#173) * Add tracing support for EntityFramework 6 (.NET Framework) (aws#171) * Added support for EntityFramework 6 for .NET framework. * Added release workflow (aws#179) * Added release workflow * Added space line * Update configuration * Switching out AppVeyor for GH workflow (aws#181) * Switching out AppVeyor for GH workflow * Running workflow on linux as well. building wth release config * Revert "Switching out AppVeyor for GH workflow (aws#181)" (aws#185) This reverts commit 34134f0. * Added public key token (aws#186) * Release commit for v2.10.0 (aws#187) * Updated API Doc (aws#188) * Add smoke test of distribution channel (aws#189) * Add smoke test of distribution channel * Updated AWSXRayRecorder.sln * Points to the latest version * Publish metric on distribution availability (aws#190) * Run test on ubuntu (aws#191) * Bump System.Net.Http from 4.3.3 to 4.3.4 in /sdk/src/Core (aws#193) Bumps System.Net.Http from 4.3.3 to 4.3.4. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Release commit for v2.10.1 (aws#194) * Update continuous-monitoring.yml * Increase continuous monitoring to 10 minutes * Add netcoreapp3.1 target framework Add a target framework moniker for .NET Core 3.1. * Create CODEOWNERS * Update appveyor.yml (aws#226) * Update appveyor.yml * Update appveyor.yml * Update .netappcore version for smoke tests (aws#228) * Update continuous-monitoring.yml * Update netcoreapp version in .csproj file * Update continuous-monitoring.yml * Update AWSXRayRecorder.SmokeTests.csproj * Use GitHub workflow for CI on PRs and master builds (aws#231) * Update continuous-monitoring.yml * Add Source Link support for enhanced debugging experience (aws#229) * Add Source Link support for enhanced debugging experience * NetFramework does not support embedded pdb Co-authored-by: Jon Armen <jarmen@bstglobal.com> * Finding the same exception in different subsegments should reuse the same cause id (aws#210) Co-authored-by: Jon Armen <jarmen@bstglobal.com> * Removed blocking call which was producing deadlocks from EC2Plugin. (aws#234) * Adding 3.x for continous build Co-authored-by: srprash <50466688+srprash@users.noreply.github.com> Co-authored-by: Anuraag Agrawal <aanuraag@amazon.co.jp> Co-authored-by: Lu Peng <61207760+lupengamzn@users.noreply.github.com> Co-authored-by: Stephen D'Olier <stephen@dolier.net> Co-authored-by: Bhautik Pipaliya <56270044+bhautikpip@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: William Armiros <54150514+willarmiros@users.noreply.github.com> Co-authored-by: martincostello <martin@martincostello.com> Co-authored-by: Jon Armen <jarmen@bstglobal.com> Co-authored-by: Scott Munro <scott.munro@yahoo.com>
jon-armen
added a commit
to jon-armen/aws-xray-sdk-dotnet
that referenced
this pull request
Apr 15, 2022
* Create stale.yml (aws#136) * Release commit for v2.9.0 (aws#137) * Added 2 seconds timeout for EC2 metadata requests (aws#138) * Update NOTICE so it only has third party licenses and no proprietary notice (aws#139) * Fixing link to EFCore usage section (aws#143) * Added EF Core installation instruction step (aws#145) * Fixing race condition for initializing and running timer in RulePoller (aws#154) * Bump AWSSDK.Core dependency version (aws#155) * Bump AWSSDK.Core dependency version * Bumped AWSSDK.Core version as range. * Removed redundant dependency * Applied HttpClient to get sampling info (aws#159) * Applied HttpWebRequest to get sampling info and removed dependency on AWSSDK.XRay * Fixed lock issue and switched to HttpClient * Added and modified documentations. * Added documentations. * Set ExpectContinue header as false by default * Adding limitation for EFCore with multiple DbCommandInterceptor (aws#164) * Adding limitation for EFCore with multiple DbCommandInterceptor * Adding doc ref for AsyncLocal * Updated xraycontext to use a concurrent dictionary (aws#157) To avoid concurrent thread access to the dictionary causing an exception * Enable tracing S3 id pairs and fix SNS naming issue. (aws#168) * Enable tracing S3 id pairs and fix SNS naming issue. * Changed AWSSDK.Core version * Changed parameter names and updated comments. * Set parameters as default * Fixed appvoyer building configuration issue. (aws#173) * Add tracing support for EntityFramework 6 (.NET Framework) (aws#171) * Added support for EntityFramework 6 for .NET framework. * Added release workflow (aws#179) * Added release workflow * Added space line * Update configuration * Switching out AppVeyor for GH workflow (aws#181) * Switching out AppVeyor for GH workflow * Running workflow on linux as well. building wth release config * Revert "Switching out AppVeyor for GH workflow (aws#181)" (aws#185) This reverts commit 34134f0. * Added public key token (aws#186) * Release commit for v2.10.0 (aws#187) * Updated API Doc (aws#188) * Add smoke test of distribution channel (aws#189) * Add smoke test of distribution channel * Updated AWSXRayRecorder.sln * Points to the latest version * Publish metric on distribution availability (aws#190) * Run test on ubuntu (aws#191) * Bump System.Net.Http from 4.3.3 to 4.3.4 in /sdk/src/Core (aws#193) Bumps System.Net.Http from 4.3.3 to 4.3.4. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Release commit for v2.10.1 (aws#194) * Update continuous-monitoring.yml * Increase continuous monitoring to 10 minutes * Add netcoreapp3.1 target framework Add a target framework moniker for .NET Core 3.1. * Create CODEOWNERS * Update appveyor.yml (aws#226) * Update appveyor.yml * Update appveyor.yml * Update .netappcore version for smoke tests (aws#228) * Update continuous-monitoring.yml * Update netcoreapp version in .csproj file * Update continuous-monitoring.yml * Update AWSXRayRecorder.SmokeTests.csproj * Use GitHub workflow for CI on PRs and master builds (aws#231) * Update continuous-monitoring.yml * Add Source Link support for enhanced debugging experience (aws#229) * Add Source Link support for enhanced debugging experience * NetFramework does not support embedded pdb Co-authored-by: Jon Armen <jarmen@bstglobal.com> * Finding the same exception in different subsegments should reuse the same cause id (aws#210) Co-authored-by: Jon Armen <jarmen@bstglobal.com> * Removed blocking call which was producing deadlocks from EC2Plugin. (aws#234) * Adding 3.x for continous build Co-authored-by: srprash <50466688+srprash@users.noreply.github.com> Co-authored-by: Anuraag Agrawal <aanuraag@amazon.co.jp> Co-authored-by: Lu Peng <61207760+lupengamzn@users.noreply.github.com> Co-authored-by: Stephen D'Olier <stephen@dolier.net> Co-authored-by: Bhautik Pipaliya <56270044+bhautikpip@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: William Armiros <54150514+willarmiros@users.noreply.github.com> Co-authored-by: martincostello <martin@martincostello.com> Co-authored-by: Jon Armen <jarmen@bstglobal.com> Co-authored-by: Scott Munro <scott.munro@yahoo.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue: Our AppVeyor CI has been a hit or miss lately. Although it does run, the CI is not able to post results back to Github PR. As a result the PRs are not blocked even if the AppVeyor CI fails. This is dangerous as we are flying blind. This is good point to use native GH workflows to run CI and we don't have to maintain AppVeyor CI anymore.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.