Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for Azure is detecting aws ecxx in QESAP deployment ansible along with the Azure. #154

Merged
merged 1 commit into from
May 23, 2023

Conversation

rjagu1
Copy link
Collaborator

@rjagu1 rjagu1 commented May 22, 2023

Detect-cloud-platform.yaml is implemented to have one Dmidecode of aws has been identified by azure in qesap deployment ansible code.
Removed below lines to avoid ambiguity

ansible.builtin.command: dmidecode --string system-uuid
probe_aws_1.stdout is match("ec.*")

TEAM-7993

Verification Links:

Azure saptune: http://openqaworker15.qa.suse.cz/tests/180426
Azure sapconf : http://openqaworker15.qa.suse.cz/tests/180427
GCP : http://openqaworker15.qa.suse.cz/tests/180428

AWS: http://openqaworker15.qa.suse.cz/tests/180429
http://openqaworker15.qa.suse.cz/tests/180430
http://openqaworker15.qa.suse.cz/tests/180431
http://openqaworker15.qa.suse.cz/tests/180432
http://openqaworker15.qa.suse.cz/tests/180433

@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:12 — with GitHub Actions Inactive
@rjagu1 rjagu1 force-pushed the detect_cloudtype branch from cc51ae2 to 8b324c2 Compare May 22, 2023 13:18
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:18 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:18 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:18 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:18 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:18 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:18 — with GitHub Actions Inactive
@rjagu1 rjagu1 temporarily deployed to production May 22, 2023 13:19 — with GitHub Actions Inactive
@rjagu1 rjagu1 changed the title Detect cloudtype Azure is detecting aws ecxx in QESAP deployment ansible along with the Azure. May 22, 2023
@rjagu1 rjagu1 changed the title Azure is detecting aws ecxx in QESAP deployment ansible along with the Azure. Fix for Azure is detecting aws ecxx in QESAP deployment ansible along with the Azure. May 22, 2023
@rjagu1 rjagu1 marked this pull request as ready for review May 22, 2023 17:18
Copy link
Collaborator

@mpagot mpagot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@BillAnastasiadis BillAnastasiadis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mpagot mpagot merged commit de53de1 into SUSE:main May 23, 2023
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