feat: proxy customer container registries - #358
Open
alongubkin wants to merge 10 commits into
Open
Conversation
Greptile SummaryThe PR adds Manager-local proxying for customer container registries while keeping provider credentials inside the Manager and separating data-plane access from repository lifecycle permissions.
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains in the previously reported customer upload-continuation paths. Current HEAD preserves the complete signed customer-session identity, validates it against the current route and repository target, forwards continuation paths with the required OCI prefix, and consistently rewrites customer upload Locations to an explicitly configured Manager origin.
|
| Filename | Overview |
|---|---|
| crates/alien-manager/src/routes/registry_proxy.rs | Adds the customer OCI broker boundary, signed upload sessions, target revalidation, repository translation, and trusted-origin Location rewriting; the previously reported continuation failures are addressed at current HEAD. |
| crates/alien-manager/src/builder.rs | Wires the optional customer broker, disables automatic upstream redirects, and requires an explicit origin-only Manager base URL. |
| crates/alien-manager/src/routes/bindings.rs | Adds local Artifact Registry resolution using separate access and management identities without exposing provider credentials over HTTP. |
| crates/alien-permissions/permission-sets/artifact-registry/remote-read-write.jsonc | Defines the remote Artifact Registry data-plane permissions consumed by generated infrastructure and credential materialization. |
| crates/alien-cloudformation/src/emitters/aws/artifact_registry.rs | Attaches repository-scoped remote access permissions to the Remote Bindings role while removing lifecycle actions from push credentials. |
| crates/alien-terraform/src/emitters/gcp/artifact_registry.rs | Extends generated GCP Artifact Registry infrastructure with remote-access qualification and repository-scoped bindings. |
Sequence Diagram
sequenceDiagram
participant Client as OCI Client
participant Manager as Alien Manager
participant Broker as Customer Registry Broker
participant Registry as Cloud Registry
Client->>Manager: Push customer/repository
Manager->>Broker: Authorize repository and operation
Broker-->>Manager: Pinned target and provider binding
Manager->>Registry: Forward with short-lived provider credentials
Registry-->>Manager: Upload Location
Manager-->>Client: Manager-origin signed continuation
Client->>Manager: PATCH/PUT signed continuation
Manager->>Broker: Resolve and revalidate signed session
Broker-->>Manager: Current pinned target
Manager->>Registry: Forward exact provider session path
Registry-->>Client: Proxied response
Reviews (22): Last reviewed commit: "test: qualify customer registries on AWS..." | Re-trigger Greptile
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 7, 2026 01:34
1a54ccf to
03aaae5
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 7, 2026 02:15
df56eaf to
1242369
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 7, 2026 02:15
ea4e0ac to
5cf8aea
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 03:40
1242369 to
c5aba42
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 14, 2026 03:40
52ddffb to
f381b98
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 03:50
c5aba42 to
5bfa04f
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
2 times, most recently
from
August 14, 2026 12:41
7958e5a to
9a1e7ba
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 12:53
e843f1f to
9936de8
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
2 times, most recently
from
August 14, 2026 12:59
0bd0a87 to
c574fc6
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 13:12
32b2cfd to
d2f001a
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 14, 2026 13:12
c574fc6 to
a19c5ed
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 14:02
d2f001a to
dfce3e9
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 14, 2026 14:02
a19c5ed to
fb24fdf
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 14:18
dfce3e9 to
7311c1d
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 14, 2026 14:18
fb24fdf to
80ca65d
Compare
alongubkin
force-pushed
the
alon/alien-476-build-byo-onboarding-and-management-experience
branch
from
August 14, 2026 14:53
7311c1d to
5f7b4ac
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 14, 2026 14:53
80ca65d to
b7b34ce
Compare
alongubkin
force-pushed
the
alon/alien-459-build-byo-container-registry-stacked
branch
from
August 14, 2026 15:33
b7b34ce to
e473aac
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #357.
Adds the least-privilege remote Artifact Registry permission set, Remote Bindings qualification, and the Manager-local OCI broker boundary. Provider credentials remain local and repository lifecycle stays with the management identity.
Validated locally: