Project

General

Profile

Actions

Bug #24867

open

No error toast when a pending node can't be accepted because hostname already exists

Added by François ARMAND 7 months ago. Updated 6 days ago.

Status:
New
Priority:
N/A
Category:
Web - Nodes & inventories
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I dislike using that feature
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Very Small
Priority:
78
Name check:
To do
Fix check:
To do
Regression:
No

Description

If we try to accept a pending node while its hostname matches a node already accepted, the node is refused and we have in logs:

2024-05-16 13:17:09+0000 ERROR nodes.pending - Error when trying to accept node with id 'b522c78a-3dd6-48c0-b242-71da15c99109': Inconsistency: There is already a node with hostname 'node1.rudder.local' in Rudder. You can not add it again.

This is expected and that behavior can be change with the setting:

curl -k  -H "X-API-Token: xxxxx" -H "Content-Type: application/json" -X POST 'http://localhost:8082/rudder-web/api/latest/settings/node_accept_duplicated_hostname' -d '{"value":true}'

But we don't have an error notification in the UI


Files

clipboard-202405161626-lnle7.png (22.5 KB) clipboard-202405161626-lnle7.png François ARMAND, 2024-05-16 16:26
Actions #1

Updated by François ARMAND 7 months ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions #2

Updated by François ARMAND 7 months ago

  • File clipboard-202405161626-lnle7.png clipboard-202405161626-lnle7.png added
  • Status changed from In progress to New
  • Assignee changed from François ARMAND to Raphael GAUTHIER
  • Severity set to Minor - inconvenience | misleading | easy workaround
  • UX impact set to I dislike using that feature
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Effort required set to Very Small
  • Priority changed from 0 to 85

OK, so the error is their, it's just that its style disappeared in 8.1:

<div class="tab-pane active" id="review_new_server" role="tabpanel">
                    <div class="callout-fade callout-info">
                      <div class="marker">
                        <span class="fa fa-info-circle"></span>
                      </div>
                      <p>Click on a node name to view detailed inventory information.</p>
                      <p>Click on <i class="fa fa-search"></i>to list Rules that would be applied to this node if you accept it.</p>
                    </div>
                    <div id="lift__noticesContainer__"><div id="lift__noticesContainer___error"><ul><li><span>
            Inconsistency: There is already a node with hostname 'node1.rudder.local' in Rudder. You can not add it again.
          </span></li></ul></div></div>
                    <div>
                      <div>

  <div id="acceptNodeGrid_wrapper" class="dataTables_wrapper no-footer"><div class="dataTables_wrapper_top">
....

Actions #3

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 8.1.3 to 8.1.4
Actions #4

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 8.1.4 to 8.1.5
Actions #5

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 8.1.5 to 8.1.6
Actions #6

Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 8.1.6 to 8.1.7
  • Priority changed from 85 to 83
Actions #7

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 8.1.7 to 8.1.8
  • Priority changed from 83 to 81
Actions #8

Updated by Vincent MEMBRÉ about 1 month ago

  • Target version changed from 8.1.8 to 8.1.9
  • Priority changed from 81 to 79
Actions #9

Updated by Vincent MEMBRÉ 10 days ago

  • Target version changed from 8.1.9 to 8.1.10
  • Priority changed from 79 to 78
Actions #10

Updated by Vincent MEMBRÉ 6 days ago

  • Target version changed from 8.1.10 to 8.1.11
Actions

Also available in: Atom PDF