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.

Sign up for GitHub

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

Jump to bottom

🐞 Multi-user use of MTV results in a warning about Inventory Server not reachable #1293

Closed
wkulhanek opened this issue Jul 23, 2024 · 1 comment
Closed

🐞 Multi-user use of MTV results in a warning about Inventory Server not reachable #1293

wkulhanek opened this issue Jul 23, 2024 · 1 comment
Assignees
@sgratch
Labels
bug Categorizes issue or PR as related to a bug.
Milestone
2.6.6

Comments

@wkulhanek
Copy link

wkulhanek commented Jul 23, 2024 •
edited
Loading

Screenshot 2024-07-23 at 11 35 11

Deploying MTV into openshift-mtv and then allowing regular users to migrate VMs results in the screenshot above. There is an error about the inventory server not being reachable.

Steps to reproduce:

  • install MTV as normal (2.6) in the openshift-mtv namespace
  • Configure ForkliftController as normal in the openshift-mtv namespace
  • Create a project for a regular (non admin) user - e.g. mtv-user1
  • In that project create two Providers, a host provider and a VMware provider (the second may not be necessary)
---
apiVersion: forklift.konveyor.io/v1beta1
kind: Provider
metadata:
  name: host
  namespace: mtv-user1
spec:
  secret: {}
  type: openshift
  url: ""
  • In the OpenShift console log in as the "normal" user (e.g. user1)
  • Navigate to Migrations (there's a problem on that screen as well... it never loads)
  • Select the mtv-user1 project
  • You will see the error.
@yaacov yaacov added the bug Categorizes issue or PR as related to a bug. label Jul 23, 2024
@yaacov yaacov added this to the 2.6.4 milestone Jul 23, 2024
@yaacov yaacov changed the title Multi-user use of MTV results in a warning about Inventory Server not reachable 🐞 Multi-user use of MTV results in a warning about Inventory Server not reachable Jul 23, 2024
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 14, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
1376b95 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
@sgratch sgratch mentioned this issue Aug 14, 2024
Merged
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 18, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
e13fc99 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 21, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
431ff47 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 21, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
9fdf819 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 21, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
c7fd8a0 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 22, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
42c88ec 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
@yaacov yaacov modified the milestones: 2.6.4, 2.6.5 Aug 27, 2024
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 28, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
61d9e44 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 29, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
7d003eb 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 29, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
f253582 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 29, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
4506a7b 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 29, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
8bb7e83 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 29, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
00551d2 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
sgratch added a commit to sgratch/forklift-console-plugin that referenced this issue Aug 29, 2024
@sgratch
Handle fetching from inventory for users that don't have permissions …
cc5f48d 
…to list all namespaces

Reference: kubev2v#1293

Handle providers fetching from inventory for users with limited namespace roles, who can only list their namespaces content
and don't have get permissions to all namespaces.

Before the fix, when those users tried to fetch providers inventory data, regarldess to
current namespace, the "inventory server is not reachable" error was always displayed.

After this fix, those users will not fetch all namepsaces but only
ones which they allow to list. So the fetching should now succeed and no
error should be displayed.

Signed-off-by: Sharon Gratch <[email protected]>
@yaacov yaacov modified the milestones: 2.6.5, 2.6.6 Sep 3, 2024
@yaacov
Copy link
Member

yaacov commented Sep 3, 2024

Addressed by #1307

@yaacov yaacov closed this as completed Sep 3, 2024
@yaacov yaacov modified the milestones: 2.6.6, 2.7.0 Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgratch sgratch

Labels
bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
2.6.6
Development

No branches or pull requests
3 participants
@yaacov @wkulhanek @sgratch