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

[MANIFEST] Set up a workflow to test clearing cache on the notify website post a deploy #2920

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

jzbahrai
Copy link
Collaborator

@jzbahrai jzbahrai commented Sep 5, 2024

What happens when your PR merges?

Set up a workflow to clear the cache after the kube deployment is complete.
I haven't added the workflow for production yet- but once this is merged and tested on staging, I will.

Checklist if releasing new version:

Checklist if making changes to Kubernetes:

  • I know how to get kubectl credentials in case it catches on fire

After merging this PR

  • I have verified that the tests / deployment actions succeeded
  • I have verified that any affected pods were restarted successfully
  • I have verified that I can still log into Notify production
  • I have verified that the smoke tests still pass on production
  • I have communicated the release in the #notify Slack channel.

@jzbahrai jzbahrai requested a review from jimleroyer as a code owner September 5, 2024 19:48
Copy link

github-actions bot commented Sep 5, 2024

ingress	nginx    	2       	2024-02-12 19:08:42.93215444 +0000 UTC	deployed	nginx-ingress-1.1.2	3.4.2      

xray-daemon	xray     	1       	2024-07-29 19:45:48.684608347 +0000 UTC	deployed	aws-xray-4.0.8	3.3.12     

Comparing release=karpenter-crd, chart=/tmp/helmfile929134908/karpenter/staging/karpenter-crd/karpenter-crd/0.36.1/karpenter-crd
Comparing release=karpenter, chart=/tmp/helmfile929134908/karpenter/staging/karpenter/karpenter/0.36.1/karpenter
Comparing release=karpenter-nodepool, chart=charts/karpenter-nodepool
Comparing release=priority-classes, chart=deliveryhero/priority-class
Comparing release=secrets-store-csi-driver, chart=secrets-store-csi-driver/secrets-store-csi-driver
Comparing release=aws-secrets-provider, chart=aws-secrets-manager/secrets-store-csi-driver-provider-aws
Comparing release=kube-state-metrics, chart=prometheus-community/kube-state-metrics
Comparing release=blazer, chart=stakater/application
Comparing release=ingress, chart=charts/nginx-ingress
Comparing release=xray-daemon, chart=okgolove/aws-xray

@jzbahrai jzbahrai force-pushed the task/test-notifcation-call branch from afffb7c to ae7164d Compare September 6, 2024 17:52
@jzbahrai jzbahrai force-pushed the task/test-notifcation-call branch from ae7164d to d6faded Compare September 6, 2024 17:54
@jzbahrai jzbahrai requested review from jimleroyer, whabanks and sastels and removed request for jimleroyer September 6, 2024 18:18
Copy link
Contributor

@whabanks whabanks left a comment

Choose a reason for hiding this comment

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

LGTM!

@jzbahrai jzbahrai merged commit 9cf40d7 into main Sep 9, 2024
4 checks passed
@jzbahrai jzbahrai deleted the task/test-notifcation-call branch September 9, 2024 13:31
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