Skip to content

Commit 1180f70

Browse files
authored
Adding credential load telemetry (#2109)
1 parent 7d4a103 commit 1180f70

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ coroutinesVersion=1.3.3
1616
ideaPluginVersion=0.4.20
1717
ktlintVersion=0.38.1
1818
jacksonVersion=2.9.8
19-
telemetryVersion=0.0.58
19+
telemetryVersion=0.0.59
2020

2121
assertjVersion=3.15.0
2222
junitVersion=4.12

jetbrains-core/src/software/aws/toolkits/jetbrains/core/credentials/CredentialManager.kt

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@ import software.aws.toolkits.core.region.AwsRegion
2121
import software.aws.toolkits.core.utils.getLogger
2222
import software.aws.toolkits.core.utils.tryOrNull
2323
import software.aws.toolkits.jetbrains.core.AwsSdkClient
24+
import software.aws.toolkits.telemetry.AwsTelemetry
2425
import java.util.concurrent.ConcurrentHashMap
26+
import java.util.concurrent.atomic.AtomicInteger
2527

2628
abstract class CredentialManager : SimpleModificationTracker() {
2729
private val providerIds = ConcurrentHashMap<String, CredentialIdentifier>()
@@ -114,10 +116,12 @@ class DefaultCredentialManager : CredentialManager() {
114116

115117
init {
116118
extensionMap.values.forEach { providerFactory ->
119+
val count = AtomicInteger(0)
117120
LOG.tryOrNull("Failed to set up $providerFactory") {
118121
providerFactory.setUp { change ->
119122
change.added.forEach {
120123
addProvider(it)
124+
count.incrementAndGet()
121125
}
122126

123127
change.modified.forEach {
@@ -126,7 +130,9 @@ class DefaultCredentialManager : CredentialManager() {
126130

127131
change.removed.forEach {
128132
removeProvider(it)
133+
count.decrementAndGet()
129134
}
135+
AwsTelemetry.loadCredentials(credentialSourceId = providerFactory.id, value = count.get().toDouble())
130136
}
131137
}
132138
}

0 commit comments

Comments
 (0)