Skip to content

Commit

Permalink
verify_manifest_consistency now returns warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
Anders M committed Jun 18, 2024
1 parent 6acf2bd commit a8cabf7
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions image_builder/image_builder/verify.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,10 +41,11 @@ def verify_manifest_consistency(submitted_yaml: str, workspace: Path, repo_dir:
logger.info(f'Submitted job manifest:\n{submitted_dict}')
logger.info(f'Manifest file in Job repository:\n{repo_dict}')
difference = _differentiate_dicts(repo_dict, submitted_dict)
raise RuntimeError('Submitted job manifest is not consistent with the file found in a repository. '
'Did you forget to do "git push"? '
f'Difference: {difference}')

warning = ('Submitted job manifest is not consistent with the file found in a repository. '
'Did you forget to do "git push"? '
f'Difference: {difference}')
logger.warning(warning)
return warning

def _find_workspace_manifest_file(workspace: Path, repo_dir: Path) -> Optional[Path]:
paths_to_check = [
Expand Down

0 comments on commit a8cabf7

Please sign in to comment.