Uploaded image for project: 'Grouper'
  1. Grouper
  2. GRP-2811

pspng log bugs

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • 2.5.31
    • None
    • None
    • None

    Description

      I sent this to the grouper-users mailing list back in February.  I see that the small log bugs in pspng haven't been fixed in the latest 2.5.29 release, so I'm posting it here, too:I sent this to the grouper-users mailing list back in February.  I see that the small log bugs in pspng haven't been fixed in the latest 2.5.29 release, so I'm posting it here, too:fix-pspng-log-entries.patch --- Provisioner.java.orig 2020-02-07 09:33:27.056252214 -0800+++ Provisioner.java 2020-02-07 09:45:46.139741165 -0800@@ -1319,8 +1319,8 @@     // Save updated list of groups selected for provisioner     selectedGroups.set(selectedGroups_now); -    LOG.info("{}: Change deselected {} groups from provisioner", deselectedGroups.size());-    LOG.info("{}: Change selected {} new groups for provisioner", newlySelectedGroups.size());+    LOG.info("{}: Change deselected {} groups from provisioner", getDisplayName(), deselectedGroups.size());+    LOG.info("{}: Change selected {} new groups for provisioner", getDisplayName(), newlySelectedGroups.size());      for (GrouperGroupInfo group : deselectedGroups) {       TSGroupClass tsGroup=null;@@ -1331,12 +1331,12 @@           continue;         }       }-      LOG.info("{}: Deleting group from target system because it is no longer selected for provisioning: {}", group);+      LOG.info("{}: Deleting group from target system because it is no longer selected for provisioning: {}", getDisplayName(), group);       deleteGroup(group, tsGroup);     }      for (GrouperGroupInfo group : newlySelectedGroups) {-      LOG.info("{}: Scheduling full sync of group because it is now selected for provisioning: {}", group);+      LOG.info("{}: Scheduling full sync of group because it is now selected for provisioning: {}", getDisplayName(), group);        getFullSyncer().scheduleGroupForSync(               FullSyncProvisioner.QUEUE_TYPE.CHANGELOG,@@ -1769,7 +1769,7 @@       if ( shouldGroupBeProvisioned(grouperGroupInfo) )         result.add(grouperGroupInfo);     }-    LOG.info("{}: There are {} groups selected for provisioning (found in %s)",+    LOG.info("{}: There are {} groups selected for provisioning (found in {})",         getDisplayName(), result.size(), PspUtils.formatElapsedTime(start, null));      return result;Collapse 

      Attachments

        Activity

          People

            chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
            chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: