Skip to content

Commit

Permalink
Refactor: pacemaker-schedulerd: don't add error status to IPC reply
Browse files Browse the repository at this point in the history
Nothing uses these attributes and apparently never did
  • Loading branch information
kgaillot committed Jun 13, 2024
1 parent bcdce66 commit 2b7245c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
4 changes: 0 additions & 4 deletions daemons/schedulerd/schedulerd_messages.c
Original file line number Diff line number Diff line change
Expand Up @@ -148,10 +148,6 @@ handle_pecalc_request(pcmk__request_t *request)
}

crm_xml_add(reply, PCMK__XA_CRM_TGRAPH_IN, filename);
crm_xml_add_int(reply, PCMK__XA_GRAPH_ERRORS, was_processing_error);
crm_xml_add_int(reply, PCMK__XA_GRAPH_WARNINGS, was_processing_warning);
crm_xml_add_int(reply, PCMK__XA_CONFIG_ERRORS, crm_config_error);
crm_xml_add_int(reply, PCMK__XA_CONFIG_WARNINGS, crm_config_warning);

pcmk__log_transition_summary(filename);

Expand Down
4 changes: 0 additions & 4 deletions include/crm/common/xml_names_internal.h
Original file line number Diff line number Diff line change
Expand Up @@ -140,8 +140,6 @@ extern "C" {
#define PCMK__XA_CIB_USER "cib_user"
#define PCMK__XA_CLIENT_NAME "client_name"
#define PCMK__XA_CLIENT_UUID "client_uuid"
#define PCMK__XA_CONFIG_ERRORS "config-errors"
#define PCMK__XA_CONFIG_WARNINGS "config-warnings"
#define PCMK__XA_CONFIRM "confirm"
#define PCMK__XA_CONNECTION_HOST "connection_host"
#define PCMK__XA_CONTENT "content"
Expand All @@ -162,8 +160,6 @@ extern "C" {
#define PCMK__XA_ELECTION_ID "election-id"
#define PCMK__XA_ELECTION_OWNER "election-owner"
#define PCMK__XA_GRANTED "granted"
#define PCMK__XA_GRAPH_ERRORS "graph-errors"
#define PCMK__XA_GRAPH_WARNINGS "graph-warnings"
#define PCMK__XA_HIDDEN "hidden"
#define PCMK__XA_HTTP_EQUIV "http-equiv"
#define PCMK__XA_IN_CCM "in_ccm"
Expand Down

0 comments on commit 2b7245c

Please sign in to comment.