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

.Net Core Agent provider #356

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ChNorman
Copy link

updated with lwrp to install New Relic .Net Core Agent

@ChNorman
Copy link
Author

Resolves Issue #359

@djoos
Copy link
Member

djoos commented Nov 1, 2018

Hi @ChNorman,

thanks for this PR - is there any chance to rebase the PR so it only includes the New Relic .Net Core Agent changes? It seems other changes are included as well currently...

Thanks in advance!
David

@ChNorman
Copy link
Author

ChNorman commented Nov 1, 2018

hi @djoos

I could attempt to but outside of the changes for the .net core the only other thing in the pull request was changes required to fet integrated testing to pass and to get the request to pass your CI build

@ChNorman
Copy link
Author

ChNorman commented Dec 31, 2018

hi @djoos

I rebased as requested but it looks like some current rubocop rules are causing CI integration to fail. Please let me know how you would like to address because some of the failures are out side code that I modified

 Changes to be committed:
	modified:   .kitchen.yml
	modified:   attributes/repository.rb
	modified:   libraries/helpers.rb
	modified:   libraries/matchers.rb
	modified:   metadata.rb
	modified:   providers/agent_dotnet.rb
	new file:   providers/agent_dotnetcore.rb
	modified:   recipes/default.rb
	new file:   recipes/dotnetcore_agent.rb
	new file:   resources/agent_dotnetcore.rb
	modified:   spec/unit/agent_dotnet_spec.rb
	new file:   spec/unit/agent_dotnetcore_spec.rb
	modified:   spec/unit/agent_php_spec.rb
	modified:   spec/unit/default_spec.rb
	modified:   spec/unit/repository_spec.rb
	new file:   test/fixtures/cookbooks/newrelic_lwrp_test/recipes/agent_dotnetcore.rb
	modified:   test/fixtures/cookbooks/newrelic_lwrp_test/recipes/agent_php.rb
	new file:   test/integration/dotnetcore-agent/serverspec/default_spec.rb
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.

2 participants