Skip to content

Commit d5cfefd

Browse files
committed
Remove unused node resource group config
1 parent 2d96feb commit d5cfefd

2 files changed

Lines changed: 5 additions & 9 deletions

File tree

pkg/config/config.go

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -87,12 +87,11 @@ type BootstrapTokenConfig struct {
8787

8888
// TargetClusterConfig holds configuration for the target AKS cluster the ARC machine will connect to.
8989
type TargetClusterConfig struct {
90-
ResourceID string `json:"resourceId"` // Full resource ID of the target AKS cluster
91-
Location string `json:"location"` // Azure region of the cluster (e.g., "eastus", "westus2")
92-
Name string // will be populated from ResourceID
93-
ResourceGroup string // will be populated from ResourceID
94-
SubscriptionID string // will be populated from ResourceID
95-
NodeResourceGroup string // reserved for explicit AKS node resource group data; not derived from ResourceID
90+
ResourceID string `json:"resourceId"` // Full resource ID of the target AKS cluster
91+
Location string `json:"location"` // Azure region of the cluster (e.g., "eastus", "westus2")
92+
Name string // will be populated from ResourceID
93+
ResourceGroup string // will be populated from ResourceID
94+
SubscriptionID string // will be populated from ResourceID
9695
}
9796

9897
// ArcConfig holds Azure Arc machine configuration for registering the machine with Azure Arc.

pkg/config/config_test.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1246,9 +1246,6 @@ func TestPopulateTargetClusterInfoFromConfig(t *testing.T) {
12461246
if config.Azure.SubscriptionID != expected.SubscriptionID {
12471247
t.Errorf("Expected Azure SubscriptionID %s, got %s", expected.SubscriptionID, config.Azure.SubscriptionID)
12481248
}
1249-
if config.Azure.TargetCluster.NodeResourceGroup != "" {
1250-
t.Errorf("Expected NodeResourceGroup to remain empty, got %s", config.Azure.TargetCluster.NodeResourceGroup)
1251-
}
12521249
if config.Azure.TargetCluster.Location != expected.Location {
12531250
t.Errorf("Expected Location %s, got %s", expected.Location, config.Azure.TargetCluster.Location)
12541251
}

0 commit comments

Comments
 (0)