Skip to content
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

Sunny/nf agent #27

Closed
wants to merge 28 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
01a1f99
choose-deploy-parameters
sunnycarter Jun 6, 2023
d83dcec
optioned deployParameters for CNF
sunnycarter Jun 6, 2023
1722871
lint
sunnycarter Jun 6, 2023
229e60e
lint2
sunnycarter Jun 6, 2023
381c0cd
docs
sunnycarter Jun 7, 2023
51dc63e
docs
sunnycarter Jun 7, 2023
729874b
Merge branch 'add-aosm-extension' into sunny/choose-deploy-parameters
sunnycarter Jun 7, 2023
4a9fe89
lint
sunnycarter Jun 7, 2023
c0a8862
9.82 score
sunnycarter Jun 7, 2023
3211ad1
Fix bugs
sunnycarter Jun 8, 2023
0be3e11
more useful debug logs
sunnycarter Jun 8, 2023
9f1b203
Working publish
patrykkulik-microsoft Jun 8, 2023
ad41715
Fix bugs and logging
sunnycarter Jun 8, 2023
d8edfa9
lint
sunnycarter Jun 8, 2023
15405c2
Merge branch 'add-aosm-extension' into pk5/publish_CNF
patrykkulik-microsoft Jun 9, 2023
f37f49a
Fix the artifact upload
patrykkulik-microsoft Jun 9, 2023
6d6cf2f
Working image copy
patrykkulik-microsoft Jun 9, 2023
05e08da
minor fix
patrykkulik-microsoft Jun 9, 2023
de34077
Minor fixes
patrykkulik-microsoft Jun 9, 2023
1e29dcf
sunny merge add-aosm-extension into patryk's branch (#25)
sunnycarter Jun 12, 2023
6af0564
minor TODOs
patrykkulik-microsoft Jun 12, 2023
7165aef
deleted comment
patrykkulik-microsoft Jun 12, 2023
eea0484
fix bring your own parameters
sunnycarter Jun 14, 2023
bee2ec7
merge patryks branch
sunnycarter Jun 15, 2023
661d449
add test files so chaos can see what I'm doing
sunnycarter Jun 15, 2023
0c058d3
latest fixes
sunnycarter Jun 16, 2023
61a6e7c
forgot these
sunnycarter Jun 16, 2023
afa0c59
check in where I got to
sunnycarter Jun 16, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix the artifact upload
  • Loading branch information
patrykkulik-microsoft committed Jun 9, 2023
commit f37f49ad81bb7d36ff9bf5b7fc1f3645a134a287
21 changes: 16 additions & 5 deletions src/aosm/azext_aosm/deploy/deploy_with_arm.py
Original file line number Diff line number Diff line change
Expand Up @@ -328,18 +328,29 @@ def deploy_cnfd_from_bicep(
)
print(f"ARTIFACTS: {acr_manifest.artifacts}")

artifact_dict = {}

for artifact in acr_manifest.artifacts:
artifact_dict[artifact.artifact_name] = artifact

for package_number in range(len(self.config.helm_packages)):
helm_chart_artifact = acr_manifest.artifacts[package_number]
helm_package_name = self.config.helm_packages[package_number]["name"]

print(
f"Uploading Helm package: {self.config.helm_packages[package_number]['name']}"
)
if helm_package_name not in artifact_dict.keys():
print(
f"Artifact {self.config.helm_packages[package_number]['name']} not found in the artifact manifest"
)
continue

manifest_artifact = artifact_dict[helm_package_name]

print(f"Uploading Helm package: {helm_package_name}")

print(
f"Uploading Helm package: {self.config.helm_packages[package_number]}"
)

helm_chart_artifact.upload(self.config.helm_packages[package_number])
manifest_artifact.upload(self.config.helm_packages[package_number])
print("Done")

def deploy_nsd_from_bicep(
Expand Down