From da82aa58db57de22dad4b8ad909717b7a4393c8b Mon Sep 17 00:00:00 2001
From: Angel Montenegro <a.montenegro@orcid.org>
Date: Thu, 16 Jan 2025 13:30:17 -0600
Subject: [PATCH] More logs (#7185)

---
 .../java/org/orcid/scheduler/loader/io/OrgDataClient.java     | 1 +
 .../scheduler/loader/manager/impl/OrgLoadManagerImpl.java     | 4 ++++
 2 files changed, 5 insertions(+)

diff --git a/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/io/OrgDataClient.java b/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/io/OrgDataClient.java
index 8b1a791a14b..e3215c40d19 100644
--- a/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/io/OrgDataClient.java
+++ b/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/io/OrgDataClient.java
@@ -47,6 +47,7 @@ public <T> T get(String url, String userAgent, Class<T> type) {
      * @return boolean indicator of success
      */
     public boolean downloadFile(String url, String userAgent, String localFilePath) {
+        LOGGER.info("About to download file {}", url);
         JerseyClientResponse<InputStream, String> response = jerseyClientHelperForOrgLoaders.executeGetRequest(url, null, null, false, Map.of(), Map.of("User-Agent", userAgent), InputStream.class, String.class);
         int status = response.getStatus();
         if (status != 200) {
diff --git a/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/manager/impl/OrgLoadManagerImpl.java b/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/manager/impl/OrgLoadManagerImpl.java
index 773f52336b3..23d22029f34 100644
--- a/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/manager/impl/OrgLoadManagerImpl.java
+++ b/orcid-scheduler-web/src/main/java/org/orcid/scheduler/loader/manager/impl/OrgLoadManagerImpl.java
@@ -38,6 +38,7 @@ public class OrgLoadManagerImpl implements OrgLoadManager {
 
     @Override
     public void loadOrgs() {
+        LOGGER.info("About to start the orgs loader process");
         OrgLoadSource loader = getNextOrgLoader();
         if (loader != null) {
             loadOrg(loader);
@@ -46,6 +47,7 @@ public void loadOrgs() {
     
     @Override
     public void loadOrg(OrgLoadSource loader) {
+        LOGGER.info("About to start loading orgs from source {}", loader.getSourceName());
         if (loader != null) {
             OrgImportLogEntity importLog = getOrgImportLogEntity(loader);
             boolean success = loader.downloadOrgData();
@@ -55,8 +57,10 @@ public void loadOrg(OrgLoadSource loader) {
             logImport(importLog, success);
 
             if (success) {
+                LOGGER.info("Orgs successfully imported from {}", loader.getSourceName());
                 slackManager.sendAlert(String.format("Orgs successfully imported from %s", loader.getSourceName()), slackChannel, slackUser);
             } else {
+                LOGGER.warn("Org import FAILURE from {}", loader.getSourceName());
                 slackManager.sendAlert(String.format("Org import FAILURE from %s", loader.getSourceName()), slackChannel, slackUser);
             }
         } else {