Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG-2.11] UI prompts tenant selection at every first login #1717

Closed
hengirmen opened this issue Jan 4, 2024 · 11 comments · Fixed by #1759
Closed

[BUG-2.11] UI prompts tenant selection at every first login #1717

hengirmen opened this issue Jan 4, 2024 · 11 comments · Fixed by #1759
Assignees
Labels
bug Something isn't working triaged ux / ui User Experience / Inferface related issues v2.12.0 Items targeting 2.12.0

Comments

@hengirmen
Copy link

hengirmen commented Jan 4, 2024

What is the bug?

  • Whenever the OpenSearch Dashboards UI is accessed on a new machine "Select your tenant" window pops up, even though multi-tenancy is disabled. We don't want the customers and common users to see this prompt since it could be confusing.

  • Also, when the multi-tenancy is enabled and default tenant is set to "Private", UI still prompts tenant selection.

How can one reproduce the bug?
Steps to reproduce the behavior:

  1. Open OpenSearch Dashboards UI in a browser.
  2. Log in with user credentials.
  3. UI prompts the user to select the tenant.

What is the expected behavior?

  • When multi-tenancy is disabled, we expect the users to access the dashboards directly without facing this prompt.

  • Also when the multi-tenancy is enabled and a default tenant is set, we don't want the users to see the tenant selection window.

Do you have any screenshots?
Figure 1: Multi-tenancy is disabled, yet the prompt to set tenants still pops up.
image

Figure 2: Multi-tenancy is enabled, default tenant is set, yet the prompt pops up with every first login.
image

@hengirmen hengirmen added bug Something isn't working untriaged labels Jan 4, 2024
@hengirmen hengirmen changed the title [BUG-2.11] UI prompts tenant selection at every login even though multi-tenancy is disabled [BUG-2.11] UI prompts tenant selection at every first login even though multi-tenancy is disabled Jan 4, 2024
@hengirmen hengirmen changed the title [BUG-2.11] UI prompts tenant selection at every first login even though multi-tenancy is disabled [BUG-2.11] UI prompts tenant selection at every first login Jan 4, 2024
@stephen-crawford
Copy link
Contributor

[Triage] Hi @hengirmen, thank you for filing this issue and the detailed info. Someone will reach out for some UX/UI guidance from another project member and hopefully we can improve this for you! You are also encouraged to open a pull request with the change in behavior should you be interested in seeing this change expedited.

@stephen-crawford stephen-crawford added ux / ui User Experience / Inferface related issues triaged and removed untriaged labels Jan 8, 2024
@hengirmen
Copy link
Author

Hi @scrawfor99 , thanks for the response. Any updates yet?

@peternied
Copy link
Member

peternied commented Jan 19, 2024

Thanks for checking in @hengirmen, my team is entering it a new sprint next week and I've earmarked this issue. More updates forthcoming...

@wbeckler
Copy link

wbeckler commented Feb 1, 2024

Will this be backported into prior versions that include the bug? Also, what is the workaround for someone who has security enabled and wants to suppress the popup? Is it enough to map a role to a default tenant?

@cwperks
Copy link
Member

cwperks commented Feb 1, 2024

@wbeckler There is no planned patch releases for any of the impacted branches so while it can be backported to those branches, it would never be officially released. This bug impacts versions 2.7, 2.8, 2.9, 2.10 and 2.11.

The workaround is to disable multi-tenancy through opensearch_dashboards.yml instead of disabling through the UI dynamically.

> opensearch_dashboards.yml

opensearch_security.multitenancy.enabled: false

Please note that this settings acts as a feature flag and to re-enable multi-tenancy would require a reboot of OSD. When this setting is set to false, you cannot enable multi-tenancy through the Security pages in OSD.

Screenshot 2024-02-01 at 9 45 43 AM

@wbeckler
Copy link

wbeckler commented Feb 1, 2024

Can this be backported to at least 2.11, so that future patches will include that fix?

@DarshitChanpura
Copy link
Member

DarshitChanpura commented Feb 1, 2024

@wbeckler the change is ready to be backported to 2.11: #1765
Is there a planned patch release for 2.11.1?

@cwperks
Copy link
Member

cwperks commented Feb 1, 2024

Is there a planned patch release for 2.11.1

2.11.1 is already released. There is no current plan for 2.11.2. Generally only security fixes are included for patch releases.

@DarshitChanpura
Copy link
Member

oops.. I meant 2.11.2

@davidlago
Copy link

Once 2.12 goes out, there will not be 2.11 patches anyway.

@DarshitChanpura
Copy link
Member

In that case I will close: #1765

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triaged ux / ui User Experience / Inferface related issues v2.12.0 Items targeting 2.12.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants