Skip to content

Commit d6d0782

Browse files
enable networkflowlogs through helm chart (#1649)
1 parent 97c2f90 commit d6d0782

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

.pipelines/azure_pipeline_mergedbranches.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1055,6 +1055,8 @@ extends:
10551055
imageRepository: $(ImageRepositoryOverride)
10561056
environment: 'CI-Agent-NetworkFlowLogs'
10571057
azureSubscription: 'ContainerInsights_Build_Subscription_CI'
1058+
# TODO: Use OmsAgent.retinaFlowLogsEnabled=true once we use the merged chart
1059+
additionalOverrides: 'OmsAgent.isRetinaFlowLogsEnabled=true'
10581060

10591061
# Cluster 6: ci-logs-prod-aks-networkflowlogs — Run E2E Tests
10601062
- template: /.pipelines/e2e-test-templates/test-ci-image-in-aks-cluster.yml@self

.pipelines/helm-deploy-templates/ama-logs-helm-deploy.yaml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ parameters:
5151
- name: releaseName
5252
type: string
5353
default: 'azuremonitor-containers'
54+
- name: additionalOverrides
55+
type: string
56+
default: ''
57+
displayName: 'Additional Helm override values (comma-separated key=value pairs)'
5458
- name: dependsOn
5559
type: object
5660
default: []
@@ -101,7 +105,10 @@ jobs:
101105
# TODO: When it is merged to ci_prod, the following chartPath will be updated accordingly.
102106
chartPath: '$(Build.SourcesDirectory)/charts/azuremonitor-containerinsights-for-prod-clusters'
103107
releaseName: '${{ parameters.releaseName }}'
104-
overrideValues: 'global.commonGlobals.CloudEnvironment=${{ parameters.cloudEnvironment }},global.commonGlobals.Region=${{ parameters.region }},OmsAgent.aksResourceID=$(AKS_RESOURCE_ID),OmsAgent.workspaceID=${{ parameters.workspaceId }},OmsAgent.imageRepository=${{ parameters.imageRepository }},OmsAgent.imageTagLinux=${{ parameters.amalogsLinuxImage }},OmsAgent.imageTagWindows=${{ parameters.amalogsWindowsImage }}'
108+
${{ if ne(parameters.additionalOverrides, '') }}:
109+
overrideValues: 'global.commonGlobals.CloudEnvironment=${{ parameters.cloudEnvironment }},global.commonGlobals.Region=${{ parameters.region }},OmsAgent.aksResourceID=$(AKS_RESOURCE_ID),OmsAgent.workspaceID=${{ parameters.workspaceId }},OmsAgent.imageRepository=${{ parameters.imageRepository }},OmsAgent.imageTagLinux=${{ parameters.amalogsLinuxImage }},OmsAgent.imageTagWindows=${{ parameters.amalogsWindowsImage }},${{ parameters.additionalOverrides }}'
110+
${{ else }}:
111+
overrideValues: 'global.commonGlobals.CloudEnvironment=${{ parameters.cloudEnvironment }},global.commonGlobals.Region=${{ parameters.region }},OmsAgent.aksResourceID=$(AKS_RESOURCE_ID),OmsAgent.workspaceID=${{ parameters.workspaceId }},OmsAgent.imageRepository=${{ parameters.imageRepository }},OmsAgent.imageTagLinux=${{ parameters.amalogsLinuxImage }},OmsAgent.imageTagWindows=${{ parameters.amalogsWindowsImage }}'
105112
waitForExecution: false
106113
arguments: '--timeout 10m --install'
107114
- task: AzureCLI@2

0 commit comments

Comments
 (0)