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.

Sign up for GitHub

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

Jump to bottom

[6.13.z] Fix domain read in discovery auto provision test #13710

Merged
merged 1 commit into from
Jan 10, 2024
Merged

[6.13.z] Fix domain read in discovery auto provision test #13710

merged 1 commit into from
Jan 10, 2024

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #13708

Problem Statement

tests/foreman/ui/test_discoveredhost.py:197: in test_positive_auto_provision_host_with_rule
    domain_name = provisioning_hostgroup.domain.name
E   AttributeError: 'Domain' object has no attribute 'name'

Solution

read Domain first after that name attribute is accessible, same is present in other tests

@Gauravtalreja1 @web-flow
Fix domain read in discovery auto provision test (#13708)
2093042 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 6d75a02)
@Satellite-QE Satellite-QE added 6.13.z Introduced in or relating directly to Satellite 6.13 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Jan 10, 2024
@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Jan 10, 2024
@adarshdubey-star adarshdubey-star removed the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Jan 10, 2024
@adarshdubey-star adarshdubey-star merged commit 4efe97e into 6.13.z Jan 10, 2024
22 of 23 checks passed
@adarshdubey-star adarshdubey-star deleted the cherry-pick-6.13.z-6d75a026be2e234c88f39328408e716a2d91f51c branch January 10, 2024 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adarshdubey-star adarshdubey-star adarshdubey-star approved these changes

Assignees

@Gauravtalreja1 Gauravtalreja1

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Satellite-QE @adarshdubey-star @Gauravtalreja1