simplyfy example by removing changed_when: false

it is recomendation by @felixfontein to avoid changed_when: false as it may confuse users and they mayht assume that you have to use that parameter

most probably no-changed-when linter rule was enforced with ansible-lint v24.6.0 released yesterday https://github.com/ansible/ansible-lint/releases/tag/v24.6.0

docs/kubernetes.core.kubectl_connection.rst

@@ -382,8 +382,7 @@ Examples
         # be aware that the command is executed as the user that started the container
         # and requires python to be installed in the image
         - name: Run a command in a pod
-          ansible.builtin.command: echo "Hello, World!"
-          changed_when: false
+          ansible.builtin.command: /run/some/script.sh
 
     - name: Run a command in a pod using local kubectl with inventory variables
       # Example inventory:
@@ -407,8 +406,7 @@ Examples
         # be aware that the command is executed as the user that started the container
         # and requires python to be installed in the image
         - name: Run a command in a pod
-          ansible.builtin.command: echo "Hello, World!"
-          changed_when: false
+          ansible.builtin.command: /run/some/script.sh
 
     - name: Run a command in a pod using dynamic inventory
       hosts: localhost
@@ -441,9 +439,8 @@ Examples
         - name: Run a command in My App pod
           # be aware that the command is executed as the user that started the container
           # and requires python to be installed in the image
-          ansible.builtin.command: echo "Hello, World!"
+          ansible.builtin.command: /run/some/script.sh
           delegate_to: "{{ my_app_pod_name }}"
-          changed_when: false
 
 
 

plugins/connection/kubectl.py

@@ -195,8 +195,7 @@
     # be aware that the command is executed as the user that started the container
     # and requires python to be installed in the image
     - name: Run a command in a pod
-      ansible.builtin.command: echo "Hello, World!"
-      changed_when: false
+      ansible.builtin.command: /run/some/script.sh
 
 - name: Run a command in a pod using local kubectl with inventory variables
   # Example inventory:
@@ -220,8 +219,7 @@
     # be aware that the command is executed as the user that started the container
     # and requires python to be installed in the image
     - name: Run a command in a pod
-      ansible.builtin.command: echo "Hello, World!"
-      changed_when: false
+      ansible.builtin.command: /run/some/script.sh
 
 - name: Run a command in a pod using dynamic inventory
   hosts: localhost
@@ -254,9 +252,8 @@
     - name: Run a command in My App pod
       # be aware that the command is executed as the user that started the container
       # and requires python to be installed in the image
-      ansible.builtin.command: echo "Hello, World!"
+      ansible.builtin.command: /run/some/script.sh
       delegate_to: "{{ my_app_pod_name }}"
-      changed_when: false
 """
 
 import json