Skip to content

Conversation

@mnivedithaa
Copy link
Contributor

@mnivedithaa mnivedithaa commented Dec 2, 2025

Description

After ansible version is updated, efs id value returned is null but since we are only checking for not empty, it is trying to get efs detail for null efs id and failed

`TASK [ibm.mas_devops.ocp_deprovision : Filter-efsId : EfsId] *******************
ok: [localhost] => {"ansible_facts": {"efs_id": null}, "changed": false}

TASK [ibm.mas_devops.ocp_deprovision : rosa : Get-mountId : MountId] ***********
[ERROR]: Task failed: Module failed: non-zero return code
Origin: /opt/app-root/lib64/python3.12/site-packages/ansible_collections/ibm/mas_devops/roles/ocp_deprovision/tasks/providers/rosa.yml:115:7

113 when: efs_id != ""
114 block:
115 - name: "rosa : Get-mountId : MountId"
^ column 7

fatal: [localhost]: FAILED! => {"changed": true, "cmd": "aws efs describe-mount-targets --file-system-id --query "MountTargets[*].MountTargetId"", "delta": "0:00:00.937337", "end": "2025-12-02 09:32:57.214194", "msg": "non-zero return code", "rc": 252, "start": "2025-12-02 09:32:56.276857", "stderr": "\naws: [ERROR]: argument --file-system-id: expected one argument\n\nusage: aws [options] [ ...] [parameters]\nTo see help text, you can run:\n\n aws help\n aws help\n aws help", "stderr_lines": ["", "aws: [ERROR]: argument --file-system-id: expected one argument", "", "usage: aws [options] [ ...] [parameters]", "To see help text, you can run:", "", " aws help", " aws help", " aws help"], "stdout": "", "stdout_lines": []}
`

@mnivedithaa mnivedithaa requested a review from a team as a code owner December 2, 2025 11:01
@whitfiea whitfiea changed the title [patch] check if efs id is not None [patch] check if efs id is not None in ocp_deprovision for rosa Dec 2, 2025
@whitfiea whitfiea added this pull request to the merge queue Dec 2, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 2, 2025
@whitfiea whitfiea added this pull request to the merge queue Dec 2, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 2, 2025
@whitfiea whitfiea merged commit 430ab25 into master Dec 2, 2025
2 checks passed
@whitfiea whitfiea deleted the mnivedithaa-patch-1 branch December 2, 2025 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants