Skip to content
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

NVSHAS-9357 Adding new env variable for CTRL_SEARCH_REGISTRIES #425

Closed
wants to merge 3 commits into from
Closed

NVSHAS-9357 Adding new env variable for CTRL_SEARCH_REGISTRIES #425

wants to merge 3 commits into from

Conversation

venkateshjayagopal
Copy link
Contributor

@venkateshjayagopal venkateshjayagopal commented Aug 23, 2024

Adding new env variable for controller, CTRL_SEARCH_REGISTRIES

Tested with single and multiple URL. Note: When using multiple URLs we need to use backslash "\" to escape ",".

Example,

root@Host-01-RKE2:/home/neuvector/helm_master/neuvector-helm/charts/core# helm install neuvector -n neuvector --set tag=jenkins-nv-build-fleet-5296,controller.image.repository=nvpublic/co,enforcer.image.repository=nvpublic/en,manager.image.repository=nvpublic/ma,controller.apisvc.type=NodePort --set manager.image.hash=sha256:4e2b70e991e8fad42009378c2fac7a2415b8c164efbe75d73f1addef66d379d8 --set controller.federation.managedsvc.type=NodePort,controller.apisvc.type=NodePort --set 'controller.certupgrader.env[0].name=EXPIRY_CERT_THRESHOLD,controller.certupgrader.env[0].value=3816h' --set "controller.certupgrader.schedule=10 * * * *,internal.autoRotateCert=true" --set "controller.search_registries=https://recovery.quay.io/\,https://aws.repository.com/" ./
NAME: neuvector
LAST DEPLOYED: Fri Aug 23 22:18:57 2024
NAMESPACE: neuvector
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
Get the NeuVector URL by running these commands:
  NODE_PORT=$(kubectl get --namespace neuvector -o jsonpath="{.spec.ports[0].nodePort}" services neuvector-service-webui)
  NODE_IP=$(kubectl get nodes --namespace neuvector -o jsonpath="{.items[0].status.addresses[0].address}")
  echo https://$NODE_IP:$NODE_PORT

controller YAML after executing above HELM changes,

      containers:
      - env:
        - name: CLUSTER_JOIN_ADDR
          value: neuvector-svc-controller.neuvector
        - name: CLUSTER_ADVERTISED_ADDR
          valueFrom:
            fieldRef:
              apiVersion: v1
              fieldPath: status.podIP
        - name: CLUSTER_BIND_ADDR
          valueFrom:
            fieldRef:
              apiVersion: v1
              fieldPath: status.podIP
        - name: CTRL_SEARCH_REGISTRIES
          value: https://recovery.quay.io/,https://aws.repository.com/

@venkateshjayagopal venkateshjayagopal changed the base branch from master to pre-5.4 August 27, 2024 18:18
@venkateshjayagopal
Copy link
Contributor Author

Will close this PR, there is a conflict when trying to merge in pre5.4 branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants