You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We should probably revisit the behavior and the interfaces (already slightly modified into such PR).
Furthermore HTTP test suite is pretty thin and this needs to be addressed too.
As note for future reference I copy here what emerged during chat with @desa.
/api/v2/telegrafs?id=1&id=2 -> should return telegrafs 1 and 2 /api/v2/telegrafs?orgID=5 -> all telegrafs that have organizationID 5 /api/v2/telegrafs?ownerID=10 -> all telegrafs that belong to that user /api/v2/telegrafs?orgID=5&ownerID=10 -> all telegrafs from org 5 that belong to user 10 /api/v2/telegrafs?orgID=5&id=1 -> errors out, as id and orgID should be mutually exclusive
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically closed because it has not had recent activity. Please reopen if this issue is still important to you. Thank you for your contributions.
Doing https://github.com/influxdata/platform/pull/2376 in order to scope telegraf configs to an organization I noticed that we should probably clarify better what their intended behavior is.
We should probably revisit the behavior and the interfaces (already slightly modified into such PR).
Furthermore HTTP test suite is pretty thin and this needs to be addressed too.
As note for future reference I copy here what emerged during chat with @desa.
/api/v2/telegrafs?id=1&id=2
-> should return telegrafs 1 and 2/api/v2/telegrafs?orgID=5
-> all telegrafs that have organizationID 5/api/v2/telegrafs?ownerID=10
-> all telegrafs that belong to that user/api/v2/telegrafs?orgID=5&ownerID=10
-> all telegrafs from org 5 that belong to user 10/api/v2/telegrafs?orgID=5&id=1
-> errors out, as id and orgID should be mutually exclusivePR https://github.com/influxdata/platform/pull/2376 does not sync swagger for these exact reasons.
The text was updated successfully, but these errors were encountered: