fix: add azure dns to request object (#286)

Co-authored-by: Heba Elayoty <hebaelayoty@gmail.com>

client/aci/client_test.go

@@ -27,7 +27,7 @@ import (
 
 var (
 	client                   *Client
-	location                 = "westus"
+	location                 = "eastus2"
 	resourceGroup            = "virtual-node-test-rg"
 	containerGroup           = "virtual-kubelet-test-container-group"
 	virtualNetwork           = "virtual-kubelet-tests-vnet"
@@ -218,8 +218,8 @@ func TestCreateContainerGroupWithoutResourceLimit(t *testing.T) {
 						},
 						Resources: ResourceRequirements{
 							Requests: &ComputeResources{
-								CPU:        1,
-								MemoryInGB: 1,
+								CPU:        0.5,
+								MemoryInGB: 0.5,
 							},
 						},
 					},

provider/aci.go

@@ -772,8 +772,13 @@ func (p *ACIProvider) getDNSConfig(pod *v1.Pod) *aci.DNSConfig {
 	nameServers := make([]string, 0)
 	searchDomains := []string{}
 
+	// Adding default Azure dns name explicitly
+	// if any other dns names are provided by the user ACI will use those instead of azure dns
+	// which may cause issues while looking up other Azure resources
+	AzureDNSIP := "168.63.129.16"
 	if pod.Spec.DNSPolicy == v1.DNSClusterFirst || pod.Spec.DNSPolicy == v1.DNSClusterFirstWithHostNet {
 		nameServers = append(nameServers, p.kubeDNSIP)
+		nameServers = append(nameServers, AzureDNSIP)
 		searchDomains = p.generateSearchesForDNSClusterFirst(pod.Spec.DNSConfig, pod)
 	}