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 1 commit 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.
applications/wg-easy/charts/cert-manager/templates/_supportbundle.tpl
Outdated
Show resolved
Hide resolved
- Bump Embedded Cluster from 2.1.3 to 2.13.4+k8s-1.33 - Configure EC custom domains (proxyRegistryDomain, replicatedAppDomain) - Update cert-manager chart: 1.14.5 -> v1.19.3 - Update traefik chart: 28.0.0 -> 39.0.2 - Update replicated SDK: 1.7.0 -> 1.16.0 - Fix traefik values: replace deprecated redirectTo with http.redirections.entryPoint for 39.x compatibility - Fix support bundle collector: namespace -> namespaces - Add clusterResources collector to wg-easy support bundle - Add JSON schema reference to EC cluster config
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 |



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