Merge pull request #66 from stuartwdouglas/aws-logging

Log all AWS requests
konflux-ci · Oct 30, 2023 · 1101156 · 1101156
2 parents e1aed31 + 6ae9ece
commit 1101156
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/aws/ec2.go b/pkg/aws/ec2.go
@@ -36,6 +36,7 @@ func Ec2Provider(platformName string, config map[string]string, systemNamespace
 }
 
 func (configMapInfo AwsDynamicConfig) LaunchInstance(kubeClient client.Client, log *logr.Logger, ctx context.Context, name string, instanceTag string) (cloud.InstanceIdentifier, error) {
+	log.Info(fmt.Sprintf("attempting to launch AWS instance for %s", name))
 	// Load AWS credentials and configuration
 
 	cfg, err := config.LoadDefaultConfig(ctx,
@@ -82,6 +83,7 @@ func (configMapInfo AwsDynamicConfig) LaunchInstance(kubeClient client.Client, l
 }
 
 func (configMapInfo AwsDynamicConfig) CountInstances(kubeClient client.Client, log *logr.Logger, ctx context.Context, instanceTag string) (int, error) {
+	log.Info("attempting to count AWS instances")
 	cfg, err := config.LoadDefaultConfig(ctx,
 		config.WithCredentialsProvider(SecretCredentialsProvider{Name: configMapInfo.Secret, Namespace: configMapInfo.SystemNamespace, Client: kubeClient}),
 		config.WithRegion(configMapInfo.Region))
@@ -109,6 +111,7 @@ func (configMapInfo AwsDynamicConfig) CountInstances(kubeClient client.Client, l
 }
 
 func (configMapInfo AwsDynamicConfig) GetInstanceAddress(kubeClient client.Client, log *logr.Logger, ctx context.Context, instanceId cloud.InstanceIdentifier) (string, error) {
+	log.Info(fmt.Sprintf("attempting to get AWS instance address %s", instanceId))
 	// Load AWS credentials and configuration
 
 	cfg, err := config.LoadDefaultConfig(ctx,
@@ -145,6 +148,7 @@ func (configMapInfo AwsDynamicConfig) GetInstanceAddress(kubeClient client.Clien
 }
 
 func (configMapInfo AwsDynamicConfig) TerminateInstance(kubeClient client.Client, log *logr.Logger, ctx context.Context, instance cloud.InstanceIdentifier) error {
+	log.Info(fmt.Sprintf("attempting to terminate AWS instance %s", instance))
 
 	// Load AWS credentials and configuration