What is the reason the Open VI Ref needs to be in the loop?
As it sits, every time the Open VI Ref runs, it generates a different refnum for the same VI. Place a probe on the refnum wire and you'll see it change each time the loop iterates. So each time it checks the Execution State, it's checking a different instance of the VI.
Unless this is what you want, you should move the Open Reference outside the loop so you check the same refnum each time.
Ed

Ed Dickens - Certified LabVIEW Architect
Lockheed Martin Space
Using the Abort button to stop your VI is like using a tree to stop your car. It works, but there may be consequences.