You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/10/20 12:38:30 UTC

[GitHub] [cloudstack] borisstoyanov opened a new issue #5596: UI - As User - Broadcast URI column available to user, but blank

borisstoyanov opened a new issue #5596:
URL: https://github.com/apache/cloudstack/issues/5596


   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and main branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete the comments.
   -->
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Bug Report
   
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   UI
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on main branch.
   -->
   
   ~~~
   4.15
   ~~~
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   As User, at the networks page the table contains Broadcast URI field, but the user does not have access to the values through API. It would be good if the UI would crop the columns when that are not available/have values. Open for discussions. 
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] sureshanaparti closed issue #5596: UI - As User - Broadcast URI column available to user, but blank

Posted by GitBox <gi...@apache.org>.
sureshanaparti closed issue #5596:
URL: https://github.com/apache/cloudstack/issues/5596


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on issue #5596: UI - As User - Broadcast URI column available to user, but blank

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #5596:
URL: https://github.com/apache/cloudstack/issues/5596#issuecomment-948343068


   Since this is good to have, not exposing any data - moved to 4.16.1.0. Generally it's a good idea that except root-admin users aren't aware of their vlans, I'm not sure about other isolation methods. Maybe for some non-VLAN isolation method, exposing broadcase URI makes sense. This can be checked what we set/return in API at the backend, and if we don't for any case then simply hide/remove the column by checking user role type.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] sureshanaparti commented on issue #5596: UI - As User - Broadcast URI column available to user, but blank

Posted by GitBox <gi...@apache.org>.
sureshanaparti commented on issue #5596:
URL: https://github.com/apache/cloudstack/issues/5596#issuecomment-995828022


   Verified the fix in PR #5745.
   
   @borisstoyanov Closing this issue as the fix PR is merged. Reopen it if you still see this issue. Thanks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] weizhouapache commented on issue #5596: UI - As User - Broadcast URI column available to user, but blank

Posted by GitBox <gi...@apache.org>.
weizhouapache commented on issue #5596:
URL: https://github.com/apache/cloudstack/issues/5596#issuecomment-983467034


   The URI should not be visible for non-root users.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on issue #5596: UI - As User - Broadcast URI column available to user, but blank

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on issue #5596:
URL: https://github.com/apache/cloudstack/issues/5596#issuecomment-983430724


   we will have to decide to show or not show. In a mixed environment, i.e. with multiple isolation methods, we will end up with a mix of networks, some requiring the user to see the broadcast uri and other forbidding it.
   I suggest a setting allowing the operator to control it @rhtyd ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org