Skip to content

Commit 2441454

Browse files
authored
Fix e2e_tests (#308)
1 parent 7cf54f2 commit 2441454

File tree

4 files changed

+23
-10
lines changed

4 files changed

+23
-10
lines changed

e2e_tests/test_suites/inventory/inventory.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ func runGatherInventoryTest(ctx context.Context, testSetup *inventoryTestSetup,
9191
metadataItems = append(metadataItems, testSetup.startup)
9292
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("enable-osconfig", "true"))
9393
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("osconfig-disabled-features", "tasks,guestpolicies"))
94-
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("osconfig-poll-interval", "30s"))
94+
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("osconfig-poll-interval", "1"))
9595

9696
testProjectConfig := testconfig.GetProject()
9797
zone := testProjectConfig.AcquireZone()

e2e_tests/test_suites/inventoryreporting/inventory_reporting.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ func runInventoryReportingTest(ctx context.Context, testSetup *inventoryTestSetu
8484
metadataItems = append(metadataItems, testSetup.startup)
8585
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("enable-osconfig", "true"))
8686
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("osconfig-disabled-features", "tasks,guestpolicies"))
87-
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("osconfig-poll-interval", "30s"))
87+
metadataItems = append(metadataItems, compute.BuildInstanceMetadataItem("osconfig-poll-interval", "1"))
8888

8989
testProjectConfig := testconfig.GetProject()
9090
zone := testProjectConfig.AcquireZone()

e2e_tests/test_suites/ospolicies/ospolicies.go

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -121,18 +121,26 @@ func createOSPolicyAssignment(ctx context.Context, client *osconfigZonalV1alpha.
121121
defer gpMx.Unlock()
122122
op, err := client.CreateOSPolicyAssignment(ctx, req)
123123
if err != nil {
124-
return nil, err
124+
return nil, fmt.Errorf("error running CreateOSPolicyAssignment: %s", utils.GetStatusFromError(err))
125125
}
126126
//return op.Wait(ctx)
127127
_ = op
128128

129+
time.Sleep(30 * time.Second)
129130
getReq := &osconfigpb.GetOSPolicyAssignmentRequest{Name: fmt.Sprintf("%s/osPolicyAssignments/%s", req.GetParent(), req.GetOsPolicyAssignmentId())}
130-
ospa, err := client.GetOSPolicyAssignment(ctx, getReq)
131-
if err != nil {
132-
return nil, err
133-
}
134-
135-
return ospa, nil
131+
for i := 0; i < 20; i++ {
132+
ospa, err := client.GetOSPolicyAssignment(ctx, getReq)
133+
if err != nil {
134+
return nil, fmt.Errorf("error running GetOSPolicyAssignment: %s", utils.GetStatusFromError(err))
135+
}
136+
switch ospa.GetRolloutState() {
137+
case osconfigpb.OSPolicyAssignment_IN_PROGRESS, osconfigpb.OSPolicyAssignment_CANCELLED:
138+
return ospa, nil
139+
}
140+
time.Sleep(10 * time.Second)
141+
}
142+
143+
return nil, fmt.Errorf("timed out waiting for rollout to finish")
136144
}
137145

138146
func runTest(ctx context.Context, testCase *junitxml.TestCase, testSetup *osPolicyTestSetup, logger *log.Logger) {
@@ -180,7 +188,7 @@ func runTest(ctx context.Context, testCase *junitxml.TestCase, testSetup *osPoli
180188
testCase.Logf("Creating OSPolicyAssignment")
181189
ospa, err := createOSPolicyAssignment(ctx, client, req)
182190
if err != nil {
183-
testCase.WriteFailure("Error running CreateOSPolicyAssignment: %s", utils.GetStatusFromError(err))
191+
testCase.WriteFailure("Error running createOSPolicyAssignment: %s", err)
184192
return
185193
}
186194
defer cleanupOSPolicyAssignment(ctx, testCase, ospa.GetName())

e2e_tests/utils/utils.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,16 @@ systemctl restart google-osconfig-agent` + CurlPost
6969

7070
// CurlPost indicates agent is installed.
7171
CurlPost = `
72+
uri=http://metadata.google.internal/computeMetadata/v1/instance/guest-attributes/guestInventory/LastUpdated
73+
curl -X DELETE $uri -H "Metadata-Flavor: Google"
74+
7275
uri=http://metadata.google.internal/computeMetadata/v1/instance/guest-attributes/osconfig_tests/install_done
7376
curl -X PUT --data "1" $uri -H "Metadata-Flavor: Google"
7477
`
7578

7679
windowsPost = `
80+
$uri = 'http://metadata.google.internal/computeMetadata/v1/instance/guest-attributes/guestInventory/LastUpdated'
81+
Invoke-RestMethod -Method DELETE -Uri $uri -Headers @{"Metadata-Flavor" = "Google"}
7782
Start-Sleep 10
7883
$uri = 'http://metadata.google.internal/computeMetadata/v1/instance/guest-attributes/osconfig_tests/install_done'
7984
Invoke-RestMethod -Method PUT -Uri $uri -Headers @{"Metadata-Flavor" = "Google"} -Body 1

0 commit comments

Comments
 (0)