WIP: release locks and try to read CONFIGURING during reconf

robot/impl/local_robot.go

@@ -1422,13 +1422,13 @@ func (r *localRobot) Shutdown(ctx context.Context) error {
 func (r *localRobot) MachineStatus(ctx context.Context) (robot.MachineStatus, error) {
 	var result robot.MachineStatus
 
-	r.manager.resourceGraphLock.Lock()
+	// r.manager.resourceGraphLock.Lock()
 	result.Resources = append(result.Resources, r.manager.resources.Status()...)
-	r.manager.resourceGraphLock.Unlock()
+	// r.manager.resourceGraphLock.Unlock()
 
-	r.configRevisionMu.RLock()
+	// r.configRevisionMu.RLock()
 	result.Config = r.configRevision
-	r.configRevisionMu.RUnlock()
+	// r.configRevisionMu.RUnlock()
 
 	return result, nil
 }

robot/impl/local_robot_test.go

@@ -3541,7 +3541,6 @@ func TestMachineStatus(t *testing.T) {
 	})
 
 	t.Run("poll during reconfiguration", func(t *testing.T) {
-		t.Skip()
 		rev4 := "rev4"
 		lr := setupLocalRobot(t, ctx, &config.Config{
 			Revision:   rev4,
@@ -3556,13 +3555,14 @@ func TestMachineStatus(t *testing.T) {
 			defer wg.Done()
 			lr.Reconfigure(ctx, &config.Config{
 				Revision:   rev5,
-				Components: []resource.Config{newMockConfig("m", 300, false, "1s")},
+				Components: []resource.Config{newMockConfig("m", 300, false, "5s")},
 			})
 		}()
+		time.Sleep(time.Millisecond * 100)
 		// while reconfiguring
 		mStatus, err := lr.MachineStatus(ctx)
 		test.That(t, err, test.ShouldBeNil)
-		test.That(t, mStatus.Config.Revision, test.ShouldEqual, rev4)
+		test.That(t, mStatus.Config.Revision, test.ShouldEqual, rev5)
 		expectedStatuses := rtestutils.ConcatResourceStatuses(
 			getExpectedDefaultStatuses(rev4),
 			[]resource.Status{