Skip to content

Commit

Permalink
chore: cleanup, logs
Browse files Browse the repository at this point in the history
  • Loading branch information
mjnagel committed Aug 2, 2024
1 parent 4300e7d commit 77810ae
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 19 deletions.
18 changes: 0 additions & 18 deletions src/pepr/operator/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -38,24 +38,6 @@ When(a.Service)
.WithName("kubernetes")
.Reconcile(updateAPIServerCIDRFromService);

// We can't watch for deletions here because of our finalizer - deletions will appear as "updates",
// with a deletionTimestamp so they end up in the normal reconcile flow

// // Watch for changes to the UDSPackage CRD and cleanup the namespace mutations
// When(UDSPackage)
// .IsDeleted()
// .Watch(async pkg => {
// // Cleanup the namespace
// await cleanupNamespace(pkg);

// // Remove any SSO clients
// await purgeSSOClients(pkg, []);
// await purgeAuthserviceClients(pkg, []);

// // Remove Finalizer
// await handleFinalizer(pkg, FinalizerOperation.Remove);
// });

// Watch for changes to the UDSPackage CRD to enqueue a package for processing
When(UDSPackage)
.IsCreatedOrUpdated()
Expand Down
2 changes: 1 addition & 1 deletion src/pepr/operator/reconcilers/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ export async function updateStatus(cr: UDSPackage, status: PkgStatus) {
* @param status The new status
*/
export async function handleFinalizer(cr: UDSPackage, operation: FinalizerOperation) {
log.debug(`${operation}ing finalizer for ${cr.metadata?.name}/${cr.metadata?.namespace}`);
log.debug(`Updating finalizer for ${cr.metadata?.name}/${cr.metadata?.namespace}, operation: ${operation}`);

let patch: Operation;
if (operation === FinalizerOperation.Add) {
Expand Down

0 comments on commit 77810ae

Please sign in to comment.