Workbench fixes - #4405
Workbench fixes#4405michaeljguarino wants to merge 4 commits into
Conversation
|
plural deploy this |
Soffi AI SummaryThis PR delivers a set of fixes and improvements to the AI Workbench, focusing on OCI registry compatibility, Docker tools reliability, and build/CI optimizations. OCI registry robustness: The OCI client was significantly refactored to handle edge cases found in registries like Docker workbench tool improvements: The HTTP utility additions: A new Build and CI optimizations: The Dockerfile was restructured for better layer caching and build efficiency. Several CI/CD workflows (test, publish, nexus, observability-proxy) were consolidated and streamlined. Unused Mix dependencies were pruned from Monitoring workbench improvements: The dashboard upsert tool schema and monitoring prompt were updated with minor enhancements. ChangesOCI registry pagination and proxy compatibility fixes
Docker workbench search_tags tool refactor
Build and CI pipeline optimizations
Updated: 2026-09-24 03:40 UTC |
There was a problem hiding this comment.
Deployment PR created: https://github.com/pluralsh/plrl-up-demos/pull/3047 (pins Console to sha-df25317; Plural Service: mgmt/apps).
|
| def proxy_options(%{url: proxy} = config, url) when is_binary(proxy) and proxy != "" do | ||
| case no_proxy?(config, url) do | ||
| true -> [] | ||
| false -> req_options(proxy: proxy) |
There was a problem hiding this comment.
When an OCI proxy URL contains credentials, such as http://user:pass@proxy.example.com:8080, this path passes it through req_options/1. The proxy parser copies only the scheme, host, and port and uses empty proxy options, so the username and password are dropped. Registry and token requests therefore cannot authenticate with the proxy and will fail, typically with a 407 response. Preserve the credentials in the generated proxy options and add a test for authenticated proxy URLs.
Comments Outside DiffThese findings sit on lines the diff does not cover, so they could not be posted inline. Each one leaves this list once its file changes.
|
|
plural deploy this |
|
Deployment PR created: https://github.com/pluralsh/plrl-up-demos/pull/3048 — promotes this PR head (df25317) as Console image tag sha-df25317. It is annotated Plural Service: mgmt/apps. |
|
plural deploy this |
|
Follow-up: deployment GitOps PR pluralsh/plrl-up-demos#3052 promotes this PR’s current head as |
8fd1b30 to
7effc54
Compare
* proxy config on docker, etc tools * ...
7effc54 to
c47ad8a
Compare
|
plural deploy this |
There was a problem hiding this comment.
Deployment PR created: https://github.com/pluralsh/plrl-up-demos/pull/3053
|
plural deploy this |
|
GitOps promotion PR: https://github.com/pluralsh/plrl-up-demos/pull/3054 — pins the Console image to |
Test Plan
Test environment: https://console.your-env.onplural.sh/
Checklist
Plural Flow: console