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

Tfa fix compute resources azure #15742

Merged

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Jul 24, 2024

Problem Statement

Encountering an assertion error that is causing three test cases to fail.

Solution

Fixed the error, update the assertion

@amolpati30 amolpati30 requested review from a team as code owners July 24, 2024 09:33
@amolpati30 amolpati30 added Easy Fix :) Easiest Fix to review and quick merge request. 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Jul 24, 2024
@amolpati30 amolpati30 force-pushed the TFA_fix_ComputeResources-Azure branch from b6ef56a to e6a4c47 Compare July 24, 2024 09:40
@amolpati30 amolpati30 added Stream Introduced in or relating directly to Satellite Stream/Master and removed 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Jul 24, 2024
@amolpati30 amolpati30 force-pushed the TFA_fix_ComputeResources-Azure branch from e6a4c47 to 5daef63 Compare July 24, 2024 11:04
@amolpati30 amolpati30 added the No-CherryPick PR doesnt need CherryPick to previous branches label Jul 24, 2024
@amolpati30
Copy link
Contributor Author

"trigger": "test-robottelo"
"pytest": "tests/foreman/cli/test_computeresource_azurerm.py -k 'test_positive_azurerm_host_provisioned'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7837
Build Status: SUCCESS
PRT Comment: pytest /opt/app-root/src/robottelo/tests/foreman -v -m build_sanity --external-logging
Test Result : ======= 12 passed, 5531 deselected, 5639 warnings in 1946.30s (0:32:26) ========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 24, 2024
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/cli/test_computeresource_azurerm.py::{TestAzureRMFinishTemplateProvisioning,TestAzureRMUserDataProvisioning}

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7846
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_computeresource_azurerm.py::{TestAzureRMFinishTemplateProvisioning,TestAzureRMUserDataProvisioning} --external-logging
Test Result : ================= 3 passed, 102 warnings in 2583.61s (0:43:03) =================

@Gauravtalreja1 Gauravtalreja1 merged commit bbb836e into SatelliteQE:master Jul 25, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Easy Fix :) Easiest Fix to review and quick merge request. No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants