Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
The goal of this PR is to cherry pick:
Longhaul Arm: Decrease frequency of test events on arm (Azure#4979)

But in order to do this I needed denis' one liner that was never ported:
Change default direct method sender interval to 10s (Azure#4708)
  • Loading branch information
and-rewsmith authored May 27, 2021
1 parent 1947aea commit 0a7e655
Show file tree
Hide file tree
Showing 18 changed files with 111 additions and 125 deletions.
2 changes: 0 additions & 2 deletions builds/e2e/connectivity.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,6 @@ jobs:
eventHub.connectionString: '$(IotHub-EventHubConnStr)'
deploymentFileName: '$(deploymentFileName)'
upstream.protocol: '$(upstream.protocol)'
loadGen.message.frequency: '$(loadGen.message.frequency.amd64)'
edgeHubRestartTest.restartPeriod: '$(edgeHubRestartTest.restartPeriod)'
edgeHubRestartTest.sdkOperationTimeout: '$(edgeHubRestartTest.sdkOperationTimeout)'
testDuration: '$(testrun.duration)'
Expand Down Expand Up @@ -367,7 +366,6 @@ jobs:
eventHub.connectionString: '$(EdgeConnectivityEventHubARM32ConnString)'
deploymentFileName: '$(deploymentFileName)'
upstream.protocol: '$(upstream.protocol)'
loadGen.message.frequency: '$(loadGen.message.frequency.arm32)'
edgeHubRestartTest.restartPeriod: '$(edgeHubRestartTest.restartPeriod)'
edgeHubRestartTest.sdkOperationTimeout: '$(edgeHubRestartTest.sdkOperationTimeout)'
testDuration: '$(testrun.duration)'
Expand Down
11 changes: 0 additions & 11 deletions builds/e2e/longhaul.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,17 +18,13 @@ variables:
#test.startDelay.amd
#edgelet.package.branchName
#images.branchName
#loadGen.message.frequency.amd64: 'xx:xx:xx.xx'
#loadGen.message.frequency.arm32v7: 'xx:xx:xx.xx'
#loadGen.message.frequency.arm64v8: 'xx:xx:xx.xx'
#run.linux.amd64.moby: 'true/false'
#run.linux.arm32v7.moby: 'true/false'
#run.linux.arm64v8.docker: 'true/false'
#longhaul.desiredModulesToRestartCSV: ''
#longhaul.restartIntervalInMins: ''
#longHaul.sendReportFrequency: 'xx:xx:xx.xx'
#twinTester.twinUpdateSize: ''
#twinTester.twinUpdateFrequency: 'xx:xx:xx.xx'
#twinTester.twinUpdateFailureThreshold: 'xx:xx:xx.xx'
#twinTester.edgeHubRestartFailureTolerance: 'xx:xx:xx.xx'
#metricsCollector.metricsEndpointsCSV
Expand Down Expand Up @@ -143,7 +139,6 @@ jobs:
snitch.alert.url: '$(SnitchLongHaulAlertUrl)'
snitch.storage.account: '$(snitch.storage.account)'
snitch.storage.masterKey: '$(StorageAccountMasterKeyStress)'
loadGen.message.frequency: '$(loadGen.message.frequency.amd64)'
longHaul.desiredModulesToRestartCSV: '$(longHaul.desiredModulesToRestartCSV)'
longHaul.restartIntervalInMins: '$(longHaul.restartIntervalInMins)'
longHaul.sendReportFrequency: '$(longHaul.sendReportFrequency)'
Expand All @@ -153,7 +148,6 @@ jobs:
logAnalytics.sharedKey: '$(kvLogAnalyticSharedKey)'
logAnalytics.logType: '$(logAnalyticsLogType)'
twinTester.twinUpdateSize: '$(twinTester.twinUpdateSize)'
twinTester.twinUpdateFrequency: '$(twinTester.twinUpdateFrequency)'
twinTester.twinUpdateFailureThreshold: '$(twinTester.twinUpdateFailureThreshold)'
twinTester.edgeHubRestartFailureTolerance: '$(twinTester.edgeHubRestartFailureTolerance)'
metricsCollector.metricsEndpointsCSV: '$(metricsCollector.metricsEndpointsCSV)'
Expand Down Expand Up @@ -272,7 +266,6 @@ jobs:
snitch.alert.url: '$(SnitchLongHaulAlertUrl)'
snitch.storage.account: '$(snitch.storage.account)'
snitch.storage.masterKey: '$(StorageAccountMasterKeyStress)'
loadGen.message.frequency: '$(loadGen.message.frequency.arm32v7)'
longHaul.desiredModulesToRestartCSV: '$(longHaul.desiredModulesToRestartCSV)'
longHaul.restartIntervalInMins: '$(longHaul.restartIntervalInMins)'
longHaul.sendReportFrequency: '$(longHaul.sendReportFrequency)'
Expand All @@ -282,8 +275,6 @@ jobs:
logAnalytics.sharedKey: '$(kvLogAnalyticSharedKey)'
logAnalytics.logType: '$(logAnalyticsLogType)'
twinTester.twinUpdateSize: '$(twinTester.twinUpdateSize)'
twinTester.twinUpdateFrequency: '$(twinTester.twinUpdateFrequency)'
twinTester.twinUpdateFailureThreshold: '$(twinTester.twinUpdateFailureThreshold)'
twinTester.edgeHubRestartFailureTolerance: '$(twinTester.edgeHubRestartFailureTolerance)'
metricsCollector.metricsEndpointsCSV: '$(metricsCollector.metricsEndpointsCSV)'
metricsCollector.scrapeFrequencyInSecs: '$(metricsCollector.scrapeFrequencyInSecs)'
Expand Down Expand Up @@ -401,7 +392,6 @@ jobs:
snitch.alert.url: '$(SnitchLongHaulAlertUrl)'
snitch.storage.account: '$(snitch.storage.account)'
snitch.storage.masterKey: '$(StorageAccountMasterKeyStress)'
loadGen.message.frequency: '$(loadGen.message.frequency.arm64v8)'
longHaul.desiredModulesToRestartCSV: '$(longHaul.desiredModulesToRestartCSV)'
longHaul.restartIntervalInMins: '$(longHaul.restartIntervalInMins)'
longHaul.sendReportFrequency: '$(longHaul.sendReportFrequency)'
Expand All @@ -411,7 +401,6 @@ jobs:
logAnalytics.sharedKey: '$(kvLogAnalyticSharedKey)'
logAnalytics.logType: '$(logAnalyticsLogType)'
twinTester.twinUpdateSize: '$(twinTester.twinUpdateSize)'
twinTester.twinUpdateFrequency: '$(twinTester.twinUpdateFrequency)'
twinTester.twinUpdateFailureThreshold: '$(twinTester.twinUpdateFailureThreshold)'
twinTester.edgeHubRestartFailureTolerance: '$(twinTester.edgeHubRestartFailureTolerance)'
metricsCollector.metricsEndpointsCSV: '$(metricsCollector.metricsEndpointsCSV)'
Expand Down
1 change: 0 additions & 1 deletion builds/e2e/nested-connectivity.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,6 @@ stages:
eventHub.connectionString: '$(IotHub-EventHubConnStr)'
deploymentFileName: 'nestededge_bottomLayerBaseDeployment_connectivity_mqtt.template.json'
upstream.protocol: '$(upstream.protocol)'
loadGen.message.frequency: '$(loadGen.message.frequency.amd64)'
edgeHubRestartTest.restartPeriod: '$(edgeHubRestartTest.restartPeriod)'
edgeHubRestartTest.sdkOperationTimeout: '$(edgeHubRestartTest.sdkOperationTimeout)'
testDuration: '$(testrun.duration)'
Expand Down
2 changes: 0 additions & 2 deletions builds/e2e/templates/connectivity-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ parameters:
iotHub.connectionString: ''
eventHub.connectionString: ''
upstream.protocol: ''
loadGen.message.frequency: ''
networkController.description: ''
networkController.frequencies: ''
networkController.runProfile: ''
Expand Down Expand Up @@ -122,7 +121,6 @@ steps:
-upstreamProtocol "${{ parameters['upstream.protocol'] }}" \
-testDuration "${{ parameters['testDuration'] }}" \
-testStartDelay "${{ parameters['testStartDelay'] }}" \
-loadGenMessageFrequency "${{ parameters['loadGen.message.frequency'] }}" \
-networkControllerFrequency "${{ parameters['networkController.frequencies'] }}" \
-networkControllerRunProfile "${{ parameters['networkController.runProfile'] }}" \
-logAnalyticsWorkspaceId "${{ parameters['logAnalyticsWorkspaceId'] }}" \
Expand Down
4 changes: 0 additions & 4 deletions builds/e2e/templates/longhaul-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ parameters:
snitch.alert.url: ''
snitch.storage.account: ''
snitch.storage.masterKey: ''
loadGen.message.frequency: ''
longHaul.desiredModulesToRestartCSV: ''
longHaul.restartIntervalInMins: ''
longHaul.sendReportFrequency: ''
Expand All @@ -27,7 +26,6 @@ parameters:
logAnalytics.sharedKey: ''
logAnalytics.logType: ''
twinTester.twinUpdateSize: ''
twinTester.twinUpdateFrequency: ''
twinTester.twinUpdateFailureThreshold: ''
twinTester.edgeHubRestartFailureTolerance: ''
metricsCollector.metricsEndpointsCSV: ''
Expand Down Expand Up @@ -104,7 +102,6 @@ steps:
-eventHubConnectionString "${{ parameters['eventHub.connectionString'] }}" \
-testStartDelay "${{ parameters['test.startDelay'] }}" \
-restartIntervalInMins "${{ parameters['longHaul.restartIntervalInMins'] }}" \
-loadGenMessageFrequency "${{ parameters['loadGen.message.frequency'] }}" \
-logAnalyticsWorkspaceId "${{ parameters['logAnalytics.workspaceId'] }}" \
-logAnalyticsSharedKey "${{ parameters['logAnalytics.sharedKey'] }}" \
-logAnalyticsLogType "${{ parameters['logAnalytics.logType'] }}" \
Expand All @@ -116,7 +113,6 @@ steps:
-testRuntimeLogLevel "${{ parameters['test.runtimeLogLevel'] }}" \
-testInfo "$testInfo" \
-twinUpdateSize "${{ parameters['twinTester.twinUpdateSize'] }}" \
-twinUpdateFrequency "${{ parameters['twinTester.twinUpdateFrequency'] }}" \
-edgeHubRestartFailureTolerance "${{ parameters['twinTester.edgeHubRestartFailureTolerance'] }}" \
-desiredModulesToRestartCSV "${{ parameters['longHaul.desiredModulesToRestartCSV'] }}" \
-sendReportFrequency "${{ parameters['longHaul.sendReportFrequency'] }}" \
Expand Down
2 changes: 0 additions & 2 deletions builds/e2e/templates/nested-longhaul-deploy-amd64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ jobs:
container.registry.password: '$(edgebuilds-azurecr-io-pwd)'
iotHub.connectionString: '$(IotHub-ConnStr)'
eventHub.connectionString: '$(IotHub-EventHubConnStr)'
loadGen.message.frequency: '$(loadGen.message.frequency)'
longHaul.desiredModulesToRestartCSV: '$(longHaul.desiredModulesToRestartCSV)'
longHaul.restartIntervalInMins: '$(longHaul.restartIntervalInMins)'
longHaul.sendReportFrequency: '$(longHaul.sendReportFrequency)'
Expand All @@ -54,7 +53,6 @@ jobs:
logAnalytics.sharedKey: '$(kvLogAnalyticSharedKey)'
logAnalytics.logType: '$(logAnalyticsLogType)'
twinTester.twinUpdateSize: '$(twinTester.twinUpdateSize)'
twinTester.twinUpdateFrequency: '$(twinTester.twinUpdateFrequency)'
twinTester.twinUpdateFailureThreshold: '$(twinTester.twinUpdateFailureThreshold)'
twinTester.edgeHubRestartFailureTolerance: '$(twinTester.edgeHubRestartFailureTolerance)'
metricsCollector.metricsEndpointsCSV: '$(metricsCollector.metricsEndpointsCSV)'
Expand Down
2 changes: 0 additions & 2 deletions builds/e2e/templates/nested-longhaul-deploy-arm32.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ jobs:
container.registry.password: '$(edgebuilds-azurecr-io-pwd)'
iotHub.connectionString: '$(IotHub-ConnStr)'
eventHub.connectionString: '$(IotHub-EventHubConnStr)'
loadGen.message.frequency: '$(loadGen.message.frequency)'
longHaul.desiredModulesToRestartCSV: '$(longHaul.desiredModulesToRestartCSV)'
longHaul.restartIntervalInMins: '$(longHaul.restartIntervalInMins)'
longHaul.sendReportFrequency: '$(longHaul.sendReportFrequency)'
Expand All @@ -55,7 +54,6 @@ jobs:
logAnalytics.sharedKey: '$(kvLogAnalyticSharedKey)'
logAnalytics.logType: '$(logAnalyticsLogType)'
twinTester.twinUpdateSize: '$(twinTester.twinUpdateSize)'
twinTester.twinUpdateFrequency: '$(twinTester.twinUpdateFrequency)'
twinTester.twinUpdateFailureThreshold: '$(twinTester.twinUpdateFailureThreshold)'
twinTester.edgeHubRestartFailureTolerance: '$(twinTester.edgeHubRestartFailureTolerance)'
metricsCollector.metricsEndpointsCSV: '$(metricsCollector.metricsEndpointsCSV)'
Expand Down
2 changes: 0 additions & 2 deletions builds/e2e/templates/nested-longhaul-deploy-arm64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ jobs:
container.registry.password: '$(edgebuilds-azurecr-io-pwd)'
iotHub.connectionString: '$(IotHub-ConnStr)'
eventHub.connectionString: '$(IotHub-EventHubConnStr)'
loadGen.message.frequency: '$(loadGen.message.frequency)'
longHaul.desiredModulesToRestartCSV: '$(longHaul.desiredModulesToRestartCSV)'
longHaul.restartIntervalInMins: '$(longHaul.restartIntervalInMins)'
longHaul.sendReportFrequency: '$(longHaul.sendReportFrequency)'
Expand All @@ -55,7 +54,6 @@ jobs:
logAnalytics.workspaceId: '$(kvLogAnalyticWorkspaceId)'
logAnalytics.sharedKey: '$(kvLogAnalyticSharedKey)'
twinTester.twinUpdateSize: '$(twinTester.twinUpdateSize)'
twinTester.twinUpdateFrequency: '$(twinTester.twinUpdateFrequency)'
twinTester.twinUpdateFailureThreshold: '$(twinTester.twinUpdateFailureThreshold)'
twinTester.edgeHubRestartFailureTolerance: '$(twinTester.edgeHubRestartFailureTolerance)'
metricsCollector.metricsEndpointsCSV: '$(metricsCollector.metricsEndpointsCSV)'
Expand Down
3 changes: 0 additions & 3 deletions builds/e2e/templates/nested-longhaul-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ parameters:
container.registry.password: ''
iotHub.connectionString: ''
eventHub.connectionString: ''
loadGen.message.frequency: ''
longHaul.desiredModulesToRestartCSV: ''
longHaul.restartIntervalInMins: ''
longHaul.sendReportFrequency: ''
Expand All @@ -24,7 +23,6 @@ parameters:
logAnalytics.sharedKey: ''
logAnalytics.logType: ''
twinTester.twinUpdateSize: ''
twinTester.twinUpdateFrequency: ''
twinTester.twinUpdateFailureThreshold: ''
twinTester.edgeHubRestartFailureTolerance: ''
metricsCollector.metricsEndpointsCSV: ''
Expand Down Expand Up @@ -172,7 +170,6 @@ steps:
-eventHubConnectionString "${{ parameters['eventHub.connectionString'] }}" \
-testStartDelay "${{ parameters['test.startDelay'] }}" \
-restartIntervalInMins "${{ parameters['longHaul.restartIntervalInMins'] }}" \
-loadGenMessageFrequency "${{ parameters['loadGen.message.frequency'] }}" \
-logAnalyticsWorkspaceId "${{ parameters['logAnalytics.workspaceId'] }}" \
-logAnalyticsSharedKey "${{ parameters['logAnalytics.sharedKey'] }}" \
-logAnalyticsLogType "${{ parameters['logAnalytics.logType'] }}" \
Expand Down
9 changes: 9 additions & 0 deletions e2e_deployment_files/connectivity_deployment.template.json
Original file line number Diff line number Diff line change
Expand Up @@ -156,6 +156,9 @@
},
"trackingId": {
"value": "<TrackingId>"
},
"DirectMethodFrequency": {
"value": "<DirectMethodFrequency>"
}
},
"settings": {
Expand Down Expand Up @@ -213,6 +216,9 @@
},
"trackingId": {
"value": "<TrackingId>"
},
"DirectMethodFrequency": {
"value": "<DirectMethodFrequency>"
}
},
"settings": {
Expand Down Expand Up @@ -273,6 +279,9 @@
},
"trackingId": {
"value": "<TrackingId>"
},
"DirectMethodFrequency": {
"value": "<DirectMethodFrequency>"
}
},
"settings": {
Expand Down
6 changes: 6 additions & 0 deletions e2e_deployment_files/long_haul_deployment.template.json
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,9 @@
},
"trackingId": {
"value": "<TrackingId>"
},
"DirectMethodFrequency": {
"value": "<DirectMethodFrequency>"
}
},
"settings": {
Expand Down Expand Up @@ -312,6 +315,9 @@
},
"trackingId": {
"value": "<TrackingId>"
},
"DirectMethodFrequency": {
"value": "<DirectMethodFrequency>"
}
},
"settings": {
Expand Down
Loading

0 comments on commit 0a7e655

Please sign in to comment.