You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2022/02/21 12:51:00 UTC

[GitHub] [cloudstack] DaanHoogland commented on pull request #5970: Improve default network offering setting for egress policy

DaanHoogland commented on pull request #5970:
URL: https://github.com/apache/cloudstack/pull/5970#issuecomment-1046844354


   > > clgtm, can you expand on testing for this @Pearl1594
   > 
   > @DaanHoogland Sorry for the delay in response. So to test this -
   > 
   >     1. Create an Isolated Network offering setting the default egress policy (if firewall service is checked) - true / false
   > 
   >     2. Create a network using that offering
   > 
   >     3. Notice in the details tab of the created network you will find details of the policy:
   >        ![image](https://user-images.githubusercontent.com/10495417/154954642-53d67a68-333f-42cc-9827-9e2a957e2162.png)
   > 
   >     4. Go to the egress rules tab, there's a note for user to understand the implication of the policy / rules they add:
   >        ![image](https://user-images.githubusercontent.com/10495417/154954974-7aee5024-9081-43e4-a298-2b18a51d5ef2.png)
   
   I meant for the test description to go into the PR description, @Pearl1594 


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

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

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