[GitHub] [cloudstack-primate] rhtyd opened a new issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

rhtyd opened a new issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528


   List of issues raised by Alex wrt https://github.com/apache/cloudstack-primate/issues/519
   
   - Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier
   - Fix button name in VM deployment form in its network section to "Create New Network"
   - Hide the 'create private gateway' button for Users in VPC view (users not allowed to create private gateway)
   - Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)
   - Don't allow users in UI to delete/archive events
   - Fix the add and edit users form to show timezone as list of searchable dropdown like in the add account form
   - From VRs or systemvms, click on compute offering the URL/path is not added with ?issystem=true, and show an empty resource view
   
   Usability (optional?):
   - Show buttons with labels on resource view when the number of action buttons are very few                                                                                   
   - Add resources tab like from zone, but for pod and cluster (listCapacity)
   - All icons UI-wide look hanging at the bottom in Firefox but not Chrome (could be AntD bug)
   - Whether we should show the reset password button for all VMs, including those not password enabled? (hidden in Primate based on password enabled option)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   - Show HA provider only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported 
   - Missing capacity fields while creating primary storage
   - Contextual help is missing for creating service offerings
   - When migrating a vm / systemvm, exclude the current host from the list
   - While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search settings for account / domain
   - When editing a role, resource admin is not in the 


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Template actions not showing while accessing a project template~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   - ~Add loading while fetching tags and annotations (OK, David - Infocard)~
   - ~Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`~
   - ~Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?) - It is the same as state : `metricsResponse.setState(zoneResponse.getAllocationState());`~
   
   TO FIX :
   - Deploy VM doesn't show SG when a zone has Security Groups enabled
   - Can't take a vm storage snap in the vm detail view
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - List VPC and other offerings based on zone in create vpc form
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] utchoang edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

utchoang edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661708393


   @davidjumani @rhtyd 
   > - Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)
   
   This means that if there is no zone with SG enabled, `networks> security group` will not be displayed?
   
   > - Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier [preferably the former, to add existing vms]
   
   I still don't know this requirement very well, can you describe it more clearly?


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api
   - Missing capacity fields while creating primary storage
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search in account / domain level settings
   - When editing a role, resource admin is not in the dropdown
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - VR router offering link leads to an empty page
   - In add ACL Rule, the Rule# field should be a number, not text
   - In the migrate snapshot to different storage, exclude the current host storage
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] rhtyd commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

rhtyd commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656586024


   With @davidjumani - please check and add in comments if I missed anything


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] rhtyd commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

rhtyd commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-665541229


   okay @davidjumani 


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] utchoang edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

utchoang edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661708393


   @davidjumani @rhtyd 
   > - Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)
   This means that if there is no zone with SG enabled, `networks> security group` will not be displayed?
   
   > - Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier [preferably the former, to add existing vms]
   I still don't know this requirement very well, can you describe it more clearly?


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] utchoang commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

utchoang commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661708393


   @davidjumani @rhtyd 
   - Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)
   >> This means that if there is no zone with SG enabled, `networks> security group` will not be displayed?
   
   - Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier [preferably the former, to add existing vms]
   >> I still don't know this requirement very well, can you describe it more clearly?


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Backend Fixes :
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Pagination broken on roles - Broken in the backend
   - ListVMMetrics doesn't respect hostid param
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Shows only one template setting if it is within a project : Related to showunique flag
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Shows only one template setting if it is within a project : Related to showunique flag
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Backend Fixes :
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Pagination broken on roles - Broken in the backend
   - Search by tag doesn't work for vm snapshots
   - ListVMMetrics doesn't respect hostid param
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Backend Fixes :
   - Pagination broken on roles - Broken in the backend
   - ListVMMetrics doesn't respect hostid param
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Refresh after successfully uploading an iso (OK, David to check and call parent refresh)
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-665531367


   @rhtyd Can close this. Have created another issue of things todo from this


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]??
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Refresh after successfully uploading an iso (OK, David to check and call parent refresh)
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Backend Fixes :
   - ListVMMetrics doesn't respect hostid param
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   - Add loading while fetching tags and annotations
   - Root Domain shows as '/' in list & detail view infocard
   - Report bug at the bottom should redirect to bug or just say provide feedback
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms)
   - Why do IPs always point to the resource ?
   - Account in user info card doesn't directly go to account but to the account listview ?
   
   Mobile View :
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - In add ACL Rule, the Rule# field should be a number, not text
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661712601


   > This means that if there is no zone with SG enabled, `networks> security group` will not be displayed?
   
   @utchoang Exactly!
   
   > I still don't know this requirement very well, can you describe it more clearly?
   ![Screenshot from 2020-07-21 13-52-29](https://user-images.githubusercontent.com/8244774/88030989-945ef380-cb59-11ea-9473-a953a9268582.png)
   
   In the VPC View > Networks > Instances, when 'Add Instance' is selected, preferably open a new component to select a VM from the existing vms. If that would be too time-consuming, just rename it to deploy vm
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search in account / domain level settings
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - In add ACL Rule, the Rule# field should be a number, not text
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - In add ACL Rule, the Rule# field should be a number, not text
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]??
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Refresh after successfully uploading an iso (OK, David to check and call parent refresh)
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   - Missing capacity fields while creating primary storage
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search in account / domain level settings
   - When editing a role, resource admin is not in the dropdown
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - VR router offering link leads to an empty page
   - In add ACL Rule, the Rule# field should be a number, not text
   - In the migrate snapshot to different storage, exclude the current host storage
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Backend Fixes :
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Pagination broken on roles - Broken in the backend
   - Search by tag doesn't work for vm snapshots
   - ListVMMetrics doesn't respect hostid param
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Shows only one template setting if it is within a project : Related to showunique flag
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] rhtyd commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

rhtyd commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-664251031


   @davidjumani can you strike off anything from the list that got merged from the consolidate bugfix PR


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]??
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   - Add loading while fetching tags and annotations
   - Root Domain shows as '/' in list & detail view infocard
   - Report bug at the bottom should redirect to bug or just say provide feedback
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms)
   - Why do IPs always point to the resource ?
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   - Refresh after successfully uploading an iso
   - Make external Id required for backup offerings
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate
   
   
   Mobile View :
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Backend Fixes :
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Template actions not showing while accessing a project template~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   - ~Add loading while fetching tags and annotations (OK, David - Infocard)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - List VPC and other offerings based on zone in create vpc form
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   - ~Add loading while fetching tags and annotations (OK, David - Infocard)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - List VPC and other offerings based on zone in create vpc form
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661687110


   @utchoang Could you work on the following :
   - Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)
   - Fix the add and edit users form to show timezone as list of searchable dropdown like in the add account form
   - Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier [preferably the former, to add existing vms]
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661712601


   > This means that if there is no zone with SG enabled, `networks> security group` will not be displayed?
   
   @utchoang Exactly!
   
   > I still don't know this requirement very well, can you describe it more clearly?
   ![Screenshot from 2020-07-21 13-52-29](https://user-images.githubusercontent.com/8244774/88030989-945ef380-cb59-11ea-9473-a953a9268582.png)
   
   In the VPC View > Networks > Instances, when 'Add Instance' is selected, preferably open a component to select a VM from the existing vms. If that would be too time-consuming, just rename it to deploy vm
   Could extract the modal from LoadBalancing.vue and reuse it


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Refresh after successfully uploading an iso (OK, David to check and call parent refresh)
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] rhtyd closed issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

rhtyd closed issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528


   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656986237


   ~While in a project view, the account column in the list view / info card shows the project account. It should be changed to project itself.
   This is seen in ssh keys, instance groups, private gateways, vpn gateways, and perhaps others too~
   Fixed in #539
   
   ![Screenshot from 2020-07-11 09-57-13](https://user-images.githubusercontent.com/8244774/87216421-2ea28880-c35d-11ea-87b7-fe7ce064ab57.png)
   ![Screenshot from 2020-07-11 09-57-24](https://user-images.githubusercontent.com/8244774/87216424-2fd3b580-c35d-11ea-8a8f-96477811fca0.png)
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656986237


   While in a project view, the account column in the list view / info card shows the project account. It should be changed to project itself.
   This is seen in ssh keys, instance groups, private gateways, vpn gateways, and perhaps others too
   
   ![Screenshot from 2020-07-11 09-57-13](https://user-images.githubusercontent.com/8244774/87216421-2ea28880-c35d-11ea-87b7-fe7ce064ab57.png)
   ![Screenshot from 2020-07-11 09-57-24](https://user-images.githubusercontent.com/8244774/87216424-2fd3b580-c35d-11ea-8a8f-96477811fca0.png)
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported 
   - Missing capacity fields while creating primary storage
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Show host HA only for KVM since that's the only one available
   - When migrating a vm / systemvm, exclude the current host from the list
   - While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search settings for account / domain
   - When editing a role, resource admin is not in the dropdown
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate 


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Template actions not showing while accessing a project template~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   - ~Add loading while fetching tags and annotations (OK, David - Infocard)~
   - ~Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`~
   
   TO FIX :
   - Deploy VM doesn't show SG when a zone has Security Groups enabled
   - Can't take a vm storage snap in the vm detail view
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - List VPC and other offerings based on zone in create vpc form
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]??
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMs rather than metrics since it doesn't respect hostid param
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   - Add loading while fetching tags and annotations
   - Root Domain shows as '/' in list & detail view infocard
   - Report bug at the bottom should redirect to bug or just say provide feedback
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms)
   - Why do IPs always point to the resource ?
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   - Refresh after successfully uploading an iso
   - Make external Id required for backup offerings
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate
   
   
   Mobile View :
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   - ~Add loading while fetching tags and annotations (OK, David - Infocard)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported 
   - Missing capacity fields while creating primary storage
   - Contextual help is missing for creating service offerings
   - When migrating a vm / systemvm, exclude the current host from the list
   - While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search settings for account / domain
   - When editing a role, resource admin is not in the 


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Missing capacity fields while creating primary storage
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Template actions not showing while accessing a project template~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   - ~Add loading while fetching tags and annotations (OK, David - Infocard)~
   - ~Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`~
   
   TO FIX :
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - List VPC and other offerings based on zone in create vpc form
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions) 
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template adding setting removes the prior existing setting after refresh for xen & vmware : Relate to showunique flag
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Deleteing all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMMetrics doesn't respect hostid param
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   - Refresh after successfully uploading an iso (OK, David to check and call parent refresh)
   - Make external Id required for backup offerings (David to check and remove/address the issue)
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   TO FIX :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   - Add loading while fetching tags and annotations
   - Root Domain shows as '/' in list & detail view infocard
   - Report bug at the bottom should redirect to bug or just say provide feedback
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms)
   - Why do IPs always point to the resource ?
   - Account in user info card doesn't directly go to account but to the account listview ?
   
   Mobile View :
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Shows only one template setting~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)~
   - ~When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)~
   - ~If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)~
   - ~The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state~
   - ~Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state~
   - ~Enter to advance search rather than submit (OK, if this is a quick one)~
   
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   
   Usability :
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] utchoang edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

utchoang edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661708393


   @davidjumani @rhtyd 
   - Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)
   > This means that if there is no zone with SG enabled, `networks> security group` will not be displayed?
   
   - Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier [preferably the former, to add existing vms]
   > I still don't know this requirement very well, can you describe it more clearly?


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting removes the prior existing setting after refresh for xen & vmware
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - On first time listing isos, the account column is missing
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   - ~Since the get health check results for VR doesn't download, consider changing the icon (OK)~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Shows only one template setting if it is within a project : Related to showunique flag
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] utchoang commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

utchoang commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-661713471


   @davidjumani I got it. 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.

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

[GitHub] [cloudstack-primate] davidjumani commented on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani commented on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-664281254


   Fixed from the original list :
   
   Via #552
   - ~Change or remove (explain?) what "Add new VM" is from VPC tiers tab; (a) allow users to add existing VMs or (b) deploy new VMs on the specific VPC tier, the deployment form does not auto-select the specific VPC tier~
   - ~Hide SG feature when there are no zones with SG enabled (may be done during API discovery, don't create the SG route if no zones have SG enabled)~
   - ~Fix the add and edit users form to show timezone as list of searchable dropdown like in the add account form~
   
   Fixed in #539 
   - ~Fix button name in VM deployment form in its network section to "Create New Network"~
   - ~Hide the 'create private gateway' button for Users in VPC view (users not allowed to create private gateway)~ : Users do not have access to the create gateway API so already covered
   - ~Don't allow users in UI to delete/archive events~
   - ~From VRs or systemvms, click on compute offering the URL/path is not added with ?issystem=true, and show an empty resource view~
   - ~Add resources tab like from zone, but for pod and cluster (listCapacity)~
   
   TODO : 
   - Check and don't allow domain admins to see VRs tab in guest networks and in VPCs : But domain admins do have access to listrouters. Perhaps just remove the router-link ?
   - Show buttons with labels on resource view when the number of action buttons are very few
   - All icons UI-wide look hanging at the bottom in Firefox but not Chrome (could be AntD bug)
   - Whether we should show the reset password button for all VMs, including those not password enabled? (hidden in Primate based on password enabled option)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api
   - Missing capacity fields while creating primary storage
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search in account / domain level settings
   - When editing a role, resource admin is not in the dropdown
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - VR router offering link leads to an empty page
   - In add ACL Rule, the Rule# field should be a number, not text
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]??
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMs rather than metrics since it doesn't respect hostid param
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   - Add loading while fetching tags and annotations
   - Root Domain shows as '/' in list & detail view infocard
   - Report bug at the bottom should redirect to bug or just say provide feedback
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms)
   - Why do IPs always point to the resource ?
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   - Refresh after successfully uploading an iso
   - Make external Id required for backup offerings
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate
   
   
   Mobile View :
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   Fixed in #544
   - ~Scrolling on an autogenerated options, it scrolls back to the top automatically~
   
   Fixed in #545
   - ~Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR~
   
   Fixed in https://github.com/apache/cloudstack/pull/4213
   - ~Search by tag doesn't work for vm snapshots~
   
   Fixed in https://github.com/apache/cloudstack/pull/4214
   - ~Adding pagination for listNetworkAclLists~
   - ~Adding pagination for listRoles~
   - ~Returning mshost uuid rather than msid in list hosts response~
   - ~ListVMMetrics doesn't respect hostid param~
   
   Fixed in #539
   - ~Refresh after successfully uploading an iso (OK, David to check and call parent refresh)~
   - ~Make external Id required for backup offerings (David to check and remove/address the issue)~
   - ~Traffic type on the network offerings redirects to offering. Could remove the router link~
   - ~Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate (David - based on states show/hide the button)~
   - ~Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh~
   - ~On first time listing isos, the account column is missing~
   - ~Report bug at the bottom should redirect to bug or just say provide feedback (change string from "Report Bug" to "report issue")~
   - ~Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)~
   
   TO FIX :
   - Deleting all types of Network's tags removes it on the current page but persists on refresh (Perhaps just within a project) : Delete tags api returns an error : `Unable to execute API command deletetags due to missing parameter resourcetype`
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Shows only one template setting if it is within a project : Related to showunique flag
   - Templates: On refresh, lists only first tag but on adding a tag, shows all (probably just in project view)
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]?? Probably has to do with the fact that deleting them is an async job
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present) -> David - hide the button based on state
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present) -> hide/disable buttons based on state
   
   Usability :
   - Can perform the listHostHaProvider to check whether the hypervisor is supported (OK, show()-> only show for KVM and Simulator)
   - When migrating a VR/systemvm, exclude the current host from the list (OK, check/reuse the migrate user-VM component... concern: keep it simple, don't expand scope)
   - In the migrate snapshot to different storage, exclude the current host storage (OK... -> quick framework solution -> TBD with Rohit/David)
   - Is there a need to show state as well as allocated state in list cluster and zones ? (David to investigate what that allocation and managed states are, and then remove from list view... add them in detail view and/or the info card?)
   - Add account on project dashboard (OK with this, add button on the accounts/project component and remove the add button from project actions)
   - Enter to advance search rather than submit (OK, if this is a quick one)
   - Since the get health check results for VR doesn't download, consider changing the icon (OK)
   - No point in having update template permissions if inside project since you can't modify them (TBD - David/Pearl/Rohit to review)
   - Sort the projects in the projects dropdown (OK with sorting the list... David - review the logic of loading projects)
   - Check/ensure 500 as max page size across views/components (David?)
   - Add loading while fetching tags and annotations (OK, David - Infocard)
   - Root Domain shows as '/' in list & detail view infocard (David - to check in what views or check in InfoCard - domainpath....?)
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms; David - to check and remove/hide/show buttons based on hypervisor and old UI)
   
   After 1.0/GA:
   - Enter to submit a form (Alex's point - should be consistent)
   - Search for IPAddress firewall rule ?
   - While listing domains, on full page reload, it goes back to the root domain (context is lost; we don't keep selected domain in router path/url)
   - Why do IPs always point to the resource ? (David - OK to remove)
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   
   Mobile View/1.1:
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   
   Undecided:
   - Keep the current router path while switching projects (David can explore, can we use the current router path/name to reload the page)


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   - Contextual help is missing for creating service offerings
   - Contextual help missing in Infra > Add SSL certificate
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api
   - Missing capacity fields while creating primary storage
   - In the event listview, clicking the username redirects to the event not the user
   - While listing domains, on refresh, it goes back to the root domain (context is lost)
   - Add option to search in account / domain level settings
   - When editing a role, resource admin is not in the dropdown
   - No pagination for security groups ingress / egress rules
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - VR router offering link leads to an empty page
   - In add ACL Rule, the Rule# field should be a number, not text
   - In the migrate snapshot to different storage, exclude the current host storage
   - The delete snapshot icons shows even while the snapshot is being created (BackingUp state)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   


----------------------------------------------------------------
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.

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

[GitHub] [cloudstack-primate] davidjumani edited a comment on issue #528: Consolidated Bug List from Testing/Review

[GitBox <gi...@apache.org>]

davidjumani edited a comment on issue #528:
URL: https://github.com/apache/cloudstack-primate/issues/528#issuecomment-656987179


   Bugs :
   Fixed in #534
   - ~While changing a systemvm service offering, need to fetch system offerings by passing `issystem=true`, `virtualmachineid` and `systemvmtype` to the listServiceOfferings api~
   - ~In the event listview, clicking the username redirects to the event not the user~
   - ~When editing a role, resource admin is not in the dropdown~
   - ~VR router offering link leads to an empty page~
   - ~In DeployVM, disk offering search doesn't show 'No records' if none present but keeps the old rows~
   - ~In add ACL Rule, the Rule# field should be a number, not text~
   - ~No pagination for security groups ingress / egress rules~
   
   Fixed in #540
   - ~Contextual help is missing for creating service offerings~
   - ~Contextual help missing in Infra > Add SSL certificate~
   
   TO FIX :
   - Contextual help missing in Add Zone, although not there in the legacy UI either
   - Scrolling on an autogenerated options, it scrolls back to the top automatically
   - Missing capacity fields while creating primary storage
   - Deleteing all types of Network's tags removes it on the current page but prrsists on refresh (Perhaps just within a project)
   - Template order column appears if you go back from details view to listview while in a project but disappears on refresh or just disappears on refresh
   - Template adding setting disappears after refresh for non standard input. Eg: cpu.corespersocket: 4
   - Pagination broken on VPN > ACL : `listnetworkacllistsresponse` does not support pagination so broken on primate
   - Traffic type on the network offerings redirects to offering. Could remove the router link ?
   - Management server on the host info card is wrong ?
   - Pagination broken on roles - Broken in the backend
   - Save settings in templates, should save all, not just the one added
   - Templates: On refresh, lists only first tag but on adding a tag, shows all
   - Add issystem=true systemvmtype=domainrouter to listserviceofferings called within VR
   - Tags won't go unless deleted twice [vpc, ipaddress, snapshots]??
   - Search by tag doesn't work for vm snapshots
   - On first time listing isos, the account column is missing
   - Call ListVMs rather than metrics since it doesn't respect hostid param
   
   
   Usability :
   - Show EnableHA as a uservm action only if it is supported. Can perform the listHostHaProvider to check whether the hypervisor is supported
   - When migrating a vm / systemvm, exclude the current host from the list
   - In the migrate snapshot to different storage, exclude the current host storage
   - Default select a zone in deployVM
   - Is there a need to show state as well as allocated state in list zones ?
   - Add account on project dashboard
   - Enter to submit a form
   - Enter to advance search rather than submit
   - Keep the current router path while switching projects
   - Since the get health check results for VR doesn't download, consider changing the icon
   - No point in having update template permissions if inside project since you can't modify them
   - Sort the projects in the projects dropdown
   - Search for IPAddress firewall rule ?
   - The delete snapshot icons show even while the snapshot is being created (BackingUp state. Copies existing functionality but would be better if this check is present)
   - Actions for templates / isos show even while they're being uploaded (clicking them tells you that they're not in the right state to perform any action. Copies existing functionality but would be better if this check is present)
   - While listing domains, on page refresh, it goes back to the root domain (context is lost)
   - Add loading while fetching tags and annotations
   - Root Domain shows as '/' in list & detail view infocard
   - Report bug at the bottom should redirect to bug or just say provide feedback
   - If dynamic scalling not supported on the hypervisor, don't show it as an action button (systemvms)
   - Why do IPs always point to the resource ?
   - Account in user info card doesn't directly go to account but to the account listview ?
   - LB Rule is just plain json in ilb
   - Refresh after successfully uploading an iso
   - Make external Id required for backup offerings
   - Assign to backup shows even while vm is starting in legacy UI as opposed to while running in primate
   
   
   Mobile View :
   - Project selector not visible in mobile mode ?
   - Create zone form is not aligned in mobile view
   - Header notification should be clearable (obstrucs in mobile view)
   - Edit setting icon misaligned in mobile (accounts, domain, zones, etc)
   - Page size changer doesn't show up on mobile mode
   


----------------------------------------------------------------
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.

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