Skip to content

Commit 9700a13

Browse files
Improvements for dependency report (#653)
1 parent 5762b72 commit 9700a13

File tree

1 file changed

+18
-9
lines changed

1 file changed

+18
-9
lines changed

src/package_report.py

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -270,11 +270,16 @@ def _generate_python_package_dependency_report(image_config, base_version_dir, t
270270

271271
results = dict()
272272
for package, version in new_packages.items():
273-
# Pull package metadata from conda-forge and dump into json file
274-
search_result = conda.cli.python_api.run_command("search", f"{package}=={version}", "--json")
275-
package_metadata = json.loads(search_result[0])[package][0]
276-
results[package] = {"version": package_metadata["version"], "depends": package_metadata["depends"]}
277-
273+
try:
274+
# Pull package metadata from conda-forge and dump into json file
275+
search_result = conda.cli.python_api.run_command(
276+
"search", "-c", "conda-forge", f"{package}=={version}", "--json"
277+
)
278+
package_metadata = json.loads(search_result[0])[package][0]
279+
results[package] = {"version": package_metadata["version"], "depends": package_metadata["depends"]}
280+
except Exception as e:
281+
print(f"Error in report generation: {str(e)}")
282+
results[package] = "N/A - see logs"
278283
print(
279284
create_markdown_table(
280285
["Package", "Version in the Target Image", "Dependencies"],
@@ -339,6 +344,11 @@ def generate_package_dependency_report(args):
339344
target_version = get_semver(args.target_patch_version)
340345
target_version_dir = get_dir_for_version(target_version)
341346

347+
# No new packages for patch versions, so skip dependency report.
348+
if target_version.patch != 0:
349+
print("Skipping dependency report for patch release.")
350+
return
351+
342352
base_version = None
343353
source_version_txt_file_path = f"{target_version_dir}/source-version.txt"
344354
if os.path.exists(source_version_txt_file_path):
@@ -352,10 +362,9 @@ def generate_package_dependency_report(args):
352362
print("\n### Target Image Version: " + str(target_version) + " | Base Image Version: " + str(base_version) + "\n")
353363
if not base_version:
354364
print("WARNING: No base version or base version directory found, will generate full report for target version.")
355-
for _, configs in _image_generator_configs.items():
356-
for image_config in configs:
357-
print("## Image Type: " + "(" + image_config["image_type"].upper() + ")")
358-
_generate_python_package_dependency_report(image_config, base_version_dir, target_version_dir)
365+
for image_config in _image_generator_configs[target_version.major]:
366+
print("## Image Type: " + "(" + image_config["image_type"].upper() + ")")
367+
_generate_python_package_dependency_report(image_config, base_version_dir, target_version_dir)
359368

360369

361370
def get_image_size(image_tag, staging_repo_name, staging_account):

0 commit comments

Comments
 (0)