Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Github Workflow] Logging Failure During Add PR link to Related Jira Issue #4649

Merged
merged 2 commits into from
Nov 10, 2024

Conversation

shmuel44
Copy link
Contributor

@shmuel44 shmuel44 commented Nov 10, 2024

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-12138

Description

  File "/home/runner/work/demisto-sdk/demisto-sdk/Utils/github_workflow_scripts/jira_integration_scripts/./link_pr_to_jira_issue.py", line 85, in trigger_generic_webhook
    logger.info(f"Detected Pr: {pr_title=}, {pr_link=}, {pr_body=}")
  File "/home/runner/work/demisto-sdk/demisto-sdk/.venv/lib/python3.9/site-packages/loguru/_logger.py", line 2044, in info
    __self._log("INFO", False, __self._options, __message, args, kwargs)
  File "/home/runner/work/demisto-sdk/demisto-sdk/.venv/lib/python3.9/site-packages/loguru/_logger.py", line 2017, in _log
    colored_message = Colorizer.prepare_simple_message(str(message))
  File "/home/runner/work/demisto-sdk/demisto-sdk/.venv/lib/python3.9/site-packages/loguru/_colorizer.py", line 3[68](https://github.com/demisto/demisto-sdk/actions/runs/11764096196/job/32768859984?pr=4648#step:7:69), in prepare_simple_message
    parser.feed(string)
  File "/home/runner/work/demisto-sdk/demisto-sdk/.venv/lib/python3.9/site-packages/loguru/_colorizer.py", line 252, in feed
    raise ValueError(
ValueError: Tag "<2.0.0,>" does not correspond to any known color directive, make sure you did not misspelled it (or prepend '\' to escape it)

Copy link

Changelog(s) in markdown:

  • Fixed an issue where logging would fail when linking PR to Jira issue in the GitHub workflow step. #4649

@shmuel44 shmuel44 self-assigned this Nov 10, 2024
@shmuel44 shmuel44 requested a review from dorschw November 10, 2024 10:12
@shmuel44 shmuel44 marked this pull request as ready for review November 10, 2024 10:12
@shmuel44 shmuel44 merged commit 04b158c into master Nov 10, 2024
27 of 28 checks passed
@shmuel44 shmuel44 deleted the sk_fix_add_pr_step_logger branch November 10, 2024 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants