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.

Sign up for GitHub

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

Jump to bottom

OpenAI configuration updates - part 2 #1866

Closed
bizob2828 opened this issue Nov 14, 2023 · 1 comment · Fixed by #1873
Closed

OpenAI configuration updates - part 2 #1866

bizob2828 opened this issue Nov 14, 2023 · 1 comment · Fixed by #1873
Assignees
@jsumners-nr

Comments

@bizob2828
Copy link
Member

Description

In #1848 we added a feature flag for openai instrumentation. Based on changes in requirements we are changing this to ai_monitoring.enabled. This will default to false until we finish the openai feature. When high security mode is enabled this should be set to false.

Acceptance Criteria

As a user of Node.js agent I want the ability to disable openai instrumentation and the sending of LLM events.
@workato-integration
Copy link

https://new-relic.atlassian.net/browse/NR-180009

jsumners-nr added a commit to jsumners-nr/node-newrelic that referenced this issue Nov 16, 2023
@jsumners-nr
feat: Add ai_monitoring configuration
1cf3ca4 
This change resolves newrelic#1866 by removing the previously added
feature flag and replacing it with the ai_monitoring configuration
directive.
@jsumners-nr jsumners-nr mentioned this issue Nov 16, 2023
Merged
jsumners-nr added a commit to jsumners-nr/node-newrelic that referenced this issue Nov 16, 2023
@jsumners-nr
feat: Add ai_monitoring configuration
4bc7192 
This change resolves newrelic#1866 by removing the previously added
feature flag and replacing it with the ai_monitoring configuration
directive.
@bizob2828 bizob2828 moved this to Done: Issues recently completed in Node.js Engineering Board Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jsumners-nr jsumners-nr

Labels
None yet
Projects
Node.js Engineering Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Add ai_monitoring configuration jsumners-nr/node-newrelic
2 participants
@bizob2828 @jsumners-nr