Skip to content

Edit docs on using the proxy registry #3366

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

Merged
merged 8 commits into from
Jul 15, 2025

Conversation

paigecalvert
Copy link
Contributor

@paigecalvert paigecalvert commented Jul 8, 2025

Previews:

Copy link

netlify bot commented Jul 8, 2025

Deploy Preview for replicated-docs ready!

Name Link
🔨 Latest commit d710e96
🔍 Latest deploy log https://app.netlify.com/projects/replicated-docs/deploys/687667a1d5530b0008be6db4
😎 Deploy Preview https://deploy-preview-3366--replicated-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jul 8, 2025

Deploy Preview for replicated-docs-upgrade ready!

Name Link
🔨 Latest commit d710e96
🔍 Latest deploy log https://app.netlify.com/projects/replicated-docs-upgrade/deploys/687667a15d618e00087791fa
😎 Deploy Preview https://deploy-preview-3366--replicated-docs-upgrade.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@replicated-ci replicated-ci added type::docs Improvements or additions to documentation type::feature labels Jul 8, 2025
@paigecalvert paigecalvert marked this pull request as ready for review July 9, 2025 19:30
@paigecalvert paigecalvert requested a review from a team as a code owner July 9, 2025 19:30
@@ -1,64 +1,32 @@
import StepCreds from "../partials/proxy-service/_step-creds.mdx"
import StepCustomDomain from "../partials/proxy-service/_step-custom-domain.mdx"
import RewriteHelmValues from "../partials/proxy-service/_step-rewrite-helm-values.mdx"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

pulls in the new partial that has the step on how to modify image refs in your helm values


## Overview

With the Replicated proxy registry, each customer's unique license can grant proxy access to images in an external private registry.
During Helm installations with Replicated, after customers provide their unique license ID, a `global.replicated.dockerconfigjson` field that contains a base64 encoded Docker configuration file is automatically injected in the Helm chart values.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

minor rewording in overview section mainly to try to remove extra info/words


1. <StepCreds/>

1. <StepCustomDomain/>

1. In your Helm chart values file, set your image repository URL to the location of the image on the proxy registry. If you added a custom domain, use your custom domain. Otherwise, use `proxy.replicated.com`.
1. <RewriteHelmValues/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

^ replaced with the partial


# Use the Proxy Registry with KOTS Installations
# Use the Proxy Registry with Replicated Installers
Copy link
Contributor Author

Choose a reason for hiding this comment

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

with KOTS Installations --> with Replicated Installers

Open to other suggestions on wording. Goal is to avoid using "KOTS" to actually mean KOTS/EC/kURL

Copy link
Member

Choose a reason for hiding this comment

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

sounds good


## Overview
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Did some deleting/shuffling so this no longer needs such a big overview section

Instead, jumps right into the steps on how to configure your app to use the proxy, with a couple other subheadings for the non-HelmChart v2 apps


<details>

<summary>How does KOTS patch image names?</summary>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

^ adds all this info to a details element

## Enable the Proxy Registry

This section describes how to enable the proxy registry for applications deployed with KOTS, including how to ensure that image names are rewritten and that the required image pull secret is provided.
### Kubernetes Operators
Copy link
Contributor Author

@paigecalvert paigecalvert Jul 10, 2025

Choose a reason for hiding this comment

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

Left in the k8s operator stuff. Not sure if anyone's still using them/if this could be removed yet

If it can be removed, that would of course make it easier to further simplify this page

Copy link
Member

Choose a reason for hiding this comment

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

people do use operators, maybe even increasingly so

Copy link
Member

Choose a reason for hiding this comment

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

but often people deploy their operator via Helm. if that makes sense.

basically they deploy the operator via helm, but the operator can deploy other things, which is why the extra steps are needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Interesting, yeah that makes sense.


### Image Pull Secret
If you are installing with the HelmChart v1 custom resource, or if your application is not packaged with Helm, there are different steps for configuring your application to use the proxy registry. See [Configure Other Application Types](#other) below.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

^ I landed on a main "Configure Your App" section that assumes Helm app and HelmChart v2. And then links people that use other packaging/install methods down to the "other" options

My thought is that it highlights the main path that we expect most people to follow while minimizing noise of other possibilities

Open to suggestions on this

Copy link
Member

Choose a reason for hiding this comment

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

i think that makes sense. sometimes we try to cater to everyone and it gets confusing. maybe better to keep the main thing the main thing and put other stuff separately.

@paigecalvert paigecalvert requested a review from ajp-io July 10, 2025 00:17
Copy link
Member

@ajp-io ajp-io left a comment

Choose a reason for hiding this comment

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

still need to review the replicated installers page, but might be headed to the hopsital this morning, so i'll submit this for now.

@@ -1,64 +1,32 @@
import StepCreds from "../partials/proxy-service/_step-creds.mdx"
import StepCustomDomain from "../partials/proxy-service/_step-custom-domain.mdx"
import RewriteHelmValues from "../partials/proxy-service/_step-rewrite-helm-values.mdx"

# Use the Proxy Registry with Helm Installations
Copy link
Member

Choose a reason for hiding this comment

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

one thing that could clarify the docs a bit is if we said something like "with Helm CLI installations." i know helm is all over the place now, and people are using helm for all installation methods, so maybe that would clear up the fact that this is when you're using helm install directly. just a thought.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that sounds good to me. For awhile I wasn't sure if "Helm CLI" was overly explanatory compared to just "Helm", but couldn't hurt if you think it would be a little clearer

Copy link
Member

Choose a reason for hiding this comment

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

i'm up for a discussion. i just felt that Helm is all over the place now, and maybe this would be clearer (since every install method uses helm at this point)

Copy link
Member

@ajp-io ajp-io left a comment

Choose a reason for hiding this comment

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

left some comments but lgtm

@paigecalvert paigecalvert merged commit 87eb215 into main Jul 15, 2025
5 checks passed
@paigecalvert paigecalvert deleted the modify-image-refs-in-values-file branch July 15, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type::docs Improvements or additions to documentation type::feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants