Skip to content

Hybrid agent perf testing#3449

Closed
kaylareopelle wants to merge 13 commits intodevfrom
hybrid-agent-perf-testing
Closed

Hybrid agent perf testing#3449
kaylareopelle wants to merge 13 commits intodevfrom
hybrid-agent-perf-testing

Conversation

@kaylareopelle
Copy link
Contributor

@kaylareopelle kaylareopelle commented Feb 10, 2026

Goals to run:

version 1 - no otel

hybrid-agent

version 2 - otel enabled, but instrumentation only from NR

hybrid-agent:NEW_RELIC_OPENTELEMETRY_ENABLED=true

version 3 - otel enabled, otel instrumentation included, nr instrumentation for same library disabled

hybrid-agent:NEW_RELIC_OPENTELEMETRY_ENABLED=true;NEW_RELIC_OPENTELEMETRY_TRACES_INCLUDE="OpenTelemetry::Instrumentation::HTTP::Instrumentation";NEW_RELIC_INSTRUMENTATION_HTTP=disabled

# frozen_string_literal: true

source 'https://rubygems.org'
source 'http://rubygems.org'

Check failure

Code scanning / CodeQL

Dependency download using unencrypted communication channel High test

Dependency source URL uses the unencrypted protocol HTTP. Use HTTPS instead.

Copilot Autofix

AI 9 days ago

In general, the fix is to ensure that all gem sources (and any other dependency repositories) use encrypted protocols such as HTTPS instead of HTTP. For Ruby Gemfiles, that means replacing any source 'http://...' entries with source 'https://...'.

For this specific file, the best fix without changing existing functionality is to modify line 3 of test/perfverse/rails7/Gemfile to use HTTPS: change source 'http://rubygems.org' to source 'https://rubygems.org'. No additional code changes, imports, or configuration are required; Bundler and RubyGems already support HTTPS for rubygems.org. The rest of the file (Ruby version and gem declarations) remains unchanged.

Suggested changeset 1
test/perfverse/rails7/Gemfile

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/test/perfverse/rails7/Gemfile b/test/perfverse/rails7/Gemfile
--- a/test/perfverse/rails7/Gemfile
+++ b/test/perfverse/rails7/Gemfile
@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-source 'http://rubygems.org'
+source 'https://rubygems.org'
 
 ruby '3.3.3'
 
EOF
@@ -1,6 +1,6 @@
# frozen_string_literal: true

source 'http://rubygems.org'
source 'https://rubygems.org'

ruby '3.3.3'

Copilot is powered by AI and may make mistakes. Always verify output.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is only a temporary workaround to get Docker to run on my machine.

@kaylareopelle kaylareopelle force-pushed the hybrid-agent-perf-testing branch from 6f404f9 to be0d084 Compare February 11, 2026 17:47
@github-actions
Copy link
Contributor

SimpleCov Report

Coverage Threshold
Line 93.33% 93%

@kaylareopelle
Copy link
Contributor Author

Using the git tag/env vars didn't behave as expected. We need unique git tags for every run, since the git tag is used for the column name in the output graph.

Instead, the git tags created to run these tests were:

  • hybrid-agent
  • hybrid-agent-disabled
  • ha-1
  • ha-1a
  • ha-2
  • ha-2a
  • ha-3
  • ha-3a
  • ha-3b

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.

1 participant

Comments