Tags: ansible/awx-operator
Tags
Wait for instance ready in molecule test (#1901) Sometimes a job is launched through the web api before the instance is in a ready state. This throws a 500 internal server error, causing CI to fail. Adds a task to query the instances endpoint and check that at least one control node is in a ready state. Signed-off-by: Seth Foster <[email protected]>
Add explicit list filter after rejectattr (#1845) With ansible 2.9.27 (operator-sdk v1.27.0) then the rejectattr filter returns a generator so we need to cast it to list. The behavior doesn't exist when using a more recent operator-sdk version like v1.34.0 (ansible-core 2.15.8) but using the list filter on that version works too (even if not needed) "<generator object select_or_reject at 0x7fbbf0443728>" This is a similar issue as 80a9e8c. TASK [Get the new resource pod information after updating resource.] ******************************** FAILED! => {"msg": "The conditional check '_new_pod['resources'] | rejectattr('metadata.deletionTimestamp', 'defined') | length' failed. The error was: Unexpected templating type error occurred on ({% if _new_pod['resources'] | rejectattr('metadata.deletionTimestamp', 'defined') | length %} True {% else %} False {% endif %}): object of type 'generator' has no len() This also removes the unneeded quotes on the when conditions. Signed-off-by: Dimitri Savineau <[email protected]>
backup: Remove default parameter from jinja map (#1839) The default paramater from the jinja map filter has been added in the 2.11.0 release. However, the downstream ansible operator is still using ansible 2.9 with jinja 2.10.x so using the default parameter leads to the following error: TASK [Dump ingress tls secret names from awx spec and data into file] ******************************** The error was: jinja2.exceptions.FilterArgumentError: Unexpected keyword argument 'default' fatal: [localhost]: FAILED! => { "msg": "Unexpected failure during module execution.", "stdout": "" } Rather than using the default parameter with the map filter then add the selectattr filter to get only the items with tls_secret defined and then get the tls_secret attribute with the map filter. This also gets rid of the when statement since we always get an empty list when no tls_secret are present in ingress_hosts so the loop statement will be skipped on the empty list. Finally this changes the default value from the ingress_hosts field because it's a list rather than a string. https://jinja.palletsprojects.com/en/latest/templates/#jinja-filters.map Signed-off-by: Dimitri Savineau <[email protected]>
csv: Fix metrics utility fields (#1783) The metrics utility fields were configured under the statusDescriptors section rather than specDescriptors so displaying those fields in the UI wasn't done correctly (not under the Advanced section nor using the correct field type). This also changes the `metrics_utility_configmap` descriptor from `urn:alm:descriptor:com.tectonic.ui:selector:core:v1:ConfigMap` to `urn:alm:descriptor:io.kubernetes:ConfigMap` because the first value doesn't work. Finally, all metrics utility fields are only displayed (in the Advanced section) when `metrics_utility_enabled` is enabled (not default). Signed-off-by: Dimitri Savineau <[email protected]>
Revert change to cast settings values to strings as they could be nes… …ted (#1756) - We cast the settings value to a string so that it would display properly in the Openshift UI. Unfortunately, the k8s validator will no longer allow arrays for settings values.
PreviousNext