Skip to content

Commit

Permalink
fix changelog
Browse files Browse the repository at this point in the history
  • Loading branch information
QuentinBisson committed Dec 17, 2024
1 parent c945501 commit e42cca1
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed

- Use `giantswarm` tenant by default instead of cluster name.
- Replace multi-tenant-proxy with ingress auth map.
- Replace multi-tenant-proxy with ingress auth map on CAPI.

### Removed

Expand Down
7 changes: 6 additions & 1 deletion pkg/resource/proxy-auth/proxy-auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -67,12 +67,17 @@ func GenerateProxyAuthSecret(lc loggedcluster.Interface, credentialsSecret *v1.S
return v1.Secret{}, errors.WithStack(err)
}

tenant := writeUser
if lc.IsCAPI() {
tenant = lc.GetTenant()
}

authCfg.Users = append(authCfg.Users, config.User{
Username: writeUser,
Password: writePassword,
// we set the default tenant even though it may be given by the sender
// depending of grafana-multi-teant-proxy config
OrgID: common.DefaultWriteTenant,
OrgID: tenant,
})

// Add write user to allowed tenants for read user
Expand Down

0 comments on commit e42cca1

Please sign in to comment.