Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

List label values of allowed tenants only #1427

Merged
merged 1 commit into from
Jun 14, 2022
Merged

List label values of allowed tenants only #1427

merged 1 commit into from
Jun 14, 2022

Conversation

niksajakovljevic
Copy link
Contributor

If allowed tenants are configured we only list those when showing __tenant__ values.

Fixes #1194

@niksajakovljevic niksajakovljevic added the Bug Something isn't working label Jun 13, 2022
@niksajakovljevic niksajakovljevic requested a review from a team as a code owner June 13, 2022 09:41
@niksajakovljevic niksajakovljevic self-assigned this Jun 13, 2022
@niksajakovljevic niksajakovljevic requested a review from a team as a code owner June 13, 2022 10:31
@niksajakovljevic niksajakovljevic mentioned this pull request Jun 13, 2022
Merged
14 tasks
Harkishen-Singh
Harkishen-Singh suggested changes Jun 14, 2022
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
pkg/runner/runner.go Show resolved Hide resolved
@niksajakovljevic niksajakovljevic merged commit 307350b into master Jun 14, 2022
@niksajakovljevic niksajakovljevic deleted the niksa/list-only-allowed-tenants branch June 14, 2022 09:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@antekresic antekresic antekresic approved these changes

@Harkishen-Singh Harkishen-Singh Harkishen-Singh approved these changes

Assignees

@niksajakovljevic niksajakovljevic

Labels
Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

.../api/v1/label/__tenant__/values shows all tenants even though -multi-tenancy-valid-tenants=XYZ is set
3 participants
@niksajakovljevic @antekresic @Harkishen-Singh