From 7e417b13df6784974a490dd42bde28db9153838b Mon Sep 17 00:00:00 2001
From: Jason O'Donnell <2160810+jasonodonnell@users.noreply.github.com>
Date: Fri, 19 Jul 2024 16:13:42 -0400
Subject: [PATCH 1/4] provider/azure: update vmss doc only supporting unique
 deployment mode

---
 provider/azure/azure_discover.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/provider/azure/azure_discover.go b/provider/azure/azure_discover.go
index 2106b663..ef291a91 100644
--- a/provider/azure/azure_discover.go
+++ b/provider/azure/azure_discover.go
@@ -56,6 +56,8 @@ func (p *Provider) Help() string {
    When using tags the only permission needed is Microsoft.Network/networkInterfaces/*
 
    When using Virtual Machine Scale Sets the only role action needed is Microsoft.Compute/virtualMachineScaleSets/*/read.
+   The Azure provider only supports Virtual Machine Scale Sets deployed in [Uniform mode](https://learn.microsoft.com/en-us/azure/virtual-machine-scale-sets/virtual-machine-scale-sets-orchestration-modes#scale-sets-with-uniform-orchestration).
+   As of 2023 VMSS deploys using Flexible mode by default.
 
    It is recommended you make a dedicated key used only for auto-joining.
 `
@@ -130,7 +132,6 @@ func (p *Provider) Addrs(args map[string]string, l *log.Logger) ([]string, error
 		l.Printf("[ERROR] discover-azure: resource_group %s, vm_scale_set %s", resourceGroup, vmScaleSet)
 		return nil, fmt.Errorf("discover-azure: unclear configuration. use (tag name and value) or (resouce_group and vm_scale_set)")
 	}
-
 }
 
 func fetchAddrsWithTags(tagName string, tagValue string, vmnet network.InterfacesClient, l *log.Logger) ([]string, error) {
@@ -139,7 +140,6 @@ func fetchAddrsWithTags(tagName string, tagValue string, vmnet network.Interface
 
 	ctx := context.Background()
 	netres, err := vmnet.ListAll(ctx)
-
 	if err != nil {
 		return nil, fmt.Errorf("discover-azure: %s", err)
 	}

From ed80574c3d3a7692241acadba591135bcb35a74b Mon Sep 17 00:00:00 2001
From: Jason O'Donnell <2160810+jasonodonnell@users.noreply.github.com>
Date: Fri, 26 Jul 2024 15:12:13 -0400
Subject: [PATCH 2/4] Add descriptions

---
 .github/actions/acctest/action.yml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/.github/actions/acctest/action.yml b/.github/actions/acctest/action.yml
index 208a1a9c..7f2e6ada 100644
--- a/.github/actions/acctest/action.yml
+++ b/.github/actions/acctest/action.yml
@@ -2,13 +2,17 @@ name: acctest
 
 inputs:
   provider-test-infra-dir:
+    description: 'Specifies a directory for test infrastructure'
     required: false
   provider-go-test-dir:
+    description: 'Specifies a directory for Go tests'
     required: false
   provider-go-test-tags:
+    description: 'Test flags for go'
     required: false
     default: ''
   provider-tf-apply:
+    description: 'Applies terraform with auto-approve enabled'
     required: false
     default: terraform apply -auto-approve
   aws-region:

From 443f37249514fc26e0c67aee42b0e6629c05f9d6 Mon Sep 17 00:00:00 2001
From: Jason O'Donnell <2160810+jasonodonnell@users.noreply.github.com>
Date: Fri, 26 Jul 2024 15:14:42 -0400
Subject: [PATCH 3/4] Try again

---
 .github/workflows/acceptance.yml | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/acceptance.yml b/.github/workflows/acceptance.yml
index 169ec61e..439ee6e7 100644
--- a/.github/workflows/acceptance.yml
+++ b/.github/workflows/acceptance.yml
@@ -83,7 +83,7 @@ jobs:
         with:
           gotestsum_version: 1.9.0
 
-      - uses: "./.github/actions/acctest"
+      - uses: './.github/actions/acctest'
         with:
           provider-test-infra-dir: aliyun
           provider-go-test-dir: aliyun
@@ -313,13 +313,13 @@ jobs:
   # - be placed after the fanout of a workflow so that everything fans back in
   #   to this job.
   # - "need" any job that is part of the fan out / fan in
-  # - implement the if logic because we have conditional jobs 
-  #   (go-test-enteprise) that this job needs and this would potentially get 
-  #   skipped if a previous job got skipped.  So we use the if clause to make 
+  # - implement the if logic because we have conditional jobs
+  #   (go-test-enteprise) that this job needs and this would potentially get
+  #   skipped if a previous job got skipped.  So we use the if clause to make
   # sure it does not get skipped.
 
   acceptance-success:
-    needs: 
+    needs:
     - lint
     - go-test
     - alicloud-provider

From 8e06159db732145b6feff2bae7a1ed1eda6ee0bf Mon Sep 17 00:00:00 2001
From: Jason O'Donnell <2160810+jasonodonnell@users.noreply.github.com>
Date: Fri, 26 Jul 2024 17:17:19 -0400
Subject: [PATCH 4/4] Description

---
 .github/actions/acctest/action.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/actions/acctest/action.yml b/.github/actions/acctest/action.yml
index 7f2e6ada..77064bcc 100644
--- a/.github/actions/acctest/action.yml
+++ b/.github/actions/acctest/action.yml
@@ -1,5 +1,5 @@
 name: acctest
-
+description: "Acctest runs acceptance tests for go-discover"
 inputs:
   provider-test-infra-dir:
     description: 'Specifies a directory for test infrastructure'