chore(wg-easy): bump EC to 2.13.4 and update chart dependencies#82
chore(wg-easy): bump EC to 2.13.4 and update chart dependencies#82adamancini wants to merge 5 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull Request Overview
Updates the Embedded Cluster configuration to version 2.8.1 with Kubernetes 1.31 and adds domain configuration for proxy registry and replicated app domains.
- Upgrade Embedded Cluster from version 2.1.3+k8s-1.29 to 2.8.1+k8s-1.31
- Add domain configuration with proxy registry and replicated app domains
- Include YAML language server schema reference for better IDE support
|
Changes look OK but in my testing I'm still seeing that my images are not sourced from the registry proxy |
|
7cbae51 to
1b35dac
Compare
|
this needs to be brought up to date, this PR has lagged |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 9 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (1)
applications/wg-easy/charts/wg-easy/templates/_supportbundle.tpl:14
- The
logs.selectorlist is not indented underselector:(the- app.kubernetes.io/name=wg-easyline is at the same indentation asselector:). This makes the rendered SupportBundle YAML invalid. Indent the selector items so they are children ofselector:.
- logs:
namespace: {{ .Release.Namespace }}
selector:
- app.kubernetes.io/name=wg-easy
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2524c19 to
6d08925
Compare
|
@adamancini the replicated pod is in crashloop after EC deploy with a license issue (I have set my VP test customer license to never expire) I also notice it's sitting in its own replicated namespace rather than residing in the kotsadm ns, I presume this is by design?
k -n replicated describe po replicated-5b494897f7-896v5
Environment:
REPLICATED_NAMESPACE: replicated (v1:metadata.namespace)
REPLICATED_POD_NAME: replicated-5b494897f7-896v5 (v1:metadata.name)
IS_HELM_MANAGED: true
HELM_RELEASE_NAME: replicated
HELM_RELEASE_NAMESPACE: replicated
HELM_DRIVER: secret
REPLICATED_SECRET_NAME: replicated
REPLICATED_DEPLOYMENT_NAME: replicated
REPLICATED_CONFIG_FILE: /etc/replicated/config.yamlI'm seeing some warnings on the deploy logs:
and a missing informer:
The deployment name is wg-easy as opposed to public k -n wg-easy get deploy
NAME READY UP-TO-DATE AVAILABLE AGE
wg-easy 1/1 1 1 12m |
|
Looking quickly at the support bundle I notice the cert-manager Pods and logs are not included, is it something to do with: applications/wg-easy/charts/cert-manager/templates/_supportbundle.tpl:9 The logs collector field was changed from namespace to namespaces, but Adam already noted in the review thread that the troubleshoot.sh logs schema defines this field as namespace (singular). This change will cause the logs collector to silently ignore the namespace filter or error out. |
yeah the original design here was to deploy the replicated SDK as a standalone chart, but this may run counter to the SDK design, today. The SDK can be deployed "by itself" in integration mode https://docs.replicated.com/vendor/replicated-sdk-installing#install-the-sdk-in-integration-mode but I think I may move it to a subchart under wg-easy to make it work as expected |
6d08925 to
30014b4
Compare
- Remove standalone charts/replicated/ chart directory - Add replicated SDK 1.16.0 as dependency in wg-easy Chart.yaml - Configure SDK values (image, pullSecrets, createPullSecret) and builder in helmChart-wg-easy.yaml - Remove standalone replicated release and replicatedSDK references from helmfile.yaml.gotmpl (both default and replicated environments) - Add development-config-values.yaml (ConfigValues) for headless installs - Add config-validate task to validate the four-way contract between values.yaml, HelmChart CR, KOTS Config, and ConfigValues - Include config-validate in chart-validate pipeline - Simplify allowedIps default to 0.0.0.0/0 - Update docs: README, chart-structure, replicated-integration, task-reference, task-dependency-graph
30014b4 to
9c84bb8
Compare



Summary
proxyRegistryDomain,replicatedAppDomain)namespace->namespaces)clusterResourcescollector to wg-easy support bundleDependency Updates
EC Domain Configuration
Test plan
cluster.yamlpinsversion: 2.13.4+k8s-1.33proxyRegistryDomainandreplicatedAppDomainadded to cluster specproxy.xdrcft.netconfigured in cluster spec domainsv1.19.3; supportbundle template fixed (namespace→namespaces)39.0.2; values.yaml updated redirect syntax for 39.x compatibility (redirectTo.port→http.redirections.entryPoint)1.16.0from OCI registry