Project

General

Profile

Actions

Bug #27498

open

Make group tree API not include system by default

Added by Clark ANDRIANASOLO 2 days ago. Updated about 18 hours ago.

Status:
Pending technical review
Priority:
1 (highest)
Category:
API
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Small
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
No

Description

In #27486, an API query parameter includeSystem was introduced in /groups/tree, and for compatibility (in a patch release) it was set to true
But in the directives API the behavior is to not include system directives/techniques by default.
So it makes sense to not have system elements by default in our API (since they cannot really be acted on).

We need to change the default in the new 9.0 API version (upon merge of #27486).
We also need to change API calls in our UI to include the system groups where needed.


Subtasks 1 (1 open0 closed)

Rudder plugins - Bug #27503: Use groups API to include system groups in private pluginsPending technical reviewClark ANDRIANASOLOActions

Related issues 1 (1 open0 closed)

Related to Rudder - Enhancement #27486: Add includeSystem parameter to filter system groups in APIPending technical reviewFrançois ARMANDActions
Actions #1

Updated by Clark ANDRIANASOLO 2 days ago

  • Related to Enhancement #27486: Add includeSystem parameter to filter system groups in API added
Actions #2

Updated by Clark ANDRIANASOLO 1 day ago

  • Status changed from New to In progress
Actions #3

Updated by Clark ANDRIANASOLO 1 day ago

  • Description updated (diff)
  • Status changed from In progress to New
Actions #4

Updated by Clark ANDRIANASOLO about 18 hours ago

  • Subtask #27503 added
Actions #5

Updated by Clark ANDRIANASOLO about 18 hours ago

  • Status changed from New to Pending technical review
  • Pull Request set to https://github.com/Normation/rudder/pull/6579
Actions

Also available in: Atom PDF