Skip to content

Add aosm extension#36

Open
sunnycarter wants to merge 299 commits intomainfrom
add-aosm-extension
Open

Add aosm extension#36
sunnycarter wants to merge 299 commits intomainfrom
add-aosm-extension

Conversation

@sunnycarter
Copy link
Collaborator

@sunnycarter sunnycarter commented Jun 30, 2023

  • Various fixes to NFD build of deployParameters schema and interactive mode create of deployParameters mappings file. (plus additional logging)
  • Fix CNF NFD publish so that it doesn't render the ACR unuseable for future Artifact publishing.
  • Allow CNF NFD image copy from a source ACR using a namespace.
  • Fix - Add new CGSchema parameters not from the NFD to the required section of the schema.
  • Add the ability to skip bicep publish or artifact upload during publish commands.
  • Fix Manifest name for NSDs so it isn't the same as that for NFDs
  • Add validation of source_registry_id format for CNF configuration

Required for https://dev.azure.com/msazuredev/AzureForOperators/_git/pez-networkservicedesigns/pullrequest/62630

jddarby and others added 30 commits May 16, 2023 11:00
@jddarby
push example cnf bicep
89636e5
@PerfectChaos
Add temporary build workflow for AOSM extension (#4)
715c925 
* Add temporary build workflow for AOSM extension

* Add Releaser to maintain a release with the latest build in a consistent place
added output dir; fixing getchartmappingschema
0f50403
Merge branch 'jl/add-cnf-generation' of github.com:jddarby/azure-cli-…
fbba297 
…extensions into jl/add-cnf-generation
@PerfectChaos
Add empty init files to inner modules so setuptools recognises them a…
f92435c 
…s modules
@PerfectChaos
Merge pull request #5 from jddarby/chaos/fixpackages
0b1266f 
Add empty init files to inner modules so setuptools recognises them
@PerfectChaos
Use latest RG model to remove unnecessary dependency on version
599ac38
@PerfectChaos
Merge pull request #6 from jddarby/chaos/generalrgmodel
5437fac 
Use latest RG model to remove unnecessary dependency on version
@PerfectChaos
Use latest deployment model to remove unnecessary dependency on version
2efdba6
@PerfectChaos
Merge pull request #7 from jddarby/chaos/generaldeploymodel
c80ced8 
Use latest Deployment model to remove unnecessary dependency on version
fixed parameter mappings + copied needed files to output folder
dfd5790
@jddarby
jinja2 and find value paths
4747224
@jddarby
Merge branch 'jl/add-cnf-generation' of github.com:jddarby/azure-cli-…
58d525d 
…extensions into jl/add-cnf-generation
fixed typos + self.aritfacts only has unique artifacts
3188399
@jddarby
use regex instead of string to find deploy params
30921b8
@jddarby
delete accidentaly commited input.json
3501b82
fixed deploy params; added indent to mapping file
7064890
Merge branch 'jl/add-cnf-generation' of github.com:jddarby/azure-cli-…
ae4f8fe 
…extensions into jl/add-cnf-generation
@PerfectChaos
Update readme with install/bug reporting instructions
2604a71
@PerfectChaos
Adjust headers so rendered readme looks nicer
961daad
renamed values.nondef + added cli errors
8274996
@PerfectChaos
Split help command example lines
d4fbc16
@PerfectChaos
Clarify that bug process is internal
671fb64
@PerfectChaos
Merge pull request #9 from jddarby/chaos/installinstructions
b3573e8 
Update readme with install/bug reporting instructions
fixed parameter vals not updating
eb3c1a5
@jddarby
delete unuseed temoplate and update jinja2 templates
1a29b43
@jordlay
Merge pull request #8 from jddarby/jl/add-cnf-generation
13f72f4 
Add CNF NFD generation to build command
@jordlay
Update README.md
6099b7a
@jordlay
Update README.md
3c52481
added docstrings; added basic error catching with file handling
8c94905
Jamieparsons and others added 29 commits October 6, 2023 09:56
@Jamieparsons
Fix unauthorized error bug
5f0e04f
@Jamieparsons
Add spacing
68132c5
@sunnycarter
fix unexpected symbol when parsing lists (#107)
b9dc90a
@jamiedparsons
Merge pull request #106 from jddarby/JDP/unauthorized_error_bug
2e426fb 
Fix unauthorized error bug
@sunnycarter
Validation requirements on helm names (#109)
2779e8e 
* Validation requirements on helm names

* lint

* markups

* lint
@Jamieparsons
Fix error parsing
68f6458
@jamiedparsons
Merge pull request #111 from jddarby/JDP/fix_error_parsing
bf13498 
Fix error parsing
@sunnycarter
Lower case acr names (#112)
848ee2e 
* Lower case ACR names and fix interactive mode for lists

* lint

* remove duplicate else branch
@jddarby
always allow azureDeployLocation to be configurable
c756deb
@jddarby
Add new optional VHD parameters
759727e
@jddarby
Make VNF publish and delete a live test only
c42157b
@jddarby
Validate VHD parameters in build UT
c745388
@Jamieparsons
Markups from the CLI team
ba4c4c3
@jamiedparsons
Merge pull request #117 from jddarby/JDP/cli_team_markups
07273b8 
Markups from the CLI team
@jddarby
mark-ups
f4b278e
@jddarby
remove azureDeployLocation as a configurable vhd parameter
744d415
@jddarby
add explanitory comments
907e27e
@jddarby
Merge pull request #114 from jddarby/jdarby/image-value-mappings-fixes
69e03d1 
Image value mappings fixes
@Cyclam
Updates for 2023-09-01 API (#84)
24c9849 
* Regen Python SDK from 2023-09-01 API 
  - also uses latest AutoRest client
  - fix for HybridNetworkManagementClient init signature (swap order of subscription_id and credential parameters)

* Update CLI extension code to use new SDK

* added SAMI to publisher pre deploy

* Update bicep templates to use 2023-09-01

* Update NF templates

* Update metaschema

* Add Allow-Publisher to required feature flags

* Use secure objects for deployment parameters
@jddarby
Merge branch 'Azure:main' into add-aosm-extension
17f2e6a
@PerfectChaos
Correctly get array item types in CNF deployment parameter schemas
8d35266
@PerfectChaos
Merge pull request #118 from jddarby/chaos/schemarrays
bd1274f 
Correctly get array item types in CNF deployment parameter schemas
Revert master .flake8 config
171a873
@Cyclam @zhoxing-ms
Fix double space in user output.
288c8f8 
Co-authored-by: Xing Zhou <Zhou.Xing@microsoft.com>
Remove delete message to be consistent with other CLI extensions
793adcd
Move "Deleted <resource>" messages to logger.info().
84c22ed
@jordlay @sunnycarter
Remove Blob_SAS_URL Option from ArmTemplate in input file (#119)
cf4bc47 
* moved blob url to vhd config only; untested

* stopped error in post innit before validate

* changed ordering of inputs so that blob and filepath are next to each other; helptext for filepath different for each option

* Markups from sunny

* appease mypy

---------

Co-authored-by: Jordan <jordan.layton@metaswitch.com>
Co-authored-by: Sunny Carter <sunny.carter@metaswitch.com>
Use default_factory with dataclass hashable defaults (Python 3.11 com…
ac58f17 
…patibility)
Changelog and version bump
4cf7b96
@jordlay
Copy link
Collaborator

jordlay commented Mar 11, 2024

This is outdated but may have useful elements such as testing and logic, we should look through properly before closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@sunnycarter @jordlay @jddarby @PerfectChaos @patrykkulik-microsoft @Jamieparsons @jamiedparsons @Cyclam