Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: future release
    • Component/s: configuration
    • Labels:
      None
    • Environment:

      build: b38, v3.1

      Description

      When creating a config node, Node Host is not a required field and node can be create without it. However, if user enters a host name and creates a node, if user then wants to clear this field, the following error is displayed on Save:

      An error has occurred
      javax.validation.ConstraintViolationException: Constraints for this Node configuration have been violated: on property [ nodeHost ] violation reason [ Invalid nodehost name. Name must start with a letter or number and may contain only letters, numbers, and certain other characters. ]

      User should be able to clear node host field.

        Activity

        Hide
        Anissa Lam added a comment -

        The restriction comes from the backend.
        Assign to Joe to evaluate.

        Show
        Anissa Lam added a comment - The restriction comes from the backend. Assign to Joe to evaluate.
        Hide
        Joe Di Pol added a comment -

        This is being caused by a bean validation constraint that does not allow an empty string to be set as the host name:

        @Pattern(regexp=NAME_REGEX, message="

        {nodehost.invalid.name}

        ", payload=Node.class)
        String getNodeHost();

        where NAME_REGEX = "[A-Za-z0-9_][A-Za-z0-9\\-_\\./;#]*";

        Show
        Joe Di Pol added a comment - This is being caused by a bean validation constraint that does not allow an empty string to be set as the host name: @Pattern(regexp=NAME_REGEX, message=" {nodehost.invalid.name} ", payload=Node.class) String getNodeHost(); where NAME_REGEX = " [A-Za-z0-9_] [A-Za-z0-9\\-_\\./;#] *";
        Hide
        Joe Di Pol added a comment -

        Not a must fix for the next update release. Lowering priority.

        Show
        Joe Di Pol added a comment - Not a must fix for the next update release. Lowering priority.

          People

          • Assignee:
            Joe Di Pol
            Reporter:
            lidiam
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: