Skip to content

Fix omnia_sh_tag_url to use _omnia_branch instead of _core_tag in URL path - #624

Merged
abhishek-sa1 merged 1 commit into
dell:automation-v2.2.0.0from
saifudheene:fix/omnia-sh-tag-url-variable
Aug 19, 2026
Merged

abhishek-sa1 merged 1 commit into
dell:automation-v2.2.0.0from
saifudheene:fix/omnia-sh-tag-url-variable

Conversation

@saifudheene

Copy link
Copy Markdown
Collaborator

Bug Fix

File: automation_library/omnia_sh/vars/omnia_sh_vars.py (line 79)

Problem

The omnia_sh_tag_url was using _core_tag for both the URL path segment
and the guard condition. This caused the tag-based download URL to resolve
to the wrong Git ref when _core_tag and _omnia_branch differ.

Change

- "omnia_sh_tag_url": f"{_GIT_RAW_BASE}/refs/tags/{_core_tag}/omnia.sh" if _core_tag else "",
+ "omnia_sh_tag_url": f"{_GIT_RAW_BASE}/refs/tags/{_omnia_branch}/omnia.sh" if _core_tag else "",

… path The omnia_sh_tag_url was using _core_tag for both the URL path segment and the guard condition, causing the tag download URL to resolve to the wrong Git ref. The branch_url already uses _omnia_branch correctly, so the tag_url path should also use _omnia_branch to stay consistent while the _core_tag guard still controls whether the URL is populated. Before: refs/tags/{_core_tag}/omnia.sh After: refs/tags/{_omnia_branch}/omnia.sh Generated with [Devin](https://devin.ai)  Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@abhishek-sa1
abhishek-sa1 merged commit 7bbe218 into dell:automation-v2.2.0.0 Aug 19, 2026
3 checks passed
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.

3 participants