-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add fragment reference from_image command #9459
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
base: main
Are you sure you want to change the base?
Conversation
…standalone fragments
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| 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': {}}} |
||
| confcom acifragmentgen | cmd confcom acifragmentgen added parameter out_signed_fragment |
||
| confcom acipolicygen | cmd confcom acipolicygen added parameter fragment_definitions |
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
Hi @DomAyre, |
|
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>
|
3612a9b to
19e9e16
Compare
|
5884f6c to
c31c420
Compare
c31c420 to
5640958
Compare
5640958 to
819a575
Compare
819a575 to
403c16f
Compare
Why
Allows the user to generate the fragment reference rego for building policies and insert that in the final policy.
How
--with-fragmentsflag to acipolicygenThis 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)