Merge pull request #69 from depot/logging

depot · Sep 25, 2023 · cb9e10f · cb9e10f
2 parents 02c30ea + 08e215c
commit cb9e10f
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/handlers/state.ts b/src/handlers/state.ts
@@ -14,6 +14,8 @@ export async function startStateStream(signal: AbortSignal) {
     try {
       logger.info('Getting current AWS state to report')
       let currentState = await getCurrentState()
+
+      logger.info('Reporting current AWS state')
       await reportCurrentState(currentState)
 
       logger.info('Getting desired state')
@@ -48,6 +50,7 @@ let stateCache: StateCache | null = null
 
 export async function reportCurrentState(currentState: CurrentState) {
   if (stateCache) {
+    logger.info('Computing state diff')
     const diff = compare(stateCache.state, currentState)
 
     // If there is no difference, don't send a request
@@ -70,8 +73,10 @@ export async function reportCurrentState(currentState: CurrentState) {
     }
 
     try {
+      logger.info('Sending state diff to API')
       const res = await client.reportCurrentState(request)
       stateCache = {state: currentState, generation: res.generation}
+      return
     } catch {
       // Ignore an error here and fall down to below
     }
@@ -92,7 +97,9 @@ export async function reportCurrentState(currentState: CurrentState) {
       },
     },
   }
+  logger.info('Sending full state to API')
   const res = await client.reportCurrentState(request)
+  logger.info('Saving state in cache')
   stateCache = {state: currentState, generation: res.generation}
 }