APPENG-4285 - Intercept Tracing Context in ExploitIQ Agent and save to DB#198
Conversation
|
/test-heavy |
- Added tracing context interception - Persisted context data to database - Updated confusion matrix and Tekton run handling
RedTanny
force-pushed
the
MLOpsAgentSide
branch
from
5bf28df to
d186e83
Compare
zvigrinberg
left a comment
zvigrinberg
left a comment
There was a problem hiding this comment.
@RedTanny This is a good job, that intercepts the spans and traces from the agent pipeline, in an independent manner from the agent!.
We'll need to do something similar for the online , that will not be enabled by default ( the default will be sending the traces to arize phoenix without the sidecars topology of otel collector and single broker kafka, we'll talk about how to do it.)
| TRACE_VERSION = 1 | ||
|
|
||
| # Default dispatch endpoint configuration | ||
| DEFAULT_URL = "https://exploit-iq-client-exploit-iq-tests.apps.ai-dev03.kni.syseng.devcluster.openshift.com" |
There was a problem hiding this comment.
@RedTanny Please change to service, this will become very critical very soon.
https://exploit-iq-client.exploit-iq-tests.svc.cluster.local:8443
Also, please verify using certificate ( ConfigMap - openshift-service-ca.crt - mount it to a volume in the container in order to access it from code for TLS Verification).
There was a problem hiding this comment.
- change to url to service -- will be done
- certificate -- do u mean we should use it in the tekton scripts ? why ? i understand we need to use it for the online / ui flow
There was a problem hiding this comment.
@RedTanny We need to do it anyway as a matter of fact...
There was a problem hiding this comment.
I Just wanted you not to focus on it from the beginning, rather on doing the requirement itself, and then get back to it..
There was a problem hiding this comment.
ok understand
will added it to the server (agent), test automation, and tracing script pod
scripts/HttpProvider.py
Outdated
| from urllib3.util.url import parse_url | ||
|
|
||
| # Default dispatch endpoint configuration | ||
| DEFAULT_URL = "https://exploit-iq-client-exploit-iq-tests.apps.ai-dev03.kni.syseng.devcluster.openshift.com" |
There was a problem hiding this comment.
@RedTanny Please change to service, this will become very critical very soon.
https://exploit-iq-client.exploit-iq-tests.svc.cluster.local:8443
Also, please verify using certificate ( ConfigMap - openshift-service-ca.crt - mount it to a volume in the container in order to access it from code for TLS Verification).
2 participants
No description provided.