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

Fix typo in capsule_setup helper #17657

Merged
merged 1 commit into from
Feb 24, 2025
Merged

Fix typo in capsule_setup helper #17657

merged 1 commit into from
Feb 24, 2025

Conversation

Gauravtalreja1
Copy link
Collaborator

Problem Statement

Recent change in PR #17489 updated capsule_setup helper, which now fails with below error

failed on setup with "dynaconf.vendor.box.exceptions.BoxKeyError: "'DynaBox' object has no attribute 'rhelsource'". Did you mean: 'rhel_source'?"

Solution

Fix typo in capsule_setup helper

Related Issues

@Gauravtalreja1
Fix typo in capsule_setup helper
e6c2c5b 
Signed-off-by: Gaurav Talreja <[email protected]>
@Gauravtalreja1 Gauravtalreja1 added TestFailure Issues and PRs related to a test failing in automation CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.16.z Introduced in or relating directly to Satellite 6.16 6.17.z labels Feb 24, 2025
@Gauravtalreja1 Gauravtalreja1 self-assigned this Feb 24, 2025
@Gauravtalreja1 Gauravtalreja1 requested a review from a team as a code owner February 24, 2025 07:51
@Gauravtalreja1 Gauravtalreja1 added the Easy Fix :) Easiest Fix to review and quick merge request. label Feb 24, 2025
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/destructive/test_puppetplugin.py::test_positive_enable_disable_logic

@pondrejk
Copy link
Contributor

~260 failures related to this in a recent downstream run

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10271
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/destructive/test_puppetplugin.py::test_positive_enable_disable_logic --external-logging
Test Result : ================= 1 passed, 17 warnings in 2426.20s (0:40:26) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 24, 2025
@pondrejk pondrejk merged commit ab40a81 into SatelliteQE:master Feb 24, 2025
16 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 24, 2025
@Gauravtalreja1 @web-flow
Fix typo in capsule_setup helper (#17657)
2a67cba 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit ab40a81)
github-actions bot pushed a commit that referenced this pull request Feb 24, 2025
@Gauravtalreja1 @web-flow
Fix typo in capsule_setup helper (#17657)
3383d14 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit ab40a81)
This was referenced Feb 24, 2025
Merged
Merged
@Gauravtalreja1 Gauravtalreja1 deleted the fix-typo-capsupe-setup branch February 24, 2025 10:03
ogajduse pushed a commit that referenced this pull request Feb 24, 2025
@Satellite-QE
[6.17.z] Fix typo in capsule_setup helper (#17658)
db4eaaa 
Co-authored-by: Gaurav Talreja <[email protected]>
Fix typo in capsule_setup helper (#17657)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pondrejk pondrejk pondrejk approved these changes

Assignees

@Gauravtalreja1 Gauravtalreja1

Labels
6.16.z Introduced in or relating directly to Satellite 6.16 6.17.z AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request. PRT-Passed Indicates that latest PRT run is passed for the PR TestFailure Issues and PRs related to a test failing in automation
Projects
None yet
Milestone
No milestone
Development

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