Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-3885

enhance stability of clusterNodeInfo's machineId

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.0.25, 1.2.9, 1.3.13
    • 1.0.26, 1.2.10, 1.3.14, 1.4
    • documentmk
    • None

    Description

      We currently use network interface information to derive a unique machine ID (ClusterNodeInfo.getMachineId()). Among the 6-byte addresses, we use the "smallest".

      At least on Windows machines, connecting through a VPN inserts a new low machineID into the list, causing the machineID to vary depending on whether the VPN is connected or not.

      I don't see a clean way to filter these addresses. We could inspect the names of the interfaces and treat those containing "VPN" or "Virtual" to be less relevant. Of course that would be an ugly hack, but it would fix the problem for now.

      Attachments

        1. OAK-3885.diff
          2 kB
          Julian Reschke

        Issue Links

          Activity

            reschke Julian Reschke added a comment -

            Proposed change.

            reschke Julian Reschke added a comment - Proposed change.

            Agreed, the solution is not really nice, but if it solves the immediate problem I'm fine with the change.

            mreutegg Marcel Reutegger added a comment - Agreed, the solution is not really nice, but if it solves the immediate problem I'm fine with the change.

            Bulk close for 1.3.14

            edivad Davide Giannella added a comment - Bulk close for 1.3.14
            reschke Julian Reschke added a comment -

            1.4: (1.3.14) r1725216
            1.2: (1.2.10) r1725270 r1725229
            1.0: (1.0.26) r1725272 r1725238

            reschke Julian Reschke added a comment - 1.4: (1.3.14) r1725216 1.2: (1.2.10) r1725270 r1725229 1.0: (1.0.26) r1725272 r1725238

            People

              reschke Julian Reschke
              reschke Julian Reschke
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: