ravening opened a new pull request #4230:
URL: https://github.com/apache/cloudstack/pull/4230


   ## Description
   <!--- Describe your changes in detail -->
   Provide reset button to zone,cluster,domain,account,
   primary and secondary storage so that config values
   can be reset to the default value
   
   Depends on https://github.com/apache/cloudstack/pull/4215
   ## Types of changes
   <!--- What types of changes does your code introduce? Put an `x` in all the 
boxes that apply: -->
   - [X] Enhancement (improves an existing feature and functionality)
   
   
   ## Screenshots (if appropriate):
   Reset button has been provided under zone,cluster, domain, account, primary 
storage and secondary storage level
   
   ![Screenshot 2021-03-23 at 14 07 
26](https://user-images.githubusercontent.com/10645273/112151086-1ae43d00-8be1-11eb-9ffd-3adec8a4ded2.png)
   
   
   
   ## How Has This Been Tested?
   <!-- Please describe in detail how you tested your changes. -->
   <!-- Include details of your testing environment, and the tests you ran to 
-->
   <!-- see how your change affects other areas of the code, etc. -->
   
   Tested throguh cmk also
   
   ```
   (local) mgt01 > list configurations name=allow.user.expunge.recover.vm 
accountid=4048bfec-c28e-11ea-9a43-0617bc003377
   {
     "configuration": [
       {
         "category": "Advanced",
         "description": "Determines whether users can expunge or recover their 
vm",
         "isdynamic": true,
         "name": "allow.user.expunge.recover.vm",
         "scope": "account",
         "value": "false"
       }
     ],
     "count": 1
   }
   (local) mgt01 > update configuration name=allow.user.expunge.recover.vm 
accountid=4048bfec-c28e-11ea-9a43-0617bc003377 value=true
   {
     "configuration": {
       "category": "Advanced",
       "description": "Determines whether users can expunge or recover their 
vm",
       "isdynamic": true,
       "name": "allow.user.expunge.recover.vm",
       "scope": "account",
       "value": "true"
     }
   }
   
   (local) mgt01 > reset configuration name=allow.user.expunge.recover.vm 
accountid=4048bfec-c28e-11ea-9a43-0617bc003377
   {
     "configuration": {
       "category": "Advanced",
       "description": "Determines whether users can expunge or recover their 
vm",
       "isdynamic": true,
       "name": "allow.user.expunge.recover.vm",
       "scope": "account",
       "value": "false"
     }
   }
   (local) mgt01 > list configurations name=allow.user.expunge.recover.vm 
accountid=4048bfec-c28e-11ea-9a43-0617bc003377
   {
     "configuration": [
       {
         "category": "Advanced",
         "description": "Determines whether users can expunge or recover their 
vm",
         "isdynamic": true,
         "name": "allow.user.expunge.recover.vm",
         "scope": "account",
         "value": "false"
       }
     ],
     "count": 1
   }
   ```
   
   
   Run integration test case using
   
   ```
   nosetests --with-marvin --marvin-config=<cfg file> 
test_reset_configuration_settings.py
   ```


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to