-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[confcom] Fix opa installation on windows and in strict networking environments #9440
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
Conversation
️✔️Azure CLI Extensions Breaking Change Test
|
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR. Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
|
Hi @DomAyre
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes OPA (Open Policy Agent) installation issues on Windows and in strict networking environments by switching from openpolicyagent.org to GitHub releases and correcting the Windows binary URL to include the .exe extension.
Key Changes:
- Updated OPA download source from
openpolicyagent.orgtogithub.comfor better network compatibility - Fixed Windows OPA binary URL by adding
.exeextension - Refactored code to use platform-specific dictionaries for URLs and SHA256 hashes
- Bumped extension version from 1.4.2 to 1.4.3
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/confcom/setup.py | Version bump to 1.4.3 |
| src/confcom/azext_confcom/lib/opa.py | Refactored to use GitHub releases with platform-specific URL and SHA256 mappings |
| src/confcom/HISTORY.rst | Added changelog entry for version 1.4.3 |
|
Supporting MacOS and ARM would require changes beyond this and the extension generally doesn't support it for now. Doing so would be a wider effort so I think it's outside the scope for this PR |
|
[Release] Update index.json for extension [ confcom ] : https://dev.azure.com/msazure/One/_build/results?buildId=144408448&view=results |

Why
Attempts to build confcom's release package failed for two reasons:
openpolicyagent.org.exeon the endHow
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired)python scripts/ci/test_index.py -qlocally? (pip install wheel==0.30.0required)