Merge branch '270130-policy-preview-for-egress-deny-all-reports-egress-allow-all' into 'master'

Correct preview for egress & ingress `deny all` policy which incorrectly reports `allow all`

See merge request gitlab-org/gitlab!52370

changelogs/unreleased/270130-policy-preview-for-egress-deny-all-reports-egress-allow-all.yml

+---
+title: Correct the egress 'deny all' policy preview from Allow to Deny
+merge_request: 52370
+author:
+type: fixed

ee/app/assets/javascripts/threat_monitoring/components/policy_editor/lib/humanize.js

@@ -129,10 +129,10 @@ export default function humanizeNetworkPolicy(policy) {
     const template =
       direction === RuleDirectionInbound
         ? s__(
-            'NetworkPolicies|Allow all inbound traffic to %{selector} from %{ruleSelector} on %{ports}',
+            'NetworkPolicies|Deny all inbound traffic to %{selector} from %{ruleSelector} on %{ports}',
           )
         : s__(
-            'NetworkPolicies|Allow all outbound traffic from %{selector} to %{ruleSelector} on %{ports}',
+            'NetworkPolicies|Deny all outbound traffic from %{selector} to %{ruleSelector} on %{ports}',
           );
     const ruleSelector = humanizeNetworkPolicyRule(rule);
     const ports = humanizeNetworkPolicyRulePorts(rule);

ee/spec/frontend/threat_monitoring/components/policy_editor/lib/humanize_spec.js

@@ -36,7 +36,7 @@ describe('humanizeNetworkPolicy', () => {
 
   it('returns policy description', () => {
     expect(humanizeNetworkPolicy(policy)).toEqual(
-      'Allow all inbound traffic to <strong>all</strong> pods ' +
+      'Deny all inbound traffic to <strong>all</strong> pods ' +
         'from <strong>all</strong> pods ' +
         'on <strong>any</strong> port',
     );
@@ -50,7 +50,7 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to pods <strong>[one: , two: another]</strong> ' +
+        'Deny all inbound traffic to pods <strong>[one: , two: another]</strong> ' +
           'from <strong>all</strong> pods ' +
           'on <strong>any</strong> port',
       );
@@ -66,9 +66,9 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to <strong>all</strong> pods from <strong>all</strong> pods on <strong>any</strong> port' +
+        'Deny all inbound traffic to <strong>all</strong> pods from <strong>all</strong> pods on <strong>any</strong> port' +
           '<br><br>AND<br><br>' +
-          'Allow all outbound traffic from <strong>all</strong> pods to <strong>all</strong> pods on <strong>any</strong> port',
+          'Deny all outbound traffic from <strong>all</strong> pods to <strong>all</strong> pods on <strong>any</strong> port',
       );
     });
   });
@@ -81,7 +81,7 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to <strong>all</strong> pods ' +
+        'Deny all inbound traffic to <strong>all</strong> pods ' +
           'from <strong>all</strong> pods ' +
           'on ports <strong>80/TCP, 81/UDP</strong>',
       );
@@ -95,7 +95,7 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to <strong>all</strong> pods ' +
+        'Deny all inbound traffic to <strong>all</strong> pods ' +
           'from pods <strong>[one: , two: another]</strong> ' +
           'on <strong>any</strong> port',
       );
@@ -111,7 +111,7 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to <strong>all</strong> pods ' +
+        'Deny all inbound traffic to <strong>all</strong> pods ' +
           'from <strong>host, world</strong> ' +
           'on <strong>any</strong> port',
       );
@@ -127,7 +127,7 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to <strong>all</strong> pods ' +
+        'Deny all inbound traffic to <strong>all</strong> pods ' +
           'from <strong>0.0.0.0/32, 1.1.1.1/24</strong> ' +
           'on <strong>any</strong> port',
       );
@@ -143,7 +143,7 @@ describe('humanizeNetworkPolicy', () => {
 
     it('returns policy description', () => {
       expect(humanizeNetworkPolicy(policy)).toEqual(
-        'Allow all inbound traffic to <strong>all</strong> pods ' +
+        'Deny all inbound traffic to <strong>all</strong> pods ' +
           'from <strong>some-service.com, another-service.com</strong> ' +
           'on <strong>any</strong> port',
       );

locale/gitlab.pot

@@ -18836,12 +18836,6 @@ msgstr ""
 msgid "NetworkPolicies|Allow"
 msgstr ""
 
-msgid "NetworkPolicies|Allow all inbound traffic to %{selector} from %{ruleSelector} on %{ports}"
-msgstr ""
-
-msgid "NetworkPolicies|Allow all outbound traffic from %{selector} to %{ruleSelector} on %{ports}"
-msgstr ""
-
 msgid "NetworkPolicies|Are you sure you want to delete this policy? This action cannot be undone."
 msgstr ""
 
@@ -18860,6 +18854,12 @@ msgstr ""
 msgid "NetworkPolicies|Delete policy: %{policy}"
 msgstr ""
 
+msgid "NetworkPolicies|Deny all inbound traffic to %{selector} from %{ruleSelector} on %{ports}"
+msgstr ""
+
+msgid "NetworkPolicies|Deny all outbound traffic from %{selector} to %{ruleSelector} on %{ports}"
+msgstr ""
+
 msgid "NetworkPolicies|Deny all traffic"
 msgstr ""