Compare inodes for running and configured dirs

Summary:
On Devservers that are setup in a similar way as OD we use placeholder user to spin up Eden and clone repos, because of that the running directory looks like
```
/var/localhome/devenv-user-5f6c/local/.eden
```
The configured directory on the other hand looks like
```
${HOME}/local/.eden
```

When real user reserves a new machine and tries to run `eden doctor` it fails:
```
- Found problem:
The running configuration for /data/users/svcscm/configerator is different than "
the on-disk state in Eden's configuration file:
- Running state directory:    /var/localhome/devenv-user-5f6c/local/.eden/clients/configerator
- Configured state directory: /home/oresk/local/.eden/clients/configerator
Running `eden restart` will cause EdenFS to restart and use the data from the
on-disk configuration.
```

However it's not a real issue, because both directories are the same actually as that's because of how we setup real user on reservation, they still point to the same directory just via mount:
```
[[email protected] ~/fbsource/fbcode (387c3a67d)]$ findmnt -T /var/localhome/devenv-user-5f6c/local
TARGET SOURCE    FSTYPE OPTIONS
/      /dev/vda3 btrfs  rw,relatime,compress-force=zstd:3,ssd,discard,space_cache=v2,subvolid=5,subvol=/
[[email protected] ~/fbsource/fbcode (387c3a67d)]$ findmnt -T /home/oresk/local
TARGET      SOURCE                                     FSTYPE OPTIONS
/home/oresk /dev/vda3[/var/localhome/devenv-user-5f6c] btrfs  rw,relatime,compress-force=zstd:3,ssd,discard,space_cache=v2,subvolid=5,subvol=/
```

Let me know if it makes sense or I need to do extra testing on a different platform like Mac/Windows

Differential Revision: D73083956

fbshipit-source-id: c61087c48454b599e515b46d30217e56445f4100

Author: Orest Kyrylchuk

eden/fs/cli/doctor/__init__.py

@@ -880,6 +880,7 @@ def check_mount(
     elif checkout.state == MountState.RUNNING:
         try:
             check_running_mount(
+                out,
                 tracker,
                 instance,
                 checkout,
@@ -992,6 +993,7 @@ def check_starting_mount(
 
 
 def check_running_mount(
+    out: ui.Output,
     tracker: ProblemTracker,
     instance: EdenInstance,
     checkout_info: CheckoutInfo,
@@ -1004,6 +1006,28 @@ def check_running_mount(
         tracker.add_problem(CheckoutNotConfigured(checkout_info))
         return
     elif checkout_info.configured_state_dir != checkout_info.running_state_dir:
+        try:
+            configured_state_dir_inode = (
+                checkout_info.configured_state_dir.stat().st_ino
+                if checkout_info.configured_state_dir is not None
+                and checkout_info.configured_state_dir.exists()
+                else None
+            )
+            running_state_dir_inode = (
+                checkout_info.running_state_dir.stat().st_ino
+                if checkout_info.running_state_dir is not None
+                and checkout_info.running_state_dir.exists()
+                else None
+            )
+            if (
+                configured_state_dir_inode is not None
+                and configured_state_dir_inode == running_state_dir_inode
+            ):
+                return
+        except OSError as ex:
+            # If we fail to confirm they have the same inode, we'll just
+            # assume they're different and report the problem.
+            out.write(f"Error reading inode for state directories: {ex}", flush=True)
         tracker.add_problem(CheckoutConfigurationMismatch(checkout_info))
         return
 

eden/fs/cli/doctor/test/doctor_test.py

@@ -2559,6 +2559,7 @@ def test_missing_config_fix(
         os.unlink(checkout.state_dir / "config.toml")
 
         check_running_mount(
+            out,
             fixer,
             # pyre-fixme[6]: For 2nd param expected `EdenInstance` but got
             # `FakeEdenInstance`.
@@ -2621,6 +2622,7 @@ def test_corrupted_config_fix(
             print(f.read())
 
         check_running_mount(
+            out,
             fixer,
             # pyre-fixme[6]: For 2nd param expected `EdenInstance` but got
             # `FakeEdenInstance`.
@@ -2681,6 +2683,7 @@ def test_corrupted_config_fail(
         watchman_info = check_watchman.WatchmanCheckInfo(watchman_roots)
 
         check_running_mount(
+            out,
             fixer,
             # pyre-fixme[6]: For 2nd param expected `EdenInstance` but got
             # `FakeEdenInstance`.
@@ -2753,6 +2756,7 @@ def test_corrupted_config_fail_oss(
         watchman_info = check_watchman.WatchmanCheckInfo(watchman_roots)
 
         check_running_mount(
+            out,
             fixer,
             # pyre-fixme[6]: For 2nd param expected `EdenInstance` but got
             # `FakeEdenInstance`.
@@ -2815,6 +2819,7 @@ def test_corrupted_snapshot(
         watchman_info = check_watchman.WatchmanCheckInfo(watchman_roots)
 
         check_running_mount(
+            out,
             fixer,
             # pyre-fixme[6]: For 2nd param expected `EdenInstance` but got
             # `FakeEdenInstance`.