Skip to content

Conversation

@DomAyre
Copy link
Contributor

@DomAyre DomAyre commented Dec 7, 2025

Why

Allows the user to generate the fragment reference rego for building policies and insert that in the final policy.

How

  • Add a command to generate the fragment reference
  • Add the --with-fragments flag to acipolicygen
  • Add testing

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Dec 7, 2025

⚠️Azure CLI Extensions Breaking Change Test
⚠️confcom
rule cmd_name rule_message suggest_message
⚠️ 1015 - SubgroupPropUpdate confcom sub group confcom updated property sub_groups from {} to {'confcom fragment': {'name': 'confcom fragment', 'commands': {'confcom fragment attach': {'name': 'confcom fragment attach', 'is_aaz': False, 'is_preview': True, 'parameters': [{'name': 'signed_fragment', 'options': [], 'type': 'custom_type', 'required': True, 'nargs': '?', 'default': "<_io.BufferedReader name='<stdin>'>"}, {'name': 'manifest_tag', 'options': ['--manifest-tag'], 'required': True}]}, 'confcom fragment push': {'name': 'confcom fragment push', 'is_aaz': False, 'is_preview': True, 'parameters': [{'name': 'signed_fragment', 'options': [], 'type': 'custom_type', 'required': True, 'nargs': '?', 'default': "<_io.BufferedReader name='<stdin>'>"}, {'name': 'manifest_tag', 'options': ['--manifest-tag'], 'required': True}]}}, 'sub_groups': {'confcom fragment references': {'name': 'confcom fragment references', 'commands': {'confcom fragment references from_image': {'name': 'confcom fragment references from_image', 'is_aaz': False, 'parameters': [{'name': 'image', 'options': [], 'type': 'string', 'required': True}, {'name': 'minimum_svn', 'options': ['--minimum-svn'], 'type': 'string'}]}}, 'sub_groups': {}}}}, 'confcom containers': {'name': 'confcom containers', 'commands': {'confcom containers from_image': {'name': 'confcom containers from_image', 'is_aaz': False, 'parameters': [{'name': 'image', 'options': [], 'type': 'string', 'required': True}, {'name': 'platform', 'options': ['--platform'], 'type': 'string', 'default': 'aci'}]}}, 'sub_groups': {}}}
⚠️ 1006 - ParaAdd confcom acifragmentgen cmd confcom acifragmentgen added parameter out_signed_fragment
⚠️ 1006 - ParaAdd confcom acipolicygen cmd confcom acipolicygen added parameter fragment_definitions

@yonzhan
Copy link
Collaborator

yonzhan commented Dec 7, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@azure-client-tools-bot-prd
Copy link

Hi @DomAyre,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

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).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@DomAyre DomAyre force-pushed the fragment-reference-from-image branch from 3612a9b to 19e9e16 Compare December 8, 2025 10:17
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

@DomAyre DomAyre force-pushed the fragment-reference-from-image branch from 5884f6c to c31c420 Compare December 8, 2025 15:11
@DomAyre DomAyre force-pushed the fragment-reference-from-image branch from c31c420 to 5640958 Compare December 8, 2025 15:58
@DomAyre DomAyre force-pushed the fragment-reference-from-image branch from 5640958 to 819a575 Compare December 8, 2025 17:24
@DomAyre DomAyre force-pushed the fragment-reference-from-image branch from 819a575 to 403c16f Compare December 8, 2025 17:51
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