[GitHub] [cloudstack-primate] rhtyd commented on issue #487: [BUG] Backup tab is shown when Backup Framework is not enabled

2020-07-06 Thread GitBox


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


   With @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:
us...@infra.apache.org




[GitHub] [cloudstack-primate] rhtyd commented on issue #487: [BUG] Backup tab is shown when Backup Framework is not enabled

2020-07-06 Thread GitBox


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


   Legacy UI also show the backups view and offering even when the feature is 
disabled globally and for all zones
   ![Screenshot from 2020-07-06 
14-09-25](https://user-images.githubusercontent.com/95203/86573648-53c87d00-bf92-11ea-9548-7c72a7d7a54d.png)
   
   Using 
http://docs.cloudstack.apache.org/en/latest/adminguide/backup_and_recovery.html 
as a reference, if the feature is not globally enabled 
(backup.framework.enabled) then it must not be considered enabled; therefore 
the feature is considered enabled for a zone when it's globally enabled and 
enabled for a zone https://github.com/apache/cloudstack/pull/4202
   This will fix exporting the APIs, if feature is disabled it won't export 
APIs and clients such as cmk, Primate won't show APIs or views.
   



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:
us...@infra.apache.org




[GitHub] [cloudstack-primate] rhtyd commented on issue #487: [BUG] Backup tab is shown when Backup Framework is not enabled

2020-07-05 Thread GitBox


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


   @PaulAngus I've already said show/hide "depending on user-role access" and 
"availability of feature" so why is that unacceptable? Why is this a release 
blocker? 



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:
us...@infra.apache.org




[GitHub] [cloudstack-primate] rhtyd commented on issue #487: [BUG] Backup tab is shown when Backup Framework is not enabled

2020-07-04 Thread GitBox


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


   With @davidjumani 
   - Remove the view backup and vmsnapshot button via config::related for 
compute.js
   - Instead add tabs to the VM instance/detail component to show/hide both 
backup and vm snapshots (similar to the volumes, nics etc tabs) depending on 
user-role access and availability of feature
   
   @PaulAngus Lowering severity to Major, we'll fix the issue for 1.0/GA but 
let's use blocker limited to issues where a feature is broken (i.e. does not 
work, than a visibility/usabilty issue)



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:
us...@infra.apache.org