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

E2E test deployment issues #228

Closed
drcgjung opened this issue Jul 24, 2024 · 0 comments · Fixed by #229
Closed

E2E test deployment issues #228

drcgjung opened this issue Jul 24, 2024 · 0 comments · Fixed by #229
Assignees
Labels
bug Something isn't working

Comments

@drcgjung
Copy link
Contributor

drcgjung commented Jul 24, 2024

Current Behavior

Helm-Deployed Agent Planes cannot communicate with connectors/control planes over http (only https).
This is due to a default url allowance configuration which requires https as the protocol scheme.
This cannot be easily overriden (need to use envVar section).

Furthermore, calling a consumer-only skill with a sparql-result body will fail because of unbound value bindings (sended text is not bound, parameters in body have been read from request and will not be bound again).

Calling a consumer-only skill with a runMode ALL will work, but is executed on the PROVIDER side. That is wrong

see eclipse-tractusx/knowledge-agents#150

Expected Behavior

the helm chart of the agent planes should have explicit values in the agent.services.connector section in order to override the default allowance regexes.

Steps To Reproduce

Deploy helm chart an use non-ingress control plane reference

@drcgjung drcgjung added the bug Something isn't working label Jul 24, 2024
@drcgjung drcgjung self-assigned this Jul 24, 2024
@drcgjung drcgjung moved this from 🆕 New to 👀 In review in Knowledge Agent Kanban Board Jul 24, 2024
drcgjung added a commit to drcgjung/knowledge-agents-edc that referenced this issue Jul 24, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Knowledge Agent Kanban Board Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant