Merge remote-tracking branch

'origin/issues/93_validation_startup_log_levels' into develop

Author: hhund

codex-process-data-transfer/src/main/java/de/netzwerk_universitaetsmedizin/codex/processes/data_transfer/validation/ValidationPackageManagerImpl.java

@@ -232,16 +232,18 @@ private void expandExternal(List<ValueSet> expandedValueSets, ValueSet v)
 		}
 		catch (WebApplicationException e)
 		{
-			logger.error("Error while expanding ValueSet {}|{}: {} - {}", v.getUrl(), v.getVersion(),
-					e.getClass().getName(), e.getMessage());
+			logger.warn(
+					"Error while expanding ValueSet {}|{} externally, this may result in incomplete validation: {} - {}",
+					v.getUrl(), v.getVersion(), e.getClass().getName(), e.getMessage());
 			getOutcome(e).ifPresent(m -> logger.debug("Expansion error response: {}", m));
 			logger.debug("ValueSet with error while expanding: {}",
 					fhirContext.newJsonParser().encodeResourceToString(v));
 		}
 		catch (Exception e)
 		{
-			logger.error("Error while expanding ValueSet {}|{}: {} - {}", v.getUrl(), v.getVersion(),
-					e.getClass().getName(), e.getMessage());
+			logger.warn(
+					"Error while expanding ValueSet {}|{} externally, this may result in incomplete validation: {} - {}",
+					v.getUrl(), v.getVersion(), e.getClass().getName(), e.getMessage());
 			logger.debug("ValueSet with error while expanding: {}",
 					fhirContext.newJsonParser().encodeResourceToString(v));
 		}
@@ -254,15 +256,15 @@ private void expandInternal(List<ValueSet> expandedValueSets, ValueSetExpander e
 			ValueSetExpansionOutcome expansion = expander.expand(v);
 
 			if (expansion.getError() != null)
-				logger.warn("Error while expanding ValueSet {}|{}: {}", v.getUrl(), v.getVersion(),
+				logger.warn("Error while expanding ValueSet {}|{} internally: {}", v.getUrl(), v.getVersion(),
 						expansion.getError());
 			else
 				expandedValueSets.add(expansion.getValueset());
 		}
 		catch (Exception e)
 		{
 			logger.info(
-					"Error while expanding ValueSet {}|{}: {} - {}, trying to expand via external terminology server next",
+					"Error while expanding ValueSet {}|{} internally: {} - {}, trying to expand via external terminology server next",
 					v.getUrl(), v.getVersion(), e.getClass().getName(), e.getMessage());
 
 			expandExternal(expandedValueSets, v);

codex-process-data-transfer/src/main/java/de/netzwerk_universitaetsmedizin/codex/processes/data_transfer/validation/ValidationPackageWithDepedencies.java

@@ -255,7 +255,7 @@ private void logMissingValueSets(Set<String> neededValueSets, List<ValueSet> fou
 		if (!missingValueSets.isEmpty())
 		{
 			logger.warn(
-					"The following ValueSet are required for validation but could not be found in validation package {}|{} or its dependencies: {}",
+					"The following ValueSet are required for validation but could not be found in validation package {}|{} or its dependencies, this may result in incomplete valdidation: {}",
 					getName(), getVersion(), missingValueSets);
 		}
 	}

codex-process-data-transfer/src/main/java/de/netzwerk_universitaetsmedizin/codex/processes/data_transfer/validation/structure_definition/ClosedTypeSlicingRemover.java

@@ -27,7 +27,7 @@ public StructureDefinition modify(StructureDefinition sd)
 				ElementDefinitionSlicingDiscriminatorComponent discriminator = slicing.getDiscriminator().get(0);
 				if (DiscriminatorType.TYPE.equals(discriminator.getType()) && "$this".equals(discriminator.getPath()))
 				{
-					logger.warn(
+					logger.info(
 							"Removing Type slicing with slicing.rules != closed from validation rule with id {} in StructureDefinition {}|{}",
 							e.getId(), sd.getUrl(), sd.getVersion());
 

codex-process-data-transfer/src/main/java/de/netzwerk_universitaetsmedizin/codex/processes/data_transfer/validation/structure_definition/IdentifierRemover.java

@@ -35,7 +35,7 @@ public StructureDefinition modify(StructureDefinition sd)
 
 			if (filteredRules.size() < sd.getDifferential().getElement().size())
 			{
-				logger.warn("Removing validation rules with ids {} from StructureDefinition {}|{}",
+				logger.info("Removing validation rules with ids {} from StructureDefinition {}|{}",
 						sd.getDifferential().getElement().stream().filter(toRemove).map(ElementDefinition::getId)
 								.collect(Collectors.joining(", ", "[", "]")),
 						sd.getUrl(), sd.getVersion());

codex-process-data-transfer/src/main/java/de/netzwerk_universitaetsmedizin/codex/processes/data_transfer/validation/structure_definition/SliceMinFixer.java

@@ -27,7 +27,7 @@ public StructureDefinition modify(StructureDefinition sd)
 						.anyMatch(e1 -> Objects.equals(e.getPath(), e1.getPath()) && e1.hasSlicing()))
 				.forEach(e ->
 				{
-					logger.warn("Adding min=0 to rule with id {} in StructureDefinition {}|{}", e.getId(), sd.getUrl(),
+					logger.info("Adding min=0 to rule with id {} in StructureDefinition {}|{}", e.getId(), sd.getUrl(),
 							sd.getVersion(), sd.getBaseDefinition(), sd.getDifferential().getElement().stream()
 									.anyMatch(e1 -> Objects.equals(e.getPath(), e1.getPath()) && e1.hasSlicing()));
 					e.setMin(0);

codex-process-data-transfer/src/main/java/de/netzwerk_universitaetsmedizin/codex/processes/data_transfer/validation/value_set/MissingEntriesIncluder.java

@@ -38,7 +38,7 @@ public ValueSet modifyPostExpansion(ValueSet vsWithComposition, ValueSet vsWithE
 							if (!expandedEntries.contains(toEntry(system, concept.getCode()))
 									&& !expandedEntries.contains(toEntry(system, concept.getCode())))
 							{
-								logger.warn(
+								logger.info(
 										"Adding missing concept to ValueSet {}|{}: system: '{}', version: '{}', code: '{}', display: '{}'",
 										vsWithExpansion.getUrl(), vsWithExpansion.getVersion(), system, version,
 										concept.getCode(), concept.getDisplay());