Ambari
  1. Ambari
  2. AMBARI-3733

Register Host: need to check host OS type if specific os group been selected at Advanced Repo in step1

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.4.2
    • Fix Version/s: 1.4.2
    • Component/s: ambari-web
    • Labels:
      None

      Description

      Since we are introducing the concept of an "enabled" repo (in #1), during host registration (on the next page of the wizard), we should also confirm based on the hosts that register that their OS repo is valid. So after the host registers, we should check the repos API that for that OS, we have a valid repo. If not, we shouldn't let them continue. We should inform the user to either a) remove the hosts since they do not have access to a repository with packages for that OS or b) go back and enter the repo URL for that OS.

      1. AMBARI-3733.patch
        10 kB
        Xi Wang
      2. AMBARI-3733_2.patch
        4 kB
        Xi Wang

        Activity

        Hide
        Yusaku Sako added a comment -

        1. Please externalize the string "Repository for OS not available" to messages.js.
        2. Please remove references to "Step 1" in the messages, as "Step 1" has no meaning to the end user. I'm referring to:

        +  'installer.step3.hostOsTypeCheck.popup.body' : 'Host registered successfully, but the host operating system type NOT match the selected group in step 1:Advanced Repository Option.<br> You can go back to step 1 OR remove this host. <br>' +
        +    'The host type is {0}, but you selected group {1} in step 1.',
        
        Show
        Yusaku Sako added a comment - 1. Please externalize the string "Repository for OS not available" to messages.js. 2. Please remove references to "Step 1" in the messages, as "Step 1" has no meaning to the end user. I'm referring to: + 'installer.step3.hostOsTypeCheck.popup.body' : 'Host registered successfully, but the host operating system type NOT match the selected group in step 1:Advanced Repository Option.<br> You can go back to step 1 OR remove this host. <br>' + + 'The host type is {0}, but you selected group {1} in step 1.',
        Hide
        Xi Wang added a comment -

        Committed to trunk.

        Show
        Xi Wang added a comment - Committed to trunk.
        Hide
        Xi Wang added a comment -

        Accidentally committed before review, so made a second patch to address the review comments. We need both patches.

        Show
        Xi Wang added a comment - Accidentally committed before review, so made a second patch to address the review comments. We need both patches.
        Hide
        Yusaku Sako added a comment -

        Thanks.
        +1 for the patches.

        Show
        Yusaku Sako added a comment - Thanks. +1 for the patches.
        Hide
        Xi Wang added a comment -

        Committed all patches to trunk.

        Show
        Xi Wang added a comment - Committed all patches to trunk.

          People

          • Assignee:
            Xi Wang
            Reporter:
            Xi Wang
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development