Skip to content

Commit c0c988c

Browse files
committed
Fix wrong telemetry properties
1 parent b8af93f commit c0c988c

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azure/toolkit/intellij/springcloud/deplolyment/SpringCloudDeploymentConfigurationState.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,8 @@ public AppResourceInner executeSteps(@NotNull RunProcessHandler processHandler,
8080

8181
final boolean toCreateApp = !app.exists();
8282
final boolean toCreateDeployment = !deployment.exists();
83+
operation.trackProperty("isCreateNewApp", String.valueOf(toCreateApp));
84+
operation.trackProperty("isCreateNewDeployment", String.valueOf(toCreateDeployment));
8385
final List<AzureTask<?>> tasks = new ArrayList<>();
8486
if (toCreateApp) {
8587
setText(processHandler, String.format("Creating app(%s)...", app.name()));
@@ -139,7 +141,7 @@ protected Map<String, String> getTelemetryMap() {
139141
props.put("runtime", config.getAppConfig().getRuntimeVersion());
140142
props.put("subscriptionId", config.getAppConfig().getSubscriptionId());
141143
props.put("public", String.valueOf(config.getAppConfig().isPublic()));
142-
props.put("jvmOptions", String.valueOf(StringUtils.isEmpty(config.getAppConfig().getDeployment().getJvmOptions())));
144+
props.put("jvmOptions", String.valueOf(StringUtils.isNotEmpty(config.getAppConfig().getDeployment().getJvmOptions())));
143145
props.put("instanceCount", String.valueOf(config.getAppConfig().getDeployment().getInstanceCount()));
144146
props.put("memory", String.valueOf(config.getAppConfig().getDeployment().getMemoryInGB()));
145147
props.put("cpu", String.valueOf(config.getAppConfig().getDeployment().getCpu()));

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azure/toolkit/lib/webapp/WebAppService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public WebApp createWebApp(final WebAppConfig config) {
3636
final Operation operation = TelemetryManager.createOperation(WEBAPP, CREATE_WEBAPP);
3737
try {
3838
operation.start();
39-
EventUtil.logEvent(EventType.info, operation, properties);
39+
operation.trackProperties(properties);
4040
return AzureWebAppMvpModel.getInstance().createWebApp(settings);
4141
} catch (final RuntimeException e) {
4242
EventUtil.logError(operation, ErrorType.userError, e, properties, null);
@@ -84,6 +84,7 @@ public static WebAppSettingModel convertConfig2Settings(final WebAppConfig confi
8484
settings.setEnableDetailedErrorMessage(monitorConfig.isEnableDetailedErrorMessage());
8585
settings.setEnableFailedRequestTracing(monitorConfig.isEnableFailedRequestTracing());
8686
}
87+
settings.setTargetPath(config.getApplication() == null ? null : config.getApplication().toString());
8788
return settings;
8889
}
8990
}

0 commit comments

Comments
 (0)