Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-44844

netconfig_updater not managed properly by it's supervisor

    XMLWordPrintable

Details

    • Untriaged
    • 1
    • Unknown

    Description

      netconfig_updater has behaviour of gen_server but treated as a supervisor by it's supervisor ns_server_cluster_sup as here . 

      The code looks like it's been this way since inception

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          Build couchbase-server-6.6.2-9564 contains ns_server commit 9eeb672 with commit message:
          MB-44844: Start netconfig_updater as a worker ...

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.2-9564 contains ns_server commit 9eeb672 with commit message: MB-44844 : Start netconfig_updater as a worker ...

          Ritam Sharma - Agree. But if we need to produce another RC build, we recommend to include it this fix due to the issue it addresses.

          meni.hillel Meni Hillel (Inactive) added a comment - Ritam Sharma  - Agree. But if we need to produce another RC build, we recommend to include it this fix due to the issue it addresses.

          Timofey Barmin and Meni Hillel - Please help with step to validate.

          ritam.sharma Ritam Sharma added a comment - Timofey Barmin and Meni Hillel - Please help with step to validate.

          From Timofey Barmin comment - "hypothetically possible that node will take longer to shutdown if shutdown and change of netconfig settings (address family or node-to-node encryption) are being done simultaneously. But it will be very hard to test that." Ritam Sharma - can we try this scenario? If issue is not reproducible, we can should be able to close.

          meni.hillel Meni Hillel (Inactive) added a comment - From Timofey Barmin comment - "hypothetically possible that node will take longer to shutdown if shutdown and change of netconfig settings (address family or node-to-node encryption) are being done simultaneously. But it will be very hard to test that." Ritam Sharma - can we try this scenario? If issue is not reproducible, we can should be able to close.

          I think we should close it. There is not much to test.

          timofey.barmin Timofey Barmin added a comment - I think we should close it. There is not much to test.

          People

            timofey.barmin Timofey Barmin
            Abhijeeth.Nuthan Abhijeeth Nuthan
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty