-
Notifications
You must be signed in to change notification settings - Fork 63
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
RHCLOUD-36142: Update Clowder to reconcile on changes to non-app secrets/configmaps #1124
base: master
Are you sure you want to change the base?
Conversation
8c4a386
to
73cb5be
Compare
test-config-secret-restarter is broken working on fix now. |
This is looking great so far, awesome work on this @nesty156 ! |
e58d0e0
to
bbb031d
Compare
9b8dcdd
to
64be3dd
Compare
ae123ab
to
8f89c78
Compare
…gmaps * Add hashCache to the ClowdEnvironmentReconciliation * Remove all Secrets/ConfigMaps from Environment at start of reconciliation cycle * Embue HashObject with the always flag * Change updateHashCacheForConfigMapAndSecret to try to read in the sec/config and return true if always set * Make managed_kafka add the secret to the cache at the start of reconciliation * Update HashCache tests * Add kuttl tests for managed kafka secrets cc: @psav
…AddClowdObjectToObject
8f89c78
to
add22ee
Compare
e2e test fail: FAIL: kuttl/harness/test-clowdapp-watcher-kafka-msk (821.82s) |
cc: @psav