Skip to content

Conversation

@saeedfazal
Copy link
Contributor

No description provided.

@saeedfazal saeedfazal added exclude from release notes Use this label to exclude a PR from the release notes team/pesto labels Nov 14, 2022
@saeedfazal saeedfazal self-assigned this Nov 14, 2022
@saeedfazal saeedfazal requested a review from foot November 14, 2022 12:37
helm install my-mccp wkpv3/mccp --version "${CHART_VERSION}" --namespace flux-system ${helmArgs[@]}

# Wait for cluster to settle
kubectl wait --for=condition=Ready --timeout=300s -n flux-system --all pod
Copy link
Collaborator

@foot foot Nov 14, 2022

Choose a reason for hiding this comment

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

Other thing that comes to mind is to helm install --wait above which will wait for pods / services to be available,

--wait if set, will wait until all Pods, PVCs, Services, and minimum number of Pods of a Deployment, StatefulSet, or ReplicaSet are in a ready state before marking the release as successful. It will wait for as long as --timeout

https://helm.sh/docs/helm/helm_install/

Which I would assume would be in the main chart at least, maybe the subcharts too (like policy-agent), not sure exactly.

- Create a new clean kind cluster.

`kind create cluster --config test/utils/data/local-kind-config.yaml`
`./test/utils/scripts/mgmt-cluster-setup.sh kind $(pwd) mgmt-kind`
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@saeedfazal saeedfazal merged commit 6230851 into main Nov 14, 2022
@saeedfazal saeedfazal deleted the ingress-nginx-failure branch November 14, 2022 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exclude from release notes Use this label to exclude a PR from the release notes team/pesto

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants